Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-30 Thread Lorenzo Pieralisi
7; ; > >linux-arm-ker...@lists.infradead.org; m.szyprow...@samsung.com > >Subject: Re: [PATCH V3 0/8] IOMMU probe deferral support > > > >On Mon, Nov 28, 2016 at 11:12:08PM +0530, Sricharan wrote: > > > >[...] > > > >> >Cool. We're rather hoping tha

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-29 Thread Sricharan
a...@arm.com; >tf...@chromium.org; iommu@lists.linux- >foundation.org; srinivas.kandaga...@linaro.org; >laurent.pinch...@ideasonboard.com; 'Robin Murphy' ; >linux-arm-ker...@lists.infradead.org; m.szyprow...@samsung.com >Subject: Re: [PATCH V3 0/8] IOMMU probe deferral support

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-28 Thread Lorenzo Pieralisi
On Mon, Nov 28, 2016 at 11:12:08PM +0530, Sricharan wrote: [...] > >Cool. We're rather hoping that the ACPI stuff is good to go for 4.10 > >now, so it's probably worth pulling the rest of that in (beyond the one > >patch I picked) to make sure the of_dma_configure/acpi_dma_configure > >paths don'

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-28 Thread Sricharan
Hi Robin, >> >> >>> iommu_group_get_for_dev which gets called in the add_device callback, increases the reference count of the iommu_group, so we do an iommu_group_put after that. iommu_group_get_for_dev inturn calls device_group callback and in the case of arm

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-24 Thread Robin Murphy
On 24/11/16 16:10, Sricharan wrote: > Hi Robin, > > > >> >>> iommu_group_get_for_dev which gets called in the add_device >>> callback, increases the reference count of the iommu_group, >>> so we do an iommu_group_put after that. iommu_group_get_for_dev >>> inturn calls device_gro

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-24 Thread Sricharan
Hi Robin, > >> iommu_group_get_for_dev which gets called in the add_device >> callback, increases the reference count of the iommu_group, >> so we do an iommu_group_put after that. iommu_group_get_for_dev >> inturn calls device_group callback and in the case of arm-smmu >> we

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-23 Thread Robin Murphy
On 20/11/16 15:11, Sricharan wrote: > Hi Robin, > >> Hi Robin, >> >>> Hi Robin, >>> On 04/11/16 15:16, Sricharan wrote: > Hi Robin, > Yikes, on second look, that definitely shouldn't be happening. Everything below is probably the resulting fallout. >>> >>> [

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-20 Thread Sricharan
Hi Robin, >Hi Robin, > >>Hi Robin, >> >>>On 04/11/16 15:16, Sricharan wrote: Hi Robin, >>> Yikes, on second look, that definitely shouldn't be happening. >>> Everything below is probably the resulting fallout. >> >> [ 40.206703] vfio-pci :08:00.0: Failed to setup io

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-13 Thread Sricharan
Hi Robin, >Hi Robin, > >>On 04/11/16 15:16, Sricharan wrote: >>> Hi Robin, >>> >> Yikes, on second look, that definitely shouldn't be happening. >> Everything below is probably the resulting fallout. > > [ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops > > I

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-09 Thread Will Deacon
On Wed, Nov 09, 2016 at 11:54:20AM +0530, Sricharan wrote: > >> diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c > >> index 71ce4b6..a1d0b3c 100644 > >> --- a/drivers/iommu/arm-smmu.c > >> +++ b/drivers/iommu/arm-smmu.c > >> @@ -1516,8 +1516,10 @@ static struct iommu_group > >> *ar

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-08 Thread Sricharan
Hi Robin, >On 04/11/16 15:16, Sricharan wrote: >> Hi Robin, >> > Yikes, on second look, that definitely shouldn't be happening. > Everything below is probably the resulting fallout. [ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops I think the above print

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-07 Thread Robin Murphy
Hi Sricharan, On 04/11/16 15:16, Sricharan wrote: > Hi Robin, > Yikes, on second look, that definitely shouldn't be happening. Everything below is probably the resulting fallout. >>> >>> [ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops >>> >>> I think the above print whic

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-07 Thread Will Deacon
On Fri, Nov 04, 2016 at 08:46:06PM +0530, Sricharan wrote: > >>>Yikes, on second look, that definitely shouldn't be happening. > >>>Everything below is probably the resulting fallout. > >> > >>[ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops > >> > >>I think the above print which say

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-04 Thread Sricharan
Hi Robin, >>>Yikes, on second look, that definitely shouldn't be happening. >>>Everything below is probably the resulting fallout. >> >>[ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops >> >>I think the above print which says "failed to setup iommu_ops" >>because the call ops->add_de

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-11-03 Thread Sricharan
Hi Robin, [ 39.901592] iommu: Removing device :08:00.0 from group 0 >> >>Yikes, on second look, that definitely shouldn't be happening. >>Everything below is probably the resulting fallout. > >[ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops > >I think the above print whi

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-27 Thread Sricharan
Hi Robin, >> >> >> >>> [ 39.901592] iommu: Removing device :08:00.0 from group 0 > >Yikes, on second look, that definitely shouldn't be happening. >Everything below is probably the resulting fallout. [ 40.206703] vfio-pci :08:00.0: Failed to setup iommu ops I think the above print wh

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-26 Thread Robin Murphy
On 26/10/16 15:44, Sricharan wrote: > Hi Robin, > >> On 04/10/16 18:03, Sricharan R wrote: >>> Initial post from Laurent Pinchart[1]. This is >>> series calls the dma ops configuration for the devices >>> at a generic place so that it works for all busses. >>> The dma_configure_ops for a device is

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-26 Thread Sricharan
Hi Robin, >On 04/10/16 18:03, Sricharan R wrote: >> Initial post from Laurent Pinchart[1]. This is >> series calls the dma ops configuration for the devices >> at a generic place so that it works for all busses. >> The dma_configure_ops for a device is now called during >> the device_attach callba

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-25 Thread Robin Murphy
Hi Sricharan, On 04/10/16 18:03, Sricharan R wrote: > Initial post from Laurent Pinchart[1]. This is > series calls the dma ops configuration for the devices > at a generic place so that it works for all busses. > The dma_configure_ops for a device is now called during > the device_attach callback

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-24 Thread Archit Taneja
On 10/04/2016 10:33 PM, Sricharan R wrote: Initial post from Laurent Pinchart[1]. This is series calls the dma ops configuration for the devices at a generic place so that it works for all busses. The dma_configure_ops for a device is now called during the device_attach callback just before the

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-24 Thread Sricharan
Hi Marek, Initial post from Laurent Pinchart[1]. This is series calls the dma ops configuration for the devices at a generic place so that it works for all busses. The dma_configure_ops for a device is now called during the device_attach callback just before the probe of t

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-23 Thread Marek Szyprowski
Hi Sricharan, On 2016-10-12 08:24, Sricharan wrote: On 2016-10-04 19:03, Sricharan R wrote: Initial post from Laurent Pinchart[1]. This is series calls the dma ops configuration for the devices at a generic place so that it works for all busses. The dma_configure_ops for a device is now called

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-17 Thread Sricharan
bin.mur...@arm.com; j...@8bytes.org; iommu@lists.linux- >foundation.org; linux-arm-ker...@lists.infradead.org; >linux-arm-...@vger.kernel.org; laurent.pinch...@ideasonboard.com; >tf...@chromium.org; srinivas.kandaga...@linaro.org >Subject: Re: [PATCH V3 0/8] IOMMU probe deferral support

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-16 Thread Sricharan
;iommu@lists.linux-foundation.org' foundation.org>; 'linux-arm-ker...@lists.infradead.org' >; 'linux-arm-...@vger.kernel.org' >; 'laurent.pinch...@ideasonboard.com' >; >'tf...@chromium.org' ; 'srinivas.kandaga...@linaro.org' &

RE: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-11 Thread Sricharan
Hi Marek, >Hi Sricharan, > > >On 2016-10-04 19:03, Sricharan R wrote: >> Initial post from Laurent Pinchart[1]. This is >> series calls the dma ops configuration for the devices >> at a generic place so that it works for all busses. >> The dma_configure_ops for a device is now called during >> the

Re: [PATCH V3 0/8] IOMMU probe deferral support

2016-10-10 Thread Marek Szyprowski
Hi Sricharan, On 2016-10-04 19:03, Sricharan R wrote: Initial post from Laurent Pinchart[1]. This is series calls the dma ops configuration for the devices at a generic place so that it works for all busses. The dma_configure_ops for a device is now called during the device_attach callback just

[PATCH V3 0/8] IOMMU probe deferral support

2016-10-04 Thread Sricharan R
Initial post from Laurent Pinchart[1]. This is series calls the dma ops configuration for the devices at a generic place so that it works for all busses. The dma_configure_ops for a device is now called during the device_attach callback just before the probe of the bus/driver is called. Similarly d