diff options
| author | Jian J Wang <jian.j.wang@intel.com> | 2018-09-14 13:40:37 +0800 |
|---|---|---|
| committer | Jian J Wang <jian.j.wang@intel.com> | 2018-09-17 10:45:53 +0800 |
| commit | 16d8465756ad29a17a082108dae55d254b91bf2c (patch) | |
| tree | f938e59e5595d1602d65fa76358ea5bdc2b27d58 /UefiCpuPkg/PiSmmCpuDxeSmm | |
| parent | a95c9cfd18b691d9694b17f032ffefd1e905cc4a (diff) | |
| download | edk2-16d8465756ad29a17a082108dae55d254b91bf2c.tar.gz edk2-16d8465756ad29a17a082108dae55d254b91bf2c.tar.bz2 edk2-16d8465756ad29a17a082108dae55d254b91bf2c.zip | |
UefiCpuPkg/PiSmmCpuDxeSmm: move InitSmmS3Cr3() into else block
BZ#: https://bugzilla.tianocore.org/show_bug.cgi?id=1165
InitSmmS3Cr3 () will update SmmS3ResumeState so moving the calling of
it into else block to keep the logic consistency.
Cc: Star Zeng <star.zeng@intel.com>
Cc: Benjamin You <benjamin.you@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Laszlo Ersek <lersek@redhat.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>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Diffstat (limited to 'UefiCpuPkg/PiSmmCpuDxeSmm')
| -rw-r--r-- | UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c b/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c index fcbd56240d..52ff9679d5 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c @@ -752,12 +752,12 @@ InitSmmS3ResumeState ( if (sizeof (UINTN) == sizeof (UINT32)) {
SmmS3ResumeState->Signature = SMM_S3_RESUME_SMM_32;
}
- }
- //
- // Patch SmmS3ResumeState->SmmS3Cr3
- //
- InitSmmS3Cr3 ();
+ //
+ // Patch SmmS3ResumeState->SmmS3Cr3
+ //
+ InitSmmS3Cr3 ();
+ }
//
// Allocate safe memory in ACPI NVS for AP to execute hlt loop in
|