public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
To: libc-alpha@sourceware.org
Subject: [PATCH 0/3] Improve lld support and current status
Date: Tue, 26 Oct 2021 17:03:43 -0300	[thread overview]
Message-ID: <20211026200346.3371750-1-adhemerval.zanella@linaro.org> (raw)

Ths patch allows x86_64, i686, aarch64, powerpc64le, powerpc64, and
powerpc to build, including the tests, with LLD packed in LLVM 13 [1].

For x86_64 and aarch64 there is no regression in testssuite.  On 
i686 there is only one:

  FAIL: elf/ifuncmain6pie

Which segfaults calling the 'foo' functions:

  Program received signal SIGSEGV, Segmentation fault.
  0xfb010101 in ?? ()
  (gdb) bt
  #0  0xfb010101 in ?? ()
  #1  0xf7fba620 in call_foo () at ifuncmod6.c:18
  #2  0xf7fc0857 in ?? ()
  #3  0xf7e1b195 in __libc_start_call_main (main=main@entry=0xf7fc0830,
  argc=argc@entry=2, argv=argv@entry=0xffffc610) at
  ../sysdeps/nptl/libc_start_call_main.h:58
  #4  0xf7e1b286 in __libc_start_main_impl (main=0xf7fc0830, argc=2,
  argv=0xffffc610, init=0x0, fini=0x0, rtld_fini=0xf7fe79b0 <_dl_fini>,
  stack_end=0xffffc60c) at ../csu/libc-start.c:409
  #5  0xf7fc06d7 in ?? ()

The powerpc64le fails at libc.so start:

  (gdb) bt
  #0  0x00007ffff7f2a980 in __gep_setup___vmx__sigjmp_save ()
    from
  /home/azanella/glibc/build/powerpc64le-linux-gnu-power9-lld/libc.so.6
  #1  0x00007ffff7d7ac5c in __libc_start_call_main ()
     from
  /home/azanella/glibc/build/powerpc64le-linux-gnu-power9-lld/libc.so.6
  #2  0x00007ffff7d7ae90 in __libc_start_main_impl ()
     from
  /home/azanella/glibc/build/powerpc64le-linux-gnu-power9-lld/libc.so.6
  #3  0x0000000000000000 in ?? ()
  (gdb) disas
  Dump of assembler code for function __gep_setup___vmx__sigjmp_save:
  => 0x00007ffff7f2a980 <+0>:     .long 0x613ffe6
     0x00007ffff7f2a984 <+4>:     li      r12,-1280
     0x00007ffff7f2a988 <+8>:     mtctr   r12
     0x00007ffff7f2a98c <+12>:    bctr

And powerpc64-linux-gnu fails on dynamic loaders start:

  (gdb) bt
  #0  0x00007ffff7ffa9a8 in _start ()
  (gdb) disas
  Dump of assembler code for function _start:
  => 0x00007ffff7ffa9a8 <+0>:     .long 0x0
     0x00007ffff7ffa9ac <+4>:     .long 0x0
     0x00007ffff7ffa9b0 <+8>:     .long 0x0
     0x00007ffff7ffa9b4 <+12>:    .long 0x0


The arm, sparcv9, mips, and riscv fail to build due different issues. On
arm the loader fails to build:

  ld.so fails with
  ld.lld: error: relocation R_ARM_GOTOFF32 cannot be used against symbol
  _dl_argv; recompile with -fPIC
  >>> defined in
  >>> /home/azanella/Projects/glibc/build/arm-linux-gnueabihf-lld/elf/librtld.os
  >>> referenced by rtld.c:164
  >>>               /home/azanella/Projects/glibc/build/arm-linux-gnueabihf-lld/elf/librtld.os:(.text+0xA8)

On sparcv9, lld handles --relax option different than ld.bfd: it throws
an error instead of silent ignoring it:

  $ sparc64-glibc-linux-gnu-gcc -mcpu=niagara -fuse-ld=lld -Bclang+llvm-13.0.0-x86_64-linux-gnu-ubuntu-20.04/bin
    -g -O2 -fPIC -shared -fno-stack-protector -o conftest.so conftest.c -nostdlib -nostartfiles -Wl,-z,combreloc
  ld.lld: error: unknown argument '-relax'

And even when -mno-relax is explicit add, lld does not support some relocations
generated by GCC:

  $ sparc64-glibc-linux-gnu-readelf -Wr elf/librtld.os | grep _dl_skip_args
  | head -n2
  00000000000000a0  0000012200000052 R_SPARC_GOTDATA_OP_HIX22
  0000000000000058 _dl_skip_args + 0
  00000000000000a4  0000012200000053 R_SPARC_GOTDATA_OP_LOX10
  0000000000000058 _dl_skip_args + 0

The mips/mipsel also fails to build the loader:

  ld.lld: error: can't create dynamic relocation R_MIPS_32 against local
  symbol in readonly segment; recompile object files with -fPIC or pass
  '-Wl,-z,notext' to allow text relocations in the output
  >>> defined in
  >>> /home/azanella/Projects/glibc/build/mips-linux-gnu-lld/elf/librtld.os
  >>> referenced by /home/azanella/toolchain/src/gcc/libgcc/libgcc2.c
  >>>               /home/azanella/Projects/glibc/build/mips-linux-gnu-lld/elf/librtld.os:(.eh_frame+0x1C)

  ld.lld: error: can't create dynamic relocation R_MIPS_32 against local
  symbol in readonly segment; recompile object files with -fPIC or pass
  '-Wl,-z,notext' to allow text relocations in the output
  >>> defined in
  >>> /home/azanella/Projects/glibc/build/mips-linux-gnu-lld/elf/librtld.os
  >>> referenced by /home/azanella/toolchain/src/gcc/libgcc/libgcc2.c
  >>>               /home/azanella/Projects/glibc/build/mips-linux-gnu-lld/elf/librtld.os:(.eh_frame+0x54)

Similar to mips64/mips64el:

  ld.lld: warning: ../sysdeps/unix/sysv/linux/setitimer.c:(function
  _dl_runtime_resolve: .text+0x18954): found R_MIPS_JALR relocation
  against non-function symbol . This is invalid and most likely a compiler
  bug.
  ld.lld: error: can't create dynamic relocation R_MIPS_64 against local
  symbol in readonly segment; recompile object files with -fPIC or pass
  '-Wl,-z,notext' to allow text relocations in the output
  >>> defined in
  >>> /home/azanella/Projects/glibc/build/mips64-linux-gnu-lld/elf/librtld.os
  >>> referenced by ../sysdeps/unix/sysv/linux/setitimer.c
  >>>               /home/azanella/Projects/glibc/build/mips64-linux-gnu-lld/elf/librtld.os:(.eh_frame+0x20)

The riscv32/riscv64 fails to the loader, but it is due missing support
to relaxation:

  ld.lld: error:
  /home/azanella/Projects/glibc/glibc-git/elf/rtld.c:2584:(.text+0xEC84):
  relocation R_RISCV_ALIGN requires unimplemented linker relaxation;
  recompile with -mno-relax
  ld.lld: error:
  /home/azanella/Projects/glibc/glibc-git/elf/rtld.c:2589:(.text+0x13DEC):
  relocation R_RISCV_ALIGN requires unimplemented linker relaxation;
  recompile with -mno-relax
  ld.lld: error:
  /home/azanella/Projects/glibc/glibc-git/elf/rtld.c:2589:(.text+0x13DEE):
  relocation R_RISCV_ALIGN requires unimplemented linker relaxation;
  recompile with -mno-rela

Adding -mno-relax to sysdep-CFLAGS seems to be ineffective.


[1] https://releases.llvm.org/download.html#13.0.0

Adhemerval Zanella (3):
  elf: Disable ifuncmain{1,5,5pic,5pie} when using LLD
  Fix LIBC_PROG_BINUTILS for -fuse-ld=lld
  Check if linker also support -mtls-dialect=gnu2

 aclocal.m4   |  6 +++++-
 configure    | 10 +++++++---
 configure.ac |  4 ++--
 elf/Makefile | 17 +++++++++++++----
 4 files changed, 27 insertions(+), 10 deletions(-)

-- 
2.32.0


             reply	other threads:[~2021-10-26 20:03 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-26 20:03 Adhemerval Zanella [this message]
2021-10-26 20:03 ` [PATCH 1/3] elf: Disable ifuncmain{1,5,5pic,5pie} when using LLD Adhemerval Zanella
2021-10-29 19:49   ` Fangrui Song
2021-10-26 20:03 ` [PATCH 2/3] Fix LIBC_PROG_BINUTILS for -fuse-ld=lld Adhemerval Zanella
2021-10-26 20:48   ` Fangrui Song
2021-10-27 11:42     ` Adhemerval Zanella
2021-10-26 20:03 ` [PATCH 3/3] Check if linker also support -mtls-dialect=gnu2 Adhemerval Zanella
2021-10-27  2:04   ` Fāng-ruì Sòng
2021-10-29  0:56     ` Fāng-ruì Sòng
2021-10-26 20:33 ` [PATCH 0/3] Improve lld support and current status Fangrui Song
2021-10-27 13:11   ` Adhemerval Zanella
2021-10-28  1:06     ` Fangrui Song
2021-10-28  1:18       ` Fangrui Song
2021-10-28 17:40         ` Adhemerval Zanella
2021-10-28 11:48       ` Adhemerval Zanella
2021-10-27 22:39   ` Tulio Magno Quites Machado Filho
2021-10-27 22:50     ` Tulio Magno Quites Machado Filho
2021-11-05  7:23       ` lld status with powerpc64 Fangrui Song
2021-11-05  7:45         ` Fangrui Song
2021-11-05 13:58         ` Adhemerval Zanella
2021-11-05 19:32           ` Adhemerval Zanella
2021-11-05 19:38             ` H.J. Lu
2021-11-05 19:40               ` H.J. Lu
2021-11-05 19:50               ` Fāng-ruì Sòng
2021-11-07 14:24           ` Bill Schmidt
2021-11-08 11:37             ` Adhemerval Zanella
2021-11-08 13:26               ` Bill Schmidt
2021-11-08 13:54                 ` Adhemerval Zanella
2021-11-08 13:59                   ` Bill Schmidt
2021-11-08 14:11                     ` Adhemerval Zanella
2021-11-08 14:12                       ` Bill Schmidt
     [not found]                       ` <OFD215FC7A.323066CE-ON00258787.0051DA95-00258787.00532945@ibm.com>
2021-11-08 22:38                         ` Fangrui Song
2021-11-09 12:20                           ` Adhemerval Zanella
2021-10-27 23:37     ` [PATCH 0/3] Improve lld support and current status Fangrui Song
2021-10-28 17:27       ` Tulio Magno Quites Machado Filho

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211026200346.3371750-1-adhemerval.zanella@linaro.org \
    --to=adhemerval.zanella@linaro.org \
    --cc=libc-alpha@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).