>>> On 30.11.16 at 14:04, <daniel.ki...@oracle.com> 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 > + > +/* > + * TODO: Enable EFI boot allocator on ARM. > + * This code can be common for x86 and ARM. > + * Things TODO on ARM before enabling ebmalloc: > + * - estimate required EBMALLOC_SIZE value, > + * - where (in which section) ebmalloc_mem[] should live; if in > .bss.page_aligned > + * then whole BSS zeroing have to be disabled in > xen/arch/arm/arm64/head.S; > + * though BSS should be initialized somehow before use of variables > living there, > + * - call free_ebmalloc_unused_mem() sowehere in init code. > + */ > + > +#define EBMALLOC_SIZE MB(1)
The previous communication with Julien ended in it being acceptable to him for this to be zero for ARM for now, eliminating (or at least reducing) the #ifndef CONFIG_ARM guarded region(s). Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel