回复: [edk2-devel] 回复: [edk2][PATCH 0/2] Add Events introduced in UEFI 2.9

Wed Dec 21 01:16:04 UTC 2022

Rob:
  I suggest to only update EfiSignalEventReadyToBoot () API in UefiLib to trig ReadyToBoot and AfterReadyToBoot event both. 

Thanks
Liming
> -----邮件原件-----
> 发件人: devel at edk2.groups.io <devel at edk2.groups.io> 代表 Robert Phelps
> via groups.io
> 发送时间: 2022年12月15日 1:50
> 收件人: gaoliming <gaoliming at byosoft.com.cn>; 'Kinney, Michael D'
> <michael.d.kinney at intel.com>; devel at edk2.groups.io
> 抄送: 'Liu, Zhiguang' <zhiguang.liu at intel.com>; 'Bi, Dandan'
> <dandan.bi at intel.com>
> 主题: Re: [edk2-devel] 回复: [edk2][PATCH 0/2] Add Events introduced in
> UEFI 2.9
> 
> Liming,
> 
> I can add that, but I was wondering if you have a preference of one of the
> following:
> 1)  add to the EfiSignalReadyToBoot Function and create, signal, and close
> the AfterReadyToBoot Event
> 2) create a new function EfiSignalAfterReadyToBoot and call that from
> BmBoot.c
> 3) have all the code that would be in the EfiSignalAfterReadyToBoot Function
> inline in the EfiBootManagerBoot function.
> 
> Rob
> 
> -----Original Message-----
> From: gaoliming <gaoliming at byosoft.com.cn>
> Sent: Tuesday, December 06, 2022 20:08
> To: 'Kinney, Michael D' <michael.d.kinney at intel.com>; devel at edk2.groups.io;
> Robert Phelps <Robert at ami.com>
> Cc: 'Liu, Zhiguang' <zhiguang.liu at intel.com>; 'Bi, Dandan'
> <dandan.bi at intel.com>
> Subject: [EXTERNAL] 回复: [edk2][PATCH 0/2] Add Events introduced in UEFI
> 2.9
> 
> 
> **CAUTION: The e-mail below is from an external source. Please exercise
> caution before opening attachments, clicking links, or following guidance.**
> 
> Robert:
>   EFI_EVENT_GROUP_AFTER_READY_TO_BOOT should be trig in
> Edk2\MdeModulePkg\Library\UefiBootManagerLib like ReadyToBoot event.
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: Kinney, Michael D <michael.d.kinney at intel.com>
> > 发送时间: 2022年11月9日 6:45
> > 收件人: devel at edk2.groups.io; robert at ami.com; Kinney, Michael D
> > <michael.d.kinney at intel.com>
> > 抄送: Gao, Liming <gaoliming at byosoft.com.cn>; Liu, Zhiguang
> > <zhiguang.liu at intel.com>; Bi, Dandan <dandan.bi at intel.com>
> > 主题: RE: [edk2][PATCH 0/2] Add Events introduced in UEFI 2.9
> >
> > Hi Robert,
> >
> > I do not see the signal of the after exit boot services event.
> >
> > Thanks,
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of
> > > Robert
> > Phelps via groups.io
> > > Sent: Tuesday, November 8, 2022 12:28 PM
> > > To: devel at edk2.groups.io
> > > Cc: Kinney, Michael D <michael.d.kinney at intel.com>; Gao, Liming
> > <gaoliming at byosoft.com.cn>; Liu, Zhiguang
> > > <zhiguang.liu at intel.com>; Bi, Dandan <dandan.bi at intel.com>
> > > Subject: [edk2-devel] [edk2][PATCH 0/2] Add Events introduced in
> > > UEFI 2.9
> > >
> > > The following patches contain the new definitions for new events
> > > that were added in UEFI 2.9: Before Exit Boot Services and After
> > > Ready to Boot.  The second patch adds the call to signal the Before
> > > Exit Boot Services Event in DxeMain.c and the addition of the GUID
> > > in the INF file DxeMain.inf.
> > >
> > >
> > > Cc: Michael D Kinney <michael.d.kinney at intel.com> [mdkinney]
> > > Cc: Liming Gao <gaoliming at byosoft.com.cn> [lgao4]
> > > Cc: Zhiguang Liu <zhiguang.liu at intel.com> [LiuZhiguang001]
> > > Cc: Dandan Bi <dandan.bi at intel.com> [dandanbi]
> > > Signed-off-by: Robert Phelps <robert at ami.com>
> > >
> > > Robert Phelps (2):
> > >   MdePkg: Add New Event Definitions from UEFI 2.9
> > >   MdeModulePkg: Added call to signal New Event
> > >
> > >  MdeModulePkg/Core/Dxe/DxeMain.inf       |  1 +
> > >  MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c |  2 ++
> > >  MdePkg/Include/Guid/EventGroup.h        | 10 ++++++++++
> > >  MdePkg/MdePkg.dec                       |  6 ++++++
> > >  4 files changed, 19 insertions(+)
> > >
> > > --
> > > 2.36.1.windows.1
> > > -The information contained in this message may be confidential and
> > proprietary to American Megatrends (AMI). This communication
> > > is intended to be read only by the individual or entity to whom it
> > > is
> > addressed or by their designee. If the reader of this
> > > message is not the intended recipient, you are on notice that any
> > distribution of this message, in any form, is strictly
> > > prohibited. Please promptly notify the sender by reply e-mail or by
> > telephone at 770-246-8600, and then delete or destroy all
> > > copies of the transmission.
> > >
> > >
> > >
> > >
> 
> 
> 
> -The information contained in this message may be confidential and
> proprietary to American Megatrends (AMI). This communication is intended to
> be read only by the individual or entity to whom it is addressed or by their
> designee. If the reader of this message is not the intended recipient, you are
> on notice that any distribution of this message, in any form, is strictly
> prohibited. Please promptly notify the sender by reply e-mail or by telephone
> at 770-246-8600, and then delete or destroy all copies of the transmission.
> 
> 
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97643): https://edk2.groups.io/g/devel/message/97643
Mute This Topic: https://groups.io/mt/95798525/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-