I've just rebased the PR and sent a v6 version. Thanks, Sebastien ________________________________ From: Yao, Jiewen Sent: Thursday, December 9, 2021 7:41 AM To: Boeuf, Sebastien ; devel@edk2.groups.io Cc: Justen, Jordan L ; kraxel@redhat.com Subject: RE: [PATCH v5 0/5] Add Cloud Hypervisor support for x86 Hi Sebatien I got some format error when I try to merge. Have you tried latest CI? > -----Original Message----- > From: Boeuf, Sebastien > Sent: Monday, December 6, 2021 11:59 PM > To: devel@edk2.groups.io > Cc: Yao, Jiewen ; Justen, Jordan L > ; kraxel@redhat.com; Boeuf, Sebastien > > Subject: [PATCH v5 0/5] Add Cloud Hypervisor support for x86 > > From: Sebastien Boeuf > > This series aims at adding the support for the Cloud Hypervisor platform > to the OVMF firmware for x86 architecture. > > The goal is to allow the same binary to be used either by QEMU or Cloud > Hypervisor, using the Cloud Hypervisor way as a fallback if the fw_cfg > mechanism is not present. > > Sebastien Boeuf (5): > OvmfPkg: Handle Cloud Hypervisor host bridge > OvmfPkg: Create global entry point for SMBIOS parsing > OvmfPkg: Retrieve SMBIOS from Cloud Hypervisor > OvmfPkg: Generalize AcpiPlatformDxe > OvmfPkg: Install ACPI tables for Cloud Hypervisor > > ArmVirtPkg/ArmVirtQemu.dsc | 2 +- > ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 2 +- > ArmVirtPkg/ArmVirtQemuKernel.dsc | 2 +- > ...QemuFwCfgAcpiPlatform.c => AcpiPlatform.c} | 15 ++- > OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h | 6 + > ...cpiPlatformDxe.inf => AcpiPlatformDxe.inf} | 6 +- > OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c | 117 ++++++++++++++++++ > OvmfPkg/AmdSev/AmdSevX64.dsc | 2 +- > OvmfPkg/AmdSev/AmdSevX64.fdf | 2 +- > OvmfPkg/Include/IndustryStandard/CloudHv.h | 45 +++++++ > OvmfPkg/Include/OvmfPlatforms.h | 1 + > .../Library/AcpiTimerLib/BaseAcpiTimerLib.c | 3 + > .../AcpiTimerLib/BaseRomAcpiTimerLib.c | 4 + > .../Library/AcpiTimerLib/DxeAcpiTimerLib.c | 3 + > .../PlatformBootManagerLib/BdsPlatform.c | 1 + > .../ResetSystemLib/BaseResetShutdown.c | 3 + > .../Library/ResetSystemLib/DxeResetShutdown.c | 12 +- > OvmfPkg/Microvm/MicrovmX64.dsc | 2 +- > OvmfPkg/Microvm/MicrovmX64.fdf | 2 +- > OvmfPkg/OvmfPkgIa32.dsc | 2 +- > OvmfPkg/OvmfPkgIa32.fdf | 2 +- > OvmfPkg/OvmfPkgIa32X64.dsc | 2 +- > OvmfPkg/OvmfPkgIa32X64.fdf | 2 +- > OvmfPkg/OvmfPkgX64.dsc | 2 +- > OvmfPkg/OvmfPkgX64.fdf | 2 +- > OvmfPkg/PlatformPei/MemDetect.c | 9 +- > OvmfPkg/PlatformPei/Platform.c | 6 + > OvmfPkg/SmbiosPlatformDxe/CloudHv.c | 32 +++++ > OvmfPkg/SmbiosPlatformDxe/EntryPoint.c | 52 ++++++++ > OvmfPkg/SmbiosPlatformDxe/Qemu.c | 35 ------ > OvmfPkg/SmbiosPlatformDxe/SmbiosPlatformDxe.h | 22 ++++ > .../SmbiosPlatformDxe/SmbiosPlatformDxe.inf | 3 + > 32 files changed, 345 insertions(+), 56 deletions(-) > rename OvmfPkg/AcpiPlatformDxe/{QemuFwCfgAcpiPlatform.c => > AcpiPlatform.c} (52%) > rename OvmfPkg/AcpiPlatformDxe/{QemuFwCfgAcpiPlatformDxe.inf => > AcpiPlatformDxe.inf} (87%) > create mode 100644 OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c > create mode 100644 OvmfPkg/Include/IndustryStandard/CloudHv.h > create mode 100644 OvmfPkg/SmbiosPlatformDxe/CloudHv.c > create mode 100644 OvmfPkg/SmbiosPlatformDxe/EntryPoint.c > > -- > 2.30.2 --------------------------------------------------------------------- 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: 4,572,000 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.