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.
>
>
>
>
next prev parent 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