On Wed, Aug 17, 2016 at 02:07:01PM +0200, Alexander Gordeev wrote: > Hi Andrew, > > This is 7th version of PCI support. Noticeable changes since v5 & v6: > - patch "pci: Factor out pci_bar_get()" moved earlier in the series; > - 'extern' added to public functions in lib/pci.h; > - PCI test added to arm/unittests.cfg; > > Sources are avalable at: > https://github.com/a-gordeev/kvm-unit-tests.git pci-testdev-v7 This branch has two additional commits on it (for x86). I've dropped them from my branch import. > > Cc: Thomas Huth <thuth@xxxxxxxxxx> > Cc: Andrew Jones <drjones@xxxxxxxxxx> > > Alexander Gordeev (13): > pci: Fix coding style in generic PCI files > pci: x86: Rename pci_config_read() to pci_config_readl() > pci: Add 'extern' to public function declarations > pci: x86: Add remaining PCI configuration space accessors > pci: Factor out pci_bar_get() > pci: Rework pci_bar_addr() > pci: Add pci_bar_set_addr() > pci: Add pci_dev_exists() > pci: Add pci_print() > pci: Add generic ECAM host support > arm/arm64: pci: Add PCI bus operation test > pci: Add pci-testdev PCI bus test device > arm/arm64: pci: Add pci-testdev PCI device operation test I think patch 12 should be patch 11, and patch 13 should be squashed into the current 11. Current 11 by itself is pretty pointless. And now for bad news. I imported your branch and tested it. It works great over TCG, but hits the "No PCI resource found for a device" assert when running with KVM on an AArch64 server. I've only tried on a mustang so far. Thanks, drew -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
