commit 081f6416ff7c2b6a2fe91bb96cdb645c3890a6f3
author Jeff Fan <jeff.fan@intel.com> Wed Nov 23 21:33:20 2016 +0800
committer Jeff Fan <jeff.fan@intel.com> Mon Nov 28 08:56:26 2016 +0800
tree fce6ae8d44659b6c72b8b5426cb34ff08a60ea7f
parent 4575a602ca6072ee9d04150b38bfb143cbff8588

UefiCpuPkg/DxeMpLib: Get safe AP loop handler from global variable

AP loop function is already saved into global variable, needn't to get it from
AP function parameter.

Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Feng Tian <feng.tian@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jeff Fan <jeff.fan@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Tested-by: Laszlo Ersek <lersek@redhat.com>

UefiCpuPkg/Library/MpInitLib/DxeMpLib.c

diff --git a/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c b/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c
index 7f3900b..a0d5eeb 100644
--- a/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c
+++ b/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c
@@ -244,7 +244,7 @@
 

   CpuMpData    = GetCpuMpData ();

   MwaitSupport = IsMwaitSupport ();

-  AsmRelocateApLoopFunc = (ASM_RELOCATE_AP_LOOP) (UINTN) Buffer;

+  AsmRelocateApLoopFunc = (ASM_RELOCATE_AP_LOOP) (UINTN) mReservedApLoopFunc;

   AsmRelocateApLoopFunc (MwaitSupport, CpuMpData->ApTargetCState, CpuMpData->PmCodeSegment);

   //

   // It should never reach here

@@ -273,7 +273,7 @@
   CpuMpData->SaveRestoreFlag = TRUE;

   CpuMpData->PmCodeSegment = GetProtectedModeCS ();

   CpuMpData->ApLoopMode = PcdGet8 (PcdCpuApLoopMode);

-  WakeUpAP (CpuMpData, TRUE, 0, RelocateApLoop, mReservedApLoopFunc);

+  WakeUpAP (CpuMpData, TRUE, 0, RelocateApLoop, NULL);

   DEBUG ((DEBUG_INFO, "%a() done!\n", __FUNCTION__));

 }