[edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor

Boeuf, Sebastien posted 9 patches 2 years, 3 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
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
[edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor
Posted by Boeuf, Sebastien 2 years, 3 months ago
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.



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


Re: [edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor
Posted by Yao, Jiewen 2 years, 3 months ago
Thanks. For 1~2,4~9, Acked-by: Jiewen Yao <Jiewen.yao@intel.com>

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]
========

> -----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



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


Re: [edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor
Posted by Boeuf, Sebastien 2 years, 3 months ago
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.


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