[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v7 08/14] x86: add multiboot2 protocol support for EFI platforms



>>> On 26.09.16 at 16:19, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 23/09/16 22:47, Daniel Kiper wrote:
>> +        /*
>> +         * Initialize BSS (no nasty surprises!).
>> +         * It must be done earlier than in BIOS case
>> +         * because efi_multiboot2() touches it.
>> +         */
>> +        lea     .startof.(.bss)(%rip),%edi
>> +        mov     $.sizeof.(.bss),%ecx
> 
> Sorry, but you cannot use this syntax, for the same reasons why I won't
> accept Jan's patch making similar changes elsewhere.
> 
> Amongst other issues, you will break the Clang build with it.

Did you verify meanwhile that this doesn't work with llvm?

Jan


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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.