Re: [Xen-devel] [PATCH v10 06/13] efi: create new early memory allocator

2016-12-01 Thread Daniel Kiper
On Thu, Dec 01, 2016 at 09:08:45AM -0700, Jan Beulich wrote: > >>> On 01.12.16 at 16:41, wrote: > > On Thu, Dec 01, 2016 at 06:13:34AM -0700, Jan Beulich wrote: > >> >>> On 30.11.16 at 14:04, wrote: > >> > --- a/xen/common/efi/boot.c > >> > +++ b/xen/common/efi/boot.c > >> > @@ -98,6 +98,52 @@ st

Re: [Xen-devel] [PATCH v10 06/13] efi: create new early memory allocator

2016-12-01 Thread Jan Beulich
>>> On 01.12.16 at 16:41, wrote: > On Thu, Dec 01, 2016 at 06:13:34AM -0700, Jan Beulich wrote: >> >>> On 30.11.16 at 14:04, wrote: >> > --- a/xen/common/efi/boot.c >> > +++ b/xen/common/efi/boot.c >> > @@ -98,6 +98,52 @@ static CHAR16 __initdata newline[] = L"\r\n"; >> > #define PrintStr(s) Std

Re: [Xen-devel] [PATCH v10 06/13] efi: create new early memory allocator

2016-12-01 Thread Daniel Kiper
On Thu, Dec 01, 2016 at 06:13:34AM -0700, Jan Beulich wrote: > >>> On 30.11.16 at 14:04, wrote: > > --- a/xen/common/efi/boot.c > > +++ b/xen/common/efi/boot.c > > @@ -98,6 +98,52 @@ static CHAR16 __initdata newline[] = L"\r\n"; > > #define PrintStr(s) StdOut->OutputString(StdOut, s) > > #define

Re: [Xen-devel] [PATCH v10 06/13] efi: create new early memory allocator

2016-12-01 Thread Jan Beulich
>>> On 30.11.16 at 14:04, wrote: > --- a/xen/common/efi/boot.c > +++ b/xen/common/efi/boot.c > @@ -98,6 +98,52 @@ static CHAR16 __initdata newline[] = L"\r\n"; > #define PrintStr(s) StdOut->OutputString(StdOut, s) > #define PrintErr(s) StdErr->OutputString(StdErr, s) > > +#ifndef CONFIG_ARM >

[Xen-devel] [PATCH v10 06/13] efi: create new early memory allocator

2016-11-30 Thread Daniel Kiper
There is a problem with place_string() which is used as early memory allocator. It gets memory chunks starting from start symbol and goes down. Sadly this does not work when Xen is loaded using multiboot2 protocol because then the start lives on 1 MiB address and we should not allocate a memory fro