[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANiq72m+_QYtn_1gyrjXFs6yeDdiMoS4DVcWqYcTgyCFnSFXbw@mail.gmail.com>
Date: Wed, 21 Oct 2020 19:54:11 +0200
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Nick Desaulniers <ndesaulniers@...gle.com>
Cc: Peter Zijlstra <peterz@...radead.org>,
Jakub Jelinek <jakub@...hat.com>,
Florian Fainelli <f.fainelli@...il.com>,
kernel test robot <lkp@...el.com>,
Steven Rostedt <rostedt@...dmis.org>,
LKML <linux-kernel@...r.kernel.org>,
"maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)" <x86@...nel.org>,
LKP <lkp@...ts.01.org>, Kees Cook <keescook@...omium.org>,
"H.J. Lu" <hjl.tools@...il.com>,
Rasmus Villemoes <linux@...musvillemoes.dk>,
linux-toolchains@...r.kernel.org
Subject: Re: GCC section alignment, and GCC-4.9 being a weird one
On Wed, Oct 21, 2020 at 7:42 PM Nick Desaulniers
<ndesaulniers@...gle.com> wrote:
>
> If you used some of the macros from
> include/linux/compiler_attributes.h like __section and __aligned, I
> would prefer it. Please consider spelling out __attribute__(()) an
> antipattern.
+1, the shorthands should be used unless there is a reason not to (and
please write the reason in a comment in that case).
Cheers,
Miguel
Powered by blists - more mailing lists