Re: [PATCH 2/2] MAINTAINERS: Mark "vhost-user-scmi" orphaned

["Michael S. Tsirkin" <mst@redhat.com>]

On Mon, Jul 13, 2026 at 10:17:07AM +0200, Markus Armbruster wrote:
> Philippe Mathieu-Daudé <philmd@oss.qualcomm.com> writes:
> 
> > On 13/7/26 08:52, Michael S. Tsirkin wrote:
> >> On Mon, Jul 13, 2026 at 08:13:38AM +0200, Philippe Mathieu-Daudé wrote:
> >>> On 11/7/26 10:00, Michael S. Tsirkin wrote:
> >>>> On Fri, Jul 10, 2026 at 01:43:05PM +0200, Markus Armbruster wrote:
> >>>>> Commit a5dab090e1 (hw/virtio: Add boilerplate for vhost-user-scmi
> >>>>> device, 2023-06-28) added the section with S: Supported, but no M:.
> >>>>> This makes no sense.  Downgrade to S: Orphan.
> >>>>>
> >>>>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> >>>>
> >>>>
> >>>> No, this makes sense. The reason this section was added is so that Milan can
> >>>> get copied on vhost-user-scmi but not other virtio stuff:
> >>>>
> >>>> ./scripts/get_maintainer.pl -f hw/virtio/vhost-user-scmi.c
> >>>> Milan Zamazal <mzamazal@redhat.com> (reviewer:vhost-user-scmi)
> >>>> "Michael S. Tsirkin" <mst@redhat.com> (supporter:vhost)
> >>>> Stefano Garzarella <sgarzare@redhat.com> (reviewer:vhost)
> >>>> qemu-devel@nongnu.org (open list:All patches CC here)
> >>>>
> >>>> Pls drop this patch.
> >>>>
> >>>>> ---
> >>>>>    MAINTAINERS | 2 +-
> >>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>>>> index aa2df9cbe3..4d558ddee9 100644
> >>>>> --- a/MAINTAINERS
> >>>>> +++ b/MAINTAINERS
> >>>>> @@ -2622,7 +2622,7 @@ F: include/hw/virtio/vhost-user-snd.h
> >>>>>    vhost-user-scmi
> >>>
> >>> So what we want is to add here:
> >>>
> >>>    M: Michael S. Tsirkin <mst@redhat.com>
> >>>    R: Stefano Garzarella <sgarzare@redhat.com>
> >>
> >> But these files already get M from the match against hw/virtio/ -
> >> what am I missing?
> >
> > Yeah, just trying to make Markus script happy ;)
> 
> Appreciate the kindness!
> 
> In decreasing importance:
> 
> 1. Correct get_maintainers.pl output

Is anything wrong with it?

> 2. Legible MAINTAINERS

We can always come up with a new rule for such cases,
where we have a reviewer just for some specific files
in a project, but the supporters are from the bigger
project.

How about we drop S: completely in such cases?

We can also tweak the name:
vhost-user-scmi (part of vhost)


> 99. My script is happy.
> 
> [...]