On Wed, Aug 23, 2023 at 01:47:38PM -0300, Jason Gunthorpe wrote: > These drivers don't support IOMMU_DOMAIN_DMA, so this commit effectively > allows them to support that mode. > > The prior work to require default_domains makes this safe because every > one of these drivers is either compilation incompatible with dma-iommu.c, > or already establishing a default_domain. In both cases alloc_domain() > will never be called with IOMMU_DOMAIN_DMA for these drivers so it is safe > to drop the test. > > Removing these tests clarifies that the domain allocation path is only > about the functionality of a paging domain and has nothing to do with > policy of how the paging domain is used for UNMANAGED/DMA/DMA_FQ. > > Tested-by: Niklas Schnelle <schne...@linux.ibm.com> > Tested-by: Steven Price <steven.pr...@arm.com> > Tested-by: Marek Szyprowski <m.szyprow...@samsung.com> > Tested-by: Nicolin Chen <nicol...@nvidia.com> > Reviewed-by: Lu Baolu <baolu...@linux.intel.com> > Signed-off-by: Jason Gunthorpe <j...@nvidia.com> > --- > drivers/iommu/msm_iommu.c | 7 ++----- > drivers/iommu/mtk_iommu_v1.c | 7 ++----- > drivers/iommu/omap-iommu.c | 7 ++----- > drivers/iommu/s390-iommu.c | 7 ++----- > 4 files changed, 8 insertions(+), 20 deletions(-) >
Reviewed-by: Jerry Snitselaar <jsnit...@redhat.com>
