[edk2-devel] [PATCH v7 0/6] OvmfPkg/Microvm/pcie: add pcie support

Gerd Hoffmann posted 6 patches 1 year, 11 months ago
Failed in applying to current master (apply log)
OvmfPkg/Microvm/MicrovmX64.dsc                | 40 ++++++++++-------
.../PlatformInitLib/PlatformInitLib.inf       |  4 +-
OvmfPkg/PlatformPei/PlatformPei.inf           |  2 +-
.../Bus/Pci/PciHostBridgeDxe/PciHostBridge.c  |  6 +++
.../FdtPciHostBridgeLib/FdtPciHostBridgeLib.c | 45 ++++++++++---------
OvmfPkg/Library/PlatformInitLib/MemDetect.c   | 45 ++++++++++++++++++-
OvmfPkg/Library/PlatformInitLib/Platform.c    |  4 +-
OvmfPkg/PlatformPei/Platform.c                |  2 +-
OvmfPkg/QemuVideoDxe/VbeShim.c                |  2 +
OvmfPkg/Microvm/README                        |  2 +-
10 files changed, 107 insertions(+), 45 deletions(-)
[edk2-devel] [PATCH v7 0/6] OvmfPkg/Microvm/pcie: add pcie support
Posted by Gerd Hoffmann 1 year, 11 months ago
Needs two little tweaks in PCI code because microvm supports mmio only.
Other than that just wire up the existing code (the PCIe host adapter
used by microvm is the same (virtual) hardware used by the arm/aarch64
virtual machines).

v7:
 - allow non-existing io address space only in case
   there are no io reservations (Mateusz Albecki)

v6:
 - codestyle fix (Abner Chang).

v5:
 - codestyle (uncrustify) fix.

v4:
 - update PciHostBridge check (Abner Chang).

v3:
 - rebase to latest master, adapt to PlatformInitLib.
 - rework PhysMemAddressWidth handling for microvm.

v2:
 - rebase to latest master
 - pick up review tags

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3777

Gerd Hoffmann (6):
  MdeModulePkg/PciHostBridge: io range is not mandatory
  OvmfPkg/FdtPciHostBridgeLib: io range is not mandatory
  OvmfPkg/Platform: unfix PcdPciExpressBaseAddress
  OvmfPkg/Microvm/pcie: no vbeshim please
  OvmfPkg/Microvm/pcie: mPhysMemAddressWidth tweak
  OvmfPkg/Microvm/pcie: add pcie support

 OvmfPkg/Microvm/MicrovmX64.dsc                | 40 ++++++++++-------
 .../PlatformInitLib/PlatformInitLib.inf       |  4 +-
 OvmfPkg/PlatformPei/PlatformPei.inf           |  2 +-
 .../Bus/Pci/PciHostBridgeDxe/PciHostBridge.c  |  6 +++
 .../FdtPciHostBridgeLib/FdtPciHostBridgeLib.c | 45 ++++++++++---------
 OvmfPkg/Library/PlatformInitLib/MemDetect.c   | 45 ++++++++++++++++++-
 OvmfPkg/Library/PlatformInitLib/Platform.c    |  4 +-
 OvmfPkg/PlatformPei/Platform.c                |  2 +-
 OvmfPkg/QemuVideoDxe/VbeShim.c                |  2 +
 OvmfPkg/Microvm/README                        |  2 +-
 10 files changed, 107 insertions(+), 45 deletions(-)

-- 
2.36.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90140): https://edk2.groups.io/g/devel/message/90140
Mute This Topic: https://groups.io/mt/91495638/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v7 0/6] OvmfPkg/Microvm/pcie: add pcie support
Posted by Ard Biesheuvel 1 year, 10 months ago
On Thu, 2 Jun 2022 at 10:42, Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> Needs two little tweaks in PCI code because microvm supports mmio only.
> Other than that just wire up the existing code (the PCIe host adapter
> used by microvm is the same (virtual) hardware used by the arm/aarch64
> virtual machines).
>
> v7:
>  - allow non-existing io address space only in case
>    there are no io reservations (Mateusz Albecki)
>
> v6:
>  - codestyle fix (Abner Chang).
>
> v5:
>  - codestyle (uncrustify) fix.
>
> v4:
>  - update PciHostBridge check (Abner Chang).
>
> v3:
>  - rebase to latest master, adapt to PlatformInitLib.
>  - rework PhysMemAddressWidth handling for microvm.
>
> v2:
>  - rebase to latest master
>  - pick up review tags
>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3777
>
> Gerd Hoffmann (6):
>   MdeModulePkg/PciHostBridge: io range is not mandatory

Dropped this one

>   OvmfPkg/FdtPciHostBridgeLib: io range is not mandatory
>   OvmfPkg/Platform: unfix PcdPciExpressBaseAddress
>   OvmfPkg/Microvm/pcie: no vbeshim please
>   OvmfPkg/Microvm/pcie: mPhysMemAddressWidth tweak
>   OvmfPkg/Microvm/pcie: add pcie support
>

... and merged the rest as #2941

Thanks,

>  OvmfPkg/Microvm/MicrovmX64.dsc                | 40 ++++++++++-------
>  .../PlatformInitLib/PlatformInitLib.inf       |  4 +-
>  OvmfPkg/PlatformPei/PlatformPei.inf           |  2 +-
>  .../Bus/Pci/PciHostBridgeDxe/PciHostBridge.c  |  6 +++
>  .../FdtPciHostBridgeLib/FdtPciHostBridgeLib.c | 45 ++++++++++---------
>  OvmfPkg/Library/PlatformInitLib/MemDetect.c   | 45 ++++++++++++++++++-
>  OvmfPkg/Library/PlatformInitLib/Platform.c    |  4 +-
>  OvmfPkg/PlatformPei/Platform.c                |  2 +-
>  OvmfPkg/QemuVideoDxe/VbeShim.c                |  2 +
>  OvmfPkg/Microvm/README                        |  2 +-
>  10 files changed, 107 insertions(+), 45 deletions(-)
>
> --
> 2.36.1
>


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