From mboxrd@z Thu Jan  1 00:00:00 1970
From: Julien Grall <julien.grall@citrix.com>
Subject: Re: [PATCH v2] MAINTAINERS: Document maintainers for
	xen/common/
Date: Thu, 17 Sep 2015 13:04:02 +0100
Message-ID: <55FAAC32.3040806@citrix.com>
References: <1442490839-23962-1-git-send-email-ian.jackson@eu.citrix.com>
	<55FAAB19.4080405@citrix.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <55FAAB19.4080405@citrix.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Andrew Cooper <andrew.cooper3@citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, xen-devel@lists.xensource.com
Cc: Tim Deegan <tim@xen.org>, Keir Fraser <keir@xen.org>, Ian Campbell <ian.campbell@citrix.com>, Jan Beulich <jbeulich@suse.com>, David Vrabel <david.vrabel@citrix.com>
List-Id: xen-devel@lists.xenproject.org

On 17/09/15 12:59, Andrew Cooper wrote:
> 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/.

The first version of this patch was covering xen/. But the downside was
to CC all the "REST OF THE HYPERVISOR" group to any changes under xen/.
I.e you would be CCed on ARM changes for instance.

See http://lists.xen.org/archives/html/xen-devel/2015-09/msg02171.html

I know some maintainers which will complaining very quickly on the ML
and ask the contributors to use correctly scripts/get_maintainers.pl.

But the scrips doesn't do correctly his job for Xen...

Regards,

-- 
Julien Grall