Re: [PATCH] MAINTAINERS: Split out maintainers for the hypervisor

[Julien Grall <julien.grall@citrix.com>]

Hi Ian,

On 17/09/15 11:16, Ian Jackson wrote:
> This is a copy of the `THE REST' entry but with my own entry removed,
> as I do not normally review hypervisor patches.
> 
> I have chosen to be conservative and make a minimal change, rather
> than trying to describe a desirable state, or reflect reality.
> 
> The obvious questions to me, that I have not answered, are:
>  - Does Ian Campbell want to be in this list ?
>  - Should some of the arch-specific Xen maintainers be in this list ?
>  - Are there others who should be listed ?
> 
> I have tried to include in the CC list the active people listed in
> MAINTAINERS for any of the principal components under xen/.

There is a slight problem with this change. The group "REST OF THE
HYPERVISOR" is now CCed to any file under xen.

Before:

42sh> scripts/get_maintainers.pl -f xen/arch/arm/gic.c
Ian Campbell <ian.campbell@citrix.com>
Stefano Stabellini <stefano.stabellini@citrix.com>
xen-devel@lists.xen.org

After:

42sh> scripts/get_maintainers.pl -f xen/arch/arm/gic.c
Ian Campbell <ian.campbell@citrix.com>
Stefano Stabellini <stefano.stabellini@citrix.com>
Jan Beulich <jbeulich@suse.com>
Tim Deegan <tim@xen.org>
Keir Fraser <keir@xen.org>
xen-devel@lists.xen.org


I suspect this is a bug in get_maintainers.pl because we had to tweak it
for "THE REST" group. It may be necessary to do the same for "REST OF
THE HYPERVISOR"?

Regards,

> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Ian Campbell <ian.campbell@citrix.com>
> CC: Jan Beulich <jbeulich@suse.com>
> CC: Keir Fraser <keir@xen.org>
> CC: Tim Deegan <tim@xen.org>
> CC: Julien Grall <julien.grall@citrix.com>
> CC: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
> CC: Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>
> CC: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> CC: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
> CC: Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>
> CC: Stefano Stabellini <stefano.stabellini@citrix.com>
> CC: Juergen Gross <jgross@suse.com>
> CC: David Vrabel <david.vrabel@citrix.com>
> CC: Gang Wei <gang.wei@intel.com>
> CC: Shane Wang <shane.wang@intel.com>
> CC: Yang Zhang <yang.z.zhang@intel.com>
> CC: Kevin Tian <kevin.tian@intel.com>
> CC: Jun Nakajima <jun.nakajima@intel.com>
> CC: Liu Jinsong <jinsong.liu@alibaba-inc.com>
> CC: George Dunlap <george.dunlap@eu.citrix.com>
> CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> CC: Razvan Cojocaru <rcojocaru@bitdefender.com>
> CC: Tamas K Lengyel <tamas@tklengyel.com>
> CC: Daniel De Graaf <dgdegra@tycho.nsa.gov>
> ---
>  MAINTAINERS |    9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a6bece6..45fe426 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -404,6 +404,15 @@ F:  xen/include/xsm/
>  F:  xen/xsm/
>  F:  docs/misc/xsm-flask.txt
>  
> +REST OF THE HYPERVISOR
> +M:	Jan Beulich <jbeulich@suse.com>
> +M:	Tim Deegan <tim@xen.org>
> +M:	Keir Fraser <keir@xen.org>
> +M:	Ian Campbell <ian.campbell@citrix.com>
> +L:	xen-devel@lists.xen.org
> +S:	Supported
> +F:	xen/
> +
>  THE REST
>  M:	Ian Campbell <ian.campbell@citrix.com>
>  M:	Ian Jackson <ian.jackson@eu.citrix.com>
> 


-- 
Julien Grall