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

["Yao, Jiewen" <jiewen.yao@intel.com>]

Pushed 26af69d..dd3b1f2
https://github.com/tianocore/edk2/pull/2584

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao, Jiewen
> Sent: Friday, March 4, 2022 1:55 PM
> To: Boeuf, Sebastien <sebastien.boeuf@intel.com>; devel@edk2.groups.io
> Cc: Justen, Jordan L <jordan.l.justen@intel.com>; kraxel@redhat.com
> Subject: Re: [edk2-devel] [PATCH v6 0/8] CloudHv: Rely on PVH boot
> specification
> 
> 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
> 
> 
> 
> 
>