Re: [PATCH v2] iommu: Streamline iommu_iova_to_phys()

2021-07-26 Thread Joerg Roedel
On Thu, Jul 15, 2021 at 02:04:24PM +0100, Robin Murphy wrote: > --- > > v2: Lowered the priority of the ops check per Baolu's suggestion, only > even further to the point of non-existence :) Err, missed this newer version. Replaced the previous patch with this one.

Re: [PATCH v2] iommu: Streamline iommu_iova_to_phys()

2021-07-16 Thread Robin Murphy
On 2021-07-16 07:19, Christoph Hellwig wrote: On Thu, Jul 15, 2021 at 03:16:08PM +0100, Robin Murphy wrote: On 2021-07-15 15:07, Christoph Hellwig wrote: On Thu, Jul 15, 2021 at 02:04:24PM +0100, Robin Murphy wrote: If people are going to insist on calling iommu_iova_to_phys() pointlessly and

Re: [PATCH v2] iommu: Streamline iommu_iova_to_phys()

2021-07-16 Thread Christoph Hellwig
On Thu, Jul 15, 2021 at 03:16:08PM +0100, Robin Murphy wrote: > On 2021-07-15 15:07, Christoph Hellwig wrote: > > On Thu, Jul 15, 2021 at 02:04:24PM +0100, Robin Murphy wrote: > > > If people are going to insist on calling iommu_iova_to_phys() > > > pointlessly and expecting it to work, > > > >

Re: [PATCH v2] iommu: Streamline iommu_iova_to_phys()

2021-07-15 Thread Robin Murphy
On 2021-07-15 15:07, Christoph Hellwig wrote: On Thu, Jul 15, 2021 at 02:04:24PM +0100, Robin Murphy wrote: If people are going to insist on calling iommu_iova_to_phys() pointlessly and expecting it to work, Maybe we need to fix that? Feel free to try, but we didn't have much luck pushing

Re: [PATCH v2] iommu: Streamline iommu_iova_to_phys()

2021-07-15 Thread Christoph Hellwig
On Thu, Jul 15, 2021 at 02:04:24PM +0100, Robin Murphy wrote: > If people are going to insist on calling iommu_iova_to_phys() > pointlessly and expecting it to work, Maybe we need to fix that? ___ iommu mailing list iommu@lists.linux-foundation.org

[PATCH v2] iommu: Streamline iommu_iova_to_phys()

2021-07-15 Thread Robin Murphy
If people are going to insist on calling iommu_iova_to_phys() pointlessly and expecting it to work, we can at least do ourselves a favour by handling those cases in the core code, rather than repeatedly across an inconsistent handful of drivers. Since all the existing drivers implement the