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 <jiewen.yao@intel.com>
Sent: Friday, December 10, 2021 1:47 PM
To: Boeuf, Sebastien <sebastien.boeuf@intel.com>; devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Justen, Jordan L <jordan.l.justen@intel.com>; kraxel@redhat.com <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 <sebastien.boeuf@intel.com>
Sent: Friday, December 10, 2021 5:37 PM
To: Yao, Jiewen <jiewen.yao@intel.com>; devel@edk2.groups.io
Cc: Justen, Jordan L <jordan.l.justen@intel.com>; 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 <jiewen.yao@intel.com>
Sent: Thursday, December 9, 2021 7:41 AM
To: Boeuf, Sebastien <sebastien.boeuf@intel.com>; devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Justen, Jordan L <jordan.l.justen@intel.com>; kraxel@redhat.com <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 <sebastien.boeuf@intel.com>
> Sent: Monday, December 6, 2021 11:59 PM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Justen, Jordan L
> <jordan.l.justen@intel.com>; kraxel@redhat.com; Boeuf, Sebastien
> <sebastien.boeuf@intel.com>
> Subject: [PATCH v5 0/5] Add Cloud Hypervisor support for x86
>
> From: Sebastien Boeuf <sebastien.boeuf@intel.com>
>
> 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.