Re: [PATCH v5 0/5] Add Cloud Hypervisor support for x86

[sebastien.boeuf@intel.com]

[-- Attachment #1: Type: text/plain, Size: 4558 bytes --]

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.

[-- Attachment #2: Type: text/html, Size: 8416 bytes --]