diff options
| author | Jeff Fan <jeff.fan@intel.com> | 2017-04-18 10:14:08 +0800 |
|---|---|---|
| committer | Guo Mang <mang.guo@intel.com> | 2017-07-12 11:24:20 +0800 |
| commit | 67d66e218ed52f5018e4769990bd5f02fe19fc7e (patch) | |
| tree | 009f342b6281a77a03525525442ee986188d94d7 | |
| parent | 312725835b47251983579b854aa56fb96dc33919 (diff) | |
| download | edk2-platforms-67d66e218ed52f5018e4769990bd5f02fe19fc7e.tar.xz | |
UefiCpuPkg/PiSmmCpuDxeSmm: Lock should be acquired
SMM BSP's *busy* state should be acquired. We could use AcquireSpinLock()
instead of AcquireSpinLockOrFail().
Cc: Hao Wu <hao.a.wu@intel.com>
Cc: Feng Tian <feng.tian@intel.com>
Cc: Michael Kinney <michael.d.kinney@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jeff Fan <jeff.fan@intel.com>
Reviewed-by: Hao Wu <hao.a.wu@intel.com>
(cherry picked from commit 170a3c1e0fff7de43501ef851b4ad5cbe355e220)
| -rw-r--r-- | Core/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Core/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c b/Core/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c index a1d16b4570..e03f1e02c0 100644 --- a/Core/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c +++ b/Core/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c @@ -407,7 +407,7 @@ BSPHandler ( //
// The BUSY lock is initialized to Acquired state
//
- AcquireSpinLockOrFail (mSmmMpSyncData->CpuData[CpuIndex].Busy);
+ AcquireSpinLock (mSmmMpSyncData->CpuData[CpuIndex].Busy);
//
// Perform the pre tasks
|