[PATCH] MAINTAINERS: Split out maintainers for the hypervisor

[PATCH] MAINTAINERS: Split out maintainers for the hypervisor

[Ian Jackson]

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/.

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>
-- 
1.7.10.4

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

[Julien Grall]

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

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

[Ian Jackson]

Julien Grall writes ("Re: [PATCH] MAINTAINERS: Split out maintainers for the hypervisor"):
> There is a slight problem with this change. The group "REST OF THE
> HYPERVISOR" is now CCed to any file under xen.

Eeek.  (Dropping many of the CCs.)

> 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"?

I had missed us adding a special case for `THE REST'.

I will come back with a proper fix to this.

Ian.

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

[Ian Campbell]

(cutting Cc to just those listed as REST plus Julien who suggested this
change)

On Thu, 2015-09-17 at 11:16 +0100, 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 ?

I think under the second (for arm) I would want the first regardless of
other considerations.


>  - 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/.
> 
> 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>