Re: [PATCH 0/9] Create new target for Cloud Hypervisor

["Boeuf, Sebastien" <sebastien.boeuf@intel.com>]

On Mon, 2022-01-10 at 14:49 +0000, Yao, Jiewen wrote:
> Thanks. For 1~2,4~9, Acked-by: Jiewen Yao <Jiewen.yao@intel.com>

Thanks for the quick review :)

> 
> Please also update   
> https://github.com/tianocore/edk2/blob/master/Maintainers.txt
> to add a CloudHv section with you as Reviewer. Similar to below
> ========
> OvmfPkg: microvm-related modules
> F: OvmfPkg/Microvm/
> F: OvmfPkg/Include/IndustryStandard/Microvm.h
> F: OvmfPkg/Library/ResetSystemLib/*Microvm.*
> R: Gerd Hoffmann <kraxel@redhat.com> [kraxel]
> ========

Oh yes I'll add this!

Thanks,
Sebastien

> 
> > -----Original Message-----
> > From: Boeuf, Sebastien <sebastien.boeuf@intel.com>
> > Sent: Monday, January 10, 2022 10:26 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 0/9] Create new target for Cloud Hypervisor
> > 
> > From: Sebastien Boeuf <sebastien.boeuf@intel.com>
> > 
> > Since Cloud Hypervisor and QEMU pc/q35 are quite different, it
> > makes
> > more sense to create a dedicated OVMF target for Cloud Hypervisor
> > rather
> > than trying to support both VMMs from the same OvmfPkgX64 target.
> > 
> > That's the reason why this series introduces a new target called
> > CloudHvX64, meant to be used with the Cloud Hypervisor VMM only.
> > 
> > The new target is initially copied over from the OvmfPkgX64, then
> > it is
> > trimmed down by removing what is not needed from a Cloud Hypervisor
> > perspective.
> > 
> > Sebastien Boeuf (9):
> >   OvmfPkg/CloudHv: Add new target for Cloud Hypervisor
> >   OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer
> >   OvmfPkg/CloudHv: Connect serial console
> >   OvmfPkg/CloudHv: Remove legacy 8259 PIC support
> >   OvmfPkg/CloudHv: Remove Q35 specifics
> >   OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg
> >   OvmfPkg/CloudHv: Remove video support
> >   OvmfPkg/CloudHv: Remove USB support
> >   OvmfPkg/CloudHv: Remove CSM support
> > 
> >  OvmfPkg/CloudHv/CloudHvX64.dsc                | 937
> > ++++++++++++++++++
> >  OvmfPkg/CloudHv/CloudHvX64.fdf                | 503 ++++++++++
> >  .../PlatformBootManagerLib/BdsPlatform.c      |   2 +-
> >  3 files changed, 1441 insertions(+), 1 deletion(-)
> >  create mode 100644 OvmfPkg/CloudHv/CloudHvX64.dsc
> >  create mode 100644 OvmfPkg/CloudHv/CloudHvX64.fdf
> > 
> > --
> > 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.