From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [Linaro-uefi] [PATCH RFC 00/19] arm64 EFI stub Date: Sun, 29 Jun 2014 17:42:06 +0100 Message-ID: <53B041DE.1070704@linaro.org> References: <1403918735-30027-1-git-send-email-roy.franz@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1403918735-30027-1-git-send-email-roy.franz@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Roy Franz , xen-devel@lists.xen.org, ian.campbell@citrix.com, stefano.stabellini@citrix.com, tim@xen.org, jbeulich@suse.com, keir@xen.org Cc: linaro-uefi@lists.linaro.org List-Id: xen-devel@lists.xenproject.org Hi Roy, Thank you for this series. I haven't look closely to this series but I've tried to compile it for arm32 and it's fails. It looks like the EFI stub is also included for arm32, which you don't plan to support right? I would use your CONFIG_EFI_STUB you've hardcoded in arm64/head.s to know if we need to include the EFI stub or not in the compilation. On 28/06/14 02:25, Roy Franz wrote: > There are a few open issues in this patch series that I would appreciate > feedback/suggestions on: > > 1) Build system changes. The 'efi-shared.c' file should be properly > shared, rather than symbolicly linked. One complication is that the EFI > code (for both archs) needs to be compiled with "-fshort-wchar". I also > likely need to create an efi subdir for arm64. I would create a directly xen/drivers/efi so you can move efi code in there. Hence, you will be able to use -fshort-wchar only for this directory. > 2) Is it valid to start XEN with a device tree that only contains > multi-boot and EFI info? (As would be the case if the stub doesn't get a > device tree as input.) Currently this isn't supported, some libfdt > functions are missing, so I'm checking if this is desired before I add > that. > > 3) I'm not sure arm64 needs it's own copy of efibind.h. The x86_64 > version worked fine as is, but has some Microsoft related defines in > there. The arm64 version I created is a proper subset with the exception > of the EFI_STUB_ERROR define. Maybe you can create a new header in include/efi and use asm/efi.h for architectural bindings. Regards, -- Julien Grall