[Qemu-devel] [PATCH] MAINTAINERS: remove or downgrade myself to reviewer from some subsystems

Paolo Bonzini posted 1 patch 5 years, 6 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20181019122613.28663-1-pbonzini@redhat.com
Test docker-clang@ubuntu passed
Test checkpatch passed
Test asan passed
Test docker-mingw@fedora failed
Test docker-quick@centos7 passed
MAINTAINERS | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
[Qemu-devel] [PATCH] MAINTAINERS: remove or downgrade myself to reviewer from some subsystems
Posted by Paolo Bonzini 5 years, 6 months ago
Other people are doing a much better work than myself at handling some
subsystems.  For those files it is better if I downgrade myself to
reviewer or recognize that I am not actually doing any work there.

Cc: Daniel P. Berrange <berrange@redhat.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Eric Blake <eblake@redhat.com>
Cc: Thomas Huth <thuth@redhat.com>
Cc: Laurent Vivier <lvivier@redhat.com>
Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 MAINTAINERS | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index fb81b3a8eb..8c814b2e60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -105,9 +105,9 @@ Guest CPU cores (TCG):
 ----------------------
 Overall
 L: qemu-devel@nongnu.org
-M: Paolo Bonzini <pbonzini@redhat.com>
 M: Peter Crosthwaite <crosthwaite.peter@gmail.com>
 M: Richard Henderson <rth@twiddle.net>
+R: Paolo Bonzini <pbonzini@redhat.com>
 S: Maintained
 F: cpus.c
 F: exec.c
@@ -1132,7 +1132,8 @@ F: hw/pci-host/ppce500.c
 F: hw/net/fsl_etsec/
 
 Character devices
-M: Paolo Bonzini <pbonzini@redhat.com>
+M: Marc-André Lureau <marcandre.lureau@redhat.com>
+R: Paolo Bonzini <pbonzini@redhat.com>
 S: Odd Fixes
 F: hw/char/
 
@@ -1517,8 +1518,8 @@ T: git git://github.com/famz/qemu.git bitmaps
 T: git git://github.com/jnsnow/qemu.git bitmaps
 
 Character device backends
-M: Paolo Bonzini <pbonzini@redhat.com>
 M: Marc-André Lureau <marcandre.lureau@redhat.com>
+R: Paolo Bonzini <pbonzini@redhat.com>
 S: Maintained
 F: chardev/
 F: include/chardev/
@@ -1751,9 +1752,9 @@ F: tests/qmp-cmd-test.c
 T: git git://repo.or.cz/qemu/armbru.git qapi-next
 
 qtest
-M: Paolo Bonzini <pbonzini@redhat.com>
 M: Thomas Huth <thuth@redhat.com>
 M: Laurent Vivier <lvivier@redhat.com>
+R: Paolo Bonzini <pbonzini@redhat.com>
 S: Maintained
 F: qtest.c
 F: tests/libqtest.*
@@ -1860,7 +1861,6 @@ F: tests/test-io-*
 Sockets
 M: Daniel P. Berrange <berrange@redhat.com>
 M: Gerd Hoffmann <kraxel@redhat.com>
-M: Paolo Bonzini <pbonzini@redhat.com>
 S: Maintained
 F: include/qemu/sockets.h
 F: util/qemu-sockets.c
@@ -2050,13 +2050,12 @@ M: Ronnie Sahlberg <ronniesahlberg@gmail.com>
 M: Paolo Bonzini <pbonzini@redhat.com>
 M: Peter Lieven <pl@kamp.de>
 L: qemu-block@nongnu.org
-S: Supported
+S: Odd Fixes
 F: block/iscsi.c
 F: block/iscsi-opts.c
 
 Network Block Device (NBD)
 M: Eric Blake <eblake@redhat.com>
-M: Paolo Bonzini <pbonzini@redhat.com>
 L: qemu-block@nongnu.org
 S: Maintained
 F: block/nbd*
-- 
2.17.1


Re: [Qemu-devel] [PATCH] MAINTAINERS: remove or downgrade myself to reviewer from some subsystems
Posted by Daniel P. Berrangé 5 years, 6 months ago
On Fri, Oct 19, 2018 at 02:26:13PM +0200, Paolo Bonzini wrote:
> Other people are doing a much better work than myself at handling some
> subsystems.  For those files it is better if I downgrade myself to
> reviewer or recognize that I am not actually doing any work there.
> 
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Eric Blake <eblake@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Laurent Vivier <lvivier@redhat.com>
> Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  MAINTAINERS | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)

For sockets maintainer change

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


> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index fb81b3a8eb..8c814b2e60 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -105,9 +105,9 @@ Guest CPU cores (TCG):
>  ----------------------
>  Overall
>  L: qemu-devel@nongnu.org
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Peter Crosthwaite <crosthwaite.peter@gmail.com>
>  M: Richard Henderson <rth@twiddle.net>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: cpus.c
>  F: exec.c
> @@ -1132,7 +1132,8 @@ F: hw/pci-host/ppce500.c
>  F: hw/net/fsl_etsec/
>  
>  Character devices
> -M: Paolo Bonzini <pbonzini@redhat.com>
> +M: Marc-André Lureau <marcandre.lureau@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Odd Fixes
>  F: hw/char/
>  
> @@ -1517,8 +1518,8 @@ T: git git://github.com/famz/qemu.git bitmaps
>  T: git git://github.com/jnsnow/qemu.git bitmaps
>  
>  Character device backends
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Marc-André Lureau <marcandre.lureau@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: chardev/
>  F: include/chardev/
> @@ -1751,9 +1752,9 @@ F: tests/qmp-cmd-test.c
>  T: git git://repo.or.cz/qemu/armbru.git qapi-next
>  
>  qtest
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Thomas Huth <thuth@redhat.com>
>  M: Laurent Vivier <lvivier@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: qtest.c
>  F: tests/libqtest.*
> @@ -1860,7 +1861,6 @@ F: tests/test-io-*
>  Sockets
>  M: Daniel P. Berrange <berrange@redhat.com>
>  M: Gerd Hoffmann <kraxel@redhat.com>
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: include/qemu/sockets.h
>  F: util/qemu-sockets.c
> @@ -2050,13 +2050,12 @@ M: Ronnie Sahlberg <ronniesahlberg@gmail.com>
>  M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Peter Lieven <pl@kamp.de>
>  L: qemu-block@nongnu.org
> -S: Supported
> +S: Odd Fixes
>  F: block/iscsi.c
>  F: block/iscsi-opts.c
>  
>  Network Block Device (NBD)
>  M: Eric Blake <eblake@redhat.com>
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  L: qemu-block@nongnu.org
>  S: Maintained
>  F: block/nbd*
> -- 
> 2.17.1
> 

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: [Qemu-devel] [PATCH] MAINTAINERS: remove or downgrade myself to reviewer from some subsystems
Posted by Thomas Huth 5 years, 6 months ago
On 2018-10-19 14:26, Paolo Bonzini wrote:
> Other people are doing a much better work than myself at handling some
> subsystems.  For those files it is better if I downgrade myself to
> reviewer or recognize that I am not actually doing any work there.
> 
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Eric Blake <eblake@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Laurent Vivier <lvivier@redhat.com>
> Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  MAINTAINERS | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index fb81b3a8eb..8c814b2e60 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -105,9 +105,9 @@ Guest CPU cores (TCG):
>  ----------------------
>  Overall
>  L: qemu-devel@nongnu.org
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Peter Crosthwaite <crosthwaite.peter@gmail.com>
>  M: Richard Henderson <rth@twiddle.net>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: cpus.c
>  F: exec.c
> @@ -1132,7 +1132,8 @@ F: hw/pci-host/ppce500.c
>  F: hw/net/fsl_etsec/
>  
>  Character devices
> -M: Paolo Bonzini <pbonzini@redhat.com>
> +M: Marc-André Lureau <marcandre.lureau@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Odd Fixes
>  F: hw/char/
>  
> @@ -1517,8 +1518,8 @@ T: git git://github.com/famz/qemu.git bitmaps
>  T: git git://github.com/jnsnow/qemu.git bitmaps
>  
>  Character device backends
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Marc-André Lureau <marcandre.lureau@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: chardev/
>  F: include/chardev/
> @@ -1751,9 +1752,9 @@ F: tests/qmp-cmd-test.c
>  T: git git://repo.or.cz/qemu/armbru.git qapi-next
>  
>  qtest
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Thomas Huth <thuth@redhat.com>
>  M: Laurent Vivier <lvivier@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: qtest.c
>  F: tests/libqtest.*
> @@ -1860,7 +1861,6 @@ F: tests/test-io-*
>  Sockets
>  M: Daniel P. Berrange <berrange@redhat.com>
>  M: Gerd Hoffmann <kraxel@redhat.com>
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: include/qemu/sockets.h
>  F: util/qemu-sockets.c
> @@ -2050,13 +2050,12 @@ M: Ronnie Sahlberg <ronniesahlberg@gmail.com>
>  M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Peter Lieven <pl@kamp.de>
>  L: qemu-block@nongnu.org
> -S: Supported
> +S: Odd Fixes
>  F: block/iscsi.c
>  F: block/iscsi-opts.c
>  
>  Network Block Device (NBD)
>  M: Eric Blake <eblake@redhat.com>
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  L: qemu-block@nongnu.org
>  S: Maintained
>  F: block/nbd*
> 

:-(

Acked-by: Thomas Huth <thuth@redhat.com>

Re: [Qemu-devel] [PATCH] MAINTAINERS: remove or downgrade myself to reviewer from some subsystems
Posted by Laurent Vivier 5 years, 6 months ago
On 19/10/2018 14:26, Paolo Bonzini wrote:
> Other people are doing a much better work than myself at handling some
> subsystems.  For those files it is better if I downgrade myself to
> reviewer or recognize that I am not actually doing any work there.
> 
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Eric Blake <eblake@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Laurent Vivier <lvivier@redhat.com>
> Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  MAINTAINERS | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index fb81b3a8eb..8c814b2e60 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -105,9 +105,9 @@ Guest CPU cores (TCG):
>  ----------------------
>  Overall
>  L: qemu-devel@nongnu.org
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Peter Crosthwaite <crosthwaite.peter@gmail.com>
>  M: Richard Henderson <rth@twiddle.net>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: cpus.c
>  F: exec.c
> @@ -1132,7 +1132,8 @@ F: hw/pci-host/ppce500.c
>  F: hw/net/fsl_etsec/
>  
>  Character devices
> -M: Paolo Bonzini <pbonzini@redhat.com>
> +M: Marc-André Lureau <marcandre.lureau@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Odd Fixes
>  F: hw/char/
>  
> @@ -1517,8 +1518,8 @@ T: git git://github.com/famz/qemu.git bitmaps
>  T: git git://github.com/jnsnow/qemu.git bitmaps
>  
>  Character device backends
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Marc-André Lureau <marcandre.lureau@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: chardev/
>  F: include/chardev/
> @@ -1751,9 +1752,9 @@ F: tests/qmp-cmd-test.c
>  T: git git://repo.or.cz/qemu/armbru.git qapi-next
>  
>  qtest
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Thomas Huth <thuth@redhat.com>
>  M: Laurent Vivier <lvivier@redhat.com>
> +R: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: qtest.c
>  F: tests/libqtest.*
> @@ -1860,7 +1861,6 @@ F: tests/test-io-*
>  Sockets
>  M: Daniel P. Berrange <berrange@redhat.com>
>  M: Gerd Hoffmann <kraxel@redhat.com>
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
>  F: include/qemu/sockets.h
>  F: util/qemu-sockets.c
> @@ -2050,13 +2050,12 @@ M: Ronnie Sahlberg <ronniesahlberg@gmail.com>
>  M: Paolo Bonzini <pbonzini@redhat.com>
>  M: Peter Lieven <pl@kamp.de>
>  L: qemu-block@nongnu.org
> -S: Supported
> +S: Odd Fixes
>  F: block/iscsi.c
>  F: block/iscsi-opts.c
>  
>  Network Block Device (NBD)
>  M: Eric Blake <eblake@redhat.com>
> -M: Paolo Bonzini <pbonzini@redhat.com>
>  L: qemu-block@nongnu.org
>  S: Maintained
>  F: block/nbd*
> 


Acked-by: Laurent Vivier <lvivier@redhat.com>


Re: [Qemu-devel] [PATCH] MAINTAINERS: remove or downgrade myself to reviewer from some subsystems
Posted by Eric Blake 5 years, 6 months ago
On 10/19/18 7:26 AM, Paolo Bonzini wrote:
> Other people are doing a much better work than myself at handling some
> subsystems.  For those files it is better if I downgrade myself to
> reviewer or recognize that I am not actually doing any work there.
> 
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Eric Blake <eblake@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>
> Cc: Laurent Vivier <lvivier@redhat.com>
> Cc: Marc-André Lureau <marcandre.lureau@redhat.com>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   MAINTAINERS | 13 ++++++-------
>   1 file changed, 6 insertions(+), 7 deletions(-)

NBD portion:
Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org