Re: [Xen-devel] [RFC PATCH 2/2] arm64: Create xen.efi binary for arm64

[Roy Franz <roy.franz@xxxxxxxxxx>]

On Thu, Oct 2, 2014 at 7:08 AM, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> On Thu, 2014-10-02 at 14:09 +0100, Jan Beulich wrote:
>> >>> On 02.10.14 at 15:03, <Ian.Campbell@xxxxxxxxxx> wrote:
>> > On Wed, 2014-10-01 at 18:11 -0700, Roy Franz wrote:
>> >> The 'xen' binary for arm64 is both an Image file and a PE/COFF executable,
>> >> copy it to xen.efi so that the 'make install' processing is shared with
>> >> x86. Prior to this 'make install' was broken on arm64.
>> >>
>> >> Signed-off-by: Roy Franz <roy.franz@xxxxxxxxxx>
>> >
>> > If the affect of this is only to install Xen (and some symlinks)
>> > into /usr/lib64/efi and optionally into the ESP then:
>> >         Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
>> >
>> > What I don't want is for an extra xen.efi to show up in the
>> > regular /boot directory. I think this patch doesn't do that though.
>> >
>> > Jan, slight aside: what do you think about setting EFI_VENDOR to e.g.
>> > XenProject by default (i.e. installing into the ESP by default)?
>>
>> The respective directory on the EFI partition ought to exist (and
>> I don't think we should be creating it). This part of the installation
>> being only an aid for developers anyway, I'd prefer keeping
>> EFI_VENDOR undefined (and hence skipping that installation step)
>> by default.
>
> Makes sense to me, thanks.
>
> Ian.
>
Thanks.  I wasn't sure how important this was, and wanted a conscious
decision made
regarding it rather than being left out for ARM due to an oversight.

Roy

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel