[Qemu-devel] [PATCH 00/14] Various updates to the MAINTAINERS file

[Qemu-devel] [PATCH 00/14] Various updates to the MAINTAINERS file

[Thomas Huth]

I recently noticed that we missed to include many updates to the
MAINTAINERS file that have been posted to the qemu-devel mailing
list within the last weeks. This patch series is now a collection
of these patches, with some tweaks to some of them according to
the discussions that have taken place after their initial version
has been posted.

Please have especially a look at the new patch 13 ('Allow "Odd Fixes"
with sole "L:" entries, too') to see whether that makes sense.

Jeff Cody (2):
  MAINTAINERS: Replace myself with John Snow for block jobs
  MAINTAINERS: Remove myself as block maintainer

Philippe Mathieu-Daudé (9):
  MAINTAINERS: Add missing entries for the sun4u machines
  MAINTAINERS: Add missing entries for the PC machines
  MAINTAINERS: Add missing entries for the QObject section
  MAINTAINERS: Add missing test entries to the Cryptography section
  MAINTAINERS: Add an entry for the Dino machine
  MAINTAINERS: Add maintainer to the POSIX subsystem
  MAINTAINERS: Orphanize the 'GDB stub' subsystem
  MAINTAINERS: Add maintainers to the Linux subsystem
  MAINTAINERS: Add maintainer to the TCG/i386 subsystem

Thomas Huth (3):
  MAINTAINERS: Add some missing entries for the sun4m machine
  MAINTAINERS: Clean up the RISC-V TCG backend section
  MAINTAINERS: Allow "Odd Fixes" with sole "L:" entries, too

 MAINTAINERS | 84 +++++++++++++++++++++++++++++++++++++------------------------
 1 file changed, 51 insertions(+), 33 deletions(-)

-- 
1.8.3.1

[Qemu-devel] [PATCH 01/14] MAINTAINERS: Add some missing entries for the sun4m machine

[Thomas Huth]

These files / devices are only used by SPARC machines, so we can sort
them into the corresponding categories in the MAINTAINERS file.

Cc: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index b1d786c..7e4f70a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1082,12 +1082,15 @@ M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
 S: Maintained
 F: hw/sparc/sun4m.c
 F: hw/sparc/sun4m_iommu.c
+F: hw/display/cg3.c
+F: hw/display/tcx.c
 F: hw/dma/sparc32_dma.c
 F: hw/misc/eccmemctl.c
-F: hw/misc/slavio_misc.c
+F: hw/*/slavio_*.c
+F: include/hw/nvram/sun_nvram.h
 F: include/hw/sparc/sparc32_dma.h
-F: pc-bios/openbios-sparc32
 F: include/hw/sparc/sun4m_iommu.h
+F: pc-bios/openbios-sparc32
 
 Sun4u
 M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
-- 
1.8.3.1

Re: [Qemu-devel] [PATCH 01/14] MAINTAINERS: Add some missing entries for the sun4m machine

[Philippe Mathieu-Daudé]

On 2/19/19 9:36 AM, Thomas Huth wrote:
> These files / devices are only used by SPARC machines, so we can sort
> them into the corresponding categories in the MAINTAINERS file.
> 
> Cc: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> ---
>  MAINTAINERS | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b1d786c..7e4f70a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1082,12 +1082,15 @@ M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>  S: Maintained
>  F: hw/sparc/sun4m.c
>  F: hw/sparc/sun4m_iommu.c
> +F: hw/display/cg3.c
> +F: hw/display/tcx.c
>  F: hw/dma/sparc32_dma.c
>  F: hw/misc/eccmemctl.c
> -F: hw/misc/slavio_misc.c
> +F: hw/*/slavio_*.c
> +F: include/hw/nvram/sun_nvram.h
>  F: include/hw/sparc/sparc32_dma.h
> -F: pc-bios/openbios-sparc32
>  F: include/hw/sparc/sun4m_iommu.h
> +F: pc-bios/openbios-sparc32
>  
>  Sun4u
>  M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> 

[Qemu-devel] [PATCH 02/14] MAINTAINERS: Add missing entries for the sun4u machines

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 7e4f70a..feb9095 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1096,9 +1096,13 @@ Sun4u
 M: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
 S: Maintained
 F: hw/sparc64/sun4u.c
-F: pc-bios/openbios-sparc64
+F: hw/sparc64/sun4u_iommu.c
+F: include/hw/sparc/sun4u_iommu.h
 F: hw/pci-host/sabre.c
 F: include/hw/pci-host/sabre.h
+F: hw/pci-bridge/simba.c
+F: include/hw/pci-bridge/simba.h
+F: pc-bios/openbios-sparc64
 
 Sun4v
 M: Artyom Tarasenko <atar4qemu@gmail.com>
-- 
1.8.3.1

[Qemu-devel] [PATCH 03/14] MAINTAINERS: Add missing entries for the PC machines

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index feb9095..2cc112c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1181,6 +1181,10 @@ F: hw/acpi/ich9.c
 F: include/hw/acpi/ich9.h
 F: include/hw/acpi/piix4.h
 F: hw/misc/sga.c
+F: hw/isa/apm.c
+F: include/hw/isa/apm.h
+F: tests/test-x86-cpuid.c
+F: tests/test-x86-cpuid-compat.c
 
 PC Chipset
 M: Michael S. Tsirkin <mst@redhat.com>
-- 
1.8.3.1

[Qemu-devel] [PATCH 04/14] MAINTAINERS: Add missing entries for the QObject section

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 2cc112c..e7b4114 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1936,10 +1936,14 @@ F: include/qapi/qmp/
 X: include/qapi/qmp/dispatch.h
 F: scripts/coccinelle/qobject.cocci
 F: tests/check-qdict.c
-F: tests/check-qnum.c
 F: tests/check-qjson.c
 F: tests/check-qlist.c
+F: tests/check-qlit.c
+F: tests/check-qnull.c
+F: tests/check-qnum.c
+F: tests/check-qobject.c
 F: tests/check-qstring.c
+F: qdict-test-data.txt
 T: git https://repo.or.cz/qemu/armbru.git qapi-next
 
 QEMU Guest Agent
-- 
1.8.3.1

[Qemu-devel] [PATCH 05/14] MAINTAINERS: Add missing test entries to the Cryptography section

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Acked-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index e7b4114..59e4c8a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2065,6 +2065,8 @@ F: crypto/
 F: include/crypto/
 F: tests/test-crypto-*
 F: tests/benchmark-crypto-*
+F: tests/crypto-tls-*
+F: tests/pkix_asn1_tab.c
 F: qemu.sasl
 
 Coroutines
-- 
1.8.3.1

[Qemu-devel] [PATCH 06/14] MAINTAINERS: Add an entry for the Dino machine

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Add Richard as maintainer, and Helge as reviewer.

Cc: Richard Henderson <rth@twiddle.net>
Cc: Helge Deller <deller@gmx.de>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
[thuth: Add the machine entry alphabetically]
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 59e4c8a..a1772d9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -844,6 +844,15 @@ S: Maintained
 F: hw/cris/axis_dev88.c
 F: hw/*/etraxfs_*.c
 
+HP-PARISC Machines
+------------------
+Dino
+M: Richard Henderson <rth@twiddle.net>
+R: Helge Deller <deller@gmx.de>
+S: Odd Fixes
+F: hw/hppa/
+F: pc-bios/hppa-firmware.img
+
 LM32 Machines
 -------------
 EVR32 and uclinux BSP
-- 
1.8.3.1

[Qemu-devel] [PATCH 07/14] MAINTAINERS: Add maintainer to the POSIX subsystem

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Add Paolo as maintainer of the POSIX subsystem.

Remove the qemu-devel@nongnu.org entry because the list is always
selected by the 'All patches CC here' section.

Cc: Paolo Bonzini <pbonzini@redhat.com>
Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a1772d9..549f1f9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -431,9 +431,12 @@ F: linux-*
 F: linux-headers/
 
 POSIX
-L: qemu-devel@nongnu.org
+M: Paolo Bonzini <pbonzini@redhat.com>
 S: Maintained
-F: *posix*
+F: os-posix.c
+F: include/sysemu/os-posix.h
+F: util/*posix*.c
+F: include/qemu/*posix*.h
 
 NETBSD
 L: qemu-devel@nongnu.org
-- 
1.8.3.1

[Qemu-devel] [PATCH 08/14] MAINTAINERS: Orphanize the 'GDB stub' subsystem

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Nobody is looking at those files, downgrade this subsystem as orphan.

Remove the qemu-devel@nongnu.org entry because the list is always
selected by the 'All patches CC here' section.

Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 549f1f9..258ba5a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1811,8 +1811,7 @@ F: util/error.c
 F: util/qemu-error.c
 
 GDB stub
-L: qemu-devel@nongnu.org
-S: Odd Fixes
+S: Orphan
 F: gdbstub*
 F: gdb-xml/
 
-- 
1.8.3.1

[Qemu-devel] [PATCH 09/14] MAINTAINERS: Add maintainers to the Linux subsystem

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Add Michael, Cornelia and Paolo as maintainers of the Linux subsystem.

Remove the qemu-devel@nongnu.org entry because the list is always
selected by the 'All patches CC here' section.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Acked-by: Cornelia Huck <cohuck@redhat.com>
[thuth: Add update-linux-headers.sh, too]
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 258ba5a..e6e08c2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -425,10 +425,12 @@ Hosts:
 ------
 
 LINUX
-L: qemu-devel@nongnu.org
+M: Michael S. Tsirkin <mst@redhat.com>
+M: Cornelia Huck <cohuck@redhat.com>
+M: Paolo Bonzini <pbonzini@redhat.com>
 S: Maintained
-F: linux-*
 F: linux-headers/
+F: scripts/update-linux-headers.sh
 
 POSIX
 M: Paolo Bonzini <pbonzini@redhat.com>
-- 
1.8.3.1

Re: [Qemu-devel] [PATCH 09/14] MAINTAINERS: Add maintainers to the Linux subsystem

[Philippe Mathieu-Daudé]

On 2/19/19 9:36 AM, Thomas Huth wrote:
> From: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Add Michael, Cornelia and Paolo as maintainers of the Linux subsystem.
> 
> Remove the qemu-devel@nongnu.org entry because the list is always
> selected by the 'All patches CC here' section.
> 
> Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Acked-by: Cornelia Huck <cohuck@redhat.com>
> [thuth: Add update-linux-headers.sh, too]

Good idea.

> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 258ba5a..e6e08c2 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -425,10 +425,12 @@ Hosts:
>  ------
>  
>  LINUX
> -L: qemu-devel@nongnu.org
> +M: Michael S. Tsirkin <mst@redhat.com>
> +M: Cornelia Huck <cohuck@redhat.com>
> +M: Paolo Bonzini <pbonzini@redhat.com>
>  S: Maintained
> -F: linux-*
>  F: linux-headers/
> +F: scripts/update-linux-headers.sh
>  
>  POSIX
>  M: Paolo Bonzini <pbonzini@redhat.com>
> 

[Qemu-devel] [PATCH 10/14] MAINTAINERS: Add maintainer to the TCG/i386 subsystem

[Thomas Huth]

From: Philippe Mathieu-Daudé <philmd@redhat.com>

Richard obviously maintains this subdirectory, make this official :)

Remove the qemu-devel@nongnu.org entry because the list is always
selected by the 'All patches CC here' section.

Cc: Richard Henderson <rth@twiddle.net>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index e6e08c2..b00caaf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2213,7 +2213,7 @@ F: tcg/arm/
 F: disas/arm.c
 
 i386 target
-L: qemu-devel@nongnu.org
+M: Richard Henderson <rth@twiddle.net>
 S: Maintained
 F: tcg/i386/
 F: disas/i386.c
-- 
1.8.3.1

[Qemu-devel] [PATCH 11/14] MAINTAINERS: Clean up the RISC-V TCG backend section

[Thomas Huth]

The e-mail address mjc@sifive.com of Micheal is not valid anymore.
Commit 7d04ac38959f8115f2a02 removed the entry already from the main
RISC-V section, but apparently forgot to remove it from the TCG
backend section, too.

Cc: Palmer Dabbelt <palmer@sifive.com>
Cc: Alistair Francis <Alistair.Francis@wdc.com>
Fixes: 7d04ac38959f8115f2a029d81db1c8aac179aa95
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index b00caaf..1d0dcde 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2232,7 +2232,6 @@ F: tcg/ppc/
 F: disas/ppc.c
 
 RISC-V
-M: Michael Clark <mjc@sifive.com>
 M: Palmer Dabbelt <palmer@sifive.com>
 M: Alistair Francis <Alistair.Francis@wdc.com>
 L: qemu-riscv@nongnu.org
-- 
1.8.3.1

Re: [Qemu-devel] [PATCH 11/14] MAINTAINERS: Clean up the RISC-V TCG backend section

[Palmer Dabbelt]

On Tue, 19 Feb 2019 00:36:58 PST (-0800), thuth@redhat.com wrote:
> The e-mail address mjc@sifive.com of Micheal is not valid anymore.
> Commit 7d04ac38959f8115f2a02 removed the entry already from the main
> RISC-V section, but apparently forgot to remove it from the TCG
> backend section, too.
>
> Cc: Palmer Dabbelt <palmer@sifive.com>
> Cc: Alistair Francis <Alistair.Francis@wdc.com>
> Fixes: 7d04ac38959f8115f2a029d81db1c8aac179aa95
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Reviewed-by: Palmer Dabbelt <palmer@sifive.com>

I'm assuming this is going in through another tree, but let me know if you want 
me to take it.

Thanks!

> ---
>  MAINTAINERS | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b00caaf..1d0dcde 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2232,7 +2232,6 @@ F: tcg/ppc/
>  F: disas/ppc.c
>
>  RISC-V
> -M: Michael Clark <mjc@sifive.com>
>  M: Palmer Dabbelt <palmer@sifive.com>
>  M: Alistair Francis <Alistair.Francis@wdc.com>
>  L: qemu-riscv@nongnu.org

Re: [Qemu-devel] [PATCH 11/14] MAINTAINERS: Clean up the RISC-V TCG backend section

[Peter Maydell]

On Tue, 19 Feb 2019 at 08:44, Thomas Huth <thuth@redhat.com> wrote:
>
> The e-mail address mjc@sifive.com of Micheal is not valid anymore.

Typo: "Michael".


thanks
-- PMM