On Fri, Sep 23, 2022 at 02:42:57PM -0500, Tom Lendacky wrote: > On 9/23/22 14:34, Dionna Amalie Glaze wrote: > > Ah yes, I did forget to include that patch. Will add to v2. I was just > > setting the ResourceType to unaccepted and skipping the Prevalidate call > > in PlatformPei if the start address is greater or equal to SIZE_4GB. > > That seemed more self-contained than messing with PlatformInitLib. Would > > you prefer that I add SevSnp logic to PlatformInitLib? > > No, if it works and is easier / more concise, then please keep it the way > you have it.
Well, moving over the SEV code over to PlatformInitLib makes sense. That should allow the AmdSev use pei-less boot, simliar to IntelTdx, and further down the road allow unify AmdSev + IntelTdx variants into a single firmware binary supporting both SevSnp and Tdx. I don't think that should be part of the "unaccepted memory" patch series though. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94341): https://edk2.groups.io/g/devel/message/94341 Mute This Topic: https://groups.io/mt/93857638/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
