Reviewed all BDS related patches including PlatformBds libraries. All look good to me. Thanks for offline addressing my PlatformBds related questions.
Reviewed-by: Sunny Wang <sunnyw...@hpe.com> -----Original Message----- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu Ni Sent: Tuesday, November 08, 2016 8:29 PM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH v2 0/9] Defer 3rd party images loading to after EndOfDxe The patches change the default image loading policy by deferring 3rd party images loading to after EndOfDxe and add a new BDS API to dispatch the deferred images. Platform needs to call the new BDS API EfiBootManagerDispatchDeferredImages after EndOfDxe to ensure that any deferred images are loaded. v2 puts the deferred images checking in DEBUG_CODE macro, corrects a typo of function name, and puts CpuDeadLoop() after ASSERT(FALSE). v2 includes all the platform changes. Ruiyu Ni (9): MdeModulePkg/SecurityStubDxe: Defer 3rd party image before EndOfDxe MdeModulePkg/UefiBootManager: Add EfiBootManagerDispatchDeferredImages MdeModulePkg/BdsDxe: Check deferred images before booting to OS MdeModulePkg/SecurityStubDxe: Report failure if image is load earlier ArmVirPkg/PlatformBds: Dispatch deferred images after EndOfDxe OvmfPkg/PlatformBds: Dispatch deferred images after EndOfDxe CorebootPayload/PlatformBds: Dispatch deferred images after EndOfDxe QuarkPlatformPkg/PlatformBds: Dispatch deferred images after EndOfDxe Nt32Pkg/PlatformBds: Dispatch deferred images after EndOfDxe .../Library/PlatformBootManagerLib/PlatformBm.c | 5 + .../PlatformBootManagerLib/PlatformBootManager.c | 5 + MdeModulePkg/Include/Library/UefiBootManagerLib.h | 13 + MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c | 113 ++++++ .../Library/UefiBootManagerLib/InternalBm.h | 1 + .../UefiBootManagerLib/UefiBootManagerLib.inf | 1 + MdeModulePkg/Universal/BdsDxe/Bds.h | 4 +- MdeModulePkg/Universal/BdsDxe/BdsDxe.inf | 2 + MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 90 +++++ .../SecurityStubDxe/Defer3rdPartyImageLoad.c | 414 +++++++++++++++++++++ .../SecurityStubDxe/Defer3rdPartyImageLoad.h | 95 +++++ .../Universal/SecurityStubDxe/SecurityStub.c | 14 +- .../Universal/SecurityStubDxe/SecurityStubDxe.inf | 11 +- .../PlatformBootManagerLib/PlatformBootManager.c | 5 + .../Library/PlatformBootManagerLib/BdsPlatform.c | 5 + .../PlatformBootManagerLib/PlatformBootManager.c | 7 +- 16 files changed, 781 insertions(+), 4 deletions(-) create mode 100644 MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.c create mode 100644 MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.h -- 2.9.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
