[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <A50EA568-81E7-4AD7-8C1B-1029FEA92F49@raithlin.com>
Date: Fri, 11 May 2018 21:50:03 +0000
From: "Stephen Bates" <sbates@...thlin.com>
To: Logan Gunthorpe <logang@...tatee.com>,
Christian König <christian.koenig@....com>,
"Jerome Glisse" <jglisse@...hat.com>
CC: Alex Williamson <alex.williamson@...hat.com>,
Bjorn Helgaas <helgaas@...nel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"linux-nvme@...ts.infradead.org" <linux-nvme@...ts.infradead.org>,
"linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
"linux-nvdimm@...ts.01.org" <linux-nvdimm@...ts.01.org>,
"linux-block@...r.kernel.org" <linux-block@...r.kernel.org>,
Christoph Hellwig <hch@....de>, Jens Axboe <axboe@...nel.dk>,
Keith Busch <keith.busch@...el.com>,
Sagi Grimberg <sagi@...mberg.me>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Jason Gunthorpe <jgg@...lanox.com>,
Max Gurtovoy <maxg@...lanox.com>,
Dan Williams <dan.j.williams@...el.com>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>
Subject: Re: [PATCH v4 04/14] PCI/P2PDMA: Clear ACS P2P flags for all devices
behind switches
> I find this hard to believe. There's always the possibility that some
> part of the system doesn't support ACS so if the PCI bus addresses and
> IOVA overlap there's a good chance that P2P and ATS won't work at all on
> some hardware.
I tend to agree but this comes down to how IOVA addresses are generated in the kernel. Alex (or anyone else) can you point to where IOVA addresses are generated? As Logan stated earlier, p2pdma bypasses this and programs the PCI bus address directly but other IO going to the same PCI EP may flow through the IOMMU and be programmed with IOVA rather than PCI bus addresses.
> I prefer
> the option to disable the ACS bit on boot and let the existing code put
> the devices into their own IOMMU group (as it should already do to
> support hardware that doesn't have ACS support).
+1
Stephen
Powered by blists - more mailing lists