On 11/21/23 08:39, Yuanhao Xie wrote: > This patch replicates the WakeUpAp operation, encompassing both > microcode loading and MTRR synchronization, regardless of whether > MpHandOff is NULL. > > 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 | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c > b/UefiCpuPkg/Library/MpInitLib/MpLib.c > index 9a6ec5db5c..e8ffb99874 100644 > --- a/UefiCpuPkg/Library/MpInitLib/MpLib.c > +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.c > @@ -2256,9 +2256,11 @@ MpInitLibInitialize ( > // in DXE. > // > CpuMpData->InitFlag = ApInitReconfig; > + WakeUpAP (CpuMpData, TRUE, 0, ApInitializeSync, CpuMpData, TRUE); > + } else { > + WakeUpAP (CpuMpData, TRUE, 0, ApInitializeSync, CpuMpData, TRUE); > } > > - WakeUpAP (CpuMpData, TRUE, 0, ApInitializeSync, CpuMpData, TRUE); > // > // Wait for all APs finished initialization > //
Reviewed-by: Laszlo Ersek <[email protected]> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111617): https://edk2.groups.io/g/devel/message/111617 Mute This Topic: https://groups.io/mt/102724546/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
