Re: [PATCH] MAINTAINERS: Clarify the meaning of nested maintainership

[Juergen Gross <jgross@suse.com>]

On 21/04/16 18:03, George Dunlap wrote:
> Clarify the meaning of nested maintainership.
> 
> Signed-off-by: George Dunlap <george.dunlap@citrix.com>
> ---
> We had a discussion about the meaning of nested maintainership at the
> recent Xen Hackathon.  The notes of that meeting can be found on this
> list [1].  No decision is official until discussed on this list, so
> consider this patch the official proposal for this change, and object
> or ask for clarification accordingly.
> 
> [1] marc.info/?i=<EDB48431-C3EF-4461-B2D2-3AB95EA6C392@gmail.com>
> 
> CC: Ian Jackson <ian.jackson@eu.citrix.com>
> CC: Jan Beulich <jbeulich@suse.com>
> CC: Keir Fraser <keir@xen.org>
> CC: Tim Deegan <tim@xen.org>
> CC: Wei Liu <wei.liu2@citrix.com>
> CC: Konrad Wilk <konrad.wilk@oracle.com>
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
> CC: Lars Kurth <lars.kurth@citrix.com>
> ---
>  MAINTAINERS | 34 ++++++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a34685d..be901d5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -94,6 +94,40 @@ Descriptions of section entries:
>  	      printk, pr_info or pr_err
>  	   One regex pattern per line.  Multiple K: lines acceptable.
>  
> +
> +The meaning of nesting:
> +
> +Many maintanership areas are "nested": for example, there are entries
> +for xen/arch/x86 as well as xen/arch/x86/mm, and even
> +xen/arch/x86/mm/shadow; and there is a section at the end called "THE
> +REST" which lists all committers, as well as a few more.  The meaning
> +of nesting is that:
> +
> +1. Under normal circumstances, the Ack of the most specific maintaner
> +is both necessary and sufficient to get a change to a given file
> +committed.  So a change to xen/arch/x86/mm/shadow/multi.c requires the
> +the Ack of the xen/arch/x86/mm/shadow maintainer for that part of the
> +patch, but would not require the Ack of the xen/arch/x86 maintainer or
> +the xen/arch/x86/mm maintainer.
> +
> +(A patch of course needs acks from the maintainers of each file that
> +it changes; so a patch which changes xen/arch/x86/traps.c,
> +xen/arch/x86/mm/p2m.c, and xen/arch/x86/mm/multi.c would require an

xen/arch/x86/mm/shadow/multi.c


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel