Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-10-03 Thread Mathieu Poirier
On Tue, 1 Oct 2024 at 07:35, Jason Gunthorpe wrote: > > On Mon, Sep 30, 2024 at 10:40:30AM -0600, Mathieu Poirier wrote: > > On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote: > > > An iommu domain is allocated in rproc_enable_iommu() and is attached to > > > rproc->dev.parent in the same f

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-10-01 Thread Jason Gunthorpe
On Mon, Sep 30, 2024 at 10:40:30AM -0600, Mathieu Poirier wrote: > On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote: > > An iommu domain is allocated in rproc_enable_iommu() and is attached to > > rproc->dev.parent in the same function. > > > > Use iommu_paging_domain_alloc() to make it ex

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-09-30 Thread Mathieu Poirier
On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote: > An iommu domain is allocated in rproc_enable_iommu() and is attached to > rproc->dev.parent in the same function. > > Use iommu_paging_domain_alloc() to make it explicit. > > Signed-off-by: Lu Baolu > Reviewed-by: Jason Gunthorpe > Lin

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-09-23 Thread Jason Gunthorpe
On Sun, Sep 22, 2024 at 04:57:07PM +0530, Beleswar Prasad Padhi wrote: > Since commit 17de3f5fdd35, the IOMMU framework has changed how it handles > callback ops, moving away from using bus->iommu_ops. The omap-iommu driver > has not yet been updated to align with these framework changes. Therefor

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-09-22 Thread Beleswar Prasad Padhi
On 29-08-2024 11:47, Beleswar Prasad Padhi wrote: Hi All, On 22/08/24 21:47, Mathieu Poirier wrote: Hi Baolu, Sorry for the late reply, this slipped through the cracks. On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote: > An iommu domain is allocated in rproc_enable_iommu() and is a

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-09-16 Thread Mathieu Poirier
On Sun, 15 Sept 2024 at 08:09, Jason Gunthorpe wrote: > > On Thu, Aug 22, 2024 at 01:24:25PM -0300, Jason Gunthorpe wrote: > > On Thu, Aug 22, 2024 at 10:17:56AM -0600, Mathieu Poirier wrote: > > > > > > - domain = iommu_domain_alloc(dev->bus); > > > > - if (!domain) { > > > > + domain = iommu_pag

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-09-15 Thread Jason Gunthorpe
On Thu, Aug 22, 2024 at 01:24:25PM -0300, Jason Gunthorpe wrote: > On Thu, Aug 22, 2024 at 10:17:56AM -0600, Mathieu Poirier wrote: > > > > - domain = iommu_domain_alloc(dev->bus); > > > - if (!domain) { > > > + domain = iommu_paging_domain_alloc(dev); > > > > I'm a little hesitant here. Functio

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-08-28 Thread Beleswar Prasad Padhi
Hi All, On 22/08/24 21:47, Mathieu Poirier wrote: Hi Baolu, Sorry for the late reply, this slipped through the cracks. On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote: > An iommu domain is allocated in rproc_enable_iommu() and is attached to > rproc->dev.parent in the same function. >

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-08-22 Thread Jason Gunthorpe
On Thu, Aug 22, 2024 at 10:17:56AM -0600, Mathieu Poirier wrote: > > - domain = iommu_domain_alloc(dev->bus); > > - if (!domain) { > > + domain = iommu_paging_domain_alloc(dev); > > I'm a little hesitant here. Function iommu_domain_alloc() passes NULL two the > second argument of __iommu_d

Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-08-22 Thread Mathieu Poirier
Hi Baolu, Sorry for the late reply, this slipped through the cracks. On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote: > An iommu domain is allocated in rproc_enable_iommu() and is attached to > rproc->dev.parent in the same function. > > Use iommu_paging_domain_alloc() to make it explic

[PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()

2024-08-12 Thread Lu Baolu
An iommu domain is allocated in rproc_enable_iommu() and is attached to rproc->dev.parent in the same function. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu Reviewed-by: Jason Gunthorpe Link: https://lore.kernel.org/r/2024061008.88197-13-baolu...@linux.intel.