[Qemu-devel] [PATCH 0/3] s390x: maintainership reorg

[Qemu-devel] [PATCH 0/3] s390x: maintainership reorg

[Cornelia Huck]

Hi,

s390x maintainership is currently a bit like a huge pile of stuff routed
through one person who is not even listed as the maintainer for some
of the areas. Let's try to clean this up a bit.

Patch 1 is basically making official what I actually have been doing in
the last months; that is, acting as the overall s390x maintainer. There
does not seem to be the concept of "overall arch maintainer" in MAINTAINERS
so far, so I added it.

Where I want to go is basically a submaintainership model for s390x,
i.e. that I integrate the work in various areas and send it on. Patch 2
serves as a step in that direction, as Thomas has volunteered to look
after the s390-ccw bios (thanks!)

If that works out well, I plan to split out other parts as well. That
would make a lot of sense for those parts which I lack documentation
for, and also for parts where other folks could do a good job as well.

Speaking of other people: Patch 3 adds David as an additional maintainer
for tcg (thanks!).

Cornelia Huck (3):
  MAINTAINERS: add myself as overall s390x maintainer
  MAINTAINERS: reorganize s390-ccw bios maintainership
  MAINTAINERS: add David as additional tcg/s390 maintainer

 MAINTAINERS | 35 +++++++++++++++++++++++++++++++++--
 1 file changed, 33 insertions(+), 2 deletions(-)

-- 
2.13.6

[Qemu-devel] [PATCH 1/3] MAINTAINERS: add myself as overall s390x maintainer

[Cornelia Huck]

All your mainframes are belong to me.

Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
 MAINTAINERS | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index bbc3a617c2..e24273a574 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -76,6 +76,29 @@ K: ^Subject:.*(?i)trivial
 T: git git://git.corpit.ru/qemu.git trivial-patches
 T: git git://github.com/vivier/qemu.git trivial-patches
 
+Architecture support
+--------------------
+S390
+M: Cornelia Huck <cohuck@redhat.com>
+S: Supported
+F: default-configs/s390x-softmmu.mak
+F: gdb-xml/s390*.xml
+F: hw/char/sclp*.[hc]
+F: hw/char/terminal3270.c
+F: hw/intc/s390_flic.c
+F: hw/intc/s390_flic_kvm.c
+F: hw/s390x/
+F: hw/vfio/ccw.c
+F: hw/watchdog/wdt_diag288.c
+F: include/hw/s390x/
+F: include/hw/watchdog/wdt_diag288.h
+F: pc-bios/s390-ccw/
+F: pc-bios/s390-ccw.img
+F: target/s390x/
+K: ^Subject:.*(?i)s390(x|)
+T: git git://github.com/cohuck/qemu.git s390-next
+L: qemu-s390x@nongnu.org
+
 Guest CPU cores (TCG):
 ----------------------
 Overall
-- 
2.13.6

Re: [Qemu-devel] [PATCH 1/3] MAINTAINERS: add myself as overall s390x maintainer

[Philippe Mathieu-Daudé]

On 02/07/2018 12:55 PM, Cornelia Huck wrote:
> All your mainframes are belong to me.

=)

> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  MAINTAINERS | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bbc3a617c2..e24273a574 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -76,6 +76,29 @@ K: ^Subject:.*(?i)trivial
>  T: git git://git.corpit.ru/qemu.git trivial-patches
>  T: git git://github.com/vivier/qemu.git trivial-patches
>  
> +Architecture support
> +--------------------
> +S390
> +M: Cornelia Huck <cohuck@redhat.com>
> +S: Supported
> +F: default-configs/s390x-softmmu.mak
> +F: gdb-xml/s390*.xml
> +F: hw/char/sclp*.[hc]
> +F: hw/char/terminal3270.c
> +F: hw/intc/s390_flic.c
> +F: hw/intc/s390_flic_kvm.c
> +F: hw/s390x/
> +F: hw/vfio/ccw.c
> +F: hw/watchdog/wdt_diag288.c
> +F: include/hw/s390x/
> +F: include/hw/watchdog/wdt_diag288.h
> +F: pc-bios/s390-ccw/
> +F: pc-bios/s390-ccw.img
> +F: target/s390x/
> +K: ^Subject:.*(?i)s390(x|)

(x|) -> x?

> +T: git git://github.com/cohuck/qemu.git s390-next
> +L: qemu-s390x@nongnu.org
> +
>  Guest CPU cores (TCG):
>  ----------------------
>  Overall
> 

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Re: [Qemu-devel] [PATCH 1/3] MAINTAINERS: add myself as overall s390x maintainer

[Cornelia Huck]

On Wed, 7 Feb 2018 13:24:09 -0300
Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:

> On 02/07/2018 12:55 PM, Cornelia Huck wrote:
> > All your mainframes are belong to me.  
> 
> =)
> 
> > 
> > Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> > ---
> >  MAINTAINERS | 23 +++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index bbc3a617c2..e24273a574 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -76,6 +76,29 @@ K: ^Subject:.*(?i)trivial
> >  T: git git://git.corpit.ru/qemu.git trivial-patches
> >  T: git git://github.com/vivier/qemu.git trivial-patches
> >  
> > +Architecture support
> > +--------------------
> > +S390
> > +M: Cornelia Huck <cohuck@redhat.com>
> > +S: Supported
> > +F: default-configs/s390x-softmmu.mak
> > +F: gdb-xml/s390*.xml
> > +F: hw/char/sclp*.[hc]
> > +F: hw/char/terminal3270.c
> > +F: hw/intc/s390_flic.c
> > +F: hw/intc/s390_flic_kvm.c
> > +F: hw/s390x/
> > +F: hw/vfio/ccw.c
> > +F: hw/watchdog/wdt_diag288.c
> > +F: include/hw/s390x/
> > +F: include/hw/watchdog/wdt_diag288.h
> > +F: pc-bios/s390-ccw/
> > +F: pc-bios/s390-ccw.img
> > +F: target/s390x/
> > +K: ^Subject:.*(?i)s390(x|)  
> 
> (x|) -> x?

Probably (my perl regexp fu is very lacking).

> 
> > +T: git git://github.com/cohuck/qemu.git s390-next
> > +L: qemu-s390x@nongnu.org
> > +
> >  Guest CPU cores (TCG):
> >  ----------------------
> >  Overall
> >   
> 
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Thanks!

Re: [Qemu-devel] [qemu-s390x] [PATCH 1/3] MAINTAINERS: add myself as overall s390x maintainer

[Christian Borntraeger]

On 02/07/2018 04:55 PM, Cornelia Huck wrote:
> All your mainframes are belong to me.
> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>

Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>

> ---
>  MAINTAINERS | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bbc3a617c2..e24273a574 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -76,6 +76,29 @@ K: ^Subject:.*(?i)trivial
>  T: git git://git.corpit.ru/qemu.git trivial-patches
>  T: git git://github.com/vivier/qemu.git trivial-patches
> 
> +Architecture support
> +--------------------
> +S390
> +M: Cornelia Huck <cohuck@redhat.com>
> +S: Supported
> +F: default-configs/s390x-softmmu.mak
> +F: gdb-xml/s390*.xml
> +F: hw/char/sclp*.[hc]
> +F: hw/char/terminal3270.c
> +F: hw/intc/s390_flic.c
> +F: hw/intc/s390_flic_kvm.c
> +F: hw/s390x/
> +F: hw/vfio/ccw.c
> +F: hw/watchdog/wdt_diag288.c
> +F: include/hw/s390x/
> +F: include/hw/watchdog/wdt_diag288.h
> +F: pc-bios/s390-ccw/
> +F: pc-bios/s390-ccw.img
> +F: target/s390x/
> +K: ^Subject:.*(?i)s390(x|)
> +T: git git://github.com/cohuck/qemu.git s390-next
> +L: qemu-s390x@nongnu.org
> +
>  Guest CPU cores (TCG):
>  ----------------------
>  Overall
> 

Re: [Qemu-devel] [PATCH 1/3] MAINTAINERS: add myself as overall s390x maintainer

[David Hildenbrand]

On 07.02.2018 16:55, Cornelia Huck wrote:
> All your mainframes are belong to me.

This looks like an easy way to get a lot of expensive hardware :)

> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  MAINTAINERS | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bbc3a617c2..e24273a574 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -76,6 +76,29 @@ K: ^Subject:.*(?i)trivial
>  T: git git://git.corpit.ru/qemu.git trivial-patches
>  T: git git://github.com/vivier/qemu.git trivial-patches
>  
> +Architecture support
> +--------------------
> +S390
> +M: Cornelia Huck <cohuck@redhat.com>
> +S: Supported
> +F: default-configs/s390x-softmmu.mak
> +F: gdb-xml/s390*.xml
> +F: hw/char/sclp*.[hc]
> +F: hw/char/terminal3270.c
> +F: hw/intc/s390_flic.c
> +F: hw/intc/s390_flic_kvm.c
> +F: hw/s390x/
> +F: hw/vfio/ccw.c
> +F: hw/watchdog/wdt_diag288.c
> +F: include/hw/s390x/
> +F: include/hw/watchdog/wdt_diag288.h
> +F: pc-bios/s390-ccw/
> +F: pc-bios/s390-ccw.img
> +F: target/s390x/
> +K: ^Subject:.*(?i)s390(x|)
> +T: git git://github.com/cohuck/qemu.git s390-next
> +L: qemu-s390x@nongnu.org
> +
>  Guest CPU cores (TCG):
>  ----------------------
>  Overall
> 


-- 

Thanks,

David / dhildenb

Re: [Qemu-devel] [PATCH 1/3] MAINTAINERS: add myself as overall s390x maintainer

[Daniel Henrique Barboza]

On 02/07/2018 01:55 PM, Cornelia Huck wrote:
> All your mainframes are belong to me.

Zero Wing reference!

Liked-by: Daniel Henrique Barboza <danielhb@linux.vnet.ibm.com>

>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>   MAINTAINERS | 23 +++++++++++++++++++++++
>   1 file changed, 23 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bbc3a617c2..e24273a574 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -76,6 +76,29 @@ K: ^Subject:.*(?i)trivial
>   T: git git://git.corpit.ru/qemu.git trivial-patches
>   T: git git://github.com/vivier/qemu.git trivial-patches
>
> +Architecture support
> +--------------------
> +S390
> +M: Cornelia Huck <cohuck@redhat.com>
> +S: Supported
> +F: default-configs/s390x-softmmu.mak
> +F: gdb-xml/s390*.xml
> +F: hw/char/sclp*.[hc]
> +F: hw/char/terminal3270.c
> +F: hw/intc/s390_flic.c
> +F: hw/intc/s390_flic_kvm.c
> +F: hw/s390x/
> +F: hw/vfio/ccw.c
> +F: hw/watchdog/wdt_diag288.c
> +F: include/hw/s390x/
> +F: include/hw/watchdog/wdt_diag288.h
> +F: pc-bios/s390-ccw/
> +F: pc-bios/s390-ccw.img
> +F: target/s390x/
> +K: ^Subject:.*(?i)s390(x|)
> +T: git git://github.com/cohuck/qemu.git s390-next
> +L: qemu-s390x@nongnu.org
> +
>   Guest CPU cores (TCG):
>   ----------------------
>   Overall

[Qemu-devel] [PATCH 2/3] MAINTAINERS: reorganize s390-ccw bios maintainership

[Cornelia Huck]

Split it out from the s390-ccw-virtio machine, add Thomas as a
maintainer in addition to Christian.

Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
 MAINTAINERS | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index e24273a574..bd2dc7ed7c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -855,15 +855,22 @@ F: hw/char/sclp*.[hc]
 F: hw/char/terminal3270.c
 F: hw/s390x/
 F: include/hw/s390x/
-F: pc-bios/s390-ccw/
 F: hw/watchdog/wdt_diag288.c
 F: include/hw/watchdog/wdt_diag288.h
-F: pc-bios/s390-ccw.img
 F: default-configs/s390x-softmmu.mak
 T: git git://github.com/cohuck/qemu.git s390-next
 T: git git://github.com/borntraeger/qemu.git s390-next
 L: qemu-s390x@nongnu.org
 
+S390-ccw Bios
+M: Christian Borntraeger <borntraeger@de.ibm.com>
+M: Thomas Huth <thuth@redhat.com>
+S: Supported
+F: pc-bios/s390-ccw/
+F: pc-bios/s390-ccw.img
+T: git git://github.com/borntraeger/qemu.git s390-next
+L: qemu-s390x@nongnu.org
+
 UniCore32 Machines
 -------------
 PKUnity-3 SoC initramfs-with-busybox
-- 
2.13.6

Re: [Qemu-devel] [qemu-s390x] [PATCH 2/3] MAINTAINERS: reorganize s390-ccw bios maintainership

[Christian Borntraeger]

On 02/07/2018 04:55 PM, Cornelia Huck wrote:
> Split it out from the s390-ccw-virtio machine, add Thomas as a
> maintainer in addition to Christian.
> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>

Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>

Thanks Thomas for volunteering to co-maintain.
> ---
>  MAINTAINERS | 11 +++++++++--
>  1 file changed, 9 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e24273a574..bd2dc7ed7c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -855,15 +855,22 @@ F: hw/char/sclp*.[hc]
>  F: hw/char/terminal3270.c
>  F: hw/s390x/
>  F: include/hw/s390x/
> -F: pc-bios/s390-ccw/
>  F: hw/watchdog/wdt_diag288.c
>  F: include/hw/watchdog/wdt_diag288.h
> -F: pc-bios/s390-ccw.img
>  F: default-configs/s390x-softmmu.mak
>  T: git git://github.com/cohuck/qemu.git s390-next
>  T: git git://github.com/borntraeger/qemu.git s390-next
>  L: qemu-s390x@nongnu.org
> 
> +S390-ccw Bios
> +M: Christian Borntraeger <borntraeger@de.ibm.com>
> +M: Thomas Huth <thuth@redhat.com>
> +S: Supported
> +F: pc-bios/s390-ccw/
> +F: pc-bios/s390-ccw.img
> +T: git git://github.com/borntraeger/qemu.git s390-next
> +L: qemu-s390x@nongnu.org
> +
>  UniCore32 Machines
>  -------------
>  PKUnity-3 SoC initramfs-with-busybox
> 

Re: [Qemu-devel] [PATCH 2/3] MAINTAINERS: reorganize s390-ccw bios maintainership

[Thomas Huth]

On 07.02.2018 16:55, Cornelia Huck wrote:
> Split it out from the s390-ccw-virtio machine, add Thomas as a
> maintainer in addition to Christian.
> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  MAINTAINERS | 11 +++++++++--
>  1 file changed, 9 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e24273a574..bd2dc7ed7c 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -855,15 +855,22 @@ F: hw/char/sclp*.[hc]
>  F: hw/char/terminal3270.c
>  F: hw/s390x/
>  F: include/hw/s390x/
> -F: pc-bios/s390-ccw/
>  F: hw/watchdog/wdt_diag288.c
>  F: include/hw/watchdog/wdt_diag288.h
> -F: pc-bios/s390-ccw.img
>  F: default-configs/s390x-softmmu.mak
>  T: git git://github.com/cohuck/qemu.git s390-next
>  T: git git://github.com/borntraeger/qemu.git s390-next
>  L: qemu-s390x@nongnu.org
>  
> +S390-ccw Bios
> +M: Christian Borntraeger <borntraeger@de.ibm.com>
> +M: Thomas Huth <thuth@redhat.com>
> +S: Supported
> +F: pc-bios/s390-ccw/
> +F: pc-bios/s390-ccw.img
> +T: git git://github.com/borntraeger/qemu.git s390-next
> +L: qemu-s390x@nongnu.org
> +
>  UniCore32 Machines
>  -------------
>  PKUnity-3 SoC initramfs-with-busybox
> 

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

[Qemu-devel] [PATCH 3/3] MAINTAINERS: add David as additional tcg/s390 maintainer

[Cornelia Huck]

Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index bd2dc7ed7c..b30d2ca23a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -236,6 +236,7 @@ F: disas/ppc.c
 S390
 M: Richard Henderson <rth@twiddle.net>
 M: Alexander Graf <agraf@suse.de>
+M: David Hildenbrand <david@redhat.com>
 S: Maintained
 F: target/s390x/
 F: hw/s390x/
-- 
2.13.6

Re: [Qemu-devel] [PATCH 3/3] MAINTAINERS: add David as additional tcg/s390 maintainer

[David Hildenbrand]

On 07.02.2018 16:55, Cornelia Huck wrote:
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index bd2dc7ed7c..b30d2ca23a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -236,6 +236,7 @@ F: disas/ppc.c
>  S390
>  M: Richard Henderson <rth@twiddle.net>
>  M: Alexander Graf <agraf@suse.de>
> +M: David Hildenbrand <david@redhat.com>
>  S: Maintained
>  F: target/s390x/
>  F: hw/s390x/
> 

Thanks for the trust!

Acked-by: David Hildenbrand <david@redhat.com>

-- 

Thanks,

David / dhildenb

Re: [Qemu-devel] [PATCH 0/3] s390x: maintainership reorg

[Cornelia Huck]

On Wed,  7 Feb 2018 16:55:28 +0100
Cornelia Huck <cohuck@redhat.com> wrote:

> Hi,
> 
> s390x maintainership is currently a bit like a huge pile of stuff routed
> through one person who is not even listed as the maintainer for some
> of the areas. Let's try to clean this up a bit.
> 
> Patch 1 is basically making official what I actually have been doing in
> the last months; that is, acting as the overall s390x maintainer. There
> does not seem to be the concept of "overall arch maintainer" in MAINTAINERS
> so far, so I added it.
> 
> Where I want to go is basically a submaintainership model for s390x,
> i.e. that I integrate the work in various areas and send it on. Patch 2
> serves as a step in that direction, as Thomas has volunteered to look
> after the s390-ccw bios (thanks!)
> 
> If that works out well, I plan to split out other parts as well. That
> would make a lot of sense for those parts which I lack documentation
> for, and also for parts where other folks could do a good job as well.
> 
> Speaking of other people: Patch 3 adds David as an additional maintainer
> for tcg (thanks!).
> 
> Cornelia Huck (3):
>   MAINTAINERS: add myself as overall s390x maintainer
>   MAINTAINERS: reorganize s390-ccw bios maintainership
>   MAINTAINERS: add David as additional tcg/s390 maintainer
> 
>  MAINTAINERS | 35 +++++++++++++++++++++++++++++++++--
>  1 file changed, 33 insertions(+), 2 deletions(-)
> 

Queued to s390-next.