From: Alex Williamson <alex.williamson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org,
aik-sLpHqDYs0B2HXe+LvDLADg@public.gmane.org,
david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org,
joerg.roedel-5C7GfCeVMHo@public.gmane.org,
dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org
Cc: kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
B07421-KZfg59tc24xl57MIdRCFDg@public.gmane.org,
linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
agraf-l3A5Bk7waGM@public.gmane.org,
qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org,
chrisw-69jw2NvuJkxg9hUCZPvPmw@public.gmane.org,
B08248-KZfg59tc24xl57MIdRCFDg@public.gmane.org,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
avi-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org,
gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org,
bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
benve-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org
Subject: [PATCH 05/13] pci: New pci_acs_enabled()
Date: Fri, 11 May 2012 16:56:02 -0600 [thread overview]
Message-ID: <20120511225602.30496.80438.stgit@bling.home> (raw)
In-Reply-To: <20120511222148.30496.68571.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
In a PCIe environment, transactions aren't always required to
reach the root bus before being re-routed. Peer-to-peer DMA
may actually not be seen by the IOMMU in these cases. For
IOMMU groups, we want to provide IOMMU drivers a way to detect
these restrictions. Provided with a PCI device, pci_acs_enabled
returns the furthest downstream device with a complete PCI ACS
chain. This information can then be used in grouping to create
fully isolated groups. ACS chain logic extracted from libvirt.
Signed-off-by: Alex Williamson <alex.williamson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
drivers/pci/pci.c | 43 +++++++++++++++++++++++++++++++++++++++++++
include/linux/pci.h | 1 +
2 files changed, 44 insertions(+), 0 deletions(-)
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 111569c..d7f05ce 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -2358,6 +2358,49 @@ void pci_enable_acs(struct pci_dev *dev)
pci_write_config_word(dev, pos + PCI_ACS_CTRL, ctrl);
}
+#define PCI_EXT_CAP_ACS_ENABLED (PCI_ACS_SV | PCI_ACS_RR | \
+ PCI_ACS_CR | PCI_ACS_UF)
+
+/**
+ * pci_acs_enabled - test ACS support in downstream chain
+ * @dev: starting PCI device
+ *
+ * Returns the furthest downstream device with an unbroken ACS chain. If
+ * ACS is enabled throughout the chain, the returned device is the same as
+ * the one passed in.
+ */
+struct pci_dev *pci_acs_enabled(struct pci_dev *dev)
+{
+ struct pci_dev *acs_dev;
+ int pos;
+ u16 ctrl;
+
+ if (!pci_is_root_bus(dev->bus))
+ acs_dev = pci_acs_enabled(dev->bus->self);
+ else
+ return dev;
+
+ /* If the chain is already broken, pass on the device */
+ if (acs_dev != dev->bus->self)
+ return acs_dev;
+
+ if (!pci_is_pcie(dev) || (dev->class >> 8) != PCI_CLASS_BRIDGE_PCI)
+ return dev;
+
+ if (dev->pcie_type != PCI_EXP_TYPE_DOWNSTREAM)
+ return dev;
+
+ pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ACS);
+ if (!pos)
+ return acs_dev;
+
+ pci_read_config_word(dev, pos + PCI_ACS_CTRL, &ctrl);
+ if ((ctrl & PCI_EXT_CAP_ACS_ENABLED) != PCI_EXT_CAP_ACS_ENABLED)
+ return acs_dev;
+
+ return dev;
+}
+
/**
* pci_swizzle_interrupt_pin - swizzle INTx for device behind bridge
* @dev: the PCI device
diff --git a/include/linux/pci.h b/include/linux/pci.h
index 9910b5c..dc25da3 100644
--- a/include/linux/pci.h
+++ b/include/linux/pci.h
@@ -1586,6 +1586,7 @@ static inline bool pci_is_pcie(struct pci_dev *dev)
}
void pci_request_acs(void);
+struct pci_dev *pci_acs_enabled(struct pci_dev *dev);
#define PCI_VPD_LRDT 0x80 /* Large Resource Data Type */
next prev parent reply other threads:[~2012-05-11 22:56 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-11 22:55 [PATCH 00/13] IOMMU Groups + VFIO Alex Williamson
[not found] ` <20120511222148.30496.68571.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-11 22:55 ` [PATCH 01/13] driver core: Add iommu_group tracking to struct device Alex Williamson
2012-05-11 23:38 ` Greg KH
[not found] ` <20120511233821.GA9335-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2012-05-11 23:58 ` Alex Williamson
2012-05-12 0:00 ` Greg KH
2012-05-11 22:55 ` [PATCH 02/13] iommu: IOMMU Groups Alex Williamson
2012-05-11 23:39 ` Greg KH
[not found] ` <20120511233950.GB9335-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2012-05-11 23:58 ` Alex Williamson
[not found] ` <20120511225541.30496.34751.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-14 1:16 ` David Gibson
[not found] ` <20120514011627.GB30229-MK4v0fQdeXQXU02nzanrWNbf9cGiqdzd@public.gmane.org>
2012-05-14 17:11 ` Alex Williamson
[not found] ` <1337015502.6954.68.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-15 2:03 ` David Gibson
[not found] ` <20120515020328.GG30229-MK4v0fQdeXQXU02nzanrWNbf9cGiqdzd@public.gmane.org>
2012-05-15 6:34 ` Alex Williamson
[not found] ` <1337063643.6954.166.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-17 3:29 ` David Gibson
2012-05-11 22:55 ` [PATCH 03/13] iommu: IOMMU groups for VT-d and AMD-Vi Alex Williamson
[not found] ` <20120511225548.30496.34428.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-17 3:37 ` David Gibson
2012-05-11 22:55 ` [PATCH 04/13] pci: New pci_dma_quirk() Alex Williamson
[not found] ` <20120511225555.30496.80525.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-17 3:39 ` David Gibson
[not found] ` <20120517033914.GF22069-MK4v0fQdeXQXU02nzanrWNbf9cGiqdzd@public.gmane.org>
2012-05-17 4:06 ` Alex Williamson
2012-05-17 7:19 ` Anonymous
[not found] ` <CAJtEV7YMrsttPfAdL0_uht-ZRwCHhYwHTvypCc6TfEzTEGaijQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-17 15:22 ` Alex Williamson
2012-05-11 22:56 ` Alex Williamson [this message]
[not found] ` <20120511225602.30496.80438.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-14 22:02 ` [PATCH 05/13] pci: New pci_acs_enabled() Bjorn Helgaas
[not found] ` <CAErSpo58hKPFb5duOi64sWuR5AJDTVp0rRHkQLotYE3CMXRRhA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-14 22:49 ` Alex Williamson
[not found] ` <1337035766.6954.74.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-15 19:56 ` Bjorn Helgaas
[not found] ` <CAErSpo48ES5iLDCh1WzmHezSEYhPjhtgD5uxoY-TCTKgUBab2g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-05-15 20:05 ` Bjorn Helgaas
2012-05-15 21:09 ` Alex Williamson
[not found] ` <1337116157.6954.212.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-16 13:29 ` Don Dutile
[not found] ` <4FB3ABCF.4030708-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-05-16 16:21 ` Alex Williamson
[not found] ` <1337185318.6954.243.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-16 19:36 ` Alex Williamson
2012-05-18 23:00 ` RESEND3: " Don Dutile
[not found] ` <4FB6D47C.4010003-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-05-19 2:47 ` Alex Williamson
[not found] ` <1337395622.28883.43.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-21 13:31 ` Don Dutile
[not found] ` <4FBA43C9.1080509-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2012-05-21 14:59 ` Alex Williamson
[not found] ` <1337612353.28883.66.camel-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-21 18:14 ` Don Dutile
2012-05-11 22:56 ` [PATCH 06/13] iommu: Make use of DMA quirking and ACS enabled check for groups Alex Williamson
2012-05-11 22:56 ` [PATCH 07/13] vfio: VFIO core Alex Williamson
2012-05-11 22:56 ` [PATCH 08/13] vfio: Add documentation Alex Williamson
2012-05-11 22:56 ` [PATCH 09/13] vfio: x86 IOMMU implementation Alex Williamson
2012-05-11 22:56 ` [PATCH 10/13] pci: export pci_user functions for use by other drivers Alex Williamson
[not found] ` <20120511225636.30496.26907.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-14 21:20 ` Bjorn Helgaas
2012-05-11 22:56 ` [PATCH 11/13] pci: Create common pcibios_err_to_errno Alex Williamson
[not found] ` <20120511225643.30496.36823.stgit-xdHQ/5r00wBBDLzU/O5InQ@public.gmane.org>
2012-05-21 17:55 ` Konrad Rzeszutek Wilk
2012-05-11 22:56 ` [PATCH 12/13] pci: Misc pci_reg additions Alex Williamson
2012-05-11 22:56 ` [PATCH 13/13] vfio: Add PCI device driver Alex Williamson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120511225602.30496.80438.stgit@bling.home \
--to=alex.williamson-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
--cc=B07421-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
--cc=B08248-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
--cc=agraf-l3A5Bk7waGM@public.gmane.org \
--cc=aik-sLpHqDYs0B2HXe+LvDLADg@public.gmane.org \
--cc=avi-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \
--cc=benve-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org \
--cc=bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
--cc=chrisw-69jw2NvuJkxg9hUCZPvPmw@public.gmane.org \
--cc=david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org \
--cc=dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
--cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
--cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
--cc=joerg.roedel-5C7GfCeVMHo@public.gmane.org \
--cc=kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).