[PATCH v2 0/4] MAINTAINERS: update kraxel's entries.

[PATCH v2 0/4] MAINTAINERS: update kraxel's entries.

[Gerd Hoffmann]

I have not found much time to work on qemu due to being busy with
firmware (edk2 for the most part).  Time to update the MAINTAINERS
file entries to match reality.

I drop spice, ui, audio and usb due to lack of time.

I drop virtio-gpu, I don't follow recent development (venus etc.)
close enough to be able to actually review the changes.  Looking at
qemu-devel traffic for virtio-gpu it seems to be in good hands with
multiple people working on it, even though this is not reflected in
the MAINTAINERS file.

I keep the firmware bits (edk2, fw_cfg).

I also keep some other pieces which don't see much development
activity such as stdvga and cirrus for now.  I might revisit
this later.

v2 changes:
 - flip entries without maintainer to orphan even if there is
   a reviewer left.
 - add/upgrade volunteers from replies to audio sections.

take care,
  Gerd

Gerd Hoffmann (4):
  MAINTAINERS: drop audio maintainership
  MAINTAINERS: drop usb maintainership
  MAINTAINERS: drop virtio-gpu maintainership
  MAINTAINERS: drop spice+ui maintainership

 MAINTAINERS | 42 +++++++++++++-----------------------------
 1 file changed, 13 insertions(+), 29 deletions(-)

-- 
2.45.1

[PATCH v2 1/4] MAINTAINERS: drop audio maintainership

[Gerd Hoffmann]

Remove myself from audio (both devices and backend) entries.
Flip status to "Orphan" for entries which have nobody else listed.

Cc: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
Cc: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
Cc: Thomas Huth <huth@tuxfamily.org>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 MAINTAINERS | 30 ++++++++++--------------------
 1 file changed, 10 insertions(+), 20 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 448dc951c509..58e44885ce94 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1913,8 +1913,7 @@ F: include/hw/xtensa/mx_pic.h
 Devices
 -------
 Overall Audio frontends
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: hw/audio/
 F: include/hw/audio/
 F: tests/qtest/ac97-test.c
@@ -2389,8 +2388,8 @@ F: hw/virtio/virtio-mem-pci.c
 F: include/hw/virtio/virtio-mem.h
 
 virtio-snd
-M: Gerd Hoffmann <kraxel@redhat.com>
-R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
+M: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
+R: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
 S: Supported
 F: hw/audio/virtio-snd.c
 F: hw/audio/virtio-snd-pci.c
@@ -2768,7 +2767,6 @@ F: include/hw/hyperv/hv-balloon.h
 Subsystems
 ----------
 Overall Audio backends
-M: Gerd Hoffmann <kraxel@redhat.com>
 M: Marc-André Lureau <marcandre.lureau@redhat.com>
 S: Odd Fixes
 F: audio/
@@ -2784,13 +2782,11 @@ X: audio/spiceaudio.c
 F: qapi/audio.json
 
 ALSA Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Christian Schoenebeck <qemu_oss@crudebyte.com>
-S: Odd Fixes
+S: Orphan
 F: audio/alsaaudio.c
 
 Core Audio framework backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 M: Philippe Mathieu-Daudé <philmd@linaro.org>
 R: Christian Schoenebeck <qemu_oss@crudebyte.com>
 R: Akihiko Odaki <akihiko.odaki@daynix.com>
@@ -2798,36 +2794,30 @@ S: Odd Fixes
 F: audio/coreaudio.c
 
 DSound Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: audio/dsound*
 
 JACK Audio Connection Kit backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Christian Schoenebeck <qemu_oss@crudebyte.com>
-S: Odd Fixes
+S: Orphan
 F: audio/jackaudio.c
 
 Open Sound System (OSS) Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: audio/ossaudio.c
 
 PulseAudio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: audio/paaudio.c
 
 SDL Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
-R: Thomas Huth <huth@tuxfamily.org>
+M: Thomas Huth <huth@tuxfamily.org>
 S: Odd Fixes
 F: audio/sdlaudio.c
 
 Sndio Audio backend
-M: Gerd Hoffmann <kraxel@redhat.com>
 R: Alexandre Ratchov <alex@caoua.org>
-S: Odd Fixes
+S: Orphan
 F: audio/sndioaudio.c
 
 Block layer core
-- 
2.45.1

[PATCH v2 2/4] MAINTAINERS: drop usb maintainership

[Gerd Hoffmann]

Remove myself from usb entries.
Flip status to "Orphan" for entries which have nobody else listed.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 MAINTAINERS | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 58e44885ce94..a03756274017 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2140,8 +2140,7 @@ F: tests/qtest/fuzz-sdcard-test.c
 F: tests/qtest/sdhci-test.c
 
 USB
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: hw/usb/*
 F: stubs/usb-dev-stub.c
 F: tests/qtest/usb-*-test.c
@@ -2150,7 +2149,6 @@ F: include/hw/usb.h
 F: include/hw/usb/
 
 USB (serial adapter)
-R: Gerd Hoffmann <kraxel@redhat.com>
 M: Samuel Thibault <samuel.thibault@ens-lyon.org>
 S: Maintained
 F: hw/usb/dev-serial.c
-- 
2.45.1

[PATCH v2 3/4] MAINTAINERS: drop virtio-gpu maintainership

[Gerd Hoffmann]

Remove myself from virtio-gpu entries.
Flip status to "Orphan" for entries which have nobody else listed.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 MAINTAINERS | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a03756274017..37f2be570cc7 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2574,8 +2574,7 @@ F: hw/display/ramfb*.c
 F: include/hw/display/ramfb.h
 
 virtio-gpu
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: hw/display/virtio-gpu*
 F: hw/display/virtio-vga.*
 F: include/hw/virtio/virtio-gpu.h
@@ -2597,7 +2596,6 @@ F: include/hw/virtio/virtio-blk-common.h
 
 vhost-user-gpu
 M: Marc-André Lureau <marcandre.lureau@redhat.com>
-R: Gerd Hoffmann <kraxel@redhat.com>
 S: Maintained
 F: docs/interop/vhost-user-gpu.rst
 F: contrib/vhost-user-gpu
-- 
2.45.1

[PATCH v2 4/4] MAINTAINERS: drop spice+ui maintainership

[Gerd Hoffmann]

Remove myself from spice and ui entries.
Flip status to "Orphan" for entries which have nobody else listed.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 MAINTAINERS | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 37f2be570cc7..8479e8146470 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3044,8 +3044,7 @@ F: stubs/memory_device.c
 F: docs/nvdimm.txt
 
 SPICE
-M: Gerd Hoffmann <kraxel@redhat.com>
-S: Odd Fixes
+S: Orphan
 F: include/ui/qemu-spice.h
 F: include/ui/spice-display.h
 F: ui/spice-*.c
@@ -3055,7 +3054,6 @@ F: qapi/ui.json
 F: docs/spice-port-fqdn.txt
 
 Graphics
-M: Gerd Hoffmann <kraxel@redhat.com>
 M: Marc-André Lureau <marcandre.lureau@redhat.com>
 S: Odd Fixes
 F: ui/
-- 
2.45.1

Re: [PATCH v2 1/4] MAINTAINERS: drop audio maintainership

[Manos Pitsidianakis]

On Tue, 28 May 2024 at 11:39, Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> Remove myself from audio (both devices and backend) entries.
> Flip status to "Orphan" for entries which have nobody else listed.
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  MAINTAINERS | 30 ++++++++++--------------------
>  1 file changed, 10 insertions(+), 20 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 448dc951c509..58e44885ce94 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1913,8 +1913,7 @@ F: include/hw/xtensa/mx_pic.h
>  Devices
>  -------
>  Overall Audio frontends
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -S: Odd Fixes
> +S: Orphan
>  F: hw/audio/
>  F: include/hw/audio/
>  F: tests/qtest/ac97-test.c
> @@ -2389,8 +2388,8 @@ F: hw/virtio/virtio-mem-pci.c
>  F: include/hw/virtio/virtio-mem.h
>
>  virtio-snd
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> +M: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> +R: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
>  S: Supported
>  F: hw/audio/virtio-snd.c
>  F: hw/audio/virtio-snd-pci.c

While extra reviewers are always helpful, someone like Volker would
make sense, not someone without any contributions:

$ git log --format="%an" hw/audio/virtio-snd.c
hw/audio/virtio-snd-pci.c include/hw/audio/virtio-snd.h
docs/system/devices/virtio-snd.rst | sort -u
Manos Pitsidianakis
Michael Tokarev
Philippe Mathieu-Daudé
Richard Henderson
Stefan Hajnoczi
Volker Rümelin
Zheyu Ma

I'd suggest leaving adding reviewers here for a different patch submission.

Otherwise:

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>


> @@ -2768,7 +2767,6 @@ F: include/hw/hyperv/hv-balloon.h
>  Subsystems
>  ----------
>  Overall Audio backends
> -M: Gerd Hoffmann <kraxel@redhat.com>
>  M: Marc-André Lureau <marcandre.lureau@redhat.com>
>  S: Odd Fixes
>  F: audio/
> @@ -2784,13 +2782,11 @@ X: audio/spiceaudio.c
>  F: qapi/audio.json
>
>  ALSA Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>  R: Christian Schoenebeck <qemu_oss@crudebyte.com>
> -S: Odd Fixes
> +S: Orphan
>  F: audio/alsaaudio.c
>
>  Core Audio framework backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>  M: Philippe Mathieu-Daudé <philmd@linaro.org>
>  R: Christian Schoenebeck <qemu_oss@crudebyte.com>
>  R: Akihiko Odaki <akihiko.odaki@daynix.com>
> @@ -2798,36 +2794,30 @@ S: Odd Fixes
>  F: audio/coreaudio.c
>
>  DSound Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -S: Odd Fixes
> +S: Orphan
>  F: audio/dsound*
>
>  JACK Audio Connection Kit backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>  R: Christian Schoenebeck <qemu_oss@crudebyte.com>
> -S: Odd Fixes
> +S: Orphan
>  F: audio/jackaudio.c
>
>  Open Sound System (OSS) Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -S: Odd Fixes
> +S: Orphan
>  F: audio/ossaudio.c
>
>  PulseAudio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -S: Odd Fixes
> +S: Orphan
>  F: audio/paaudio.c
>
>  SDL Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
> -R: Thomas Huth <huth@tuxfamily.org>
> +M: Thomas Huth <huth@tuxfamily.org>
>  S: Odd Fixes
>  F: audio/sdlaudio.c
>
>  Sndio Audio backend
> -M: Gerd Hoffmann <kraxel@redhat.com>
>  R: Alexandre Ratchov <alex@caoua.org>
> -S: Odd Fixes
> +S: Orphan
>  F: audio/sndioaudio.c
>
>  Block layer core
> --
> 2.45.1
>

Re: [PATCH v2 1/4] MAINTAINERS: drop audio maintainership

[Gerd Hoffmann]

  Hi,

> >  virtio-snd
> > -M: Gerd Hoffmann <kraxel@redhat.com>
> > -R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> > +M: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> > +R: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
> >  S: Supported
> >  F: hw/audio/virtio-snd.c
> >  F: hw/audio/virtio-snd-pci.c
> 
> While extra reviewers are always helpful, someone like Volker would
> make sense, not someone without any contributions:

Matias volunteered to help (via reply to v1 of the series), and for
'reviewer' role I don't see a reason to be strict.  'Maintainer' would
be a different story of course.

If Volker wants step up (I see you cc'ed him already) I happily add
him too.

take care,
  Gerd

Re: [PATCH v2 1/4] MAINTAINERS: drop audio maintainership

[Volker Rümelin]

Am 28.05.24 um 12:02 schrieb Gerd Hoffmann:
>   Hi,
>
>>>  virtio-snd
>>> -M: Gerd Hoffmann <kraxel@redhat.com>
>>> -R: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
>>> +M: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
>>> +R: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
>>>  S: Supported
>>>  F: hw/audio/virtio-snd.c
>>>  F: hw/audio/virtio-snd-pci.c
>> While extra reviewers are always helpful, someone like Volker would
>> make sense, not someone without any contributions:
> Matias volunteered to help (via reply to v1 of the series), and for
> 'reviewer' role I don't see a reason to be strict.  'Maintainer' would
> be a different story of course.
>
> If Volker wants step up (I see you cc'ed him already) I happily add
> him too.

Hi,

I'm sorry, but at the moment I can't step up as reviewer. As you
probably know, I'm doing this in my free time and the time I can
dedicate to QEMU is quite limited at the moment. I don't think the QEMU
developers need an unresponsive reviewer.

However, I continue to read the qemu-devel mailing list and will speak
up if I think something is wrong.

With best regards,
Volker

Re: [PATCH v2 2/4] MAINTAINERS: drop usb maintainership

[Philippe Mathieu-Daudé]

On 28/5/24 10:38, Gerd Hoffmann wrote:
> Remove myself from usb entries.
> Flip status to "Orphan" for entries which have nobody else listed.
> 
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   MAINTAINERS | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)

Thanks Gerd for your contributions in USB! You taught me a lot
there, unfortunately this is still a tech & subsystem I don't
feel comfortable hacking around...

I'll queue this patch via my hw-misc tree.

Regards,

Phil.

Re: [PATCH v2 0/4] MAINTAINERS: update kraxel's entries.

[Philippe Mathieu-Daudé]

On 28/5/24 10:38, Gerd Hoffmann wrote:

> Gerd Hoffmann (4):
>    MAINTAINERS: drop audio maintainership

I couldn't figure if the discussion on patch #1 is resolved,
so I only queued patches 3 & 4 to avoid you getting spammed.

>    MAINTAINERS: drop virtio-gpu maintainership
>    MAINTAINERS: drop spice+ui maintainership
> 
>   MAINTAINERS | 42 +++++++++++++-----------------------------
>   1 file changed, 13 insertions(+), 29 deletions(-)
>