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

Gerd Hoffmann posted 6 patches 2 years ago
Failed in applying to current master (apply log)
There is a newer version of this series
OvmfPkg/Microvm/MicrovmX64.dsc                | 40 ++++++++++-------
.../PlatformInitLib/PlatformInitLib.inf       |  4 +-
OvmfPkg/PlatformPei/PlatformPei.inf           |  2 +-
.../Bus/Pci/PciHostBridgeDxe/PciHostBridge.c  |  3 ++
.../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, 104 insertions(+), 45 deletions(-)
[edk2-devel] [PATCH v6 0/6] OvmfPkg/Microvm/pcie: add pcie support
Posted by Gerd Hoffmann 2 years 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).

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  |  3 ++
 .../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, 104 insertions(+), 45 deletions(-)

-- 
2.35.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89257): https://edk2.groups.io/g/devel/message/89257
Mute This Topic: https://groups.io/mt/90681714/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v6 0/6] OvmfPkg/Microvm/pcie: add pcie support
Posted by Ard Biesheuvel 2 years ago
On Mon, 25 Apr 2022 at 12:34, 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).
>
> 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
>

Thanks Gerd. I'm happy to take these, but not without an ack from the
MdeModulePkg maintainers.



>  OvmfPkg/Microvm/MicrovmX64.dsc                | 40 ++++++++++-------
>  .../PlatformInitLib/PlatformInitLib.inf       |  4 +-
>  OvmfPkg/PlatformPei/PlatformPei.inf           |  2 +-
>  .../Bus/Pci/PciHostBridgeDxe/PciHostBridge.c  |  3 ++
>  .../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, 104 insertions(+), 45 deletions(-)
>
> --
> 2.35.1
>


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