diff options
| author | Jeff Fan <jeff.fan@intel.com> | 2017-04-18 10:14:08 +0800 |
|---|---|---|
| committer | Jeff Fan <jeff.fan@intel.com> | 2017-04-19 11:18:41 +0800 |
| commit | 170a3c1e0fff7de43501ef851b4ad5cbe355e220 (patch) | |
| tree | 9e00305b78e856c3c34780d73fe1410d0c05b126 | |
| parent | 33cc487c263384451aef77fa5d749fd4f3d78b7d (diff) | |
| download | edk2-170a3c1e0fff7de43501ef851b4ad5cbe355e220.tar.gz edk2-170a3c1e0fff7de43501ef851b4ad5cbe355e220.tar.bz2 edk2-170a3c1e0fff7de43501ef851b4ad5cbe355e220.zip | |
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>
| -rw-r--r-- | UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c b/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c index a1d16b4570..e03f1e02c0 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c +++ b/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
|