Re: [Qemu-devel] [PATCH v2 10/12] MAINTAINERS: Orphanize the 'CPU (QOM)' subsystem

[Paolo Bonzini <pbonzini@redhat.com>]

On 18/12/18 16:57, Eduardo Habkost wrote:
> On Tue, Dec 18, 2018 at 04:44:25PM +0100, Thomas Huth wrote:
>> On 2018-12-18 16:31, Philippe Mathieu-Daudé wrote:
>>> Hi Paolo, Eduardo,
>>>
>>> On 12/18/18 4:22 PM, Eduardo Habkost wrote:
>>>> On Tue, Dec 18, 2018 at 01:34:52PM +0100, Paolo Bonzini wrote:
>>>>> On 18/12/18 12:56, Philippe Mathieu-Daudé wrote:
>>>>>> Nobody is looking at those files, downgrade this subsystem as orphan.
>>>>>>
>>>>>> Remove the qemu-devel@nongnu.org entry because the list is always
>>>>>> selected by the 'All patches CC here' section.
>>>>>>
>>>>>> Suggested-by: Markus Armbruster <armbru@redhat.com>
>>>>>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>>>>>> ---
>>>>>>  MAINTAINERS | 3 +--
>>>>>>  1 file changed, 1 insertion(+), 2 deletions(-)
>>>>>>
>>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>>> index e50f8c6b97..e6a73820f1 100644
>>>>>> --- a/MAINTAINERS
>>>>>> +++ b/MAINTAINERS
>>>>>> @@ -1745,8 +1745,7 @@ S: Supported
>>>>>>  F: scripts/coverity-model.c
>>>>>>  
>>>>>>  CPU
>>>>>> -L: qemu-devel@nongnu.org
>>>>>> -S: Supported
>>>>>> +S: Orphan
>>>>>>  F: qom/cpu.c
>>>>>>  F: include/qom/cpu.h
>>>>>>  
>>>>>>
>>>>>
>>>>> I don't think that's accurate.  Simply there's not much going on.  If
>>>>> patches are sent, get-maintainers's git fallback will do something
>>>>> (probably sending the patch to Eduardo, Igor or me).
>>>
>>> I followed Markus suggestion from
>>> https://lists.gnu.org/archive/html/qemu-devel/2018-11/msg05414.html:
>>>
>>>   If we take the definition of S: in MAINTAINERS seriously,
>>>   we need to appoint a maintainer (a person, not a mailing list),
>>>   or downgrade to S: Orphan.
>>
>> I agree with Markus and Philippe. Having "S: Supported" but no names
>> available is just confusing. So either put a maintainer name in here, or
>> downgrade to "Orphan".
> 
> I volunteer to maintain it, but I don't want to be the only
> maintainer.  Any other volunteer?

I can volunteer since I'd take the patches anyway.  We can also mark it
as Odd Fixes, and also make it a single thing with all of qom/ and
include/qom/.

Paolo