[v6] CloudHv: Rely on PVH boot specification

[edk2-devel] [PATCH v6 0/8] CloudHv: Rely on PVH boot specification

Posted by Boeuf, Sebastien 2 years, 2 months ago

From: Sebastien Boeuf <sebastien.boeuf@intel.com>

Cloud Hypervisor aims at emulating the minimal amount of legacy devices
and this is why the PVH boot specification is supported. The point is to
be able to share some information with the guest without the need for
emulating devices that would be present on real hardware.

Since Cloud Hypervisor supports loading a PVH ELF binary, the CloudHv
target is updated to be generated as such. Relying on the PVH boot
specification, we don't need to hardcode the location of the ACPI tables
anymore since we can retrieve the RSDP address from the hvm_start_info
structure. Same thing for the RAM below 4G, we can find this information
through the PVH memmap entries rather than relying on the emulated CMOS.

Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>

Sebastien Boeuf (8):
  OvmfPkg: Make the Xen ELF header generator more flexible
  OvmfPkg: Xen: Use a new fdf include for the PVH ELF header
  OvmfPkg: Xen: Generate fdf include file from ELF header generator
  OvmfPkg: CloudHv: Remove VARS and CODE sections
  OvmfPkg: Generate CloudHv as a PVH ELF binary
  OvmfPkg: CloudHv: Retrieve RSDP address from PVH
  OvmfPkg: CloudHv: Rely on PVH memmap instead of CMOS
  OvmfPkg: CloudHv: Add README

 OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf |   2 +
 OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c       |  39 ++++--
 OvmfPkg/CloudHv/CloudHvElfHeader.fdf.inc    |  54 ++++++++
 OvmfPkg/CloudHv/CloudHvX64.dsc              |   2 +-
 OvmfPkg/CloudHv/CloudHvX64.fdf              |  40 ++----
 OvmfPkg/CloudHv/README                      |  67 ++++++++++
 OvmfPkg/Include/IndustryStandard/CloudHv.h  |   5 -
 OvmfPkg/OvmfXen.fdf                         |  57 +-------
 OvmfPkg/OvmfXenElfHeaderGenerator.c         | 141 +++++++++++++++-----
 OvmfPkg/PlatformPei/MemDetect.c             |  73 ++++++++++
 OvmfPkg/PlatformPei/PlatformPei.inf         |   2 +
 OvmfPkg/XenElfHeader.fdf.inc                |  42 ++++++
 12 files changed, 390 insertions(+), 134 deletions(-)
 create mode 100644 OvmfPkg/CloudHv/CloudHvElfHeader.fdf.inc
 create mode 100644 OvmfPkg/CloudHv/README
 create mode 100644 OvmfPkg/XenElfHeader.fdf.inc

-- 
2.32.0

---------------------------------------------------------------------
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 (#87224): https://edk2.groups.io/g/devel/message/87224
Mute This Topic: https://groups.io/mt/89500817/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v6 0/8] CloudHv: Rely on PVH boot specification

Posted by Yao, Jiewen 2 years, 2 months ago

Series Reviewed-by: Jiewen.Yao@intel.com.

> -----Original Message-----
> From: Boeuf, Sebastien <sebastien.boeuf@intel.com>
> Sent: Wednesday, March 2, 2022 9:31 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 v6 0/8] CloudHv: Rely on PVH boot specification
> 
> From: Sebastien Boeuf <sebastien.boeuf@intel.com>
> 
> Cloud Hypervisor aims at emulating the minimal amount of legacy devices
> and this is why the PVH boot specification is supported. The point is to
> be able to share some information with the guest without the need for
> emulating devices that would be present on real hardware.
> 
> Since Cloud Hypervisor supports loading a PVH ELF binary, the CloudHv
> target is updated to be generated as such. Relying on the PVH boot
> specification, we don't need to hardcode the location of the ACPI tables
> anymore since we can retrieve the RSDP address from the hvm_start_info
> structure. Same thing for the RAM below 4G, we can find this information
> through the PVH memmap entries rather than relying on the emulated CMOS.
> 
> Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
> 
> Sebastien Boeuf (8):
>   OvmfPkg: Make the Xen ELF header generator more flexible
>   OvmfPkg: Xen: Use a new fdf include for the PVH ELF header
>   OvmfPkg: Xen: Generate fdf include file from ELF header generator
>   OvmfPkg: CloudHv: Remove VARS and CODE sections
>   OvmfPkg: Generate CloudHv as a PVH ELF binary
>   OvmfPkg: CloudHv: Retrieve RSDP address from PVH
>   OvmfPkg: CloudHv: Rely on PVH memmap instead of CMOS
>   OvmfPkg: CloudHv: Add README
> 
>  OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf |   2 +
>  OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c       |  39 ++++--
>  OvmfPkg/CloudHv/CloudHvElfHeader.fdf.inc    |  54 ++++++++
>  OvmfPkg/CloudHv/CloudHvX64.dsc              |   2 +-
>  OvmfPkg/CloudHv/CloudHvX64.fdf              |  40 ++----
>  OvmfPkg/CloudHv/README                      |  67 ++++++++++
>  OvmfPkg/Include/IndustryStandard/CloudHv.h  |   5 -
>  OvmfPkg/OvmfXen.fdf                         |  57 +-------
>  OvmfPkg/OvmfXenElfHeaderGenerator.c         | 141 +++++++++++++++-----
>  OvmfPkg/PlatformPei/MemDetect.c             |  73 ++++++++++
>  OvmfPkg/PlatformPei/PlatformPei.inf         |   2 +
>  OvmfPkg/XenElfHeader.fdf.inc                |  42 ++++++
>  12 files changed, 390 insertions(+), 134 deletions(-)
>  create mode 100644 OvmfPkg/CloudHv/CloudHvElfHeader.fdf.inc
>  create mode 100644 OvmfPkg/CloudHv/README
>  create mode 100644 OvmfPkg/XenElfHeader.fdf.inc
> 
> --
> 2.32.0



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