Re: [Xen-devel] [PATCH v5 12/16] x86/efi: create new early memory allocator

>>> On 08.09.16 at 10:29, wrote: > On Wed, Sep 07, 2016 at 08:01:31AM -0600, Jan Beulich wrote: >> >>> On 07.09.16 at 14:05, wrote: >> > On Mon, Sep 05, 2016 at 06:33:57AM -0600, Jan Beulich wrote: >> >> >>> On 20.08.16 at 00:43,

Re: [Xen-devel] [PATCH v5 12/16] x86/efi: create new early memory allocator

On Wed, Sep 07, 2016 at 08:01:31AM -0600, Jan Beulich wrote: > >>> On 07.09.16 at 14:05, wrote: > > On Mon, Sep 05, 2016 at 06:33:57AM -0600, Jan Beulich wrote: > >> >>> On 20.08.16 at 00:43, wrote: > >> > +static char __initdata *ebmalloc_free =

Re: [Xen-devel] [PATCH v5 12/16] x86/efi: create new early memory allocator

>>> On 07.09.16 at 14:05, wrote: > On Mon, Sep 05, 2016 at 06:33:57AM -0600, Jan Beulich wrote: >> >>> On 20.08.16 at 00:43, wrote: >> > +static char __initdata *ebmalloc_free = NULL; >> > + >> > +/* EFI boot allocator. */ >> > +static void

Re: [Xen-devel] [PATCH v5 12/16] x86/efi: create new early memory allocator

On Mon, Sep 05, 2016 at 06:33:57AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: [...] > > +static char __initdata *ebmalloc_free = NULL; > > + > > +/* EFI boot allocator. */ > > +static void __init *ebmalloc(size_t size) > > +{ > > +void *ptr; > > +

Re: [Xen-devel] [PATCH v5 12/16] x86/efi: create new early memory allocator

>>> On 20.08.16 at 00:43, wrote: > --- a/xen/arch/x86/efi/efi-boot.h > +++ b/xen/arch/x86/efi/efi-boot.h > @@ -103,9 +103,56 @@ static void __init relocate_trampoline(unsigned long > phys) > *(u16 *)(*trampoline_ptr + (long)trampoline_ptr) = phys >> 4; > } >

[Xen-devel] [PATCH v5 12/16] x86/efi: create new early memory allocator

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