On Thu, Aug 29, 2019 at 12:56 AM Nick Desaulniers <[email protected]> wrote: > > diff --git a/arch/x86/include/asm/iommu_table.h > b/arch/x86/include/asm/iommu_table.h > index 1fb3fd1a83c2..7d190710eb92 100644 > --- a/arch/x86/include/asm/iommu_table.h > +++ b/arch/x86/include/asm/iommu_table.h > @@ -50,9 +50,8 @@ struct iommu_table_entry { > > #define __IOMMU_INIT(_detect, _depend, _early_init, _late_init, _finish)\ > static const struct iommu_table_entry \ > - __iommu_entry_##_detect __used \ > - __attribute__ ((unused, __section__(".iommu_table"), \ > - aligned((sizeof(void *))))) \ > + __iommu_entry_##_detect __used __section(.iommu_table) \ > + __aligned((sizeof(void *))) \ > = {_detect, _depend, _early_init, _late_init, \ > _finish ? IOMMU_FINISH_IF_DETECTED : 0} > /*
I see other patches that reduce unused -> to __unused, but is this unused -> __used intended? Cheers, Miguel
