From: Leon Romanovsky leonro@nvidia.com
P2PDMA assumes every client issues Untranslated Requests. A device with ATS enabled may translate any address it is handed, including a bus address, and Translation Blocking can reject its Translated Requests on a route the Untranslated answer called usable.
Unless the client declared per-mapping ATS, take the Address Type from its ATS Enable bit. Never hand such a client bus addresses: only an IOVA survives translation, and whatever it issues untranslated goes through the host bridge, whose route must therefore be usable. Apply this after the topology-only cache, and in pci_p2pdma_distance_many() too, so that provider selection agrees with the mapping.
Reviewed-by: Thomas Hellström thomas.hellstrom@linux.intel.com Signed-off-by: Leon Romanovsky leonro@nvidia.com --- Documentation/driver-api/pci/p2pdma.rst | 8 +++ drivers/pci/p2pdma.c | 92 ++++++++++++++++++++++++++++++--- 2 files changed, 93 insertions(+), 7 deletions(-)
diff --git a/Documentation/driver-api/pci/p2pdma.rst b/Documentation/driver-api/pci/p2pdma.rst index ff7f619616e8e..7760ff64ed05b 100644 --- a/Documentation/driver-api/pci/p2pdma.rst +++ b/Documentation/driver-api/pci/p2pdma.rst @@ -63,6 +63,14 @@ P2PDMA therefore decides each class of traffic separately, and ``pci_p2pdma_map_type()`` answers for the one a DMA mapping is built for: strictly ordered Requests carrying an Untranslated address.
+The PCIe ATS Enable bit covers the whole device, and most devices translate +any address they are handed once it is set. Unless a driver has declared with +``pcim_p2pdma_set_ats_per_mapping()`` that its device chooses ATS per mapping, +that bit decides which Address Type P2PDMA assumes, and a client with ATS +enabled is never handed a bus address, which it would translate as though it +were an IOVA. Its Translated Requests may still route directly, but anything +it issues untranslated reaches the host bridge, so that route has to work too. + The two directions are decided independently. Translation Blocking (TB), Direct Translated P2P (DT), Request Redirect (RR) and Egress Control (EC) on the Requester-side port decide the Request: diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c index df66aa056dec8..e81b9070bb4ef 100644 --- a/drivers/pci/p2pdma.c +++ b/drivers/pci/p2pdma.c @@ -1467,6 +1467,59 @@ calc_map_type_and_dist(struct pci_dev *provider, struct pci_dev *client, } EXPORT_SYMBOL_IF_KUNIT(calc_map_type_and_dist);
+static unsigned int pci_p2pdma_client_tlp_flags(struct pci_dev *client, + unsigned int tlp_flags) +{ + if (client->ats_enabled) + return tlp_flags | PCI_P2PDMA_TLP_TRANSLATED; + + return tlp_flags & ~PCI_P2PDMA_TLP_TRANSLATED; +} + +/* + * A client that translates every address it is handed cannot be handed a bus + * address, which it would translate as though it were an IOVA. Its Translated + * Requests may still route directly, but a Request it issues without a + * translation carries the IOVA to the host bridge, so that route has to work + * as well. + * + * Take the Address Type from @tlp_flags rather than reading the ATS Enable + * bit again, so the policy and the class it applies to always agree. + */ +static enum pci_p2pdma_map_type +pci_p2pdma_client_map_type(struct pci_dev *provider, struct pci_dev *client, + bool per_mapping, unsigned int tlp_flags, + enum pci_p2pdma_map_type type) +{ + if (per_mapping || !(tlp_flags & PCI_P2PDMA_TLP_TRANSLATED) || + type != PCI_P2PDMA_MAP_BUS_ADDR) + return type; + + pci_dbg(client, + "P2PDMA ACS: provider=%s bus address withheld; ATS is enabled for the whole client\n", + pci_name(provider)); + + if (cpu_supports_p2pdma() || + host_bridge_whitelist(provider, client, false)) + return PCI_P2PDMA_MAP_THRU_HOST_BRIDGE; + + return PCI_P2PDMA_MAP_NOT_SUPPORTED; +} + +/* Whether the driver of @client called pcim_p2pdma_set_ats_per_mapping(). */ +static bool pci_p2pdma_client_per_mapping(struct pci_dev *client) +{ + struct pci_p2pdma *p2pdma; + bool per_mapping; + + rcu_read_lock(); + p2pdma = rcu_dereference(client->p2pdma); + per_mapping = p2pdma && p2pdma->ats_per_mapping; + rcu_read_unlock(); + + return per_mapping; +} + /** * pci_p2pdma_distance_many - Determine the cumulative distance between * a p2pdma provider and the clients in use. @@ -1482,15 +1535,19 @@ EXPORT_SYMBOL_IF_KUNIT(calc_map_type_and_dist); * * "compatible" means the provider and the clients are either all behind * the same PCI root port or the host bridges connected to each of the devices - * are listed in the 'pci_p2pdma_whitelist'. + * are listed in the 'pci_p2pdma_whitelist'. Each client is judged the way + * pci_p2pdma_map_type() maps it, so one with ATS enabled for the whole device + * also needs a host bridge that supports peer-to-peer DMA. */ int pci_p2pdma_distance_many(struct pci_dev *provider, struct device **clients, int num_clients, bool verbose) { - enum pci_p2pdma_map_type map; + enum pci_p2pdma_map_type map, type; bool not_supported = false; struct pci_dev *pci_client; + unsigned int tlp_flags; int total_dist = 0; + bool per_mapping; int i, distance;
if (num_clients == 0) @@ -1505,8 +1562,18 @@ int pci_p2pdma_distance_many(struct pci_dev *provider, struct device **clients, return -1; }
- map = calc_map_type_and_dist(provider, pci_client, &distance, 0, - verbose); + per_mapping = pci_p2pdma_client_per_mapping(pci_client); + tlp_flags = per_mapping ? 0 : + pci_p2pdma_client_tlp_flags(pci_client, 0); + type = calc_map_type_and_dist(provider, pci_client, &distance, + tlp_flags, verbose); + map = pci_p2pdma_client_map_type(provider, pci_client, + per_mapping, tlp_flags, type); + if (verbose && map != type && + map == PCI_P2PDMA_MAP_NOT_SUPPORTED) + pci_warn(pci_client, + "cannot be used for peer-to-peer DMA with %s: ATS is enabled for the whole device and the host bridge is not whitelisted\n", + pci_name(provider));
pci_dev_put(pci_client);
@@ -1813,6 +1880,11 @@ EXPORT_SYMBOL_GPL(pcim_p2pdma_set_ats_per_mapping); * ACS routes a peer-to-peer transaction by the attributes its TLPs carry, so * the answer depends on @tlp_flags. A caller that passes flags its traffic * does not match gets a mapping the fabric will not deliver. + * + * Only a client whose driver called pcim_p2pdma_set_ats_per_mapping() takes + * the Address Type from @tlp_flags. For any other client its ATS Enable bit + * decides, and a client that translates every address it is handed never gets + * %PCI_P2PDMA_MAP_BUS_ADDR. */ static enum pci_p2pdma_map_type pci_p2pdma_map_type_tlp(struct p2pdma_provider *provider, struct device *dev, @@ -1824,6 +1896,7 @@ pci_p2pdma_map_type_tlp(struct p2pdma_provider *provider, struct device *dev, struct pci_p2pdma *p2pdma; struct pci_dev *client; bool provider_state; + bool per_mapping; int dist;
if (!pdev->p2pdma) { @@ -1841,6 +1914,9 @@ pci_p2pdma_map_type_tlp(struct p2pdma_provider *provider, struct device *dev,
client = to_pci_dev(dev); cache_index = map_types_idx(client); + per_mapping = pci_p2pdma_client_per_mapping(client); + if (!per_mapping) + tlp_flags = pci_p2pdma_client_tlp_flags(client, tlp_flags);
rcu_read_lock(); p2pdma = rcu_dereference(pdev->p2pdma); @@ -1857,10 +1933,11 @@ pci_p2pdma_map_type_tlp(struct p2pdma_provider *provider, struct device *dev, pci_p2pdma_map_type_name(type), type, provider_state);
if (type == PCI_P2PDMA_MAP_UNKNOWN) - return calc_map_type_and_dist(pdev, client, &dist, tlp_flags, + type = calc_map_type_and_dist(pdev, client, &dist, tlp_flags, true);
- return type; + return pci_p2pdma_client_map_type(pdev, client, per_mapping, tlp_flags, + type); }
/** @@ -1869,7 +1946,8 @@ pci_p2pdma_map_type_tlp(struct p2pdma_provider *provider, struct device *dev, * @dev: Client device that initiates the transfer * * Same as pci_p2pdma_map_type_tlp() for a client issuing strictly ordered - * Requests that carry an Untranslated address. + * Requests. Their Address Type is Untranslated unless the client enables ATS + * for the whole device. */ enum pci_p2pdma_map_type pci_p2pdma_map_type(struct p2pdma_provider *provider, struct device *dev)