Re: [PATCH v3 5/5] xen: Put EFI machinery in place

2014-03-26 Thread Matt Fleming
On Wed, 26 Mar, at 01:46:17PM, Matt Fleming wrote: > On Wed, 26 Mar, at 01:31:04PM, Jan Beulich wrote: > > >>> On 26.03.14 at 14:12, wrote: > > > > > > Is there a reason that you can't just populate an efi_system_table_t > > > object, which could be used by efi_init(), so that we can save you the

Re: [PATCH v3 5/5] xen: Put EFI machinery in place

2014-03-26 Thread Matt Fleming
On Wed, 26 Mar, at 01:31:04PM, Jan Beulich wrote: > >>> On 26.03.14 at 14:12, wrote: > > > > Is there a reason that you can't just populate an efi_system_table_t > > object, which could be used by efi_init(), so that we can save you the > > trouble of duplicating all of this code? > > Would the

Re: [PATCH v3 5/5] xen: Put EFI machinery in place

2014-03-26 Thread Jan Beulich
>>> On 26.03.14 at 14:12, wrote: > On Tue, 25 Mar, at 09:57:56PM, Daniel Kiper wrote: >> +static void __init efi_init_xen(void) >> +{ >> +efi_char16_t vendor_c16[100]; >> +char vendor[ARRAY_SIZE(vendor_c16)]; >> +int ret, i; >> +struct xen_platform_op op; >> +union xenpf_efi_in

Re: [PATCH v3 5/5] xen: Put EFI machinery in place

2014-03-26 Thread Matt Fleming
On Tue, 25 Mar, at 09:57:56PM, Daniel Kiper wrote: > Put EFI machinery for Xen in place. > > This patch is based on Jan Beulich and Tang Liang work. > > Signed-off-by: Daniel Kiper > Signed-off-by: Tang Liang > Signed-off-by: Jan Beulich > --- > arch/x86/xen/enlighten.c | 10 ++ > drivers/x