Re: [RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-23 Thread Robin Murphy
Hi Lorenzo, On 17/06/16 15:15, Lorenzo Pieralisi wrote: I think this patch makes sense even independent of the rest of the series, one nit inline notwithstanding. Thanks. Yes I added it to this series since it is not strictly necessary (ie it does not fix anything) in the mainline, but it *is*

Re: [RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-22 Thread Marek Szyprowski
Hi Lorenzo, On 2016-06-21 18:06, Lorenzo Pieralisi wrote: Hi Marek, On Tue, Jun 21, 2016 at 09:53:20AM +0200, Marek Szyprowski wrote: Hi Robin, On 2016-06-17 11:27, Robin Murphy wrote: Hi Lorenzo, I think this patch makes sense even independent of the rest of the series, one nit inline no

Re: [RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-21 Thread Lorenzo Pieralisi
Hi Marek, On Tue, Jun 21, 2016 at 09:53:20AM +0200, Marek Szyprowski wrote: > Hi Robin, > > > On 2016-06-17 11:27, Robin Murphy wrote: > >Hi Lorenzo, > > > >I think this patch makes sense even independent of the rest of the > >series, one nit inline notwithstanding. > > > >Marek; I'm curious as

Re: [RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-21 Thread Marek Szyprowski
Hi Robin, On 2016-06-17 11:27, Robin Murphy wrote: Hi Lorenzo, I think this patch makes sense even independent of the rest of the series, one nit inline notwithstanding. Marek; I'm curious as to whether this could make the workaround in 722ec35f7 obsolete as well, or are all the drivers al

Re: [RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-17 Thread Lorenzo Pieralisi
Hi Robin, On Fri, Jun 17, 2016 at 10:27:22AM +0100, Robin Murphy wrote: > Hi Lorenzo, > > I think this patch makes sense even independent of the rest of the > series, one nit inline notwithstanding. Thanks. Yes I added it to this series since it is not strictly necessary (ie it does not fix anyt

Re: [RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-17 Thread Robin Murphy
Hi Lorenzo, I think this patch makes sense even independent of the rest of the series, one nit inline notwithstanding. Marek; I'm curious as to whether this could make the workaround in 722ec35f7 obsolete as well, or are all the drivers also bound super-early in the setup you had there? On

[RFC PATCH v2 03/15] arm64: mm: change IOMMU notifier action to attach DMA ops

2016-06-07 Thread Lorenzo Pieralisi
Current bus notifier in ARM64 (__iommu_attach_notifier) attempts to attach dma_ops to a device on BUS_NOTIFY_ADD_DEVICE action notification. This causes issues on ACPI based systems, where PCI devices can be added before the IOMMUs the devices are attached to had a chance to be probed, causing fai