Re: [PATCH v2 0/3] Quirk Intel PCH root ports for ACS-like features
On Fri, 2014-02-14 at 14:43 -0700, Bjorn Helgaas wrote: > [+cc Don] > > On Mon, Feb 03, 2014 at 02:27:27PM -0700, Alex Williamson wrote: > > v2: > > - Remove bus #0 bug in filtering matching > > - Add 2/3 introducing PCI_DEV_FLAGS_ACS_ENABLED_QUIRK, this gives > >is better tracking and addresses the theoretical hotplug issue > > - Update 3/3 for PCI_DEV_FLAGS_ACS_ENABLED_QUIRK > > - Add dev_info to print regardless of whether we changes bits > > - Add Intel cc > > > > As described in 3/3 many Intel root ports lack PCIe ACS capabilities > > which results in excessively large IOMMU groups. Many of these root > > ports do provide isolation capabilities, we just need to use device > > specific mechanisms to enable and verify. Long term, I hope we can > > round out this list (particularly to include X79 root ports) and > > more importantly, encourage proper PCIe ACS support in future > > products. I'm really hoping we can get this in during the 3.14 cycle. > > Thanks, > > > > Alex > > --- > > > > Alex Williamson (3): > > pci: Add device specific PCI ACS enable > > pci: Add pci_dev_flag for ACS enable quirks > > pci/quirks: Enable quirks for PCIe ACS on Intel PCH root ports > > I applied these (with Don's ack for 3/3) to pci/virtualization. > > I tried to figure out how to handle post-merge window patches. Per > Documentation/development-process/2.Process, "[after -rc1], only patches > which fix problems should be submitted to the mainline," so one might > conclude that a fix for any sort of problem is allowed. However, > Documentation/networking/netdev-FAQ.txt says "No new features get mainlined > after this [-rc1] -- only fixes to the rc1 content are expected," which is > how I've been operating. > > In any case, these patches look more like new functionality (enabling a > non-standard ACS feature) than a bug fix to me, so my preference is to > merge them during the v3.15 merge window. > > I understand the desire for v3.14, namely, "lots of Intel devices don't > support ACS, and that makes it hard for users to expose devices to > userspace with fine granularity, and waiting for v3.15 will mean another > two months." But this problem is really of Intel's own making: if they'd > used standard ACS, or if they'd documented their non-standard mechanism, > this wouldn't be an issue. I appreciate you taking the time to consider it for 3.14. I'll look forward to it appearing in linux-next for 3.15. Thanks for review and applying, Alex -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH v2 0/3] Quirk Intel PCH root ports for ACS-like features
In re: Feature vs. bug fix At the risk of trying to arguing about the meaning of `is' I would argue that this is more a bug fix than a feature. We're not adding a new capability, we are trying to work around deficient HW (no argument that we should have implemented ACS on all platforms). Having said that, although I would obviously prefer to see this in v3.14 we can live with this hitting v3.15. -- Don Dugger "Censeo Toto nos in Kansa esse decisse." - D. Gale Ph: 303/443-3786 -Original Message- From: Bjorn Helgaas [mailto:bhelg...@google.com] Sent: Friday, February 14, 2014 2:43 PM To: Alex Williamson Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Dugger, Donald D Subject: Re: [PATCH v2 0/3] Quirk Intel PCH root ports for ACS-like features [+cc Don] On Mon, Feb 03, 2014 at 02:27:27PM -0700, Alex Williamson wrote: > v2: > - Remove bus #0 bug in filtering matching > - Add 2/3 introducing PCI_DEV_FLAGS_ACS_ENABLED_QUIRK, this gives >is better tracking and addresses the theoretical hotplug issue > - Update 3/3 for PCI_DEV_FLAGS_ACS_ENABLED_QUIRK > - Add dev_info to print regardless of whether we changes bits > - Add Intel cc > > As described in 3/3 many Intel root ports lack PCIe ACS capabilities > which results in excessively large IOMMU groups. Many of these root > ports do provide isolation capabilities, we just need to use device > specific mechanisms to enable and verify. Long term, I hope we can > round out this list (particularly to include X79 root ports) and more > importantly, encourage proper PCIe ACS support in future products. > I'm really hoping we can get this in during the 3.14 cycle. > Thanks, > > Alex > --- > > Alex Williamson (3): > pci: Add device specific PCI ACS enable > pci: Add pci_dev_flag for ACS enable quirks > pci/quirks: Enable quirks for PCIe ACS on Intel PCH root ports I applied these (with Don's ack for 3/3) to pci/virtualization. I tried to figure out how to handle post-merge window patches. Per Documentation/development-process/2.Process, "[after -rc1], only patches which fix problems should be submitted to the mainline," so one might conclude that a fix for any sort of problem is allowed. However, Documentation/networking/netdev-FAQ.txt says "No new features get mainlined after this [-rc1] -- only fixes to the rc1 content are expected," which is how I've been operating. In any case, these patches look more like new functionality (enabling a non-standard ACS feature) than a bug fix to me, so my preference is to merge them during the v3.15 merge window. I understand the desire for v3.14, namely, "lots of Intel devices don't support ACS, and that makes it hard for users to expose devices to userspace with fine granularity, and waiting for v3.15 will mean another two months." But this problem is really of Intel's own making: if they'd used standard ACS, or if they'd documented their non-standard mechanism, this wouldn't be an issue. Bjorn
Re: [PATCH v2 0/3] Quirk Intel PCH root ports for ACS-like features
[+cc Don] On Mon, Feb 03, 2014 at 02:27:27PM -0700, Alex Williamson wrote: > v2: > - Remove bus #0 bug in filtering matching > - Add 2/3 introducing PCI_DEV_FLAGS_ACS_ENABLED_QUIRK, this gives >is better tracking and addresses the theoretical hotplug issue > - Update 3/3 for PCI_DEV_FLAGS_ACS_ENABLED_QUIRK > - Add dev_info to print regardless of whether we changes bits > - Add Intel cc > > As described in 3/3 many Intel root ports lack PCIe ACS capabilities > which results in excessively large IOMMU groups. Many of these root > ports do provide isolation capabilities, we just need to use device > specific mechanisms to enable and verify. Long term, I hope we can > round out this list (particularly to include X79 root ports) and > more importantly, encourage proper PCIe ACS support in future > products. I'm really hoping we can get this in during the 3.14 cycle. > Thanks, > > Alex > --- > > Alex Williamson (3): > pci: Add device specific PCI ACS enable > pci: Add pci_dev_flag for ACS enable quirks > pci/quirks: Enable quirks for PCIe ACS on Intel PCH root ports I applied these (with Don's ack for 3/3) to pci/virtualization. I tried to figure out how to handle post-merge window patches. Per Documentation/development-process/2.Process, "[after -rc1], only patches which fix problems should be submitted to the mainline," so one might conclude that a fix for any sort of problem is allowed. However, Documentation/networking/netdev-FAQ.txt says "No new features get mainlined after this [-rc1] -- only fixes to the rc1 content are expected," which is how I've been operating. In any case, these patches look more like new functionality (enabling a non-standard ACS feature) than a bug fix to me, so my preference is to merge them during the v3.15 merge window. I understand the desire for v3.14, namely, "lots of Intel devices don't support ACS, and that makes it hard for users to expose devices to userspace with fine granularity, and waiting for v3.15 will mean another two months." But this problem is really of Intel's own making: if they'd used standard ACS, or if they'd documented their non-standard mechanism, this wouldn't be an issue. Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH v2 0/3] Quirk Intel PCH root ports for ACS-like features
v2: - Remove bus #0 bug in filtering matching - Add 2/3 introducing PCI_DEV_FLAGS_ACS_ENABLED_QUIRK, this gives is better tracking and addresses the theoretical hotplug issue - Update 3/3 for PCI_DEV_FLAGS_ACS_ENABLED_QUIRK - Add dev_info to print regardless of whether we changes bits - Add Intel cc As described in 3/3 many Intel root ports lack PCIe ACS capabilities which results in excessively large IOMMU groups. Many of these root ports do provide isolation capabilities, we just need to use device specific mechanisms to enable and verify. Long term, I hope we can round out this list (particularly to include X79 root ports) and more importantly, encourage proper PCIe ACS support in future products. I'm really hoping we can get this in during the 3.14 cycle. Thanks, Alex --- Alex Williamson (3): pci: Add device specific PCI ACS enable pci: Add pci_dev_flag for ACS enable quirks pci/quirks: Enable quirks for PCIe ACS on Intel PCH root ports drivers/pci/pci.c| 26 +-- drivers/pci/quirks.c | 185 ++ include/linux/pci.h |4 + 3 files changed, 209 insertions(+), 6 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/