diff options
author | Dionna Glaze <dionnaglaze@google.com> | 2023-02-14 23:07:19 +0000 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-02-15 00:08:00 +0000 |
commit | 1b5420e8071b4f9ba13136f19365542dfe66bf04 (patch) | |
tree | 61ab4ff3e2792a0d3802310327dbf72e7918e879 /OvmfPkg/AmdSevDxe | |
parent | f67ec877041b18656fb0504ce86bd05af232747e (diff) | |
download | edk2-1b5420e8071b4f9ba13136f19365542dfe66bf04.tar.gz edk2-1b5420e8071b4f9ba13136f19365542dfe66bf04.tar.bz2 edk2-1b5420e8071b4f9ba13136f19365542dfe66bf04.zip |
OvmfPkg/AmdSevDxe: Close mAcceptAllMemoryEvent
This event should only trigger once. It should be idempotent, but the
allocation of the memory map itself is observable and can cause
ExitBootServices to fail with a modified map key.
Cc: Thomas Lendacky <Thomas.Lendacky@amd.com>
Cc: Erdem Aktas <erdemaktas@google.com>
Cc: James Bottomley <jejb@linux.ibm.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Min Xu <min.m.xu@intel.com>
Cc: Michael Roth <michael.roth@amd.com>
Signed-off-by: Dionna Glaze <dionnaglaze@google.com>
Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Diffstat (limited to 'OvmfPkg/AmdSevDxe')
-rw-r--r-- | OvmfPkg/AmdSevDxe/AmdSevDxe.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/OvmfPkg/AmdSevDxe/AmdSevDxe.c b/OvmfPkg/AmdSevDxe/AmdSevDxe.c index 9c4e3bb406..a726498e27 100644 --- a/OvmfPkg/AmdSevDxe/AmdSevDxe.c +++ b/OvmfPkg/AmdSevDxe/AmdSevDxe.c @@ -130,6 +130,7 @@ AcceptAllMemory ( }
gBS->FreePool (AllDescMap);
+ gBS->CloseEvent (mAcceptAllMemoryEvent);
return Status;
}
|