[Qemu-devel] Re: [PATCH] Use a Linux-style MAINTAINERS file

[Blue Swirl]

On Thu, Sep 9, 2010 at 9:18 PM, Anthony Liguori <address@hidden> wrote:
> I make no claims that this is accurate or exhaustive but I think it's a
> reasonable place to start.
>
> As the file mentions, the purpose of this file is to give contributors
> information about who they can go to with questions about a particular piece 
> of
> code or who they can ask for review.
>
> If you sign up for a piece of code and indicate that it's Maintained or
> Supported, please be prepared to be responsive to questions about that
> subsystem.
>
> Signed-off-by: Anthony Liguori <address@hidden>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 79dfc7f..3894cd8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9,89 +9,388 @@ to be CC'd when submitting a patch to obtain appropriate 
> review.
>  In general, if you have a question about inclusion of a patch, you should
>  consult qemu-devel and not any specific individual privately.
>
> -Project leaders:
> -----------------
> +Please see the MAINTAINERS file in the Linux kernel for information about how
> +to update this file.
>
> -Paul Brook
> -Anthony Liguori
> +General Project Administration
> +------------------------------
> +M: Anthony Liguori <address@hidden>
> +M: Paul Brook <address@hidden>
>
>  Guest CPU cores (TCG):

Perhaps the CPU names should be in alphabetical order. For
completeness, KVM CPUs should have X: target-xxx/kvm.c, though
currently TCG and KVM maintainers match.

>  ----------------------
> -
> -x86                ?
> -ARM                Paul Brook
> -SPARC              Blue Swirl
> -MIPS               ?
> -PowerPC            Alexander Graf
> -M68K               Paul Brook
> -SH4                ?
> -CRIS               Edgar E. Iglesias
> -Alpha              ?
> -MicroBlaze         Edgar E. Iglesias
> -S390               ?
> -
> -Machines (sorted by CPU):
> --------------------------
> -
>  x86
> -  pc.c                    Anthony Liguori
> +M: address@hidden
> +S: Odd Fixes
> +F: target-i386/
> +
>  ARM
> -  integratorcp.c          Paul Brook
> -  versatilepb.c           Paul Brook
> -  Real View               Paul Brook
> -  spitz.c                 Andrzej Zaborowski
> -  palm.c                  Andrzej Zaborowski
> -  nseries.c               Andrzej Zaborowski
> -  stellaris.c             Paul Brook
> -  gumstix.c               Thorsten Zitterell
> -  mainstone.c             Armin Kuster
> -  musicpal.c              Jan Kiszka
> +M: Paul Brook <address@hidden>
> +S: Maintained
> +F: target-arm/
> +
>  SPARC
> -  sun4u.c                 Blue Swirl
> -  sun4m.c                 Blue Swirl
> +M: Blue Swirl <address@hidden>
> +S: Maintained
> +F: target-sparc/
> +
>  MIPS
> -  mips_r4k.c              Aurelien Jarno
> -  mips_malta.c            Aurelien Jarno
> -  mips_jazz.c             Hervé Poussineau
> -  mips_mipssim.c          ?
> +M: address@hidden
> +S: Orphan
> +F: target-mips/
> +
>  PowerPC
> -  ppc_prep.c              ?
> -  ppc_oldworld.c          Alexander Graf
> -  ppc_newworld.c          Alexander Graf
> -  ppc405_boards.c         Alexander Graf
> -M86K
> -  mcf5208.c               Paul Brook
> -  an5206.c                Paul Brook
> -  dummy_m68k.c            Paul Brook
> +M: Alexander Graf <address@hidden>
> +S: Maintained
> +F: target-ppc/
> +
> +M68K
> +M: Paul Brook <address@hidden>
> +S: Maintained
> +F: target-m68k/
> +
>  SH4
> -  shix.c                  ?
> -  r2d.c                   Magnus Damm
> +M: address@hidden
> +S: Orphan
> +F: target-sh4/
> +
>  CRIS
> -  etraxfs.c               Edgar E. Iglesias
> -  axis_dev88.c            Edgar E. Iglesias
> +M: Edgar E. Iglesias <address@hidden>
> +S: Maintained
> +F: target-cris/
> +
>  Alpha
> +M: address@hidden
> +S: Orphan
> +F: target-alpha/
> +
>  MicroBlaze
> -  petalogix_s3adsp1800.c  Edgar E. Iglesias
> +M: Edgar E. Iglesias <address@hidden>
> +S: Maintained
> +F: target-microblaze/
> +
> +S390
> +M: Alexander Graf <address@hidden>
> +S: Maintained
> +F: target-s390x/
> +
> +Guest CPU Cores (KVM):
> +----------------------

Maybe the order should be alphabetical after Overall.

> +
> +Overall
> +M: Avi Kivity <address@hidden>
> +M: Marcelo Tosatti <address@hidden>
> +L: address@hidden
> +S: Supported
> +F: kvm-*
> +F: */kvm.*
> +
> +X86
> +M: Avi Kivity <address@hidden>
> +M: Marcelo Tosatti <address@hidden>
> +L: address@hidden
> +S: Supported
> +F: target-i386/kvm.c
> +
> +PPC
> +M: Alexander Graf <address@hidden>
> +S: Maintained
> +F: target-ppc/kvm.c
> +
>  S390
> -  s390-*.c                Alexander Graf
> +M: Alexander Graf <address@hidden>
> +S: Maintained
> +F: target-s390x/kvm.c
> +
> +X86 Machines
> +------------
> +
> +PC
> +M: Anthony Liguori <address@hidden>
> +S: Supported
> +F: hw/pc.[ch] hw/pc_piix.c
> +
> +
> +ARM Machines
> +------------
> +M: Paul Brook <address@hidden>
> +S: Maintained
> +F: hw/integratorcp.c
> +
> +Versatile PB
> +M: Paul Brook <address@hidden>
> +S: Maintained
> +F: hw/versatilepb.c
> +
> +Real View
> +M: Paul Brook <address@hidden>
> +S: Maintained
> +F: hw/realview*
> +
> +Spitz
> +M: Andrzej Zaborowski <address@hidden>
> +S: Maintained
> +F: hw/spitz.c
> +
> +Palm
> +M: Andrzej Zaborowski <address@hidden>
> +S: Maintained
> +F: hw/palm.c
> +
> +nSeries
> +M: Andrzej Zaborowski <address@hidden>
> +S: Maintained
> +F: hw/nseries.c
> +
> +Stellaris
> +M: Paul Brook <address@hidden>
> +S: Maintained
> +F: hw/stellaris.c
> +
> +Gumstix
> +M: address@hidden
> +S: Orphan
> +F: hw/gumstix.c
> +
> +Mainstone
> +M: address@hidden
> +S: Orphan
> +F: hw/mainstone.c
> +
> +Musicpal
> +M: Jan Kiszka <address@hidden>
> +S: Maintained
> +F: hw/musicpal.c
> +
> +SPARC Machines
> +--------------
> +Sun4u

There is also Sun4v here, though its status could be Odd Fixes. These
are actually machine classes, but currently only few machines are
implemented.

> +M: Blue Swirl <address@hidden>
> +S: Maintained

Maybe also the technical status should be indicated, something like
Unusable, Boots BIOS, Boots a kernel, Stable?

> +F: hw/sun4u.c
> +
> +Sun4m

There's also Sun4c and Sun4d machine classes. There are 9 different
Sun4m machines, but do we want to list them? The same thing actually
applies to isapc machine.

How about also listing the ROMs under pc-bios? I'd claim openbios-*,
except if Alex wants openbios-ppc, that's fine too.