Re: [edk2-devel] [PATCH v2] OvmfPkg: Update I/O port related to ACPI devices for CloudHv

2022-09-07 Thread Boeuf, Sebastien
fo/PciSegmentInfoLibAcpiBoa > rdInfo.c#L55 >   >   > Thank you > Yao Jiewen >   >   > From: Boeuf, Sebastien > Sent: Tuesday, September 6, 2022 11:41 PM > To: Yao, Jiewen > Cc: Justen, Jordan L ; kra...@redhat.com; > devel@edk2.groups.io > Subject: Re: [PATCH v

Re: [edk2-devel] [PATCH v2] OvmfPkg: Update I/O port related to ACPI devices for CloudHv

2022-09-07 Thread Boeuf, Sebastien
Address. (from > > MCFG).https://github.com/tianocore/edk2/blob/master/UefiPayloadPk > > g/Library/PciSegmentInfoLibAcpiBoardInfo/PciSegmentInfoLibAcpiBoa > > rdInfo.c#L55 > >   > >   > > Thank you > > Yao Jiewen > >   > >   > > From: Boeuf,

Re: [edk2-devel] [PATCH v2] OvmfPkg: Update I/O port related to ACPI devices for CloudHv

2022-09-07 Thread Boeuf, Sebastien
Ok sounds good. Thanks, Sebastien From: Yao, Jiewen Sent: Wednesday, September 7, 2022 5:23 PM To: Boeuf, Sebastien Cc: kra...@redhat.com ; Justen, Jordan L ; devel@edk2.groups.io Subject: RE: [PATCH v2] OvmfPkg: Update I/O port related to ACPI devices for

Re: [edk2-devel] [PATCH v5 0/5] Add Cloud Hypervisor support for x86

2021-12-10 Thread Boeuf, Sebastien
I've just rebased the PR and sent a v6 version. Thanks, Sebastien From: Yao, Jiewen Sent: Thursday, December 9, 2021 7:41 AM To: Boeuf, Sebastien ; devel@edk2.groups.io Cc: Justen, Jordan L ; kra...@redhat.com Subject: RE: [PATCH v5 0/5] Add Cloud Hyper

Re: [edk2-devel] [PATCH v5 0/5] Add Cloud Hypervisor support for x86

2021-12-10 Thread Boeuf, Sebastien
Ah yes I'm sorry I forgot to update each patch with the uncrustify tool. I'm going to send a v7 as soon as the CI confirms the new series complies with the CI. Thanks, Sebastien From: Yao, Jiewen Sent: Friday, December 10, 2021 1:47 PM To: Boeuf,

[edk2-devel] [PATCH v7 0/5] Add Cloud Hypervisor support for x86

2021-12-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf This series aims at adding the support for the Cloud Hypervisor platform to the OVMF firmware for x86 architecture. The goal is to allow the same binary to be used either by QEMU or Cloud Hypervisor, using the Cloud Hypervisor way as a fallback if the fw_cfg mechanism is no

[edk2-devel] [PATCH v7 1/5] OvmfPkg: Handle Cloud Hypervisor host bridge

2021-12-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Handle things differently when the detected host bridge matches the Cloud Hypervisor PCI host bridge identifier. Reviewed-by: Gerd Hoffmann Reviewed-by: Jiewen Yao Signed-off-by: Rob Bradford Signed-off-by: Sebastien Boeuf --- OvmfPkg/Include/IndustryStandard/CloudHv.h

[edk2-devel] [PATCH v7 2/5] OvmfPkg: Create global entry point for SMBIOS parsing

2021-12-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Move the generic entry point part out of Qemu.c to anticipate the addition of new ways of retrieving the SMBIOS table. Reviewed-by: Gerd Hoffmann Reviewed-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/SmbiosPlatformDxe/EntryPoint.c| 42 ++

[edk2-devel] [PATCH v7 3/5] OvmfPkg: Retrieve SMBIOS from Cloud Hypervisor

2021-12-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Add a fallback on the SMBIOS code to find the SMBIOS table for Cloud Hypervisor if it couldn't be found for Qemu through fw_cfg. Reviewed-by: Gerd Hoffmann Reviewed-by: Jiewen Yao Signed-off-by: Rob Bradford Signed-off-by: Sebastien Boeuf --- OvmfPkg/Include/IndustrySt

[edk2-devel] [PATCH v7 5/5] OvmfPkg: Install ACPI tables for Cloud Hypervisor

2021-12-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding support for retrieving the Cloud Hypervisor ACPI tables as a fallback mechanism if tables are not found through fw_cfg. Reviewed-by: Gerd Hoffmann Reviewed-by: Jiewen Yao Signed-off-by: Rob Bradford Signed-off-by: Sebastien Boeuf --- OvmfPkg/AcpiPlatformDxe/Acpi

[edk2-devel] [PATCH v7 4/5] OvmfPkg: Generalize AcpiPlatformDxe

2021-12-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Don't make the package Qemu centric so that we can introduce some alternative support for other VMMs not using the fw_cfg mechanism. This patch is purely about renaming existing files with no functional change. Reviewed-by: Gerd Hoffmann Reviewed-by: Jiewen Yao Signed-of

[edk2-devel] CMOS needed for Cloud Hypervisor

2021-12-15 Thread Boeuf, Sebastien
Hi Gerd, Your patch 41d8bb30386ceab55787fc9f5aac6434e2493e27 removing the CMOS support for getting high mem and low mem is breaking the OVMF support for Cloud Hypervisor as we are still providing this information through that mechanism. Do you think it would be acceptable to revert it in order t

Re: [edk2-devel] CMOS needed for Cloud Hypervisor

2021-12-16 Thread Boeuf, Sebastien
Thank you! I really appreciate your responsiveness! Sebastien From: Yao, Jiewen Sent: Wednesday, December 15, 2021 6:14:07 PM To: devel@edk2.groups.io ; a...@kernel.org ; Boeuf, Sebastien Cc: kra...@redhat.com Subject: RE: [edk2-devel] CMOS needed for Cloud

[edk2-devel] Guidance about CI

2022-01-04 Thread Boeuf, Sebastien
Hi folks, I was wondering if you would be okay with me adding Cloud Hypervisor to the EDK2 CI. The idea would be to run a quick/simple test that Cloud Hypervisor can properly boot with the OVMF binary built from source on every pull request. And if you think that makes sense, any guidance on ho

Re: [edk2-devel] Guidance about CI

2022-01-05 Thread Boeuf, Sebastien
Wednesday, January 5, 2022 12:40 AM To: devel@edk2.groups.io ; Boeuf, Sebastien ; Yao, Jiewen ; kra...@redhat.com Subject: Re: [edk2-devel] Guidance about CI can cloud hypervisor boot on any of the free CI providers? If you look at ArmVirt, Ovmf, and even the emulatorpkg those all do similar thing

Re: [edk2-devel] Guidance about CI

2022-01-05 Thread Boeuf, Sebastien
lf of Boeuf, Sebastien Sent: Wednesday, January 5, 2022 2:32 PM To: devel@edk2.groups.io ; Yao, Jiewen ; kra...@redhat.com ; spbro...@outlook.com Subject: Re: [edk2-devel] Guidance about CI Hi Sean, Cloud Hypervisor can boot on Microsoft Azure VMs as this is what our project relies on to val

[edk2-devel] EFI shell with microvm

2022-01-06 Thread Boeuf, Sebastien
Hi Gerd, I was looking at a way to add support for EFI shell interaction with Cloud Hypervisor when I realized you added the support for microvm with commit 55f47d22998. I have been able to hack OvmfPkgX64 similarly to get it to work, but here are two follow up questions: 1. Is it possible to s

Re: [edk2-devel] EFI shell with microvm

2022-01-06 Thread Boeuf, Sebastien
On Thu, 2022-01-06 at 13:44 +0100, kra...@redhat.com wrote: > On Thu, Jan 06, 2022 at 11:25:37AM +0000, Boeuf, Sebastien wrote: > > Hi Gerd, > > > > I was looking at a way to add support for EFI shell interaction > > with Cloud Hypervisor when > > I realized you a

Re: [edk2-devel] Guidance about CI

2022-01-06 Thread Boeuf, Sebastien
On Wed, 2022-01-05 at 17:55 +0100, kra...@redhat.com wrote: > On Wed, Jan 05, 2022 at 01:44:01PM +0000, Boeuf, Sebastien wrote: > > Ah nevermind I found out QEMU was installed from packaging. > > On ubuntu. > > > We don't have packages for Cloud Hypervisor, but

Re: [edk2-devel] EFI shell with microvm

2022-01-07 Thread Boeuf, Sebastien
On Fri, 2022-01-07 at 13:07 +0100, kra...@redhat.com wrote: >   Hi, > > > > microvm has no lpc bridge, so I had to do it in a different way > > > ... > > > > Cloud Hypervisor doesn't emulate any LPC bridge or ISA bus. > > Ok, doing it microvm-style makes sense then. Ok thanks for the confirmati

Re: [edk2-devel] Guidance about CI

2022-01-07 Thread Boeuf, Sebastien
On Fri, 2022-01-07 at 13:17 +0100, kra...@redhat.com wrote: > On Thu, Jan 06, 2022 at 02:21:59PM +0000, Boeuf, Sebastien wrote: > > On Wed, 2022-01-05 at 17:55 +0100, kra...@redhat.com wrote: > > > On Wed, Jan 05, 2022 at 01:44:01PM +, Boeuf, Sebastien wrote: > > >

Re: [edk2-devel] EFI shell with microvm

2022-01-07 Thread Boeuf, Sebastien
I finally got it to work by using the XenTimerDxe. The reason why it works is because the UEFI service SetTimer() needs one Timer Architectural Protocol to be installed. In case of standard QEMU, the 8254 PIT is the one registering the service, while for microvm the Xen timer does the job. It's in

Re: [edk2-devel] EFI shell with microvm

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 00:16 +, Xu, Min M wrote: > On January 7, 2022 9:37 PM, Gerd Hoffmann wrote: > > > > > > tianocore doesn't use interrupts (other than timer). > > > > > > Yes I realized that by diving into the code. I can see microvm > > > using > > > the Xen timer while OvmfPkgX64 uses

[edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
Hi all, So far I've been able to patch the OvmfPkgX64 target to make it work for both QEMU and Cloud Hypervisor, but as I try to enable more features (EFI shell for instance) the gap is getting bigger and harder to keep them working together. That's why I'm thinking about creating an OvmfCh targe

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
g\Microvm\MicrovmX64.fdf > > And we will have OvmfPkg\IntelTdx\IntelTdxX64.fdf soon. > > I think it is OK to create: > OvmfPkg\CloudHv\CloudHvX64.fdf Sounds good, I'll start working on this. > > Thank you > Yao Jiewen > > > -Original Message- > >

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 11:45 +0100, kra...@redhat.com wrote: > On Mon, Jan 10, 2022 at 09:13:44AM +0000, Boeuf, Sebastien wrote: > > Hi all, > > > > So far I've been able to patch the OvmfPkgX64 target to make it > > work for both > > QEMU and Cloud Hypervisor

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

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf 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 C

[edk2-devel] [PATCH 1/9] OvmfPkg/CloudHv: Add new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the new target CloudHvX64, copied directly from OvmfPkgX64. The point is to create a target dedicated for Cloud Hypervisor rather than trying to support both QEMU and Cloud Hypervisor on the same target. Improvements and cleanups will be performed in follow up patche

[edk2-devel] [PATCH 2/9] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why instead of relying on it as the timer UEFI services, rely on the XenTimerDxe implementation. This is not Xen specific, as it simply uses the local APIC timer triggering interrupts on the vector 32. Signed-of

[edk2-devel] [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any LPC bridge, therefore we simply need to rely on the serial I/O port to be connected as a console. It reuses the code from Xen since it's very generic. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH 4/9] OvmfPkg/CloudHv: Remove legacy 8259 PIC support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor does not emulate any 8259 PIC, therefore there's no reason to load the corresponding driver for it. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 4 OvmfPkg/CloudHv/CloudHvX64.fdf | 1 - 2 files changed, 5 deletions(-) diff --g

[edk2-devel] [PATCH 5/9] OvmfPkg/CloudHv: Remove Q35 specifics

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Anything specific to the QEMU Q35 platform is not relevant for the CloudHv target. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 10 -- 1 file changed, 10 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc b/OvmfPkg/CloudHv/CloudHvX64.

[edk2-devel] [PATCH 6/9] OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor doesn't rely on the FwCfg mechanism, remove the libraries imports when possible. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 -- 1 file changed, 2 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc b/OvmfPkg/CloudHv/

[edk2-devel] [PATCH 7/9] OvmfPkg/CloudHv: Remove video support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf No need for video or virtio-gpu support since Cloud Hypervisor doesn't emulate any of these. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 16 OvmfPkg/CloudHv/CloudHvX64.fdf | 4 2 files changed, 20 deletions(-) diff --git a/O

[edk2-devel] [PATCH 8/9] OvmfPkg/CloudHv: Remove USB support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any USB controller or device, therefore the support can be removed. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 11 --- OvmfPkg/CloudHv/CloudHvX64.fdf | 10 -- 2 files changed, 21 deletions(-) diff

[edk2-devel] [PATCH 9/9] OvmfPkg/CloudHv: Remove CSM support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't need the support for legacy BIOS, therefore the CSM support can be removed. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 20 OvmfPkg/CloudHv/CloudHvX64.fdf | 6 -- 2 files changed, 26 deletions(-)

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

2022-01-10 Thread Boeuf, Sebastien
imilar to below > > OvmfPkg: microvm-related modules > F: OvmfPkg/Microvm/ > F: OvmfPkg/Include/IndustryStandard/Microvm.h > F: OvmfPkg/Library/ResetSystemLib/*Microvm.* > R: Gerd Hoffmann [kraxel] > Oh yes I'll add this! Thanks, Sebastien > > > ---

Re: [edk2-devel] [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Boeuf, Sebastien
e that's what you suggested :) Thanks, Sebastien > > Thank you > Yao Jiewen > > > -Original Message- > > From: Boeuf, Sebastien > > Sent: Monday, January 10, 2022 10:26 PM > > To: devel@edk2.groups.io > > Cc: Yao, Jiewen ; Jus

[edk2-devel] [PATCH v2 00/10] Create new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf 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 C

[edk2-devel] [PATCH v2 01/10] OvmfPkg/CloudHv: Add new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the new target CloudHvX64, copied directly from OvmfPkgX64. The point is to create a target dedicated for Cloud Hypervisor rather than trying to support both QEMU and Cloud Hypervisor on the same target. Improvements and cleanups will be performed in follow up patche

[edk2-devel] [PATCH v2 02/10] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why instead of relying on it as the timer UEFI services, rely on the XenTimerDxe implementation. This is not Xen specific, as it simply uses the local APIC timer triggering interrupts on the vector 32. Acked-by:

[edk2-devel] [PATCH v2 03/10] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any LPC bridge, therefore we simply need to rely on the serial I/O port to be connected as a console. It reuses the code from Xen since it's very generic. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH v2 04/10] OvmfPkg/CloudHv: Remove legacy 8259 PIC support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor does not emulate any 8259 PIC, therefore there's no reason to load the corresponding driver for it. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 4 OvmfPkg/CloudHv/CloudHvX64.fdf | 1 - 2 files changed, 5

[edk2-devel] [PATCH v2 05/10] OvmfPkg/CloudHv: Remove Q35 specifics

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Anything specific to the QEMU Q35 platform is not relevant for the CloudHv target. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 10 -- 1 file changed, 10 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc b/OvmfP

[edk2-devel] [PATCH v2 06/10] OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor doesn't rely on the FwCfg mechanism, remove the libraries imports when possible. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 -- 1 file changed, 2 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.

[edk2-devel] [PATCH v2 07/10] OvmfPkg/CloudHv: Remove video support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf No need for video or virtio-gpu support since Cloud Hypervisor doesn't emulate any of these. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 16 OvmfPkg/CloudHv/CloudHvX64.fdf | 4 2 files changed, 20 deleti

[edk2-devel] [PATCH v2 08/10] OvmfPkg/CloudHv: Remove USB support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any USB controller or device, therefore the support can be removed. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 11 --- OvmfPkg/CloudHv/CloudHvX64.fdf | 10 -- 2 files changed,

[edk2-devel] [PATCH v2 09/10] OvmfPkg/CloudHv: Remove CSM support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't need the support for legacy BIOS, therefore the CSM support can be removed. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 20 OvmfPkg/CloudHv/CloudHvX64.fdf | 6 -- 2 files chan

[edk2-devel] [PATCH v2 10/10] OvmfPkg/CloudHv: add Maintainers.txt entry

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Signed-off-by: Sebastien Boeuf --- Maintainers.txt | 5 + 1 file changed, 5 insertions(+) diff --git a/Maintainers.txt b/Maintainers.txt index 71c42bddae..3bda97ef25 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -440,6 +440,11 @@ F: OvmfPkg/Library/ResetSyste

[edk2-devel] [PATCH v3 01/11] OvmfPkg/CloudHv: Add new target for Cloud Hypervisor

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the new target CloudHvX64, copied directly from OvmfPkgX64. The point is to create a target dedicated for Cloud Hypervisor rather than trying to support both QEMU and Cloud Hypervisor on the same target. Improvements and cleanups will be performed in follow up patche

[edk2-devel] [PATCH v3 02/11] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why instead of relying on it as the timer UEFI services, rely on the XenTimerDxe implementation. This is not Xen specific, as it simply uses the local APIC timer triggering interrupts on the vector 32. Acked-by:

[edk2-devel] [PATCH v3 00/11] Create new target for Cloud Hypervisor

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf 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 C

[edk2-devel] [PATCH v3 04/11] OvmfPkg/CloudHv: Remove legacy 8259 PIC support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor does not emulate any 8259 PIC, therefore there's no reason to load the corresponding driver for it. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 4 OvmfPkg/CloudHv/CloudHvX64.fdf | 1 - 2 files changed, 5

[edk2-devel] [PATCH v3 06/11] OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor doesn't rely on the FwCfg mechanism, remove the libraries imports when possible. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 -- 1 file changed, 2 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.

[edk2-devel] [PATCH v3 05/11] OvmfPkg/CloudHv: Remove Q35 specifics

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Anything specific to the QEMU Q35 platform is not relevant for the CloudHv target. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 10 -- 1 file changed, 10 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc b/OvmfP

[edk2-devel] [PATCH v3 03/11] OvmfPkg/CloudHv: Connect serial console

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any LPC bridge, therefore we simply need to rely on the serial I/O port to be connected as a console. It reuses the code from Xen since it's very generic. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH v3 07/11] OvmfPkg/CloudHv: Remove video support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf No need for video or virtio-gpu support since Cloud Hypervisor doesn't emulate any of these. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 16 OvmfPkg/CloudHv/CloudHvX64.fdf | 4 2 files changed, 20 deleti

[edk2-devel] [PATCH v3 08/11] OvmfPkg/CloudHv: Remove USB support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any USB controller or device, therefore the support can be removed. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 11 --- OvmfPkg/CloudHv/CloudHvX64.fdf | 10 -- 2 files changed,

[edk2-devel] [PATCH v3 09/11] OvmfPkg/CloudHv: Remove CSM support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't need the support for legacy BIOS, therefore the CSM support can be removed. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 20 OvmfPkg/CloudHv/CloudHvX64.fdf | 6 -- 2 files chan

[edk2-devel] [PATCH v3 10/11] OvmfPkg/CloudHv: add Maintainers.txt entry

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Signed-off-by: Sebastien Boeuf --- Maintainers.txt | 5 + 1 file changed, 5 insertions(+) diff --git a/Maintainers.txt b/Maintainers.txt index 71c42bddae..3bda97ef25 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -440,6 +440,11 @@ F: OvmfPkg/Library/ResetSyste

[edk2-devel] [PATCH v3 11/11] OvmfPkg: Add CloudHvX64 to the CI

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the newly created target for Cloud Hypervisor to the CI, validating it can be properly built. Signed-off-by: Sebastien Boeuf --- .../.azurepipelines/Ubuntu-GCC5.yml | 9 + OvmfPkg/PlatformCI/CloudHvBuild.py| 37 +++ 2 file

[edk2-devel] [PATCH v4 00/11] Create new target for Cloud Hypervisor

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf 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 C

[edk2-devel] [PATCH v4 02/11] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why instead of relying on it as the timer UEFI services, rely on the XenTimerDxe implementation. This is not Xen specific, as it simply uses the local APIC timer triggering interrupts on the vector 32. Acked-by:

[edk2-devel] [PATCH v4 01/11] OvmfPkg/CloudHv: Add new target for Cloud Hypervisor

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the new target CloudHvX64, copied directly from OvmfPkgX64. The point is to create a target dedicated for Cloud Hypervisor rather than trying to support both QEMU and Cloud Hypervisor on the same target. Improvements and cleanups will be performed in follow up patche

[edk2-devel] [PATCH v4 03/11] OvmfPkg/CloudHv: Connect serial console

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any LPC bridge, therefore we simply need to rely on the serial I/O port to be connected as a console. It reuses the code from Xen since it's very generic. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- Ov

[edk2-devel] [PATCH v4 04/11] OvmfPkg/CloudHv: Remove legacy 8259 PIC support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor does not emulate any 8259 PIC, therefore there's no reason to load the corresponding driver for it. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 4 OvmfPkg/CloudHv/CloudHvX64.fdf |

[edk2-devel] [PATCH v4 05/11] OvmfPkg/CloudHv: Remove Q35 specifics

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Anything specific to the QEMU Q35 platform is not relevant for the CloudHv target. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 10 -- 1 file changed, 10 deletions(-) diff --git a/OvmfPkg/Cloud

[edk2-devel] [PATCH v4 06/11] OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor doesn't rely on the FwCfg mechanism, remove the libraries imports when possible. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 -- 1 file changed, 2 deletions(-) diff --git a/Ov

[edk2-devel] [PATCH v4 07/11] OvmfPkg/CloudHv: Remove video support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf No need for video or virtio-gpu support since Cloud Hypervisor doesn't emulate any of these. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 16 OvmfPkg/CloudHv/CloudHvX64.fdf | 4 2

[edk2-devel] [PATCH v4 08/11] OvmfPkg/CloudHv: Remove USB support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any USB controller or device, therefore the support can be removed. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 11 --- OvmfPkg/CloudHv/CloudHvX64.fdf | 10

[edk2-devel] [PATCH v4 10/11] OvmfPkg/CloudHv: add Maintainers.txt entry

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Acked-by: Gerd Hoffmann Reviewed-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- Maintainers.txt | 5 + 1 file changed, 5 insertions(+) diff --git a/Maintainers.txt b/Maintainers.txt index 71c42bddae..3bda97ef25 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @

[edk2-devel] [PATCH v4 09/11] OvmfPkg/CloudHv: Remove CSM support

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't need the support for legacy BIOS, therefore the CSM support can be removed. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 20 OvmfPkg/CloudHv/CloudHvX64.fdf

[edk2-devel] [PATCH v4 11/11] OvmfPkg: Add CloudHvX64 to the CI

2022-01-11 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the newly created target for Cloud Hypervisor to the CI, validating it can be properly built. Acked-by: Gerd Hoffmann Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- .../.azurepipelines/Ubuntu-GCC5.yml | 9 + OvmfPkg/PlatformCI/CloudHvBuild

Re: [edk2-devel] Guidance about CI

2022-01-11 Thread Boeuf, Sebastien
at 13:17 +0100, kra...@redhat.com wrote: > On Thu, Jan 06, 2022 at 02:21:59PM +0000, Boeuf, Sebastien wrote: > > On Wed, 2022-01-05 at 17:55 +0100, kra...@redhat.com wrote: > > > On Wed, Jan 05, 2022 at 01:44:01PM +, Boeuf, Sebastien wrote: > > > > Ah nevermind

[edk2-devel] Does CI support nested virtualization?

2022-01-11 Thread Boeuf, Sebastien
Hi, I was thinking about running a simple test with Cloud Hypervisor (VMM relying on KVM) to validate the associated Ovmf target can be properly booted. Unfortunately I get an error about /dev/kvm not being available. Is there a way to let the Azure Pipelines know that we need a machine that supp

Re: [edk2-devel] [PATCH v4 00/11] Create new target for Cloud Hypervisor

2022-01-14 Thread Boeuf, Sebastien
Great, thank you! Sebastien From: Yao, Jiewen Sent: Thursday, January 13, 2022 3:56:09 PM To: Boeuf, Sebastien ; devel@edk2.groups.io Cc: Justen, Jordan L ; kra...@redhat.com Subject: RE: [PATCH v4 00/11] Create new target for Cloud Hypervisor Merged: https

Re: [edk2-devel] Does CI support nested virtualization?

2022-01-19 Thread Boeuf, Sebastien
Any idea? From: devel@edk2.groups.io on behalf of Boeuf, Sebastien Sent: Tuesday, January 11, 2022 4:23 PM To: Kinney, Michael D ; bret.barke...@microsoft.com ; sean.bro...@microsoft.com ; devel@edk2.groups.io Subject: [edk2-devel] Does CI support nested

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

2022-02-22 Thread Boeuf, Sebastien
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 (3): OvmfPkg: Generate CloudHv as a PVH ELF binary OvmfPkg: CloudHv: Retrieve RSDP address from PVH OvmfPkg: Cl

[edk2-devel] [PATCH 1/3] OvmfPkg: Generate CloudHv as a PVH ELF binary

2022-02-22 Thread Boeuf, Sebastien
From: Sebastien Boeuf Following the model from the Xen target, CloudHv is generated as a PVH ELF binary to take advantage of the PVH specification. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvElfHeaderGenerator.c | 150 OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH 2/3] OvmfPkg: CloudHv: Retrieve RSDP address from PVH

2022-02-22 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of hardcoding the address of the RSDP in the firmware, let's rely on the PVH structure hvm_start_info to retrieve this information. Signed-off-by: Sebastien Boeuf --- OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf | 2 ++ OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c

[edk2-devel] [PATCH 3/3] OvmfPkg: CloudHv: Rely on PVH memmap instead of CMOS

2022-02-22 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of using the CMOS, the CloudHv platform relies on the list of memmap entries provided through the PVH boot protocol to determine the last RAM address below 4G. Signed-off-by: Sebastien Boeuf --- OvmfPkg/PlatformPei/MemDetect.c | 73

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

2022-02-23 Thread Boeuf, Sebastien
rget cover both use cases? Thanks, Sebastien > > Thank you > Yao, Jiewen > > > -Original Message- > > From: Boeuf, Sebastien > > Sent: Tuesday, February 22, 2022 11:53 PM > > To: devel@edk2.groups.io > > Cc: Yao, Jiewen ; Justen, Jordan

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

2022-02-23 Thread Boeuf, Sebastien
On Wed, 2022-02-23 at 13:02 +0100, kra...@redhat.com wrote: >   Hi, > > > Well that's a good question. If we expect the same target (CloudHv) > > to > > support both TDX and non-TDX, that means the generated TDVF will be > > a > > PVH ELF binary, which will require some special handling from Cloud

Re: [edk2-devel] [PATCH 1/3] OvmfPkg: Generate CloudHv as a PVH ELF binary

2022-02-23 Thread Boeuf, Sebastien
On Wed, 2022-02-23 at 12:22 +0100, Gerd Hoffmann wrote: > On Tue, Feb 22, 2022 at 04:53:04PM +0100, Boeuf, Sebastien wrote: > > From: Sebastien Boeuf > > > > Following the model from the Xen target, CloudHv is generated as a > > PVH > > ELF binary to take a

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

2022-02-23 Thread Boeuf, Sebastien
ovm is a good example - > https://github.com/tianocore/edk2/blob/master/OvmfPkg/Microvm/README. > > Thank you > Yao Jiewen > > > -Original Message- > > From: Boeuf, Sebastien > > Sent: Wednesday, February 23, 2022 8:20 PM > > To: kra...@redhat.com; deve

[edk2-devel] [PATCH v2 0/5] CloudHv: Rely on PVH boot specification

2022-02-23 Thread Boeuf, Sebastien
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 (5): OvmfPkg: Make the Xen ELF header generator more flexible OvmfPkg: Generate CloudHv as a PVH ELF binary Ov

[edk2-devel] [PATCH v2 5/5] OvmfPkg: CloudHv: Add README

2022-02-23 Thread Boeuf, Sebastien
From: Sebastien Boeuf Add some documentation to the CloudHv target in order to clarify how to use it and what to expect from it. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/README | 66 ++ 1 file changed, 66 insertions(+) create mode 100644 OvmfP

[edk2-devel] [PATCH v2 1/5] OvmfPkg: Make the Xen ELF header generator more flexible

2022-02-23 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding some flexibility to the program so that other targets can use it if needed. An optional size parameter is added so that we can provide the expected blob size of the generated binary. A global define is added so that we can choose at build time if we want to use 32-b

[edk2-devel] [PATCH v2 3/5] OvmfPkg: CloudHv: Retrieve RSDP address from PVH

2022-02-23 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of hardcoding the address of the RSDP in the firmware, let's rely on the PVH structure hvm_start_info to retrieve this information. Signed-off-by: Sebastien Boeuf --- OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf | 2 ++ OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c

[edk2-devel] [PATCH v2 4/5] OvmfPkg: CloudHv: Rely on PVH memmap instead of CMOS

2022-02-23 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of using the CMOS, the CloudHv platform relies on the list of memmap entries provided through the PVH boot protocol to determine the last RAM address below 4G. Signed-off-by: Sebastien Boeuf --- OvmfPkg/PlatformPei/MemDetect.c | 73

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

2022-02-23 Thread Boeuf, Sebastien
ou send a different email to ask the different topic - > not distract people. Sorry, I'll send a separate email. Thanks, Sebastien > > > > -----Original Message- > > From: Boeuf, Sebastien > > Sent: Wednesday, February 23, 2022 10:03 PM > > To: kra...@

[edk2-devel] [PATCH v2 2/5] OvmfPkg: Generate CloudHv as a PVH ELF binary

2022-02-23 Thread Boeuf, Sebastien
From: Sebastien Boeuf Following the model from the Xen target, CloudHv is generated as a PVH ELF binary to take advantage of the PVH specification. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 +- OvmfPkg/CloudHv/CloudHvX64.fdf | 94 +-

Re: [edk2-devel] [PATCH 1/3] OvmfPkg: Generate CloudHv as a PVH ELF binary

2022-02-23 Thread Boeuf, Sebastien
On Wed, 2022-02-23 at 13:35 +0100, Sebastien Boeuf wrote: > On Wed, 2022-02-23 at 12:22 +0100, Gerd Hoffmann wrote: > > On Tue, Feb 22, 2022 at 04:53:04PM +0100, Boeuf, Sebastien wrote: > > > From: Sebastien Boeuf > > > > > > Following the model from the

[edk2-devel] [PATCH v3 0/7] CloudHv: Rely on PVH boot specification

2022-02-24 Thread Boeuf, Sebastien
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 (7): OvmfPkg: Make the Xen ELF header generator more flexible OvmfPkg: Xen: Use a new fdf include for the PV

[edk2-devel] [PATCH v3 1/7] OvmfPkg: Make the Xen ELF header generator more flexible

2022-02-24 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding some flexibility to the program through optional parameters and global define, so that other targets can use the generator. * A global define is added so that we can choose at build time if we want to use 32-bit or 64-bit base structures. * A first optional paramet

[edk2-devel] [PATCH v3 2/7] OvmfPkg: Xen: Use a new fdf include for the PVH ELF header

2022-02-24 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of having the PVH ELF header part of the fdf file directly, we move it to a dedicated include file. This is the first step in automating the generation of the header. Signed-off-by: Sebastien Boeuf --- OvmfPkg/OvmfXen.fdf | 57 ++--

[edk2-devel] [PATCH v3 3/7] OvmfPkg: Xen: Generate fdf include file from ELF header generator

2022-02-24 Thread Boeuf, Sebastien
From: Sebastien Boeuf Updating the fdf include file based on the run of the ELF header generator. The diff from this patch is the result of: $ gcc -o elf_gen OvmfPkg/OvmfXenElfHeaderGenerator.c $ ./elf_gen 2097152 OvmfPkg/XenElfHeader.fdf.inc Signed-off-by: Sebastien Boeuf --- OvmfPkg/XenElfH

[edk2-devel] [PATCH v3 4/7] OvmfPkg: Generate CloudHv as a PVH ELF binary

2022-02-24 Thread Boeuf, Sebastien
From: Sebastien Boeuf Following the model from the Xen target, CloudHv is generated as a PVH ELF binary to take advantage of the PVH specification, which requires less emulation from the VMM. The fdf include file CloudHvElfHeader.fdf.inc has been generated from the following commands: $ gcc -D

[edk2-devel] [PATCH v3 5/7] OvmfPkg: CloudHv: Retrieve RSDP address from PVH

2022-02-24 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of hardcoding the address of the RSDP in the firmware, let's rely on the PVH structure hvm_start_info to retrieve this information. Signed-off-by: Sebastien Boeuf --- OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf | 2 ++ OvmfPkg/AcpiPlatformDxe/CloudHvAcpi.c

[edk2-devel] [PATCH v3 6/7] OvmfPkg: CloudHv: Rely on PVH memmap instead of CMOS

2022-02-24 Thread Boeuf, Sebastien
From: Sebastien Boeuf Instead of using the CMOS, the CloudHv platform relies on the list of memmap entries provided through the PVH boot protocol to determine the last RAM address below 4G. Signed-off-by: Sebastien Boeuf --- OvmfPkg/PlatformPei/MemDetect.c | 73

  1   2   >