On Mon, Jul 13, 2026 at 09:11:34AM +0200, Philippe Mathieu-Daudé wrote:
> 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 <[email protected]>
> > > > 
> > > > 
> > > > 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 <[email protected]> (reviewer:vhost-user-scmi)
> > > > "Michael S. Tsirkin" <[email protected]> (supporter:vhost)
> > > > Stefano Garzarella <[email protected]> (reviewer:vhost)
> > > > [email protected] (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 <[email protected]>
> > >    R: Stefano Garzarella <[email protected]>
> > 
> > But these files already get M from the match against hw/virtio/ -
> > what am I missing?
> 
> Yeah, just trying to make Markus script happy ;)

I don't mind though the duplication is a bit annoying.
But let's pls document what the rules are somewhere then.

> > 
> > > > >    R: Milan Zamazal <[email protected]>
> > > > > -S: Supported
> > > > > +S: Orphan
> > > > >    F: hw/virtio/vhost-user-scmi*
> > > > >    F: include/hw/virtio/vhost-user-scmi.h
> > > > >    F: tests/qtest/libqos/virtio-scmi.*
> > > > > -- 
> > > > > 2.54.0
> > > > 
> > > > 
> > 
> > 


Reply via email to