Jack, The patch removes all the sorting logic. It simplifies the logic a lot. Thanks for that! ACPI spec " 5.2.12.1 MADT Processor Local APIC / SAPIC Structure Entry Order" talked about why the BSP

CPU_EXCEPTION_INIT_DATA is now an internal implementation of CpuExceptionHandlerLib. Union can be removed since Ia32 and X64 have the same definition Cc: Eric Dong <eric.dong@...> Cc: Ray Ni

Since the API InitializeSeparateExceptionStacks is simplified and does't use the struct CPU_EXCEPTION_INIT_DATA, CPU_EXCEPTION_INIT_DATA become a inner implementation of CpuExcetionHandlerLib. Remove

Hide the Exception implementation details in CpuExcetionHandlerLib and caller only need to provide buffer Cc: Ray Ni <ray.ni@...> Cc: Leif Lindholm <quic_llindhol@...> Cc: Liming Gao

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

Yanbo: If this patch is temporarily added, you can handle it in your downstream code base. If you request to add DevicePathUtilitiesStandaloneMm for long term compatibility, this topic has been

Sean: Can you give BGRT spec link? I want to check the spec description. Thanks Liming

Sean: I add my comments below. The change in BootLogoLib is not required. Please check. Here, please use PcdGetBool(). Thanks Liming is

The change is good. But the change in Smbios type 9 will impact ShellPkg\Library\UefiShellDebug1CommandsLib\SmbiosView\PrintInfo.c. Please also update ShellPkg.

This change is good to me. Reviewed-by: Liming Gao <gaoliming@...> to their are telephone transmission. > > > >

Reviewed-by: Liming Gao <gaoliming@...> designee. If that Please then

Hi Liming, You mentioned patch rename the DevicePathUtilitiesStandaloneMm to UefiDevicePathLibBase, but there are some consumer in intel platform still use the DevicePathUtilitiesStandaloneMm, so

Thanks for your update. This change is good to me. Reviewed-by: Liming Gao <gaoliming@...> present.\n");

Yanbo: Previous change has been reviewed and merged. Please see the detail https://edk2.groups.io/g/devel/message/91799 Thanks Liming out placed specified by then Library.

Reviewed-by: Liming Gao <gaoliming@...> system enumerate

From: Yanbo Huang <yanbo.huang@...> reback the DevicePathUtilitiesStandaloneMm to unblock the downstream sync Signed-off-by: Yanbo Huang <yanbo.huang@...> CC: Michael D Kinney

Series merged via: PR -

Hi Chasel, I have a few comments for you. First, we should have a platform provided LibraryClass for running code in between multi-phase actions. Right now you just have this comment in

Hi Ard, The term "Pipeline" is not applicable in the scope of edk2-platforms. Thanks for catching it. A new version was sent for this fix: https://edk2.groups.io/g/devel/message/92128. Please let me

The new changes in SecureBootVariableLib brought in a new dependency of PlatformPKProtectionLib. This change added the new library instance from SecurityPkg to resolve PhytiumCommonPkg platforms