Re: [PATCH v2] MAINTAINERS: Document maintainers for xen/common/

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 17/09/15 12:53, 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) and Andrew Cooper's
> added (which seems appropriate given his status as x86 hypervisor
> maintainer).
>
> The effect is that patches touching xen/common/ will no longer be CC'd
> to me; but also that Ian C, Andrew, Jan, Keir and Tim will get _all_
> patches touching xen/common/, not just ones which match one of their
> existing more specific entries.
>
> 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: David Vrabel <david.vrabel@citrix.com>
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
>
> ---
> v2: A completely different approach following discussion with Ian C.
> ---
>  MAINTAINERS |   10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a6bece6..e35c1e9 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -404,6 +404,16 @@ F:  xen/include/xsm/
>  F:  xen/xsm/
>  F:  docs/misc/xsm-flask.txt
>  
> +HYPERVISOR CORE
> +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>
> +M:	Andrew Cooper <andrew.cooper3@citrix.com>
> +L:	xen-devel@lists.xen.org
> +S:	Supported
> +F:	xen/common/

I am happy with the change in principle, but it should cover xen/ rather
than xen/common/ to also include otherwise unqualified items in
xen/drivers/.

~Andrew