Detect and apply Microcode on BSP, store BSP's MTRR setting only when CpuCount > 1.
The purpose of this patch is to enhance the review process. The separation in this patch is aimed at facilitating a more straightforward review, with the ultimate goal of eliminating the microcode loading functionality for the second time Mp initialization Cc: Ray Ni <[email protected]> Cc: Eric Dong <[email protected]> Cc: Rahul Kumar <[email protected]> Cc: Tom Lendacky <[email protected]> Cc: Laszlo Ersek <[email protected]> Signed-off-by: Yuanhao Xie <[email protected]> --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index e8ffb99874..bb5d4188f0 100644 --- a/UefiCpuPkg/Library/MpInitLib/MpLib.c +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.c @@ -2236,19 +2236,19 @@ MpInitLibInitialize ( ShadowMicrocodeUpdatePatch (CpuMpData); } - // - // Detect and apply Microcode on BSP - // - MicrocodeDetect (CpuMpData, CpuMpData->BspNumber); - // - // Store BSP's MTRR setting - // - MtrrGetAllMtrrs (&CpuMpData->MtrrTable); - // // Wakeup APs to do some AP initialize sync (Microcode & MTRR) // if (CpuMpData->CpuCount > 1) { + // + // Detect and apply Microcode on BSP + // + MicrocodeDetect (CpuMpData, CpuMpData->BspNumber); + // + // Store BSP's MTRR setting + // + MtrrGetAllMtrrs (&CpuMpData->MtrrTable); + if (MpHandOff != NULL) { // // Only needs to use this flag for DXE phase to update the wake up -- 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111523): https://edk2.groups.io/g/devel/message/111523 Mute This Topic: https://groups.io/mt/102724547/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
