| Message ID | 20180730161840.13733-1-logang@deltatee.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Add parameter for disabling ACS redirection for P2P | expand |
Hey, Oops, I missed finishing this up before sending it sorry: This is mostly a resend with Alex's tags collected. Bjorn, can you please consider merging this? All patches have been reviewed and accepted by Alex. Thanks, Logan -- Changes since v7: * Collected Alex's Reviewed-by Tags * Rebased onto v4.18-rc7 (no conflicts) On 30/07/18 10:18 AM, Logan Gunthorpe wrote: > Changes since v6: > * Fixed order of operations for device specific disable function as > noticed by Alex > * Rebased onto v4.18-rc5 (no conflicts) > > Changes since v5: > * Add a quirk to handle the Intel SPT PCH case (as pointed out by Alex) > * Warn in the case that we try to disable ACS redirect on a device > that doesn't have the ACS capability (also suggested by Alex) > * Collect reviewed-by tag from Alex > * Rebased onto v4.18-rc4 (no conflicts) > > Changes since v4: > * Fixed a couple documentation mistakes spotted by Randy > > Changes since v3: > * Removed some of the cruft that was copied from the resource_alignment > paramater (per Alex) > * A number of docuemntation fixes as noticed by Alex and Willy > > Changes since v2: > * Rebased onto v4.18-rc1 (no conflicts) > * Minor tweaks to the documentation per Andy > * Removed the "path:" prefix and use the path parsing code > for simple devices (as it works the same). Per a suggestion from Alex > > Changes since v1: > * Reworked pci_dev_str_match_path using strrchr as suggested by Alex > * Collected Christian's Acks > > > Logan Gunthorpe (4): > PCI: Make specifying PCI devices in kernel parameters reusable > PCI: Allow specifying devices using a base bus and path of devfns > PCI: Introduce disable_acs_redir quirk > PCI: Introduce the disable_acs_redir parameter > > Documentation/admin-guide/kernel-parameters.txt | 41 +++- > drivers/pci/pci.c | 310 +++++++++++++++++++----- > drivers/pci/quirks.c | 78 +++++- > include/linux/pci.h | 5 + > 4 files changed, 361 insertions(+), 73 deletions(-) > > -- > 2.11.0 >
On Mon, Jul 30, 2018 at 10:18:36AM -0600, Logan Gunthorpe wrote: > Changes since v6: > * Fixed order of operations for device specific disable function as > noticed by Alex > * Rebased onto v4.18-rc5 (no conflicts) > > Changes since v5: > * Add a quirk to handle the Intel SPT PCH case (as pointed out by Alex) > * Warn in the case that we try to disable ACS redirect on a device > that doesn't have the ACS capability (also suggested by Alex) > * Collect reviewed-by tag from Alex > * Rebased onto v4.18-rc4 (no conflicts) > > Changes since v4: > * Fixed a couple documentation mistakes spotted by Randy > > Changes since v3: > * Removed some of the cruft that was copied from the resource_alignment > paramater (per Alex) > * A number of docuemntation fixes as noticed by Alex and Willy > > Changes since v2: > * Rebased onto v4.18-rc1 (no conflicts) > * Minor tweaks to the documentation per Andy > * Removed the "path:" prefix and use the path parsing code > for simple devices (as it works the same). Per a suggestion from Alex > > Changes since v1: > * Reworked pci_dev_str_match_path using strrchr as suggested by Alex > * Collected Christian's Acks > > > Logan Gunthorpe (4): > PCI: Make specifying PCI devices in kernel parameters reusable > PCI: Allow specifying devices using a base bus and path of devfns > PCI: Introduce disable_acs_redir quirk > PCI: Introduce the disable_acs_redir parameter > > Documentation/admin-guide/kernel-parameters.txt | 41 +++- > drivers/pci/pci.c | 310 +++++++++++++++++++----- > drivers/pci/quirks.c | 78 +++++- > include/linux/pci.h | 5 + > 4 files changed, 361 insertions(+), 73 deletions(-) I reorganized this a bit and split the quirk code into a couple patches an put the result on my pci/peer-to-peer branch for v4.19. I think the result is equivalent, but let me know if I made a mistake. Bjorn
On 09/08/18 05:03 PM, Bjorn Helgaas wrote: > I reorganized this a bit and split the quirk code into a couple > patches an put the result on my pci/peer-to-peer branch for v4.19. > > I think the result is equivalent, but let me know if I made a mistake. Thanks for the cleanup, Bjorn. I reviewed the patches and did a quick diff between my v8 and your branch. Everything looks good. Logan