[Qemu-devel] [PATCH v2] MAINTAINERS: Add an entry for qemu-options* files in main directory

Thomas Huth posted 1 patch 5 years, 5 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1538650419-7995-1-git-send-email-thuth@redhat.com
Test docker-clang@ubuntu failed
Test checkpatch passed
MAINTAINERS | 8 ++++++++
1 file changed, 8 insertions(+)
[Qemu-devel] [PATCH v2] MAINTAINERS: Add an entry for qemu-options* files in main directory
Posted by Thomas Huth 5 years, 5 months ago
The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h"
in the main directory are currently without maintainer according to our
get_maintainers.pl script. Considering that the command line options are
a public interface and thus quite important, this is quite a bad state.
Add an entry for these files which is maintained by Paolo, since the
option handling is tightly coupled with the code in vl.c (that Paolo
already handles via the "Main loop" entry).

And since I'm interested in the command line interface of QEMU, add
myself (and also Markus) as reviewer here.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 v2: Create a separate entry

 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3275cc6..66c13f2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1522,6 +1522,14 @@ F: tests/test-qemu-opts.c
 F: util/keyval.c
 F: util/qemu-option.c
 
+Generic command line options
+M: Paolo Bonzini <pbonzini@redhat.com>
+R: Markus Armbruster <armbru@redhat.com>
+R: Thomas Huth <thuth@redhat.com>
+S: Maintained
+F: qemu-options*
+F: vl.c
+
 Coverity model
 M: Markus Armbruster <armbru@redhat.com>
 S: Supported
-- 
1.8.3.1


Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Add an entry for qemu-options* files in main directory
Posted by Paolo Bonzini 5 years, 5 months ago
On 04/10/2018 12:53, Thomas Huth wrote:
> The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h"
> in the main directory are currently without maintainer according to our
> get_maintainers.pl script. Considering that the command line options are
> a public interface and thus quite important, this is quite a bad state.
> Add an entry for these files which is maintained by Paolo, since the
> option handling is tightly coupled with the code in vl.c (that Paolo
> already handles via the "Main loop" entry).
> 
> And since I'm interested in the command line interface of QEMU, add
> myself (and also Markus) as reviewer here.

Sometimes, a file is really only touched as part of changes to other
files.  Command line options can be changed by block device patches,
vl.c patches, or something else.  I think changes to qemu-options* alone
are quite rare, and I'd rather not have more stuff added to my misc tree
(and my inbox)...

Paolo

> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  v2: Create a separate entry
> 
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3275cc6..66c13f2 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1522,6 +1522,14 @@ F: tests/test-qemu-opts.c
>  F: util/keyval.c
>  F: util/qemu-option.c
>  
> +Generic command line options
> +M: Paolo Bonzini <pbonzini@redhat.com>
> +R: Markus Armbruster <armbru@redhat.com>
> +R: Thomas Huth <thuth@redhat.com>
> +S: Maintained
> +F: qemu-options*
> +F: vl.c
> +
>  Coverity model
>  M: Markus Armbruster <armbru@redhat.com>
>  S: Supported
> 


Re: [Qemu-devel] [PATCH v2] MAINTAINERS: Add an entry for qemu-options* files in main directory
Posted by Markus Armbruster 5 years, 5 months ago
Paolo Bonzini <pbonzini@redhat.com> writes:

> On 04/10/2018 12:53, Thomas Huth wrote:
>> The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h"
>> in the main directory are currently without maintainer according to our
>> get_maintainers.pl script. Considering that the command line options are
>> a public interface and thus quite important, this is quite a bad state.
>> Add an entry for these files which is maintained by Paolo, since the
>> option handling is tightly coupled with the code in vl.c (that Paolo
>> already handles via the "Main loop" entry).
>> 
>> And since I'm interested in the command line interface of QEMU, add
>> myself (and also Markus) as reviewer here.
>
> Sometimes, a file is really only touched as part of changes to other
> files.  Command line options can be changed by block device patches,
> vl.c patches, or something else.  I think changes to qemu-options* alone

Yes.  But when the file in question is an important interface, having
someone in charge of interface design aspects makes sense all the same.
Without that, consistency is left to chance.

Of course, you might say that command line consistency is FUBAR anyway.
You'd have a point.

> are quite rare, and I'd rather not have more stuff added to my misc tree
> (and my inbox)...

Same here.

I serve in such a role for the QAPI schema, together with Eric.  I feel
like I'm pretty consistently failing at keeping up (pardon the pun).  I
haven't thrown in the towel only because I also feel the guidance I
manage to provide there is better than nothing.  But I'd rather not take
on more.

Note that once my project to QAPIfy the CLI (on hold due to other
obligations) is complete, the CLI will be specified in the QAPI schema,
effectively dumping it on the QAPI schema maintainers.  I guess I'm a
sucker for punishment.  Just not right now, please.