[edk2-devel] [PATCH v7 3/7] MdeModulePkg: Notify BeforeExitBootServices in CoreExitBootServices
Dionna Glaze via groups.io
dionnaglaze=google.com at groups.io
Wed Oct 5 20:58:34 UTC 2022
The specification says that disabling the timer should happen right
after. Ard told me it should still work this way.
On Wed, Oct 5, 2022 at 1:50 PM Tom Lendacky <thomas.lendacky at amd.com> wrote:
>
> On 10/5/22 15:33, Dionna Glaze wrote:
> > Location of notification is has been specified in UEFI v2.9.
> >
> > Cc: Gerd Hoffmann <kraxel at redhat.com>
> > Cc: James Bottomley <jejb at linux.ibm.com>
> > Cc: Jiewen Yao <jiewen.yao at intel.com>
> > Cc: Tom Lendacky <thomas.lendacky at amd.com>
> > Cc: Ard Biesheuvel <ardb at kernel.org>
> > Cc: "Min M. Xu" <min.m.xu at intel.com>
> > Cc: Andrew Fish <afish at apple.com>
> > Cc: "Michael D. Kinney" <michael.d.kinney at intel.com>
> > Cc: Ray Ni <ray.ni at intel.com>
> >
> > Signed-off-by: Dionna Glaze <dionnaglaze at google.com>
> > ---
> > MdeModulePkg/Core/Dxe/DxeMain.inf | 1 +
> > MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c | 6 ++++++
> > 2 files changed, 7 insertions(+)
> >
> > diff --git a/MdeModulePkg/Core/Dxe/DxeMain.inf b/MdeModulePkg/Core/Dxe/DxeMain.inf
> > index e4bca89577..35d5bf0dee 100644
> > --- a/MdeModulePkg/Core/Dxe/DxeMain.inf
> > +++ b/MdeModulePkg/Core/Dxe/DxeMain.inf
> > @@ -100,6 +100,7 @@
> > gEfiEventVirtualAddressChangeGuid ## CONSUMES ## Event
> > ## CONSUMES ## Event
> > ## PRODUCES ## Event
> > + gEfiEventBeforeExitBootServicesGuid
> > gEfiEventExitBootServicesGuid
> > gEfiHobMemoryAllocModuleGuid ## SOMETIMES_CONSUMES ## HOB
> > gEfiFirmwareFileSystem2Guid ## CONSUMES ## GUID # Used to compare with FV's file system guid and get the FV's file system format
> > diff --git a/MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c b/MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c
> > index 5733f0c8ec..4683016ed7 100644
> > --- a/MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c
> > +++ b/MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c
> > @@ -763,6 +763,12 @@ CoreExitBootServices (
> > {
> > EFI_STATUS Status;
> >
> > + //
> > + // Notify other drivers of their last chance to use boot services
> > + // before the memory map is terminated.
> > + //
> > + CoreNotifySignalList (&gEfiEventBeforeExitBootServicesGuid);
>
> Isn't this supposed to be after disabling the timer?
>
> Thanks,
> Tom
>
> > +
> > //
> > // Disable Timer
> > //
--
-Dionna Glaze, PhD (she/her)
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94773): https://edk2.groups.io/g/devel/message/94773
Mute This Topic: https://groups.io/mt/94144530/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list