[PATCH] powerpc: Use ALIGN instead of BLOCK

[Nick Desaulniers]

On Fri, Jan 11, 2019 at 3:51 PM Joel Stanley <joel at 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 at jms.id.au>

Should get a suggested-by tag:
Suggested-by: George Rimar <grimar at 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 at 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
>


-- 
Thanks,
~Nick Desaulniers