UefiCpuPkg/PiSmmCpuDxeSmm: patch "gSmbase" with PatchInstructionX86()
Rename the variable to "gPatchSmbase" so that its association with PatchInstructionX86() is clear from the declaration, change its type to X86_ASSEMBLY_PATCH_LABEL, and patch it with PatchInstructionX86(). This lets us remove the binary (DB) encoding of some instructions in "SmiEntry.nasm". Cc: Eric Dong <eric.dong@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=866 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Liming Gao <liming.gao@intel.com>
This commit is contained in:
@@ -105,11 +105,11 @@ typedef struct {
|
||||
///
|
||||
/// Variables from SMI Handler
|
||||
///
|
||||
extern UINT32 gSmbase;
|
||||
extern volatile UINT32 gSmiStack;
|
||||
extern UINT32 gSmiCr3;
|
||||
extern volatile UINT8 gcSmiHandlerTemplate[];
|
||||
extern CONST UINT16 gcSmiHandlerSize;
|
||||
X86_ASSEMBLY_PATCH_LABEL gPatchSmbase;
|
||||
extern volatile UINT32 gSmiStack;
|
||||
extern UINT32 gSmiCr3;
|
||||
extern volatile UINT8 gcSmiHandlerTemplate[];
|
||||
extern CONST UINT16 gcSmiHandlerSize;
|
||||
|
||||
//
|
||||
// Variables used by SMI Handler
|
||||
@@ -718,7 +718,7 @@ InstallSmiHandler (
|
||||
//
|
||||
gSmiStack = (UINT32)((UINTN)SmiStack + StackSize - sizeof (UINTN));
|
||||
gSmiCr3 = Cr3;
|
||||
gSmbase = SmBase;
|
||||
PatchInstructionX86 (gPatchSmbase, SmBase, 4);
|
||||
gSmiHandlerIdtr.Base = IdtBase;
|
||||
gSmiHandlerIdtr.Limit = (UINT16)(IdtSize - 1);
|
||||
|
||||
|
Reference in New Issue
Block a user