public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "gaoliming" <gaoliming@byosoft.com.cn>
To: <devel@edk2.groups.io>, <robert@ami.com>,
	"'Kinney, Michael D'" <michael.d.kinney@intel.com>
Cc: "'Liu, Zhiguang'" <zhiguang.liu@intel.com>,
	"'Bi, Dandan'" <dandan.bi@intel.com>
Subject: 回复: [edk2-devel] 回复: [edk2][PATCH 0/2] Add Events introduced in UEFI 2.9
Date: Wed, 21 Dec 2022 09:16:04 +0800	[thread overview]
Message-ID: <057a01d914d9$cc0775c0$64166140$@byosoft.com.cn> (raw)
In-Reply-To: <BN0PR10MB5158539807473AE6D00F6AA2CCE09@BN0PR10MB5158.namprd10.prod.outlook.com>

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

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Robert Phelps
> via groups.io
> 发送时间: 2022年12月15日 1:50
> 收件人: gaoliming <gaoliming@byosoft.com.cn>; 'Kinney, Michael D'
> <michael.d.kinney@intel.com>; devel@edk2.groups.io
> 抄送: 'Liu, Zhiguang' <zhiguang.liu@intel.com>; 'Bi, Dandan'
> <dandan.bi@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@byosoft.com.cn>
> Sent: Tuesday, December 06, 2022 20:08
> To: 'Kinney, Michael D' <michael.d.kinney@intel.com>; devel@edk2.groups.io;
> Robert Phelps <Robert@ami.com>
> Cc: 'Liu, Zhiguang' <zhiguang.liu@intel.com>; 'Bi, Dandan'
> <dandan.bi@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@intel.com>
> > 发送时间: 2022年11月9日 6:45
> > 收件人: devel@edk2.groups.io; robert@ami.com; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > 抄送: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> > <zhiguang.liu@intel.com>; Bi, Dandan <dandan.bi@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@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > Robert
> > Phelps via groups.io
> > > Sent: Tuesday, November 8, 2022 12:28 PM
> > > To: devel@edk2.groups.io
> > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> > > <zhiguang.liu@intel.com>; Bi, Dandan <dandan.bi@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@intel.com> [mdkinney]
> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> [lgao4]
> > > Cc: Zhiguang Liu <zhiguang.liu@intel.com> [LiuZhiguang001]
> > > Cc: Dandan Bi <dandan.bi@intel.com> [dandanbi]
> > > Signed-off-by: Robert Phelps <robert@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.
> 
> 
> 
> 




  reply	other threads:[~2022-12-21  1:16 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-08 20:28 [edk2][PATCH 0/2] Add Events introduced in UEFI 2.9 Robert Phelps
2022-11-08 20:28 ` [PATCH 1/2] MdePkg: Add New Event Definitions from " Robert Phelps
2022-11-08 20:28 ` [PATCH 2/2] MdeModulePkg: Added call to signal New Event Robert Phelps
2022-11-08 23:42   ` [edk2-devel] " Dionna Glaze
2022-11-09 15:49     ` Robert Phelps
2022-11-09 16:21       ` Dionna Glaze
2022-11-09 16:52         ` Michael D Kinney
2022-11-09 16:58           ` Dionna Glaze
2022-11-09 17:31             ` Robert Phelps
2022-11-09 18:01               ` Dionna Glaze
2022-11-08 22:44 ` [edk2][PATCH 0/2] Add Events introduced in UEFI 2.9 Michael D Kinney
2022-11-09 16:10   ` [edk2-devel] " Robert Phelps
2022-12-07  1:08   ` 回复: " gaoliming
2022-12-14 17:49     ` Robert Phelps
2022-12-21  1:16       ` gaoliming [this message]
2023-01-11 19:11         ` [edk2-devel] 回复: [edk2-devel] " Robert Phelps
2023-01-11 21:16           ` Michael D Kinney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='057a01d914d9$cc0775c0$64166140$@byosoft.com.cn' \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox