Message ID | 20190111235056.27435-1-joel@jms.id.au (mailing list archive) |
---|---|
State | Accepted |
Commit | a652758ac1475f69d28d11b3528c4f489416c877 |
Headers | show |
Series | powerpc: Use ALIGN instead of BLOCK | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | next/apply_patch Successfully applied |
snowpatch_ozlabs/build-ppc64le | success | build succeeded & removed 0 sparse warning(s) |
snowpatch_ozlabs/build-ppc64be | success | build succeeded & removed 0 sparse warning(s) |
snowpatch_ozlabs/build-ppc64e | success | build succeeded & removed 0 sparse warning(s) |
snowpatch_ozlabs/build-pmac32 | success | build succeeded & removed 0 sparse warning(s) |
snowpatch_ozlabs/checkpatch | warning | total: 0 errors, 1 warnings, 0 checks, 13 lines checked |
On Fri, Jan 11, 2019 at 3:51 PM Joel Stanley <joel@jms.id.au> wrote: > > In the ld documentation under Builtin Functions: > > BLOCK(exp) > > This is a synonym for ALIGN, for compatibility with older linker scripts. > > Clang's linker (lld) doesn't know about BLOCK so remove this use of it. > > Link: https://github.com/ClangBuiltLinux/linux/issues/253 > Signed-off-by: Joel Stanley <joel@jms.id.au> Should get a suggested-by tag: Suggested-by: George Rimar <grimar@accesssoftek.com> via: https://github.com/ClangBuiltLinux/linux/issues/253#issuecomment-449715453 https://reviews.llvm.org/D56060#1340554 ftp://ftp.gnu.org/old-gnu/Manuals/ld-2.9.1/html_chapter/ld_3.html#SEC14 documents ALIGN while for BLOCK: ftp://ftp.gnu.org/old-gnu/Manuals/ld-2.9.1/html_chapter/ld_3.html#SEC21 Looks like they do the same thing. Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> > --- > arch/powerpc/kernel/vmlinux.lds.S | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/kernel/vmlinux.lds.S b/arch/powerpc/kernel/vmlinux.lds.S > index ad1c77f71f54..3ae4c959f95b 100644 > --- a/arch/powerpc/kernel/vmlinux.lds.S > +++ b/arch/powerpc/kernel/vmlinux.lds.S > @@ -86,11 +86,11 @@ SECTIONS > > #ifdef CONFIG_PPC64 > /* > - * BLOCK(0) overrides the default output section alignment because > + * ALIGN(0) overrides the default output section alignment because > * this needs to start right after .head.text in order for fixed > * section placement to work. > */ > - .text BLOCK(0) : AT(ADDR(.text) - LOAD_OFFSET) { > + .text ALIGN(0) : AT(ADDR(.text) - LOAD_OFFSET) { > #ifdef CONFIG_LD_HEAD_STUB_CATCH > KEEP(*(.linker_stub_catch)); > . = . ; > -- > 2.19.1 >
diff --git a/arch/powerpc/kernel/vmlinux.lds.S b/arch/powerpc/kernel/vmlinux.lds.S index ad1c77f71f54..3ae4c959f95b 100644 --- a/arch/powerpc/kernel/vmlinux.lds.S +++ b/arch/powerpc/kernel/vmlinux.lds.S @@ -86,11 +86,11 @@ SECTIONS #ifdef CONFIG_PPC64 /* - * BLOCK(0) overrides the default output section alignment because + * ALIGN(0) overrides the default output section alignment because * this needs to start right after .head.text in order for fixed * section placement to work. */ - .text BLOCK(0) : AT(ADDR(.text) - LOAD_OFFSET) { + .text ALIGN(0) : AT(ADDR(.text) - LOAD_OFFSET) { #ifdef CONFIG_LD_HEAD_STUB_CATCH KEEP(*(.linker_stub_catch)); . = . ;
In the ld documentation under Builtin Functions: BLOCK(exp) This is a synonym for ALIGN, for compatibility with older linker scripts. Clang's linker (lld) doesn't know about BLOCK so remove this use of it. Link: https://github.com/ClangBuiltLinux/linux/issues/253 Signed-off-by: Joel Stanley <joel@jms.id.au> --- arch/powerpc/kernel/vmlinux.lds.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)