Re: [Xen-devel] [PATCH v14 4/9] x86: add multiboot2 protocol support for EFI platforms

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 02.02.17 at 23:43, <daniel.kiper@xxxxxxxxxx> wrote:
>          /*
> -         * Now trampoline_phys points to the following structure (lowest
> -         * address is at the top):
> +         * Now trampoline_phys points to the following structure (lowest 
> address
> +         * is at the bottom):
>           *
>           * +------------------------+
> -         * |    TRAMPOLINE_SPACE    |
> -         * +- - - - - - - - - - - - +
> -         * |       mbi struct       |
> -         * +------------------------+
>           * | TRAMPOLINE_STACK_SPACE |
>           * +------------------------+
> +         * |        mbi data        |
> +         * +- - - - - - - - - - - - +
> +         * |    TRAMPOLINE_SPACE    |
> +         * +------------------------+

Well, I did like the other layout better, but anyway.

> --- a/xen/arch/x86/boot/x86_64.S
> +++ b/xen/arch/x86/boot/x86_64.S
> @@ -186,7 +186,7 @@ GLOBAL(idle_pg_table)
>  GLOBAL(__page_tables_end)
>  
>  /* Init pagetables.  Enough page directories to map into the bottom 1GB. */
> -        .section .init.data, "a", @progbits
> +        .section .init.data, "aw", @progbits

This looks to be an unrelated change?

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel