Re: [PATCH] EFI: Enable booting unified hypervisor/kernel/initrd images

[Julien Grall <julien@xxxxxxx>]

Hi Trammell,

On 04/09/2020 11:06, Trammell Hudson wrote:
> On Friday, September 4, 2020 5:29 AM, Julien Grall <julien@xxxxxxx> wrote:
>
> > On 28/08/2020 12:51, Trammell Hudson wrote:
> >
> > > -   /* PE32+ Subsystem type */
> > >      +#if defined(ARM)
> > Shouldn't this be defined(aarch64) ?
> To be honest I'm not sure and don't have a way to check if
> this code works on ARM.
The code looks generic enough. I will give a try to build it.

> Does an Xen EFI build on ARM even
> use the PE32+ format?
So far, we only support EFI and AArch64. This is built using PE32+ format.

Looking at the rest of the UEFI code, we are using CONFIG_ARM. But I 
think, in this particular instance, we want to use CONFIG_ARM_64.
Cheers,

--
Julien Grall