[PATCH] MAINTAINERS: add maintainer for docs/

Pierrick Bouvier posted 1 patch 3 weeks, 1 day ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260115203529.2713193-1-pierrick.bouvier@linaro.org
MAINTAINERS | 5 +++++
1 file changed, 5 insertions(+)
[PATCH] MAINTAINERS: add maintainer for docs/
Posted by Pierrick Bouvier 3 weeks, 1 day ago
I would like to help maintaining qemu documentation and I've been
invited by Alex to apply as maintainer.

Files in docs/ that are already maintained will continue to be under
their respective maintainer. The goal here is to have someone that can
help on all other files that don't have an official maintainer.

Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
---
 MAINTAINERS | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 4ddbfba9f01..786f3b3a456 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4505,6 +4505,11 @@ Incompatible changes
 R: devel@lists.libvirt.org
 F: docs/about/deprecated.rst
 
+General Documentation
+M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
+S: Maintained
+F: docs/
+
 Build System
 ------------
 Meson
-- 
2.47.3
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Alex Bennée 2 weeks, 2 days ago
Pierrick Bouvier <pierrick.bouvier@linaro.org> writes:

> I would like to help maintaining qemu documentation and I've been
> invited by Alex to apply as maintainer.
>
> Files in docs/ that are already maintained will continue to be under
> their respective maintainer. The goal here is to have someone that can
> help on all other files that don't have an official maintainer.

Queued to for-11.0/maintainers-update, thanks.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Pierrick Bouvier 3 weeks ago
On 1/15/26 12:35 PM, Pierrick Bouvier wrote:
> I would like to help maintaining qemu documentation and I've been
> invited by Alex to apply as maintainer.
> 
> Files in docs/ that are already maintained will continue to be under
> their respective maintainer. The goal here is to have someone that can
> help on all other files that don't have an official maintainer.
> 
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>   MAINTAINERS | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4ddbfba9f01..786f3b3a456 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4505,6 +4505,11 @@ Incompatible changes
>   R: devel@lists.libvirt.org
>   F: docs/about/deprecated.rst
>   
> +General Documentation
> +M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> +S: Maintained
> +F: docs/
> +
>   Build System
>   ------------
>   Meson

By the way, who maintains the MAINTAINERS file in cases like this?

I see mostly Acked-by in answer, so not sure who is going to have a 
final word, give a reviewed-by, and pull the change.
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Alex Bennée 3 weeks ago
Pierrick Bouvier <pierrick.bouvier@linaro.org> writes:

> I would like to help maintaining qemu documentation and I've been
> invited by Alex to apply as maintainer.
>
> Files in docs/ that are already maintained will continue to be under
> their respective maintainer. The goal here is to have someone that can
> help on all other files that don't have an official maintainer.
>
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Acked-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Daniel P. Berrangé 3 weeks ago
On Thu, Jan 15, 2026 at 12:35:29PM -0800, Pierrick Bouvier wrote:
> I would like to help maintaining qemu documentation and I've been
> invited by Alex to apply as maintainer.
> 
> Files in docs/ that are already maintained will continue to be under
> their respective maintainer. The goal here is to have someone that can
> help on all other files that don't have an official maintainer.

Thanks for volunteering - the lack of a catch-all maintainer
for docs has been a long standing gap leading to patches getting
dropped on the floor.

> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>  MAINTAINERS | 5 +++++
>  1 file changed, 5 insertions(+)

Acked-by: Daniel P. Berrangé <berrange@redhat.com>


> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4ddbfba9f01..786f3b3a456 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4505,6 +4505,11 @@ Incompatible changes
>  R: devel@lists.libvirt.org
>  F: docs/about/deprecated.rst
>  
> +General Documentation
> +M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> +S: Maintained
> +F: docs/
> +
>  Build System
>  ------------
>  Meson
> -- 
> 2.47.3
> 

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|


Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Pierrick Bouvier 3 weeks ago
On 1/16/26 1:52 AM, Daniel P. Berrangé wrote:
> On Thu, Jan 15, 2026 at 12:35:29PM -0800, Pierrick Bouvier wrote:
>> I would like to help maintaining qemu documentation and I've been
>> invited by Alex to apply as maintainer.
>>
>> Files in docs/ that are already maintained will continue to be under
>> their respective maintainer. The goal here is to have someone that can
>> help on all other files that don't have an official maintainer.
> 
> Thanks for volunteering - the lack of a catch-all maintainer
> for docs has been a long standing gap leading to patches getting
> dropped on the floor.
>

Indeed, I even experienced that myself.

As I mentioned in a private conversation with Thomas, the goal here is 
to review and pull series that concern docs/ only.
In case someone added documentation as part of a series adding a 
feature, it's expected to be reviewed/pulled as part of it.

So even though the traffic might be high, concretely the real work to do 
will be limited. Hopefully :)

Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Thomas Huth 3 weeks ago
On 15/01/2026 21.35, Pierrick Bouvier wrote:
> I would like to help maintaining qemu documentation and I've been
> invited by Alex to apply as maintainer.
> 
> Files in docs/ that are already maintained will continue to be under
> their respective maintainer. The goal here is to have someone that can
> help on all other files that don't have an official maintainer.
> 
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>   MAINTAINERS | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4ddbfba9f01..786f3b3a456 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4505,6 +4505,11 @@ Incompatible changes
>   R: devel@lists.libvirt.org
>   F: docs/about/deprecated.rst
>   
> +General Documentation
> +M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> +S: Maintained
> +F: docs/

You might trigger a lot of traffic to your inbox that way ... but if you 
don't mind:

Acked-by: Thomas Huth <thuth@redhat.com>
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Pierrick Bouvier 3 weeks ago
On 1/15/26 11:17 PM, Thomas Huth wrote:
> On 15/01/2026 21.35, Pierrick Bouvier wrote:
>> I would like to help maintaining qemu documentation and I've been
>> invited by Alex to apply as maintainer.
>>
>> Files in docs/ that are already maintained will continue to be under
>> their respective maintainer. The goal here is to have someone that can
>> help on all other files that don't have an official maintainer.
>>
>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>> ---
>>    MAINTAINERS | 5 +++++
>>    1 file changed, 5 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 4ddbfba9f01..786f3b3a456 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -4505,6 +4505,11 @@ Incompatible changes
>>    R: devel@lists.libvirt.org
>>    F: docs/about/deprecated.rst
>>    
>> +General Documentation
>> +M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>> +S: Maintained
>> +F: docs/
> 
> You might trigger a lot of traffic to your inbox that way ... but if you
> don't mind:
>
> Acked-by: Thomas Huth <thuth@redhat.com>
> 

Reading about your answer, I was thinking it would be nice one day to 
organize a BoF about everyone personal email workflow.
I feel like every dev has a different way to deal with this, and even 
though it seems basic ("Who would seriously ask advice about how to deal 
with emails?"), it's much more complex than what you can expect in the 
beginning.

Hopefully, my workflow and my email client are ready to take it now.
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Thomas Huth 2 weeks, 4 days ago
On 16/01/2026 19.31, Pierrick Bouvier wrote:
> On 1/15/26 11:17 PM, Thomas Huth wrote:
>> On 15/01/2026 21.35, Pierrick Bouvier wrote:
>>> I would like to help maintaining qemu documentation and I've been
>>> invited by Alex to apply as maintainer.
>>>
>>> Files in docs/ that are already maintained will continue to be under
>>> their respective maintainer. The goal here is to have someone that can
>>> help on all other files that don't have an official maintainer.
>>>
>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>> ---
>>>    MAINTAINERS | 5 +++++
>>>    1 file changed, 5 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 4ddbfba9f01..786f3b3a456 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -4505,6 +4505,11 @@ Incompatible changes
>>>    R: devel@lists.libvirt.org
>>>    F: docs/about/deprecated.rst
>>> +General Documentation
>>> +M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>> +S: Maintained
>>> +F: docs/
>>
>> You might trigger a lot of traffic to your inbox that way ... but if you
>> don't mind:
>>
>> Acked-by: Thomas Huth <thuth@redhat.com>
>>
> 
> Reading about your answer, I was thinking it would be nice one day to 
> organize a BoF about everyone personal email workflow.
> I feel like every dev has a different way to deal with this, and even though 
> it seems basic ("Who would seriously ask advice about how to deal with 
> emails?"), it's much more complex than what you can expect in the beginning.
> 
> Hopefully, my workflow and my email client are ready to take it now.

Since you've asked for it, here are some of my thoughts:

I don't think it is e.g. hard to filter for e-mails that have "docs/" in the 
subject, so that's certainly not a problem. It's getting tricky for huge 
series that e.g. have just one little change to the docs folder in between. 
Some people will CC: the whole big series to you for that little change in 
one of the patches - how do you filter for such series?

Having maintained a generic subsystem with a catch-all entry in the past (I 
used to be the maintainer of the qtests, and we had a "F: tests/qtest/" 
entry in MAINTAINERS for this), I often got CC:-ed on big patch series that 
should rather go through the architecture maintainer's tree, and since only 
single patches affected the qtest subsystem, this was hard to filter, so I 
always got lots of unrelated patches in my Inbox in which I was not really 
interested in.

My mitigation was to add some "X:" lines to the qtest section in MAINTAINERS 
... but if someone has some better ideas how to deal with such situations 
(e.g. if there are some clever ways to filter such series out of your 
Inbox), I'd be grateful to hear about them!

FWIW, for the functional testing framework, I organized the entry in 
MAINTAINERS differently, there is no generic "F: tests/functional/" line in 
the section, but rather only coverage for the shared files of the framework 
... the tests themselves should be covered by the individual maintainers 
instead. This works much better for me there.

So I'm interested to know how the generic "F: docs/" entry will work out for 
you once this patch got merged and some weeks have passed. Having a BoF at 
the next KVM forum about this email workflow topic might be a good idea, indeed.

  Thomas


Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Philippe Mathieu-Daudé 2 weeks, 3 days ago
On 19/1/26 09:19, Thomas Huth wrote:
> On 16/01/2026 19.31, Pierrick Bouvier wrote:
>> On 1/15/26 11:17 PM, Thomas Huth wrote:
>>> On 15/01/2026 21.35, Pierrick Bouvier wrote:
>>>> I would like to help maintaining qemu documentation and I've been
>>>> invited by Alex to apply as maintainer.
>>>>
>>>> Files in docs/ that are already maintained will continue to be under
>>>> their respective maintainer. The goal here is to have someone that can
>>>> help on all other files that don't have an official maintainer.
>>>>
>>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>> ---
>>>>    MAINTAINERS | 5 +++++
>>>>    1 file changed, 5 insertions(+)


> I don't think it is e.g. hard to filter for e-mails that have "docs/" in 
> the subject, so that's certainly not a problem. It's getting tricky for 
> huge series that e.g. have just one little change to the docs folder in 
> between. Some people will CC: the whole big series to you for that 
> little change in one of the patches - how do you filter for such series?
> 
> Having maintained a generic subsystem with a catch-all entry in the past 
> (I used to be the maintainer of the qtests, and we had a "F: tests/ 
> qtest/" entry in MAINTAINERS for this), I often got CC:-ed on big patch 
> series that should rather go through the architecture maintainer's tree, 
> and since only single patches affected the qtest subsystem, this was 
> hard to filter, so I always got lots of unrelated patches in my Inbox in 
> which I was not really interested in.

You can use lorelei, i.e.:

https://lore.kernel.org/qemu-devel/?q=dfn:/docs/+AND+-s:PULL

See how to integrate with your client workflow:
https://people.kernel.org/monsieuricon/lore-lei-part-1-getting-started#maildir-or-imap

> My mitigation was to add some "X:" lines to the qtest section in 
> MAINTAINERS ... but if someone has some better ideas how to deal with 
> such situations (e.g. if there are some clever ways to filter such 
> series out of your Inbox), I'd be grateful to hear about them!

Ditto i.e.:

https://lore.kernel.org/qemu-devel/?q=dfn:/docs/+AND+-s:PULL+AND+-dfn:/docs/devel

> 
> FWIW, for the functional testing framework, I organized the entry in 
> MAINTAINERS differently, there is no generic "F: tests/functional/" line 
> in the section, but rather only coverage for the shared files of the 
> framework ... the tests themselves should be covered by the individual 
> maintainers instead. This works much better for me there.
> 
> So I'm interested to know how the generic "F: docs/" entry will work out 
> for you once this patch got merged and some weeks have passed. Having a 
> BoF at the next KVM forum about this email workflow topic might be a 
> good idea, indeed.
> 
>   Thomas
> 


Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Pierrick Bouvier 2 weeks, 4 days ago
On 1/19/26 12:19 AM, Thomas Huth wrote:
> On 16/01/2026 19.31, Pierrick Bouvier wrote:
>> On 1/15/26 11:17 PM, Thomas Huth wrote:
>>> On 15/01/2026 21.35, Pierrick Bouvier wrote:
>>>> I would like to help maintaining qemu documentation and I've been
>>>> invited by Alex to apply as maintainer.
>>>>
>>>> Files in docs/ that are already maintained will continue to be under
>>>> their respective maintainer. The goal here is to have someone that can
>>>> help on all other files that don't have an official maintainer.
>>>>
>>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>> ---
>>>>     MAINTAINERS | 5 +++++
>>>>     1 file changed, 5 insertions(+)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index 4ddbfba9f01..786f3b3a456 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -4505,6 +4505,11 @@ Incompatible changes
>>>>     R: devel@lists.libvirt.org
>>>>     F: docs/about/deprecated.rst
>>>> +General Documentation
>>>> +M: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>> +S: Maintained
>>>> +F: docs/
>>>
>>> You might trigger a lot of traffic to your inbox that way ... but if you
>>> don't mind:
>>>
>>> Acked-by: Thomas Huth <thuth@redhat.com>
>>>
>>
>> Reading about your answer, I was thinking it would be nice one day to
>> organize a BoF about everyone personal email workflow.
>> I feel like every dev has a different way to deal with this, and even though
>> it seems basic ("Who would seriously ask advice about how to deal with
>> emails?"), it's much more complex than what you can expect in the beginning.
>>
>> Hopefully, my workflow and my email client are ready to take it now.
> 
> Since you've asked for it, here are some of my thoughts:
> 
> I don't think it is e.g. hard to filter for e-mails that have "docs/" in the
> subject, so that's certainly not a problem. It's getting tricky for huge
> series that e.g. have just one little change to the docs folder in between.
> Some people will CC: the whole big series to you for that little change in
> one of the patches - how do you filter for such series?
> 
> Having maintained a generic subsystem with a catch-all entry in the past (I
> used to be the maintainer of the qtests, and we had a "F: tests/qtest/"
> entry in MAINTAINERS for this), I often got CC:-ed on big patch series that
> should rather go through the architecture maintainer's tree, and since only
> single patches affected the qtest subsystem, this was hard to filter, so I
> always got lots of unrelated patches in my Inbox in which I was not really
> interested in.
> 
> My mitigation was to add some "X:" lines to the qtest section in MAINTAINERS
> ... but if someone has some better ideas how to deal with such situations
> (e.g. if there are some clever ways to filter such series out of your
> Inbox), I'd be grateful to hear about them!
> 
> FWIW, for the functional testing framework, I organized the entry in
> MAINTAINERS differently, there is no generic "F: tests/functional/" line in
> the section, but rather only coverage for the shared files of the framework
> ... the tests themselves should be covered by the individual maintainers
> instead. This works much better for me there.
> 
> So I'm interested to know how the generic "F: docs/" entry will work out for
> you once this patch got merged and some weeks have passed. Having a BoF at
> the next KVM forum about this email workflow topic might be a good idea, indeed.
> 
>    Thomas
> 

I really appreciate you took the time to share that and your experience 
on the topic. The "exclusion" based approach you use is definitely a 
good option in case I get notified with too many series.

I tend to see series as an atomic change, expecting them to be pulled as 
a whole. Thus, I expect, maybe wrongly, that isolated doc patches will 
be pulled with the concerned series.

For my workflow, I use a single qemu-devel folder, with a few additional 
mechanics to follow threads:
- emails where I'm a direct cc/to are starred (Imap \Flagged keyword)
- use tags (imap keywords) for series: to review, personal, to follow up
- keep cover letter unread as long as there is an action to do on this 
thread
- mark as read all the rest so it can disappear quickly

Order of series in my client are:
[tag - to review]
[tag - personal (my series)]
[tag - follow up (was reviewed or is interesting)]
[untagged series where I'm a direct cc/to]
[all the rest]

This way, top series are the one followed, and all the bottom ones are 
the new ones, and I can focus on the "starred" ones to identify quickly 
content that concerns me.

 From this, I just expect to see more starred threads, and I can easily 
see what concerns me (doc oriented series) or not (partial doc as part 
of the series), and tag the series accordingly.

We'll see in two months how this opinion changed :)

Thanks,
Pierrick

Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Markus Armbruster 2 weeks, 3 days ago
Pierrick Bouvier <pierrick.bouvier@linaro.org> writes:

[...]

> I tend to see series as an atomic change, expecting them to be pulled as a whole. Thus, I expect, maybe wrongly, that isolated doc patches will be pulled with the concerned series.

Your expectation matches how I work as overall QAPI schema maintainer.
Most patch series modifying qapi/ aren't about the QAPI schema.  I
routinely review just the QAPI part, and leave the remainder and the
merging to the actual subject matter maintainers.

[...]
Re: [PATCH] MAINTAINERS: add maintainer for docs/
Posted by Markus Armbruster 3 weeks ago
Pierrick Bouvier <pierrick.bouvier@linaro.org> writes:

> I would like to help maintaining qemu documentation and I've been
> invited by Alex to apply as maintainer.
>
> Files in docs/ that are already maintained will continue to be under
> their respective maintainer. The goal here is to have someone that can
> help on all other files that don't have an official maintainer.
>
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Quick, before you change your mind:
Acked-by: Markus Armbruster <armbru@redhat.com>