Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-11-04 Thread Stefano Stabellini
On Wed, 30 Oct 2013, Stefano Stabellini wrote: > On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: > > So... what I'm saying is please fix xgmac to use the DMA API properly > > rather than working around such problems. :) > > OK. > Would the appended patch fix your concerns? ping? > --- >

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-11-04 Thread Stefano Stabellini
On Wed, 30 Oct 2013, Stefano Stabellini wrote: On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: So... what I'm saying is please fix xgmac to use the DMA API properly rather than working around such problems. :) OK. Would the appended patch fix your concerns? ping? --- arm: make

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Stefano Stabellini
On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: > So... what I'm saying is please fix xgmac to use the DMA API properly > rather than working around such problems. :) OK. Would the appended patch fix your concerns? --- arm: make SWIOTLB available IOMMU_HELPER is needed because SWIOTLB

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Russell King - ARM Linux
On Tue, Oct 29, 2013 at 05:24:50PM +, Stefano Stabellini wrote: > On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: > > This looks like a hack. Either we want to use the streaming mask or > > the coherent mask as appropriate for the caller. That should be a choice > > the caller makes,

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Stefano Stabellini
On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: > On Tue, Oct 29, 2013 at 10:20:23AM -0400, Konrad Rzeszutek Wilk wrote: > > On Tue, Oct 29, 2013 at 04:41:40AM +, Stefano Stabellini wrote: > > > ping? > > > > You know you are pinging yourself, right ? :-) > > And the patch was only

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Russell King - ARM Linux
On Tue, Oct 29, 2013 at 10:20:23AM -0400, Konrad Rzeszutek Wilk wrote: > On Tue, Oct 29, 2013 at 04:41:40AM +, Stefano Stabellini wrote: > > ping? > > You know you are pinging yourself, right ? :-) And the patch was only Cc'd. I'm starting to read stuff which isn't flagged as having me in

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Konrad Rzeszutek Wilk
On Tue, Oct 29, 2013 at 04:41:40AM +, Stefano Stabellini wrote: > ping? You know you are pinging yourself, right ? :-) > > On Fri, 25 Oct 2013, Stefano Stabellini wrote: > > Russell, > > this is the only patch that needs an ack at the moment. > > As you commented on it before and I have

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Konrad Rzeszutek Wilk
On Tue, Oct 29, 2013 at 04:41:40AM +, Stefano Stabellini wrote: ping? You know you are pinging yourself, right ? :-) On Fri, 25 Oct 2013, Stefano Stabellini wrote: Russell, this is the only patch that needs an ack at the moment. As you commented on it before and I have already

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Russell King - ARM Linux
On Tue, Oct 29, 2013 at 10:20:23AM -0400, Konrad Rzeszutek Wilk wrote: On Tue, Oct 29, 2013 at 04:41:40AM +, Stefano Stabellini wrote: ping? You know you are pinging yourself, right ? :-) And the patch was only Cc'd. I'm starting to read stuff which isn't flagged as having me in the

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Stefano Stabellini
On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: On Tue, Oct 29, 2013 at 10:20:23AM -0400, Konrad Rzeszutek Wilk wrote: On Tue, Oct 29, 2013 at 04:41:40AM +, Stefano Stabellini wrote: ping? You know you are pinging yourself, right ? :-) And the patch was only Cc'd. I'm

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Russell King - ARM Linux
On Tue, Oct 29, 2013 at 05:24:50PM +, Stefano Stabellini wrote: On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: This looks like a hack. Either we want to use the streaming mask or the coherent mask as appropriate for the caller. That should be a choice the caller makes, not the

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-29 Thread Stefano Stabellini
On Tue, 29 Oct 2013, Russell King - ARM Linux wrote: So... what I'm saying is please fix xgmac to use the DMA API properly rather than working around such problems. :) OK. Would the appended patch fix your concerns? --- arm: make SWIOTLB available IOMMU_HELPER is needed because SWIOTLB

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-28 Thread Stefano Stabellini
ping? On Fri, 25 Oct 2013, Stefano Stabellini wrote: > Russell, > this is the only patch that needs an ack at the moment. > As you commented on it before and I have already addressed your comments > few versions ago, unless you have any complaints I am going to add it to > linux-next and I am

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-28 Thread Stefano Stabellini
ping? On Fri, 25 Oct 2013, Stefano Stabellini wrote: Russell, this is the only patch that needs an ack at the moment. As you commented on it before and I have already addressed your comments few versions ago, unless you have any complaints I am going to add it to linux-next and I am thinking

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-25 Thread Stefano Stabellini
Russell, this is the only patch that needs an ack at the moment. As you commented on it before and I have already addressed your comments few versions ago, unless you have any complaints I am going to add it to linux-next and I am thinking of merging it during the next merge window. On Fri, 25

[PATCH v9 01/18] arm: make SWIOTLB available

2013-10-25 Thread Stefano Stabellini
IOMMU_HELPER is needed because SWIOTLB calls iommu_is_span_boundary, provided by lib/iommu_helper.c. Signed-off-by: Stefano Stabellini Reviewed-by: Konrad Rzeszutek Wilk CC: will.dea...@arm.com CC: li...@arm.linux.org.uk Changes in v8: - use __phys_to_pfn and __pfn_to_phys. Changes in v7: -

[PATCH v9 01/18] arm: make SWIOTLB available

2013-10-25 Thread Stefano Stabellini
IOMMU_HELPER is needed because SWIOTLB calls iommu_is_span_boundary, provided by lib/iommu_helper.c. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com Reviewed-by: Konrad Rzeszutek Wilk konrad.w...@oracle.com CC: will.dea...@arm.com CC: li...@arm.linux.org.uk Changes in v8: -

Re: [PATCH v9 01/18] arm: make SWIOTLB available

2013-10-25 Thread Stefano Stabellini
Russell, this is the only patch that needs an ack at the moment. As you commented on it before and I have already addressed your comments few versions ago, unless you have any complaints I am going to add it to linux-next and I am thinking of merging it during the next merge window. On Fri, 25