[v5] Add Cloud Hypervisor support for x86

[edk2-devel] [PATCH v5 0/5] Add Cloud Hypervisor support for x86

Posted by sebastien.boeuf@intel.com 2 years, 4 months ago

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.



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84413): https://edk2.groups.io/g/devel/message/84413
Mute This Topic: https://groups.io/mt/87542821/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

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

Posted by Yao, Jiewen 2 years, 4 months ago

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



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84573): https://edk2.groups.io/g/devel/message/84573
Mute This Topic: https://groups.io/mt/87542821/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

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

Posted by Boeuf, Sebastien 2 years, 4 months ago

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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84633): https://edk2.groups.io/g/devel/message/84633
Mute This Topic: https://groups.io/mt/87542821/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

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

Posted by Yao, Jiewen 2 years, 4 months ago

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


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84636): https://edk2.groups.io/g/devel/message/84636
Mute This Topic: https://groups.io/mt/87542821/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

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

Posted by Boeuf, Sebastien 2 years, 4 months ago

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


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84637): https://edk2.groups.io/g/devel/message/84637
Mute This Topic: https://groups.io/mt/87542821/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-