iommu: Replace iommu_group_do_dma_first_attach with __iommu_device_set_domain

Since __iommu_device_set_domain() now knows how to handle deferred attach
we can just call it directly from the only call site.

Reviewed-by: Kevin Tian <kevin.tian@intel.com>
Reviewed-by: Lu Baolu <baolu.lu@linux.intel.com>
Tested-by: Heiko Stuebner <heiko@sntech.de>
Tested-by: Niklas Schnelle <schnelle@linux.ibm.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
Link: https://lore.kernel.org/r/8-v5-1b99ae392328+44574-iommu_err_unwind_jgg@nvidia.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>
This commit is contained in:
Jason Gunthorpe 2023-05-11 01:42:06 -03:00 committed by Joerg Roedel
parent 0046a4337e
commit 2f74198ae0

View File

@ -106,6 +106,10 @@ enum {
IOMMU_SET_DOMAIN_MUST_SUCCEED = 1 << 0, IOMMU_SET_DOMAIN_MUST_SUCCEED = 1 << 0,
}; };
static int __iommu_device_set_domain(struct iommu_group *group,
struct device *dev,
struct iommu_domain *new_domain,
unsigned int flags);
static int __iommu_group_set_domain_internal(struct iommu_group *group, static int __iommu_group_set_domain_internal(struct iommu_group *group,
struct iommu_domain *new_domain, struct iommu_domain *new_domain,
unsigned int flags); unsigned int flags);
@ -401,17 +405,6 @@ err_unlock:
return ret; return ret;
} }
static int iommu_group_do_dma_first_attach(struct device *dev, void *data)
{
struct iommu_domain *domain = data;
lockdep_assert_held(&dev->iommu_group->mutex);
if (dev->iommu->attach_deferred)
return 0;
return __iommu_attach_device(domain, dev);
}
int iommu_probe_device(struct device *dev) int iommu_probe_device(struct device *dev)
{ {
const struct iommu_ops *ops; const struct iommu_ops *ops;
@ -442,7 +435,7 @@ int iommu_probe_device(struct device *dev)
* attach the default domain. * attach the default domain.
*/ */
if (group->default_domain && !group->owner) { if (group->default_domain && !group->owner) {
ret = iommu_group_do_dma_first_attach(dev, group->default_domain); ret = __iommu_device_set_domain(group, dev, group->domain, 0);
if (ret) { if (ret) {
mutex_unlock(&group->mutex); mutex_unlock(&group->mutex);
iommu_group_put(group); iommu_group_put(group);