On 09/30/18 00:23, Laszlo Ersek wrote: > Repo: https://github.com/lersek/edk2.git > Branch: inline_asm_rw_ops_1208 > > This series mainly fixes the operand constraints (missing input-output > qualifications) in "BaseSynchronizationLib/*/GccInline.c". > > (It would be better to remove these files altogether in favor of the > already existing NASM implementation, but due to > <https://bugzilla.tianocore.org/show_bug.cgi?id=881>, we can't generally > do that in libraries yet.) > > Cc: Liming Gao <liming....@intel.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > Thanks, > Laszlo > > Laszlo Ersek (5): > MdePkg/BaseSynchronizationLib GCC: fix whitespace and comments > MdePkg/BaseSynchronizationLib GCC: fix InternalSyncCompareExchange16() > MdePkg/BaseSynchronizationLib GCC: fix InternalSyncCompareExchange32() > MdePkg/BaseSynchronizationLib GCC: fix X64 > InternalSyncCompareExchange64() > MdePkg/BaseSynchronizationLib GCC: simplify IA32 > InternalSyncCompareExchange64() > > MdePkg/Library/BaseSynchronizationLib/Ia32/GccInline.c | 42 +++++++---------- > MdePkg/Library/BaseSynchronizationLib/X64/GccInline.c | 47 > +++++++------------- > 2 files changed, 34 insertions(+), 55 deletions(-) >
Forgot to mention the testing. Ran my usual OVMF tests described at <https://github.com/tianocore/tianocore.github.io/wiki/Testing-SMM-with-QEMU,-KVM-and-libvirt> on multi-VCPU guests. I tested the series against all my longterm guests (including 32-bit and 64-bit Fedora, and 64-bit Windows 7 / Server 2008 r2, 8.1 / Server 2012 r2, 10 / Server 2016). Also covered an SMM-less build on i440fx, with 64-bit Fedora. Thanks Laszlo _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel