iommu: Remove detach_dev callback

The detach_dev callback of domain ops is not called in the IOMMU core.
Remove this callback to avoid dead code. The trace event for detaching
domain from device is removed accordingly.

Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Link: https://lore.kernel.org/r/20230110025408.667767-6-baolu.lu@linux.intel.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>

diff --git a/drivers/iommu/iommu-traces.c b/drivers/iommu/iommu-traces.c
index 1e9ca7789de105e0a0ddbaf2971ff7376e5aedba..23416bf76df97934ff011063ac99feb771ed6505 100644 (file)
--- a/drivers/iommu/iommu-traces.c
+++ b/drivers/iommu/iommu-traces.c
@@ -18,7 +18,6 @@ EXPORT_TRACEPOINT_SYMBOL_GPL(remove_device_from_group);
 
 /* iommu_device_event */
 EXPORT_TRACEPOINT_SYMBOL_GPL(attach_device_to_domain);
-EXPORT_TRACEPOINT_SYMBOL_GPL(detach_device_from_domain);
 
 /* iommu_map_unmap */
 EXPORT_TRACEPOINT_SYMBOL_GPL(map);

diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
index 85ae20c8ff5e6e02119d85ad567318b3a200b47b..9135540d7d59df67f5bc6c890e60391ac98bf650 100644 (file)
--- a/drivers/iommu/iommu.c
+++ b/drivers/iommu/iommu.c
@@ -2045,13 +2045,6 @@ int iommu_deferred_attach(struct device *dev, struct iommu_domain *domain)
        return 0;
 }
 
-static void __iommu_detach_device(struct iommu_domain *domain,
-                                 struct device *dev)
-{
-       domain->ops->detach_dev(domain, dev);
-       trace_detach_device_from_domain(dev);
-}
-
 void iommu_detach_device(struct iommu_domain *domain, struct device *dev)
 {
        struct iommu_group *group;
@@ -2156,15 +2149,6 @@ int iommu_attach_group(struct iommu_domain *domain, struct iommu_group *group)
 }
 EXPORT_SYMBOL_GPL(iommu_attach_group);
 
-static int iommu_group_do_detach_device(struct device *dev, void *data)
-{
-       struct iommu_domain *domain = data;
-
-       __iommu_detach_device(domain, dev);
-
-       return 0;
-}
-
 static int iommu_group_do_set_platform_dma(struct device *dev, void *data)
 {
        const struct iommu_ops *ops = dev_iommu_ops(dev);
@@ -2184,25 +2168,13 @@ static int __iommu_group_set_domain(struct iommu_group *group,
                return 0;
 
        /*
-        * New drivers should support default domains and so the detach_dev() op
-        * will never be called. Otherwise the NULL domain represents some
+        * New drivers should support default domains, so set_platform_dma()
+        * op will never be called. Otherwise the NULL domain represents some
         * platform specific behavior.
         */
        if (!new_domain) {
-               struct group_device *grp_dev;
-
-               grp_dev = list_first_entry(&group->devices,
-                                          struct group_device, list);
-
-               if (dev_iommu_ops(grp_dev->dev)->set_platform_dma_ops)
-                       __iommu_group_for_each_dev(group, NULL,
-                                       iommu_group_do_set_platform_dma);
-               else if (group->domain->ops->detach_dev)
-                       __iommu_group_for_each_dev(group, group->domain,
-                                       iommu_group_do_detach_device);
-               else
-                       WARN_ON_ONCE(1);
-
+               __iommu_group_for_each_dev(group, NULL,
+                                          iommu_group_do_set_platform_dma);
                group->domain = NULL;
                return 0;
        }

diff --git a/include/linux/iommu.h b/include/linux/iommu.h
index 0d10566b3cb2ff911eecc6c92862c9db7c51011a..a8063f26ff69dd5074f793acf13c7583045e9be6 100644 (file)
--- a/include/linux/iommu.h
+++ b/include/linux/iommu.h
@@ -299,7 +299,6 @@ struct iommu_ops {
  * * EBUSY     - device is attached to a domain and cannot be changed
  * * ENODEV    - device specific errors, not able to be attached
  * * <others>  - treated as ENODEV by the caller. Use is discouraged
- * @detach_dev: detach an iommu domain from a device
  * @set_dev_pasid: set an iommu domain to a pasid of device
  * @map: map a physically contiguous memory region to an iommu domain
  * @map_pages: map a physically contiguous set of pages of the same size to
@@ -320,7 +319,6 @@ struct iommu_ops {
  */
 struct iommu_domain_ops {
        int (*attach_dev)(struct iommu_domain *domain, struct device *dev);
-       void (*detach_dev)(struct iommu_domain *domain, struct device *dev);
        int (*set_dev_pasid)(struct iommu_domain *domain, struct device *dev,
                             ioasid_t pasid);
 

diff --git a/include/trace/events/iommu.h b/include/trace/events/iommu.h
index 29096fe12623d2b8537200aa441f0761e1f01f40..70743db1fb758fcc7a541d737a0776cd5544923f 100644 (file)
--- a/include/trace/events/iommu.h
+++ b/include/trace/events/iommu.h
@@ -76,13 +76,6 @@ DEFINE_EVENT(iommu_device_event, attach_device_to_domain,
        TP_ARGS(dev)
 );
 
-DEFINE_EVENT(iommu_device_event, detach_device_from_domain,
-
-       TP_PROTO(struct device *dev),
-
-       TP_ARGS(dev)
-);
-
 TRACE_EVENT(map,
 
        TP_PROTO(unsigned long iova, phys_addr_t paddr, size_t size),