From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59598) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZRGt2-0000B9-BJ for qemu-devel@nongnu.org; Mon, 17 Aug 2015 05:39:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZRGsz-0002Mc-0y for qemu-devel@nongnu.org; Mon, 17 Aug 2015 05:39:24 -0400 Received: from gate.crashing.org ([63.228.1.57]:44551) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZRGsy-0002La-Oa for qemu-devel@nongnu.org; Mon, 17 Aug 2015 05:39:20 -0400 Message-ID: <1439804354.2416.10.camel@kernel.crashing.org> From: Benjamin Herrenschmidt Date: Mon, 17 Aug 2015 19:39:14 +1000 In-Reply-To: <1436051955.3948.29.camel@kernel.crashing.org> References: <1436051955.3948.29.camel@kernel.crashing.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] pci: Fix pci_device_iommu_address_space() bus propagation List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: "Michael S. Tsirkin" On Sun, 2015-07-05 at 09:19 +1000, Benjamin Herrenschmidt wrote: > he current code walks up the bus tree for an iommu, however it passes > to the iommu_fn() callback the bus/devfn of the immediate child of > the level where the callback was found, rather than the original > bus/devfn where the search started from. Hi Michael ! Any comment on this ? I'd like to post my series for "bare metal" power8 support and the iommu implementation relies on this to work. > This prevents iommu's like POWER8 (and in fact also Q35) to properly > provide an address space for a subset of devices that aren't > immediate > children of the iommu. > > PCIe carries the originator bdfn acccross to the iommu on all DMA > transactions, so we must be able to properly identify devices at all > levels. > > This changes the function pci_device_iommu_address_space() to pass > the original pointers to the iommu_fn() callback instead. > > Signed-off-by: Benjamin Herrenschmidt > --- > > With this, I can implement PHB3's (POWER8) iommu properly, I haven't > submitted the P8 native patch series yet but if you are curious, you > can look there: > > https://github.com/ozbenh/qemu > > And more specifically: > > https://github.com/ozbenh/qemu/commit/67fe0460c75417908a2b54426cb54fe > 5a1299a13 > > For the PHB3 code. > > hw/pci/pci.c | 13 +++++-------- > 1 file changed, 5 insertions(+), 8 deletions(-) > > diff --git a/hw/pci/pci.c b/hw/pci/pci.c > index 29f0b0f..8185bbc 100644 > --- a/hw/pci/pci.c > +++ b/hw/pci/pci.c > @@ -2403,17 +2403,14 @@ static void pci_device_class_init(ObjectClass > *klass, void *data) > AddressSpace *pci_device_iommu_address_space(PCIDevice *dev) > { > PCIBus *bus = PCI_BUS(dev->bus); > + PCIBus *iommu_bus = bus; > > - if (bus->iommu_fn) { > - return bus->iommu_fn(bus, bus->iommu_opaque, dev->devfn); > + while(iommu_bus && !iommu_bus->iommu_fn && iommu_bus > ->parent_dev) { > + iommu_bus = PCI_BUS(iommu_bus->parent_dev->bus); > } > - > - if (bus->parent_dev) { > - /** We are ignoring the bus master DMA bit of the bridge > - * as it would complicate things such as VFIO for no good > reason */ > - return pci_device_iommu_address_space(bus->parent_dev); > + if (iommu_bus && iommu_bus->iommu_fn) { > + return iommu_bus->iommu_fn(bus, iommu_bus->iommu_opaque, dev > ->devfn); > } > - > return &address_space_memory; > } > >