summaryrefslogtreecommitdiffstats
path: root/StandaloneMmPkg
diff options
context:
space:
mode:
authorJian J Wang <jian.j.wang@intel.com>2018-07-13 12:46:58 +0800
committerEric Dong <eric.dong@intel.com>2018-07-20 10:34:11 +0800
commitb72f4873726b679fb4681ac26abfae227aaa13f1 (patch)
treef141ca44d9bd3c7fe36de6294bb4d0d0761c035d /StandaloneMmPkg
parent9b7242f5dec7649b878a4058f893103885520bf7 (diff)
downloadedk2-b72f4873726b679fb4681ac26abfae227aaa13f1.tar.gz
edk2-b72f4873726b679fb4681ac26abfae227aaa13f1.tar.bz2
edk2-b72f4873726b679fb4681ac26abfae227aaa13f1.zip
UefiCpuPkg/CpuDxe: fix incorrect check of SMM mode
Current IsInSmm() method makes use of gEfiSmmBase2ProtocolGuid.InSmm() to check if current processor is in SMM mode or not. But this is not correct because gEfiSmmBase2ProtocolGuid.InSmm() can only detect if the caller is running in SMRAM or from SMM driver. It cannot guarantee if the caller is running in SMM mode. Because SMM mode will load its own page table, adding an extra check of saved DXE page table base address against current CR3 register value can help to get the correct answer for sure (in SMM mode or not in SMM mode). There's indiscriminate uses of Context.X64 and Context.Ia32 in code which is not a good coding practice and will cause potential issue. In addition, the related structure type definition is not packed and has also potential issue. This will not be covered by this patch but be tracked by a bug below. https://bugzilla.tianocore.org/show_bug.cgi?id=1039 This is an issue caused by check-in at 2a1408d1d739ead00c96397549be7a9fc53c9c6e Cc: Eric Dong <eric.dong@intel.com> Cc: Laszlo Ersek <lersek@redhat.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Cc: Star Zeng <star.zeng@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jian J Wang <jian.j.wang@intel.com> Reviewed-by: Eric Dong <eric.dong@intel.com> Regression-tested-by: Laszlo Ersek <lersek@redhat.com>
Diffstat (limited to 'StandaloneMmPkg')
0 files changed, 0 insertions, 0 deletions