diff options
| author | Laszlo Ersek <lersek@redhat.com> | 2016-11-16 22:52:06 +0100 |
|---|---|---|
| committer | Laszlo Ersek <lersek@redhat.com> | 2016-11-17 10:58:07 +0100 |
| commit | 00650c531a90f29d75dadfc22656554cbea4bb72 (patch) | |
| tree | 28696f86892433637a9f5da29f86e6b9b3d99f1e | |
| parent | 5797f61c55867db25c6a6ed03ac27b946990509b (diff) | |
| download | edk2-platforms-00650c531a90f29d75dadfc22656554cbea4bb72.tar.xz | |
UefiCpuPkg/MpInitLib/X64/MpFuncs.nasm: fix fatal typo
The recent patch "UefiCpuPkg/MpInitLib: Program AP stack in fixed address"
inadvertently broke the first startup of APs during X64 PEI, because in
the TestLock section of the code, it replaced the access to the
NumApsExecuting counter with an access to the unrelated InitFlag field.
Cc: Jeff Fan <jeff.fan@intel.com>
Fixes: 845c5be1fd9bf7edfac4a103dfab70829686978f
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Jeff Fan <jeff.fan@intel.com>
| -rw-r--r-- | UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm b/UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm index 6a8794d83b..138b97312b 100644 --- a/UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm +++ b/UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm @@ -134,7 +134,7 @@ TestLock: cmp rax, NotVacantFlag
jz TestLock
- lea ecx, [esi + InitFlagLocation]
+ lea ecx, [esi + NumApsExecutingLocation]
inc dword [ecx]
mov ebx, [ecx]
|