This series is meant to rework the way the MSIX table and PBA are allocated
to prevent situations where the size allocated by kvmtool is larger than
the size of the BAR that holds them.
Patches 1-3 are fixes for stuff I found when I was investing a bug
triggered by the incorrect sizing of the table and PBA.
Patch 4 is a preparatory patch.
Patch 5 is the proper fix. More details in the commit message.
Patch 6 is there to make it easier to catch such errors if the code
regresses.
Patch 7 is an optimization for guests with larger page sizes than the host.
Testing
=======
On an AMD Seattle, host page size 4k and 64k, guest page size 4k and 64k
(so 4 tests in total for each device). Tried device passthrough with a
Realtek RTL8168 NIC and with a Intel 82574L NIC (not at the same time). No
issues encountered, MSIX table and PBA BAR has the same layout as the
physical device.
On an AMD Ryzen 3900x, host and guest 4k pages, I've assigned an Intel
82574L NIC to the guest. Realtek RTL8168 doesn't work without emulating a
PCI Express bus because the drivers falls back to CSI for device
configuration. Everything works in the guest, the shared BAR has the same
layout in the guest as the physical device.
Alexandru Elisei (7):
arm/gicv2m: Set errno when gicv2_update_routing() fails
vfio/pci.c: Remove double include for assert.h
pci: Fix pci_dev_* print macros
vfio/pci: Rename PBA offset in device descriptor to fd_offset
vfio/pci: Rework MSIX table and PBA physical size allocation
vfio/pci: Print an error when offset is outside of the MSIX table or
PBA
vfio/pci: Align MSIX Table and PBA size allocation to 64k
arm/gicv2m.c | 10 +++---
include/kvm/pci.h | 10 +++---
include/kvm/vfio.h | 3 +-
vfio/pci.c | 82 ++++++++++++++++++++++++++++++----------------
4 files changed, 66 insertions(+), 39 deletions(-)
--
2.20.1
