Gregor reported a deadlock [1] when enabling a VF that supports ATS. This series is intended to fix that. The second patch should be enough to fix the deadlock; the rest are simplification and cleanup. These are based on v4.2-rc2. [1] http://permalink.gmane.org/gmane.linux.kernel.iommu/9433 Changes between v1 and v2: - Remove use of pci_ats_enabled() (intel-iommu.c) - Call pci_ats_queue_depth() only once per device and cache result (intel-iommu.c) - Remove pci_ats_enabled() interface - Stop caching queue depth in pci_dev to save space - Add PF refcount of how many associated VFs have ATS enabled - Add comment that ATS must be enabled on PF before on VFs - Require ATS to be disabled on all VFs and PF before changing STU --- Bjorn Helgaas (11): iommu/vt-d: Cache PCI ATS state and Invalidate Queue Depth PCI: Allocate ATS struct during enumeration PCI: Embed ATS info directly into struct pci_dev PCI: Reduce size of ATS structure elements PCI: Rationalize pci_ats_queue_depth() error checking PCI: Inline the ATS setup code into pci_ats_init() PCI: Use pci_physfn() rather than looking up physfn by hand PCI: Clean up ATS error handling PCI: Move ATS declarations to linux/pci.h so they're all together PCI: Stop caching ATS Invalidate Queue Depth PCI: Remove pci_ats_enabled() drivers/iommu/intel-iommu.c | 26 +++++---- drivers/pci/ats.c | 131 +++++++++++++++---------------------------- drivers/pci/probe.c | 3 + include/linux/pci-ats.h | 49 ---------------- include/linux/pci.h | 18 ++++++ 5 files changed, 82 insertions(+), 145 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
