[edk2-devel] [PATCH v9 0/4] Add safe unaccepted memory behavior

[Yao, Jiewen]

hello
If this protocol is SEV specific, please also add prefix SEV_ or AMD_ for the protocol name.


________________________________
发件人: Ard Biesheuvel <ardb at kernel.org>
发送时间: Wednesday, January 25, 2023 10:52:42 PM
收件人: Gerd Hoffmann <kraxel at redhat.com>
抄送: devel at edk2.groups.io <devel at edk2.groups.io>; thomas.lendacky at amd.com <thomas.lendacky at amd.com>; Hansen, Dave <dave.hansen at intel.com>; dionnaglaze at google.com <dionnaglaze at google.com>; dave.hansen at linux.intel.com <dave.hansen at linux.intel.com>; Yao, Jiewen <jiewen.yao at intel.com>; Shutemov, Kirill <kirill.shutemov at intel.com>
主题: Re: [edk2-devel] [PATCH v9 0/4] Add safe unaccepted memory behavior

On Wed, 25 Jan 2023 at 13:10, Gerd Hoffmann <kraxel at redhat.com> wrote:
>
> On Wed, Jan 25, 2023 at 12:44:13PM +0100, Ard Biesheuvel wrote:
> > On Wed, 25 Jan 2023 at 10:18, Gerd Hoffmann <kraxel at redhat.com> wrote:
> > >
> > > On Wed, Jan 25, 2023 at 10:01:47AM +0100, Ard Biesheuvel wrote:
> > >
> > > > Exactly. And my Fedora kernel has those bits enabled by default.
> > > >
> > > > So I suppose the way forward here is to expose this protocol only on
> > > > OVMF builds that target SEV-SNP, instead of introducing it as a
> > > > generic CoCo feature.
> > >
> > > OVMF builds already adapt at runtime, so this needs to be a runtime
> > > check too.  But IIRC the module already checks whenever SNP-SEV or TDX
> > > is active before installing, so we only need to tweak that check to drop
> > > the TDX check.
> > >
> >
> > Sure. But Dionna's series introduces a new CocoDxe driver that is
> > intended to carry shared logic, but we should probably add this stuff
> > to AmdSevDxe instead.
>
> OvmfPkgX64.dsc can run in SEV/TDX modes too, so it is needed there as
> well.  We can probably leave it out from IntelTdx (assuming Intel's
> upstream merge plans work out as expected and tdx-guest + unaccepted
> memory actually land in the same kernel release).
>

OvmfPkgX64.dsc already incorporates AmdSevDxe, as do a few other
platforms, so adding it there is still fine afaict.


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


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230125/b68640e8/attachment-0001.htm>