I am looking at the issue and will provide the fix soon. BTW, the previous patch-set passed the EDK2 CI. It seems there is something missed in the EDK2 CI.
Thanks Min > -----Original Message----- > From: Ard Biesheuvel <a...@kernel.org> > Sent: Thursday, December 22, 2022 7:05 PM > To: devel@edk2.groups.io; Boeuf, Sebastien <sebastien.bo...@intel.com> > Cc: Yao, Jiewen <jiewen....@intel.com>; Xu, Min M <min.m...@intel.com>; > kra...@redhat.com > Subject: Re: [edk2-devel] [PATCH v2 0/3] OvmfPkg: Make IntelTdx work with > Cloud Hypervisor > > This series has broken all platforms that incorporate > OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf but do not provide a > resolution for CcProbeLib > > Please provide a fix > > https://ci.linaro.org/job/leg-virt-tianocore-edk2-upstream/4748/console > > > > On Fri, 16 Dec 2022 at 09:46, Boeuf, Sebastien <sebastien.bo...@intel.com> > wrote: > > > > Thank you Jiewen :) > > ________________________________ > > From: Yao, Jiewen <jiewen....@intel.com> > > Sent: Friday, December 16, 2022 4:03:00 AM > > To: Boeuf, Sebastien <sebastien.bo...@intel.com>; devel@edk2.groups.io > > <devel@edk2.groups.io> > > Cc: Xu, Min M <min.m...@intel.com>; kra...@redhat.com > > <kra...@redhat.com> > > Subject: RE: [PATCH v2 0/3] OvmfPkg: Make IntelTdx work with Cloud > > Hypervisor > > > > Merged: https://github.com/tianocore/edk2/pull/3778 > > > > > -----Original Message----- > > > From: Boeuf, Sebastien <sebastien.bo...@intel.com> > > > Sent: Thursday, December 15, 2022 11:10 PM > > > To: devel@edk2.groups.io > > > Cc: Yao, Jiewen <jiewen....@intel.com>; Xu, Min M > > > <min.m...@intel.com>; kra...@redhat.com; Boeuf, Sebastien > > > <sebastien.bo...@intel.com> > > > Subject: [PATCH v2 0/3] OvmfPkg: Make IntelTdx work with Cloud > > > Hypervisor > > > > > > From: Sebastien Boeuf <sebastien.bo...@intel.com> > > > > > > The IntelTdxX64 OVMF target wasn't working with Cloud Hypervisor on > > > TDX platform. This was due to the way the OVMF code expects Cloud > > > Hypervisor to rely on PVH to retrieve information like memory below > > > 4GiB as well as the ACPI tables. > > > > > > This is why this series takes care of identifying when running on > > > TDX in order to handle things differently. For the memory below > > > 4GiB, it falls back onto the CMOS to retrieve the correct > > > information, and for the ACPI tables, it relies on the HOB to obtain > > > every table individually before to expose them to the guest OS. > > > > > > With these two use cases properly handled by this series, it is now > > > possible to use the IntelTdxX64 target to build an OVMF binary that > > > works both for QEMU and Cloud Hypervisor on a TDX platform. > > > > > > Sebastien Boeuf (3): > > > OvmfPkg/PlatformInitLib: Differentiate TDX case for Cloud Hypervisor > > > OvmfPkg/PlatformInitLib: Transfer GUID Extension HOB > > > OvmfPkg/AcpiPlatformDxe: Differentiate TDX case for Cloud > > > Hypervisor > > > > > > ArmVirtPkg/ArmVirtQemu.dsc | 1 + > > > OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c | 8 +- > > > OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h | 6 ++ > > > OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf | 3 + > > > OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c | 87 > +++++++++++++++++++++ > > > OvmfPkg/Library/PlatformInitLib/IntelTdx.c | 5 ++ > > > OvmfPkg/Library/PlatformInitLib/MemDetect.c | 5 +- > > > OvmfPkg/OvmfPkg.dec | 1 + > > > 8 files changed, 114 insertions(+), 2 deletions(-) > > > > > > -- > > > 2.34.1 > > > > --------------------------------------------------------------------- > > Intel Corporation SAS (French simplified joint stock company) > > Registered headquarters: "Les Montalets"- 2, rue de Paris, > > 92196 Meudon Cedex, France > > Registration Number: 302 456 199 R.C.S. NANTERRE > > Capital: 5 208 026.16 Euros > > > > This e-mail and any attachments may contain confidential material for > > the sole use of the intended recipient(s). Any review or distribution > > by others is strictly prohibited. If you are not the intended > > recipient, please contact the sender and delete all copies. > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#97719): https://edk2.groups.io/g/devel/message/97719 Mute This Topic: https://groups.io/mt/95690234/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
