lkml.org 
[lkml]   [2023]   [Jul]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v4 1/4] iommu: Move dev_iommu_ops() to private header
    Date
    dev_iommu_ops() is essentially only used in iommu subsystem, so
    move to a private header to avoid being abused by other drivers.

    Suggested-by: Jason Gunthorpe <jgg@nvidia.com>
    Reviewed-by: Kevin Tian <kevin.tian@intel.com>
    Reviewed-by: Lu Baolu <baolu.lu@linux.intel.com>
    Signed-off-by: Yi Liu <yi.l.liu@intel.com>
    ---
    drivers/iommu/iommu-priv.h | 11 +++++++++++
    include/linux/iommu.h | 11 -----------
    2 files changed, 11 insertions(+), 11 deletions(-)

    diff --git a/drivers/iommu/iommu-priv.h b/drivers/iommu/iommu-priv.h
    index 7c8011bfd153..a6e694f59f64 100644
    --- a/drivers/iommu/iommu-priv.h
    +++ b/drivers/iommu/iommu-priv.h
    @@ -4,6 +4,17 @@

    #include <linux/iommu.h>

    +static inline const struct iommu_ops *dev_iommu_ops(struct device *dev)
    +{
    + /*
    + * Assume that valid ops must be installed if iommu_probe_device()
    + * has succeeded. The device ops are essentially for internal use
    + * within the IOMMU subsystem itself, so we should be able to trust
    + * ourselves not to misuse the helper.
    + */
    + return dev->iommu->iommu_dev->ops;
    +}
    +
    int iommu_group_replace_domain(struct iommu_group *group,
    struct iommu_domain *new_domain);

    diff --git a/include/linux/iommu.h b/include/linux/iommu.h
    index d31642596675..e0245aa82b75 100644
    --- a/include/linux/iommu.h
    +++ b/include/linux/iommu.h
    @@ -450,17 +450,6 @@ static inline void iommu_iotlb_gather_init(struct iommu_iotlb_gather *gather)
    };
    }

    -static inline const struct iommu_ops *dev_iommu_ops(struct device *dev)
    -{
    - /*
    - * Assume that valid ops must be installed if iommu_probe_device()
    - * has succeeded. The device ops are essentially for internal use
    - * within the IOMMU subsystem itself, so we should be able to trust
    - * ourselves not to misuse the helper.
    - */
    - return dev->iommu->iommu_dev->ops;
    -}
    -
    extern int bus_iommu_probe(const struct bus_type *bus);
    extern bool iommu_present(const struct bus_type *bus);
    extern bool device_iommu_capable(struct device *dev, enum iommu_cap cap);
    --
    2.34.1
    \
     
     \ /
      Last update: 2023-07-24 13:00    [W:6.960 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site