Virtualization
/
edk2
mirror of https://gitlab.com/qemu-project/edk2.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Move CommunicationBuffer from stack to global variable. 

We had put communication buffer to Runtime memory,
so that SMI handler can know it is not used by OS.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: "Yao, Jiewen" <jiewen.yao@intel.com>
Reviewed-by: "Zeng, Star" <star.zeng@intel.com>
Reviewed-by: "Kinney, Michael D" <michael.d.kinney@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18949 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
Yao, Jiewen 2015-11-25 13:14:27 +00:00 committed by jyao1
parent f45f2d4ad4
commit 670a563f28
1 changed files with 13 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
MdeModulePkg/Core/PiSmmCore/PiSmmIpl.c
View File

@ -246,6 +246,8 @@ BOOLEAN mSmmLocked = FALSE;
EFI_PHYSICAL_ADDRESS mSmramCacheBase; EFI_PHYSICAL_ADDRESS mSmramCacheBase;
UINT64 mSmramCacheSize; UINT64 mSmramCacheSize;
EFI_SMM_COMMUNICATE_HEADER mCommunicateHeader;
// //
// Table of Protocol notification and GUIDed Event notifications that the SMM IPL requires // Table of Protocol notification and GUIDed Event notifications that the SMM IPL requires
// //
@ -538,21 +540,20 @@ SmmIplGuidedEventNotify (
IN VOID *Context IN VOID *Context
) )
{ {
EFI_SMM_COMMUNICATE_HEADER CommunicateHeader;
UINTN Size; UINTN Size;
// //
// Use Guid to initialize EFI_SMM_COMMUNICATE_HEADER structure // Use Guid to initialize EFI_SMM_COMMUNICATE_HEADER structure
// //
CopyGuid (&CommunicateHeader.HeaderGuid, (EFI_GUID *)Context); CopyGuid (&mCommunicateHeader.HeaderGuid, (EFI_GUID *)Context);
CommunicateHeader.MessageLength = 1; mCommunicateHeader.MessageLength = 1;
CommunicateHeader.Data[0] = 0; mCommunicateHeader.Data[0] = 0;
// //
// Generate the Software SMI and return the result // Generate the Software SMI and return the result
// //
Size = sizeof (CommunicateHeader); Size = sizeof (mCommunicateHeader);
SmmCommunicationCommunicate (&mSmmCommunication, &CommunicateHeader, &Size); SmmCommunicationCommunicate (&mSmmCommunication, &mCommunicateHeader, &Size);
} }
/** /**
@ -569,7 +570,6 @@ SmmIplDxeDispatchEventNotify (
IN VOID *Context IN VOID *Context
) )
{ {
EFI_SMM_COMMUNICATE_HEADER CommunicateHeader;
UINTN Size; UINTN Size;
EFI_STATUS Status; EFI_STATUS Status;
@ -582,20 +582,20 @@ SmmIplDxeDispatchEventNotify (
// Clear the buffer passed into the Software SMI. This buffer will return // Clear the buffer passed into the Software SMI. This buffer will return
// the status of the SMM Core Dispatcher. // the status of the SMM Core Dispatcher.
// //
CopyGuid (&CommunicateHeader.HeaderGuid, (EFI_GUID *)Context); CopyGuid (&mCommunicateHeader.HeaderGuid, (EFI_GUID *)Context);
CommunicateHeader.MessageLength = 1; mCommunicateHeader.MessageLength = 1;
CommunicateHeader.Data[0] = 0; mCommunicateHeader.Data[0] = 0;
// //
// Generate the Software SMI and return the result // Generate the Software SMI and return the result
// //
Size = sizeof (CommunicateHeader); Size = sizeof (mCommunicateHeader);
SmmCommunicationCommunicate (&mSmmCommunication, &CommunicateHeader, &Size); SmmCommunicationCommunicate (&mSmmCommunication, &mCommunicateHeader, &Size);
// //
// Return if there is no request to restart the SMM Core Dispatcher // Return if there is no request to restart the SMM Core Dispatcher
// //
if (CommunicateHeader.Data[0] != COMM_BUFFER_SMM_DISPATCH_RESTART) { if (mCommunicateHeader.Data[0] != COMM_BUFFER_SMM_DISPATCH_RESTART) {
return; return;
} }