From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <Ian.Campbell@eu.citrix.com>
Subject: Re: [PATCH] MAINTAINERS: move hvmloader to x86
Date: Tue, 4 Nov 2014 11:29:13 +0000
Message-ID: <1415100553.11486.29.camel@eu.citrix.com>
References: <5458BBF30200007800044B69@mail.emea.novell.com>
	<5458AEAE.9080804@citrix.com>
	<5458C4830200007800044BB8@mail.emea.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta5.messagelabs.com ([195.245.231.135])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <Ian.Campbell@citrix.com>) id 1XlcIr-0008CX-Vc
	for xen-devel@lists.xenproject.org; Tue, 04 Nov 2014 11:29:38 +0000
In-Reply-To: <5458C4830200007800044BB8@mail.emea.novell.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: Jan Beulich <JBeulich@suse.com>
Cc: Andrew Cooper <andrew.cooper3@citrix.com>, Tim Deegan <tim@xen.org>, Keir Fraser <keir@xen.org>, IanJackson <Ian.Jackson@eu.citrix.com>, xen-devel <xen-devel@lists.xenproject.org>
List-Id: xen-devel@lists.xenproject.org

On Tue, 2014-11-04 at 11:20 +0000, Jan Beulich wrote:
> >>> On 04.11.14 at 11:47, <andrew.cooper3@citrix.com> wrote:
> > On 04/11/14 10:43, Jan Beulich wrote:
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -344,6 +344,7 @@ S:	Supported
> >>  L:	xen-devel@lists.xen.org 
> >>  F:	xen/arch/x86/
> >>  F:	xen/include/asm-x86/
> >> +F:	tools/firmware/hvmloader/
> > 
> > hvmloader as a binary is linked together with other bits in
> > tools/firmware, which form part of the x86 HVM boot path.
> > 
> > As a result, I would recommend this path being "tools/firmware/" to
> > include these other bits in being part of the x86 architecture code.
> 
> No, not really. SeaBIOS should remain in Ian's realm, 

Agreed (well, I'd welcome a co-maintainer, not that it's much effort to
sync every now and then).

> and I think
> e.g. the old ROM BIOS also is much better known by the tool stack
> maintainers.

Not me, that's for sure ;-)

I think things under tools/firmware should, where appropriate, have
their own independent entry overridding the tools/* default, rather than
introducing a new umbrella entry one level further down.

IOW I think this patch for tools/firmware/hvmloader is appropriate and
correct in its own right. If someone wants to propose different
maintainership for other bits of tools/firmware/* then the default then
that should be done as a separate exercise IMHO.

Ian.