From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Kiper Subject: Re: [PATCH v5 2/7] efi: Introduce EFI_NO_DIRECT flag Date: Wed, 18 Jun 2014 17:08:15 +0200 Message-ID: <20140618150815.GB28489__8882.56866387582$1403104332$gmane$org@olila.local.net-space.pl> References: <1402678823-24589-1-git-send-email-daniel.kiper@oracle.com> <1402678823-24589-3-git-send-email-daniel.kiper@oracle.com> <20140618135229.GH24049@console-pimps.org> <53A1B726020000780001B6CF@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1WxHVL-0000OI-0k for xen-devel@lists.xenproject.org; Wed, 18 Jun 2014 15:10:27 +0000 Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Stefano Stabellini Cc: Matt Fleming , jeremy@goop.org, matt.fleming@intel.com, ian.campbell@citrix.com, linux-efi@vger.kernel.org, andrew.cooper3@citrix.com, x86@kernel.org, linux-kernel@vger.kernel.org, mjg59@srcf.ucam.org, mingo@redhat.com, david.vrabel@citrix.com, Jan Beulich , hpa@zytor.com, xen-devel@lists.xenproject.org, tglx@linutronix.de, boris.ostrovsky@oracle.com, eshelton@pobox.com List-Id: xen-devel@lists.xenproject.org On Wed, Jun 18, 2014 at 03:30:25PM +0100, Stefano Stabellini wrote: > On Wed, 18 Jun 2014, Jan Beulich wrote: > > >>> On 18.06.14 at 15:52, wrote: > > > EFI_PARAVIRT will be usable by architectures other than x86, correct? If > > > your intention is for it only ever to be used by x86, then it should > > > probably be EFI_ARCH_2. > > > > I would expect ARM, once it gets UEFI support on the Xen side, to > > be able to handle most of this identically to x86. Which raises the > > question whether most of the new Xen-specific code (in one of the > > other patches) wouldn't better live under drivers/xen/. > > I was thinking the same thing. > > However this patch series doesn't add much code outside > drivers/xen/efi.c and include/xen/interface/platform.h. > I think it wouldn't be fair to ask Daniel to refactor the efi code > currently under arch/x86 to an arch-independent location. > Whoever comes in later and adds EFI Xen support for ARM can do that. I was doing those EFI patches with ARM support in mind from the beginning. That is why I moved everything, which was possible, to arch independent place. There are only some simple init calls from arch/x86. Just a few required lines. So I think that it will be quite easy to use this EFI code on ARM platform. Daniel