From: Suravee Suthikulpanit Date: Thu, 15 Jan 2026 06:08:04 +0000 (+0000) Subject: iommu/amd: Make amd_iommu_make_clear_dte() non-static inline X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=11cfa782f01b2d988175011c19906180a62dd893;p=thirdparty%2Fkernel%2Flinux.git iommu/amd: Make amd_iommu_make_clear_dte() non-static inline This will be reused in a new nested.c file for nested translation. Also, remove unused function parameter ptr. Reviewed-by: Nicolin Chen Reviewed-by: Vasant Hegde Reviewed-by: Jason Gunthorpe Signed-off-by: Suravee Suthikulpanit Signed-off-by: Joerg Roedel --- diff --git a/drivers/iommu/amd/amd_iommu.h b/drivers/iommu/amd/amd_iommu.h index b742ef1adb352..8eb5e98570798 100644 --- a/drivers/iommu/amd/amd_iommu.h +++ b/drivers/iommu/amd/amd_iommu.h @@ -190,4 +190,12 @@ void amd_iommu_domain_set_pgtable(struct protection_domain *domain, struct dev_table_entry *get_dev_table(struct amd_iommu *iommu); struct iommu_dev_data *search_dev_data(struct amd_iommu *iommu, u16 devid); +static inline void +amd_iommu_make_clear_dte(struct iommu_dev_data *dev_data, struct dev_table_entry *new) +{ + /* All existing DTE must have V bit set */ + new->data128[0] = DTE_FLAG_V; + new->data128[1] = 0; +} + #endif /* AMD_IOMMU_H */ diff --git a/drivers/iommu/amd/iommu.c b/drivers/iommu/amd/iommu.c index e3d9215244eac..ae26fbe090237 100644 --- a/drivers/iommu/amd/iommu.c +++ b/drivers/iommu/amd/iommu.c @@ -2015,14 +2015,6 @@ int amd_iommu_clear_gcr3(struct iommu_dev_data *dev_data, ioasid_t pasid) return ret; } -static void make_clear_dte(struct iommu_dev_data *dev_data, struct dev_table_entry *ptr, - struct dev_table_entry *new) -{ - /* All existing DTE must have V bit set */ - new->data128[0] = DTE_FLAG_V; - new->data128[1] = 0; -} - /* * Note: * The old value for GCR3 table and GPT have been cleared from caller. @@ -2072,7 +2064,7 @@ static void set_dte_entry(struct amd_iommu *iommu, struct dev_table_entry *dte = &get_dev_table(iommu)[dev_data->devid]; struct pt_iommu_amdv1_hw_info pt_info; - make_clear_dte(dev_data, dte, &new); + amd_iommu_make_clear_dte(dev_data, &new); if (gcr3_info && gcr3_info->gcr3_tbl) domid = dev_data->gcr3_info.domid; @@ -2153,9 +2145,8 @@ static void set_dte_entry(struct amd_iommu *iommu, static void clear_dte_entry(struct amd_iommu *iommu, struct iommu_dev_data *dev_data) { struct dev_table_entry new = {}; - struct dev_table_entry *dte = &get_dev_table(iommu)[dev_data->devid]; - make_clear_dte(dev_data, dte, &new); + amd_iommu_make_clear_dte(dev_data, &new); update_dte256(iommu, dev_data, &new); }