Patch #1 follows Laszlo's suggestion to add global NASM macros for NASM struc usage. Patch #2 changes all hardcode offset to use struc. Patch #3 doesn't have any change comparing to V1 except 1). dword/qword prefix is added. 2). the comments "program AP stack" is removed.
Ray Ni (3): MdePkg/Nasm.inc: add macros for C types used in structure definition UefiCpuPkg/MpInitLib: Use NASM struc to avoid hardcode offset UefiCpuPkg/MpInitLib: Use XADD to avoid lock acquire/release MdePkg/Include/Ia32/Nasm.inc | 38 ++++++ MdePkg/Include/X64/Nasm.inc | 38 ++++++ UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf | 5 +- UefiCpuPkg/Library/MpInitLib/Ia32/MpEqu.inc | 43 ------- .../Library/MpInitLib/Ia32/MpFuncs.nasm | 98 +++++++--------- UefiCpuPkg/Library/MpInitLib/MpEqu.inc | 99 ++++++++++++++++ UefiCpuPkg/Library/MpInitLib/MpLib.c | 1 - UefiCpuPkg/Library/MpInitLib/MpLib.h | 3 +- UefiCpuPkg/Library/MpInitLib/PeiMpInitLib.inf | 5 +- UefiCpuPkg/Library/MpInitLib/X64/MpEqu.inc | 45 -------- UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm | 108 ++++++++---------- 11 files changed, 272 insertions(+), 211 deletions(-) delete mode 100644 UefiCpuPkg/Library/MpInitLib/Ia32/MpEqu.inc create mode 100644 UefiCpuPkg/Library/MpInitLib/MpEqu.inc delete mode 100644 UefiCpuPkg/Library/MpInitLib/X64/MpEqu.inc -- 2.27.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#71344): https://edk2.groups.io/g/devel/message/71344 Mute This Topic: https://groups.io/mt/80401290/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
