[PATCH] iommu/amd: define 'amd_ivhd_dev_flags_list' in header

Ben Dooks posted 1 patch 1 month ago
drivers/iommu/amd/amd_iommu_types.h | 1 +
1 file changed, 1 insertion(+)
[PATCH] iommu/amd: define 'amd_ivhd_dev_flags_list' in header
Posted by Ben Dooks 1 month ago
The 'amd_ivhd_dev_flags_list' is used in the amd_iommu_types.h
header but not defined, so add the definition to remove the
following sparse warning:

drivers/iommu/amd/init.c:182:1: warning: symbol 'amd_ivhd_dev_flags_list' was not declared. Should it be static?

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 drivers/iommu/amd/amd_iommu_types.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/iommu/amd/amd_iommu_types.h b/drivers/iommu/amd/amd_iommu_types.h
index 320733e7d8b4..2ccb629e3243 100644
--- a/drivers/iommu/amd/amd_iommu_types.h
+++ b/drivers/iommu/amd/amd_iommu_types.h
@@ -454,6 +454,7 @@ extern bool amdr_ivrs_remap_support;
 #define for_each_pdom_dev_data_safe(pdom_dev_data, next, pdom) \
 	list_for_each_entry_safe((pdom_dev_data), (next), &pdom->dev_data_list, list)
 
+extern struct list_head amd_ivhd_dev_flags_list;
 #define for_each_ivhd_dte_flags(entry) \
 	list_for_each_entry((entry), &amd_ivhd_dev_flags_list, list)
 
-- 
2.37.2.352.g3c44437643
Re: [PATCH] iommu/amd: define 'amd_ivhd_dev_flags_list' in header
Posted by Jörg Rödel 4 weeks ago
On Thu, Jan 08, 2026 at 10:25:41AM +0000, Ben Dooks wrote:
> The 'amd_ivhd_dev_flags_list' is used in the amd_iommu_types.h
> header but not defined, so add the definition to remove the
> following sparse warning:

I think it is better to just move the for_each_ivhd_dte_flags macro over to
init.c. There is no need for this list to be visible across the entire driver.

-Joerg