I was also thinking about the same things - removing REPORT_STATUS_CODE
call.
REPORT_STATUS_CODE invokes BootServices functions (LocateProtocol() and TPL
functions are examples). And I am not sure it is 'legal' at this stage of
ExitBootServices().
I have not thought about declaring an ExitBootServices event for
ExitBootServices. But it might be a good fix.
From: Andrew Fish [mailto:af...@apple.com]
Sent: 27 January 2014 19:44
To: edk2-devel@lists.sourceforge.net
Subject: Re: [edk2] [PATCH] MdeModulePkg/DxeCore: Fixed CoreExitBootServices
- Interrupt disabling (critical defect)
I'm wondering if we should remove the call to REPORT_STATUS_CODE? I don't
see any place in the library class definition that restricts allocating
memory. Maybe it would make more sense for the ReportStatusCode lib to have
an exit boot services event?
Thanks,
Andrew Fish
------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
