Re: [edk2-devel] [PATCH 03/16] MdePkg/BaseLib: Add a new VMGEXIT instruction invocation for SVSM

["Gerd Hoffmann" <kraxel@redhat.com>]

On Wed, Jan 31, 2024 at 12:30:35PM -0600, Tom Lendacky wrote:
> On 1/30/24 05:51, Gerd Hoffmann wrote:
> > 
> > See also patch #13 sub-thread.  Maybe just move the 32-bit build to the
> > Null versions of the Libraries.
> > 
> > SEV-SNP + SVSM apparently is 64-bit only already (there is no 32-bit
> > calling convention).
> > 
> > Over the last year Intel landed full 64-bit PEI support in edk2, which
> > means the pure 64-bit build (OvmfPkgX64.dsc) has feature parity with
> > the 32-bit PEI / 64-bit DXE build (OvmfPkgIa32X64.dsc).
> > 
> > So maybe we can simply leave 32-bit support behind and remove SEV
> > support from OvmfPkgIa32X64.dsc and OvmfPkgIa32.dsc builds ...
> 
> That would break anyone that is currently using OvmfPkgIa32X64.dsc for base
> SEV (not ES or SNP), but I'm not sure if that is a real concern. For
> OvmfPkgIa32.dsc, no version of SEV will work, so that is doable.
> 
> Can we look at doing that cleanup after this series?

I thought maybe removing 32-bit support first is easier, because you
don't have to add 32-bit dummy code then.  I'd leave that do you do
decide though, doing the 32-bit cleanup as separate series is fine
with me too.

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114937): https://edk2.groups.io/g/devel/message/114937
Mute This Topic: https://groups.io/mt/103986440/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-