linux-kernel - Re: [PATCH v3 10/14] x86: prefer __section from compiler

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CANiq72mTMdwXKBOEvfqA_KzuTjwnsHQ4rNdNczAXeX_5nzYo4g@mail.gmail.com>
Date:   Thu, 29 Aug 2019 16:22:05 +0200
From:   Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To:     Nick Desaulniers <ndesaulniers@...gle.com>
Cc:     Sedat Dilek <sedat.dilek@...il.com>, Will Deacon <will@...nel.org>,
        Josh Poimboeuf <jpoimboe@...hat.com>,
        naveen.n.rao@...ux.vnet.ibm.com,
        David Miller <davem@...emloft.net>,
        Paul Burton <paul.burton@...s.com>,
        clang-built-linux <clang-built-linux@...glegroups.com>,
        linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 10/14] x86: prefer __section from compiler_attributes.h

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