Re: [edk2-devel] [PATCH v2 0/5] Enable Cloud Hypervisor support in edk2

["Laszlo Ersek" <lersek@redhat.com>]

On 05/17/21 08:50, Jianyong Wu wrote:
> Cloud Hypervisor is an open source Virtual Machine Monitor (VMM) that
> runs on top of KVM. Cloud Hypervisor is implemented in Rust and is based
> on the rust-vmm crates. See [1] to find more.
> 
> To support UEFI, Cloud Hypervisor is introduced here.
> There are three parts to be considered to do this enablements, that is:
>   1. memory initialization
> 
>   2. specific ACPI service implementation
>      compared with qemu, there is no device like Fw-cfg, so we has no
>      elegant way to get the RSDP address. A specific ACPI implementation is
>      introduced here.
> 
>   3. build configuration file
> 
> This enablement bases on the implentation for qemu so some code is
> borrowed.
> 
> [1] https://github.com/cloud-hypervisor/cloud-hypervisor
> 
> Jianyong Wu (5):
>   ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor
>   MdeMoudlePkg: introduce new PCD for Acpi/rsdp
>   ArmVirtPkg: enable ACPI for cloud hypervisor
>   ArmVirtPkg: Introduce Cloud Hypervisor to edk2 family
>   Maintainers: update Maintainers file as new files/folders created
> 
>  MdeModulePkg/MdeModulePkg.dec                 |   7 +
>  ArmVirtPkg/ArmVirtCloudHv.dsc                 | 455 ++++++++++++++++++
>  ArmVirtPkg/ArmVirtCloudHv.fdf                 | 292 +++++++++++
>  .../CloudHvAcpiPlatformDxe.inf                |  51 ++
>  .../CloudHvHasAcpiDtDxe.inf                   |  43 ++
>  .../CloudHvVirtMemInfoPeiLib.inf              |  47 ++
>  .../CloudHvAcpiPlatformDxe/CloudHvAcpi.c      |  73 +++
>  .../CloudHvHasAcpiDtDxe.c                     |  69 +++
>  .../CloudHvVirtMemInfoLib.c                   |  94 ++++
>  .../CloudHvVirtMemInfoPeiLibConstructor.c     | 100 ++++
>  ArmVirtPkg/ArmVirtCloudHvFvMain.fdf.inc       | 169 +++++++
>  Maintainers.txt                               |   7 +
>  12 files changed, 1407 insertions(+)
>  create mode 100644 ArmVirtPkg/ArmVirtCloudHv.dsc
>  create mode 100644 ArmVirtPkg/ArmVirtCloudHv.fdf
>  create mode 100644 ArmVirtPkg/CloudHvAcpiPlatformDxe/CloudHvAcpiPlatformDxe.inf
>  create mode 100644 ArmVirtPkg/CloudHvPlatformHasAcpiDtDxe/CloudHvHasAcpiDtDxe.inf
>  create mode 100644 ArmVirtPkg/Library/CloudHvVirtMemInfoLib/CloudHvVirtMemInfoPeiLib.inf
>  create mode 100644 ArmVirtPkg/CloudHvAcpiPlatformDxe/CloudHvAcpi.c
>  create mode 100644 ArmVirtPkg/CloudHvPlatformHasAcpiDtDxe/CloudHvHasAcpiDtDxe.c
>  create mode 100644 ArmVirtPkg/Library/CloudHvVirtMemInfoLib/CloudHvVirtMemInfoLib.c
>  create mode 100644 ArmVirtPkg/Library/CloudHvVirtMemInfoLib/CloudHvVirtMemInfoPeiLibConstructor.c
>  create mode 100644 ArmVirtPkg/ArmVirtCloudHvFvMain.fdf.inc
> 

Confirming that I got this in my review queue. I'll need some time to
get to it; please bear with me.

Thanks,
Laszlo