diff options
author | Jeff Fan <jeff.fan@intel.com> | 2016-07-15 14:24:28 +0800 |
---|---|---|
committer | Hao Wu <hao.a.wu@intel.com> | 2016-07-21 15:11:36 +0800 |
commit | 7586e64b59f086e96ad935c6cfcdebc0508f6896 (patch) | |
tree | 8184c49fa3228ef944538603bbd7090c5c29c5d9 /SecurityPkg | |
parent | f6879b66b55eb454e4ec9eb26907a2a627cad64b (diff) | |
download | edk2-platforms-7586e64b59f086e96ad935c6cfcdebc0508f6896.tar.xz |
UefiCpuPkg/PiSmmCpuDxeSmm: SMM_CPU_DATA_BLOCK is not cleared
The commit 8b9311 changed the zeroing of mSmmMpSyncData of type
SMM_DISPATCHER_MP_SYNC_DATA by the following patch.
- ZeroMem (mSmmMpSyncData, mSmmMpSyncDataSize);
+ mSmmMpSyncData->SwitchBsp = FALSE;
mSmmMpSyncDataSize not only includes SMM_DISPATCHER_MP_SYNC_DATA, but
also includes the SMM_CPU_DATA_BLOCK array and one BOOLEAN variable
array as shown here:
mSmmMpSyncDataSize = sizeof (SMM_DISPATCHER_MP_SYNC_DATA) +
(sizeof (SMM_CPU_DATA_BLOCK) + sizeof (BOOLEAN)) *
gSmmCpuPrivate->SmmCoreEntryContext.NumberOfCpus;
This patch restores the original ZeroMem() to clear all CPU Sync data.
The commit 8b9311 may cause unexpected behavior.
v2:
Mentioned CandidateBsp array in comments to make it more accurate.
Cc: Laszlo Ersek <lersek@redhat.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: Feng Tian <feng.tian@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Michael Kinney <michael.d.kinney@intel.com>
(cherry picked from commit e78a2a49ee6b0c0d7c6997c87ace31d7761cf636)
Diffstat (limited to 'SecurityPkg')
0 files changed, 0 insertions, 0 deletions