From: "Michael D Kinney" <michael.d.kinney@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"dionnaglaze@google.com" <dionnaglaze@google.com>,
Robert Phelps <robert@ami.com>
Subject: Re: [edk2-devel] [PATCH 2/2] MdeModulePkg: Added call to signal New Event
Date: Wed, 9 Nov 2022 16:52:42 +0000 [thread overview]
Message-ID: <CO1PR11MB4929953F58B13807C6D81F31D23E9@CO1PR11MB4929.namprd11.prod.outlook.com> (raw)
In-Reply-To: <CAAH4kHZ_8G0RmVkxrtJeMrEuuM5WHHq=C2Z+uKhhi+d7ikm=MA@mail.gmail.com>
I am confused. I see patches related to ReadyToBoot and ExitBootServices and they mix what is in description and what is in code.
I recommend you coordinate and put together a clean set of patches for these topics.
Thanks,
Mike
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dionna Glaze via groups.io
> Sent: Wednesday, November 9, 2022 8:22 AM
> To: Robert Phelps <robert@ami.com>
> Cc: devel@edk2.groups.io
> Subject: Re: [edk2-devel] [PATCH 2/2] MdeModulePkg: Added call to signal New Event
>
> > So how should we want to handle this. Leave yours in or leave mine in.
>
> Given that I'm not particularly confident in how I've implemented the
> after_ready_to_boot spec, and you haven't implemented it, I'm not
> sure. I'm pursuing the before_exit_boot_services implementation to
> solve a problem in confidential compute as part of my job, so I think
> I have the motivation to keep pushing on this. Let's go with mine
> since it's further along. I'd appreciate a Reviewed-by if you agree
> with the implementations.
>
> --
> -Dionna Glaze, PhD (she/her)
>
>
>
>
next prev parent reply other threads:[~2022-11-09 16:52 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 [this message]
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 ` 回复: [edk2-devel] " gaoliming
2023-01-11 19:11 ` [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=CO1PR11MB4929953F58B13807C6D81F31D23E9@CO1PR11MB4929.namprd11.prod.outlook.com \
--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