Re: [PATCH v3 10/14] x86: prefer __section from compiler_attributes.h

[Miguel Ojeda]

On Thu, Aug 29, 2019 at 4:14 PM Miguel Ojeda
<miguel.ojeda.sandonis@xxxxxxxxx> wrote:
>
> On Thu, Aug 29, 2019 at 12:56 AM Nick Desaulniers
> <ndesaulniers@xxxxxxxxxx> 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