Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-06-02 Thread Jan Beulich
>>> On 01.06.16 at 21:28, wrote: > On Wed, Jun 01, 2016 at 09:41:42AM -0600, Jan Beulich wrote: >> >>> On 01.06.16 at 17:23, wrote: >> > On Fri, May 27, 2016 at 02:22:39AM -0600, Jan Beulich wrote: >> >> >>> On 25.05.16 at 19:15,

Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-06-01 Thread Jan Beulich
>>> On 01.06.16 at 17:23, wrote: > On Fri, May 27, 2016 at 02:22:39AM -0600, Jan Beulich wrote: >> >>> On 25.05.16 at 19:15, wrote: >> > On Wed, May 25, 2016 at 01:20:23AM -0600, Jan Beulich wrote: >> >> >>> On 15.04.16 at 14:33,

Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-06-01 Thread Daniel Kiper
On Fri, May 27, 2016 at 02:22:39AM -0600, Jan Beulich wrote: > >>> On 25.05.16 at 19:15, wrote: > > On Wed, May 25, 2016 at 01:20:23AM -0600, Jan Beulich wrote: > >> >>> On 15.04.16 at 14:33, wrote: [...] > >> > --- a/xen/include/xen/efi.h > >>

Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-05-27 Thread Jan Beulich
>>> On 25.05.16 at 19:15, wrote: > On Wed, May 25, 2016 at 01:20:23AM -0600, Jan Beulich wrote: >> >>> On 15.04.16 at 14:33, wrote: >> > --- a/xen/arch/x86/efi/stub.c >> > +++ b/xen/arch/x86/efi/stub.c >> > @@ -4,11 +4,8 @@ >> > #include >> >

Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-05-26 Thread Andrew Cooper
>>> @@ -42,11 +38,12 @@ UINT64 __read_mostly efi_boot_remain_var_store_size; >>> UINT64 __read_mostly efi_boot_max_var_size; >>> >>> struct efi __read_mostly efi = { >>> - .acpi = EFI_INVALID_TABLE_ADDR, >>> - .acpi20 = EFI_INVALID_TABLE_ADDR, >>> - .mps= EFI_INVALID_TABLE_ADDR, >>>

Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-05-25 Thread Daniel Kiper
On Wed, May 25, 2016 at 01:20:23AM -0600, Jan Beulich wrote: > >>> On 15.04.16 at 14:33, wrote: > > --- a/xen/arch/x86/efi/stub.c > > +++ b/xen/arch/x86/efi/stub.c > > @@ -4,11 +4,8 @@ > > #include > > #include > > > > -#ifndef efi_enabled > > -const bool_t

Re: [Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-05-25 Thread Jan Beulich
>>> On 15.04.16 at 14:33, wrote: > --- a/xen/arch/x86/efi/stub.c > +++ b/xen/arch/x86/efi/stub.c > @@ -4,11 +4,8 @@ > #include > #include > > -#ifndef efi_enabled > -const bool_t efi_enabled = 0; > -#endif > - > struct efi __read_mostly efi = { > + .flags =

[Xen-devel] [PATCH v3 10/16] efi: create efi_enabled()

2016-04-15 Thread Daniel Kiper
First of all we need to differentiate between legacy BIOS and EFI platforms during runtime, not during build, because one image will have legacy and EFI code and can be executed on both platforms. Additionally, we need more fine grained knowledge about EFI environment and check for EFI platform