On October 14, 2021 1:38 PM, Gerd Hoffmann wrote:
defined in UefiCpuPkg. While BaseIoLibIntrinsicSev is in MdePkg.
Calling CPUID should not be needed, we have a new fancyThe gUefiCpuPkgTokenSpaceGuid.PcdConfidentialComputingGuestAttr is
ConfidentialComputing PCD for that now.
If the ConfidentialComputing PCD is used, then UefiCpuPkg has to be includedin BaseIoLibIntrinsicSev.inf.
I check all the *.inf under MdePkg but no one *.inf include UefiCpuPkg.Hmm, I guess we should move the pcd then so it cam be used more widely.
I am not sure if UefiCpuPkg can be included in BaseIoLibIntrinsicSev.inf.
Confidential computing has an impact beyond just cpu, it's also memory, io and
How about define ConfidentialComputingAttr PCD in MdePkg.dec?
Maybe that's something to cleanup for amd (Brijesh?) beforehand, so
the structure is there already and the tdx patches just need to add the "case
bits.Tdx patches can first use above structure. AMD can update it later. Either way
That'll work too, I don't care much about the ordering.
It will be updated in the next version.