On Thu, Aug 29, 2019 at 4:14 PM Miguel Ojeda <miguel.ojeda.sando...@gmail.com> wrote: > > On Thu, Aug 29, 2019 at 12:56 AM Nick Desaulniers > <ndesaulni...@google.com> 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
s/unused/other attrs like aligned and print/ Cheers, Miguel