Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Laszlo Ersek
gt;> D >> ; Gao, Liming >> Subject: Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix >> InternalSync[De|In]crement >> >> On 7 November 2018 at 15:31, Laszlo Ersek wrote: >>> (+Andrew) >>> >>> Hi Ray, >>> >>

Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Ni, Ruiyu
> -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Wednesday, November 7, 2018 10:46 PM > To: Laszlo Ersek > Cc: Ni, Ruiyu ; edk2-devel@lists.01.org; Kinney, Michael D > ; Gao, Liming > Subject: Re: [edk2] [PATCH] MdePkg/BaseSyn

Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Ni, Ruiyu
e: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix > InternalSync[De|In]crement > > On 11/07/18 15:31, Laszlo Ersek wrote: > > (+Andrew) > > > > Hi Ray, > > > > On 11/07/18 05:03, Ruiyu Ni wrote: > >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=130

Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Ni, Ruiyu
> -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Wednesday, November 7, 2018 10:32 PM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Cc: Gao, Liming ; Kinney, Michael D > ; Philippe Mathieu-Daudé ; > Andrew Fish > Subject: Re: [PATCH]

Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Ard Biesheuvel
On 7 November 2018 at 15:31, Laszlo Ersek wrote: > (+Andrew) > > Hi Ray, > > On 11/07/18 05:03, Ruiyu Ni wrote: >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303 >> >> XCODE disassembly code of InternalSyncDecrement with today's code is: >> >> __asm__ __volatile__ ( >> "movl

Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Laszlo Ersek
On 11/07/18 15:31, Laszlo Ersek wrote: > (+Andrew) > > Hi Ray, > > On 11/07/18 05:03, Ruiyu Ni wrote: >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303 >> >> XCODE disassembly code of InternalSyncDecrement with today's code is: >> >> __asm__ __volatile__ ( >> "movl$1, %%eax

Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-07 Thread Laszlo Ersek
(+Andrew) Hi Ray, On 11/07/18 05:03, Ruiyu Ni wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303 > > XCODE disassembly code of InternalSyncDecrement with today's code is: > > __asm__ __volatile__ ( > "movl$1, %%eax \n\t" > "lock \n\t" > "xadd

[edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE: fix InternalSync[De|In]crement

2018-11-06 Thread Ruiyu Ni
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303 XCODE disassembly code of InternalSyncDecrement with today's code is: __asm__ __volatile__ ( "movl$1, %%eax \n\t" "lock \n\t" "xadd%%eax, %1 \n\t" "inc %%eax \n\t" : "=a" (Result),