Ah yes I'm sorry I forgot to update each patch with the uncrustify tool. I'm going to send a v7 as soon as the CI confirms the new series complies with the CI. Thanks, Sebastien ________________________________ From: Yao, Jiewen Sent: Friday, December 10, 2021 1:47 PM 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 Have you tried CI? https://github.com/tianocore/edk2/pull/2286 failed. From: Boeuf, Sebastien Sent: Friday, December 10, 2021 5:37 PM To: Yao, Jiewen ; devel@edk2.groups.io Cc: Justen, Jordan L ; kraxel@redhat.com Subject: Re: [PATCH v5 0/5] Add Cloud Hypervisor support for x86 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.