[PATCH 07/13] PCI/P2PDMA: Derive peer-to-peer routing from ACS control bits
From: Leon Romanovsky <leon@kernel.org>
Date: 2026-08-02 15:10:28
Also in:
linux-pci, lkml
Subsystem:
pci peer-to-peer dma (p2pdma), pci subsystem, the rest · Maintainers:
Bjorn Helgaas, Logan Gunthorpe, Linus Torvalds
From: Leon Romanovsky <leonro@nvidia.com> The redirect test reports an upstream redirect for any ACS redirect or egress control bit. PCIe r7.0, sec 6.12.3, table 6-11 ties the outcome to the control bits and the target vector, and an enabled egress control bit alone does not redirect. Compute the result from the control bits so the vector can be honored next, using the host-bridge route while the peer target is unknown. Signed-off-by: Leon Romanovsky <leonro@nvidia.com> --- drivers/pci/p2pdma.c | 49 ++++++++++++++++++++++++++++++++++++------------- 1 file changed, 36 insertions(+), 13 deletions(-)
diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
index 801c183ae6b7..079cec6c0509 100644
--- a/drivers/pci/p2pdma.c
+++ b/drivers/pci/p2pdma.c@@ -21,6 +21,8 @@ #include <linux/seq_buf.h> #include <linux/xarray.h> +#include "pci.h" + struct pci_p2pdma { struct gen_pool *pool; bool p2pmem_published;
@@ -489,26 +491,45 @@ static struct pci_dev *find_parent_pci_dev(struct device *dev) return NULL; } -/* - * Check if a PCI bridge has its ACS redirection bits set to redirect P2P - * TLPs upstream via ACS. Returns 1 if the packets will be redirected - * upstream, 0 otherwise. - */ -static int pci_bridge_has_acs_redir(struct pci_dev *pdev) +enum pci_acs_p2pdma_state { + PCI_ACS_P2PDMA_DIRECT, + PCI_ACS_P2PDMA_REDIRECT, +}; + +static enum pci_acs_p2pdma_state +pci_acs_p2pdma_state(struct pci_dev *pdev, struct pci_dev *target) { int pos; u16 ctrl; pos = pdev->acs_cap; if (!pos) - return 0; + return PCI_ACS_P2PDMA_DIRECT; - pci_read_config_word(pdev, pos + PCI_ACS_CTRL, &ctrl); + if (pci_read_config_word(pdev, pos + PCI_ACS_CTRL, &ctrl)) + return PCI_ACS_P2PDMA_REDIRECT; - if (ctrl & (PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_EC)) - return 1; + if (!(ctrl & PCI_ACS_EC)) + return ctrl & (PCI_ACS_RR | PCI_ACS_CR) ? + PCI_ACS_P2PDMA_REDIRECT : PCI_ACS_P2PDMA_DIRECT; - return 0; + /* + * The vector cannot be read without the peer target, so redirect + * upstream until the paths diverge. + */ + if (!target) + return PCI_ACS_P2PDMA_REDIRECT; + + /* + * PCIe r7.0, sec 6.12.3, table 6-11: a set or indeterminate egress + * control vector bit keeps the request off the direct path; a clear + * bit permits it, subject only to completion redirect. + */ + if (pci_acs_egress_ctrl_set(pdev, target)) + return PCI_ACS_P2PDMA_REDIRECT; + + return ctrl & PCI_ACS_CR ? PCI_ACS_P2PDMA_REDIRECT : + PCI_ACS_P2PDMA_DIRECT; } static void seq_buf_print_bus_devfn(struct seq_buf *buf, struct pci_dev *pdev)
@@ -722,7 +743,8 @@ calc_map_type_and_dist(struct pci_dev *provider, struct pci_dev *client, while (a) { dist_b = 0; - if (pci_bridge_has_acs_redir(a)) { + if (pci_acs_p2pdma_state(a, NULL) == + PCI_ACS_P2PDMA_REDIRECT) { seq_buf_print_bus_devfn(&acs_list, a); acs_cnt++; }
@@ -755,7 +777,8 @@ calc_map_type_and_dist(struct pci_dev *provider, struct pci_dev *client, if (a == bb) break; - if (pci_bridge_has_acs_redir(bb)) { + if (pci_acs_p2pdma_state(bb, NULL) == + PCI_ACS_P2PDMA_REDIRECT) { seq_buf_print_bus_devfn(&acs_list, bb); acs_cnt++; }
--
2.55.0