The patch set is to hide the exception implementation details, so that caller don't need to know anything about IDT when separate stack for it. However, this patch set changes a library API, so I have to change multiple packages inside one patch. Otherwise, I can make sure every single commit can build and boot fine. If anyone has good idea to separate the first big patch, please tell me. Thanks in advance.
V2: Add another patch to Simplify the CPU_EXCEPTION_INIT_DATA definition Keep the memory layout picture in CpuExceptionHandlerLib. Fix some code and comment issue according to Ray's comment V3: Change the code behavior when the needed size is zero: skip instead of assert Fix the bug that treating the TSS as part of GDT Reorder the modification in the patch set. Code can be seen at https://github.com/tianocore/edk2/pull/3124 Cc: Eric Dong <eric.d...@intel.com> Cc: Ray Ni <ray...@intel.com> Cc: Rahul Kumar <rahul1.ku...@intel.com> Cc: Leif Lindholm <quic_llind...@quicinc.com> Cc: Dandan Bi <dandan...@intel.com> Cc: Liming Gao <gaolim...@byosoft.com.cn> Cc: Jian J Wang <jian.j.w...@intel.com> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> Cc: Sami Mujawar <sami.muja...@arm.com> Signed-off-by: Zhiguang Liu <zhiguang....@intel.com> Zhiguang Liu (3): UefiCpuPkg: Simplify InitializeSeparateExceptionStacks MdeModulePkg: Move CPU_EXCEPTION_INIT_DATA to UefiCpuPkg UefiCpuPkg: Simplify the struct definition of CPU_EXCEPTION_INIT_DATA .../Library/ArmExceptionLib/ArmExceptionLib.c | 15 +- MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c | 4 +- .../Include/Library/CpuExceptionHandlerLib.h | 82 +------- .../CpuExceptionHandlerLibNull.c | 15 +- UefiCpuPkg/CpuDxe/CpuMp.c | 162 ++++------------ UefiCpuPkg/CpuMpPei/CpuMpPei.c | 176 ++++-------------- .../CpuExceptionCommon.h | 57 +++++- .../CpuExceptionHandlerLib/DxeException.c | 112 ++++++++--- .../Ia32/ArchExceptionHandler.c | 71 ++++--- .../CpuExceptionHandlerLib/PeiCpuException.c | 94 +++++++++- .../PeiCpuExceptionHandlerLib.inf | 4 +- .../SecPeiCpuException.c | 15 +- .../CpuExceptionHandlerLib/SmmException.c | 15 +- .../X64/ArchExceptionHandler.c | 67 ++++--- 14 files changed, 411 insertions(+), 478 deletions(-) -- 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#92226): https://edk2.groups.io/g/devel/message/92226 Mute This Topic: https://groups.io/mt/92907048/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-