elf: Fix dynamic-link.h usage on rtld.c
Checks
Context |
Check |
Description |
dj/TryBot-apply_patch |
success
|
Patch applied to master at the time it was sent
|
dj/TryBot-32bit |
success
|
Build for i686
|
Commit Message
The 4af6982e4c fix does not fully handle RTLD_BOOTSTRAP usage on
rtld.c due two issues:
1. RTLD_BOOTSTRAP is also used on dl-machine.h on various
architectures and it changes the semantics of various machine
relocation functions.
2. The elf_get_dynamic_info() change was done sideways, previously
to 490e6c62aa get-dynamic-info.h was included by the first
dynamic-link.h include *without* RTLD_BOOTSTRAP being defined.
It means that the code within elf_get_dynamic_info() that uses
RTLD_BOOTSTRAP is in fact unused.
To fix 1. this patch now includes dynamic-link.h only once with
RTLD_BOOTSTRAP defined. The ELF_DYNAMIC_RELOCATE call will now have
the relocation fnctions with the expected semantics for the loader.
And to fix 2. part of 4af6982e4c is reverted (the check argument
elf_get_dynamic_info() is not required) and the RTLD_BOOTSTRAP
pieces are removed.
To reorganize the included the static TLS definition is moved to
its own header to avoid a circular dependency (it is defined on
dynamic-link.h and dl-machine.h requires it at same time other
dynamic-link.h definition requires dl-machine.h defitions).
Also ELF_MACHINE_NO_REL, ELF_MACHINE_NO_RELA, and ELF_MACHINE_PLT_REL
are moved to its own header. Only ancient ABIs need special values
(arm, i386, and mips), so a generic one is used as default.
The powerpc Elf64_FuncDesc is also moved to its own header, since
csu code required its definition (which would require either include
elf/ folder or add a full path with elf/).
Checked on x86_64, i686, aarch64, armhf, powerpc64, powerpc32,
and powerpc64le.
---
elf/dl-load.c | 2 +-
elf/dl-reloc-static-pie.c | 2 +-
elf/dl-static-tls.h | 51 +++++++++++++++++++++
elf/dynamic-link.h | 33 +------------
elf/get-dynamic-info.h | 25 +++-------
elf/rtld.c | 23 +++++-----
elf/setup-vdso.h | 2 +-
sysdeps/aarch64/dl-machine.h | 8 ++--
sysdeps/alpha/dl-machine.h | 6 +--
sysdeps/arc/dl-machine.h | 10 ++--
sysdeps/arm/dl-machine-rel.h | 31 +++++++++++++
sysdeps/arm/dl-machine.h | 11 +----
sysdeps/csky/dl-machine.h | 6 +--
sysdeps/generic/dl-machine-rel.h | 27 +++++++++++
sysdeps/generic/dl-machine.h | 2 +
sysdeps/hppa/dl-machine.h | 6 +--
sysdeps/i386/dl-machine-rel.h | 31 +++++++++++++
sysdeps/i386/dl-machine.h | 11 +----
sysdeps/ia64/dl-machine.h | 6 +--
sysdeps/m68k/dl-machine.h | 6 +--
sysdeps/microblaze/dl-machine.h | 6 +--
sysdeps/mips/dl-machine-rel.h | 26 +++++++++++
sysdeps/mips/dl-machine.h | 6 +--
sysdeps/nios2/dl-machine.h | 6 +--
sysdeps/powerpc/powerpc32/dl-machine.h | 6 +--
sysdeps/powerpc/powerpc64/dl-funcdesc.h | 34 ++++++++++++++
sysdeps/powerpc/powerpc64/dl-irel.h | 2 +-
sysdeps/powerpc/powerpc64/dl-machine.h | 19 ++------
sysdeps/riscv/dl-machine.h | 5 +-
sysdeps/s390/s390-32/dl-machine.h | 6 +--
sysdeps/s390/s390-64/dl-machine.h | 6 +--
sysdeps/sh/dl-machine.h | 6 +--
sysdeps/sparc/sparc32/dl-machine.h | 6 +--
sysdeps/sparc/sparc64/dl-machine.h | 6 +--
sysdeps/unix/sysv/linux/powerpc/libc-vdso.h | 2 +-
sysdeps/x86_64/dl-machine.h | 6 +--
36 files changed, 272 insertions(+), 175 deletions(-)
create mode 100644 elf/dl-static-tls.h
create mode 100644 sysdeps/arm/dl-machine-rel.h
create mode 100644 sysdeps/generic/dl-machine-rel.h
create mode 100644 sysdeps/i386/dl-machine-rel.h
create mode 100644 sysdeps/mips/dl-machine-rel.h
create mode 100644 sysdeps/powerpc/powerpc64/dl-funcdesc.h
Comments
It runs fine on i386 and x64.
Manuel
On Wed, Oct 13, 2021 at 9:25 PM Adhemerval Zanella
<adhemerval.zanella@linaro.org> wrote:
>
> The 4af6982e4c fix does not fully handle RTLD_BOOTSTRAP usage on
> rtld.c due two issues:
>
> 1. RTLD_BOOTSTRAP is also used on dl-machine.h on various
> architectures and it changes the semantics of various machine
> relocation functions.
>
> 2. The elf_get_dynamic_info() change was done sideways, previously
> to 490e6c62aa get-dynamic-info.h was included by the first
> dynamic-link.h include *without* RTLD_BOOTSTRAP being defined.
> It means that the code within elf_get_dynamic_info() that uses
> RTLD_BOOTSTRAP is in fact unused.
>
> To fix 1. this patch now includes dynamic-link.h only once with
> RTLD_BOOTSTRAP defined. The ELF_DYNAMIC_RELOCATE call will now have
> the relocation fnctions with the expected semantics for the loader.
>
> And to fix 2. part of 4af6982e4c is reverted (the check argument
> elf_get_dynamic_info() is not required) and the RTLD_BOOTSTRAP
> pieces are removed.
>
> To reorganize the included the static TLS definition is moved to
> its own header to avoid a circular dependency (it is defined on
> dynamic-link.h and dl-machine.h requires it at same time other
> dynamic-link.h definition requires dl-machine.h defitions).
>
> Also ELF_MACHINE_NO_REL, ELF_MACHINE_NO_RELA, and ELF_MACHINE_PLT_REL
> are moved to its own header. Only ancient ABIs need special values
> (arm, i386, and mips), so a generic one is used as default.
>
> The powerpc Elf64_FuncDesc is also moved to its own header, since
> csu code required its definition (which would require either include
> elf/ folder or add a full path with elf/).
>
> Checked on x86_64, i686, aarch64, armhf, powerpc64, powerpc32,
> and powerpc64le.
> ---
> elf/dl-load.c | 2 +-
> elf/dl-reloc-static-pie.c | 2 +-
> elf/dl-static-tls.h | 51 +++++++++++++++++++++
> elf/dynamic-link.h | 33 +------------
> elf/get-dynamic-info.h | 25 +++-------
> elf/rtld.c | 23 +++++-----
> elf/setup-vdso.h | 2 +-
> sysdeps/aarch64/dl-machine.h | 8 ++--
> sysdeps/alpha/dl-machine.h | 6 +--
> sysdeps/arc/dl-machine.h | 10 ++--
> sysdeps/arm/dl-machine-rel.h | 31 +++++++++++++
> sysdeps/arm/dl-machine.h | 11 +----
> sysdeps/csky/dl-machine.h | 6 +--
> sysdeps/generic/dl-machine-rel.h | 27 +++++++++++
> sysdeps/generic/dl-machine.h | 2 +
> sysdeps/hppa/dl-machine.h | 6 +--
> sysdeps/i386/dl-machine-rel.h | 31 +++++++++++++
> sysdeps/i386/dl-machine.h | 11 +----
> sysdeps/ia64/dl-machine.h | 6 +--
> sysdeps/m68k/dl-machine.h | 6 +--
> sysdeps/microblaze/dl-machine.h | 6 +--
> sysdeps/mips/dl-machine-rel.h | 26 +++++++++++
> sysdeps/mips/dl-machine.h | 6 +--
> sysdeps/nios2/dl-machine.h | 6 +--
> sysdeps/powerpc/powerpc32/dl-machine.h | 6 +--
> sysdeps/powerpc/powerpc64/dl-funcdesc.h | 34 ++++++++++++++
> sysdeps/powerpc/powerpc64/dl-irel.h | 2 +-
> sysdeps/powerpc/powerpc64/dl-machine.h | 19 ++------
> sysdeps/riscv/dl-machine.h | 5 +-
> sysdeps/s390/s390-32/dl-machine.h | 6 +--
> sysdeps/s390/s390-64/dl-machine.h | 6 +--
> sysdeps/sh/dl-machine.h | 6 +--
> sysdeps/sparc/sparc32/dl-machine.h | 6 +--
> sysdeps/sparc/sparc64/dl-machine.h | 6 +--
> sysdeps/unix/sysv/linux/powerpc/libc-vdso.h | 2 +-
> sysdeps/x86_64/dl-machine.h | 6 +--
> 36 files changed, 272 insertions(+), 175 deletions(-)
> create mode 100644 elf/dl-static-tls.h
> create mode 100644 sysdeps/arm/dl-machine-rel.h
> create mode 100644 sysdeps/generic/dl-machine-rel.h
> create mode 100644 sysdeps/i386/dl-machine-rel.h
> create mode 100644 sysdeps/mips/dl-machine-rel.h
> create mode 100644 sysdeps/powerpc/powerpc64/dl-funcdesc.h
>
> diff --git a/elf/dl-load.c b/elf/dl-load.c
> index 335c79fad5..18d3e8fe64 100644
> --- a/elf/dl-load.c
> +++ b/elf/dl-load.c
> @@ -1298,7 +1298,7 @@ _dl_map_object_from_fd (const char *name, const char *origname, int fd,
> else
> l->l_ld = (ElfW(Dyn) *) ((ElfW(Addr)) l->l_ld + l->l_addr);
>
> - elf_get_dynamic_info (l, false);
> + elf_get_dynamic_info (l);
>
> /* Make sure we are not dlopen'ing an object that has the
> DF_1_NOOPEN flag set, or a PIE object. */
> diff --git a/elf/dl-reloc-static-pie.c b/elf/dl-reloc-static-pie.c
> index 8a99c4995b..40a031f2e9 100644
> --- a/elf/dl-reloc-static-pie.c
> +++ b/elf/dl-reloc-static-pie.c
> @@ -52,7 +52,7 @@ _dl_relocate_static_pie (void)
> break;
> }
>
> - elf_get_dynamic_info (main_map, false);
> + elf_get_dynamic_info (main_map);
>
> # ifdef ELF_MACHINE_BEFORE_RTLD_RELOC
> ELF_MACHINE_BEFORE_RTLD_RELOC (main_map, main_map->l_info);
> diff --git a/elf/dl-static-tls.h b/elf/dl-static-tls.h
> new file mode 100644
> index 0000000000..730924fc01
> --- /dev/null
> +++ b/elf/dl-static-tls.h
> @@ -0,0 +1,51 @@
> +/* Inline functions for dynamic linking.
> + Copyright (C) 1995-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_STATIC_TLS_H
> +#define _DL_STATIC_TLS_H
> +
> +/* This macro is used as a callback from elf_machine_rel{a,} when a
> + static TLS reloc is about to be performed. Since (in dl-load.c) we
> + permit dynamic loading of objects that might use such relocs, we
> + have to check whether each use is actually doable. If the object
> + whose TLS segment the reference resolves to was allocated space in
> + the static TLS block at startup, then it's ok. Otherwise, we make
> + an attempt to allocate it in surplus space on the fly. If that
> + can't be done, we fall back to the error that DF_STATIC_TLS is
> + intended to produce. */
> +#define HAVE_STATIC_TLS(map, sym_map) \
> + (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET \
> + && ((sym_map)->l_tls_offset \
> + != FORCED_DYNAMIC_TLS_OFFSET), 1))
> +
> +#define CHECK_STATIC_TLS(map, sym_map) \
> + do { \
> + if (!HAVE_STATIC_TLS (map, sym_map)) \
> + _dl_allocate_static_tls (sym_map); \
> + } while (0)
> +
> +#define TRY_STATIC_TLS(map, sym_map) \
> + (__builtin_expect ((sym_map)->l_tls_offset \
> + != FORCED_DYNAMIC_TLS_OFFSET, 1) \
> + && (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET, 1) \
> + || _dl_try_allocate_static_tls (sym_map, true) == 0))
> +
> +int _dl_try_allocate_static_tls (struct link_map *map, bool optional)
> + attribute_hidden;
> +
> +#endif
> diff --git a/elf/dynamic-link.h b/elf/dynamic-link.h
> index 21cdfc88bb..ac4cc70dea 100644
> --- a/elf/dynamic-link.h
> +++ b/elf/dynamic-link.h
> @@ -16,35 +16,7 @@
> License along with the GNU C Library; if not, see
> <https://www.gnu.org/licenses/>. */
>
> -/* This macro is used as a callback from elf_machine_rel{a,} when a
> - static TLS reloc is about to be performed. Since (in dl-load.c) we
> - permit dynamic loading of objects that might use such relocs, we
> - have to check whether each use is actually doable. If the object
> - whose TLS segment the reference resolves to was allocated space in
> - the static TLS block at startup, then it's ok. Otherwise, we make
> - an attempt to allocate it in surplus space on the fly. If that
> - can't be done, we fall back to the error that DF_STATIC_TLS is
> - intended to produce. */
> -#define HAVE_STATIC_TLS(map, sym_map) \
> - (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET \
> - && ((sym_map)->l_tls_offset \
> - != FORCED_DYNAMIC_TLS_OFFSET), 1))
> -
> -#define CHECK_STATIC_TLS(map, sym_map) \
> - do { \
> - if (!HAVE_STATIC_TLS (map, sym_map)) \
> - _dl_allocate_static_tls (sym_map); \
> - } while (0)
> -
> -#define TRY_STATIC_TLS(map, sym_map) \
> - (__builtin_expect ((sym_map)->l_tls_offset \
> - != FORCED_DYNAMIC_TLS_OFFSET, 1) \
> - && (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET, 1) \
> - || _dl_try_allocate_static_tls (sym_map, true) == 0))
> -
> -int _dl_try_allocate_static_tls (struct link_map *map, bool optional)
> - attribute_hidden;
> -
> +#include <dl-machine.h>
> #include <elf.h>
>
> #ifdef RESOLVE_MAP
> @@ -91,9 +63,6 @@ elf_machine_lazy_rel (struct link_map *map, struct r_scope_elem *scope[],
> # endif
> #endif
>
> -#include <dl-machine.h>
> -
> -
> #ifdef RESOLVE_MAP
>
> # if defined RTLD_BOOTSTRAP || defined STATIC_PIE_BOOTSTRAP
> diff --git a/elf/get-dynamic-info.h b/elf/get-dynamic-info.h
> index d169099fbc..1ac0663d1f 100644
> --- a/elf/get-dynamic-info.h
> +++ b/elf/get-dynamic-info.h
> @@ -22,10 +22,11 @@
> #define _GET_DYNAMIC_INFO_H
>
> #include <assert.h>
> +#include <dl-machine-rel.h>
> #include <libc-diag.h>
>
> static inline void __attribute__ ((unused, always_inline))
> -elf_get_dynamic_info (struct link_map *l, bool check)
> +elf_get_dynamic_info (struct link_map *l)
> {
> #if __ELF_NATIVE_CLASS == 32
> typedef Elf32_Word d_tag_utype;
> @@ -33,7 +34,7 @@ elf_get_dynamic_info (struct link_map *l, bool check)
> typedef Elf64_Xword d_tag_utype;
> #endif
>
> -#if !defined RTLD_BOOTSTRAP && !defined STATIC_PIE_BOOTSTRAP
> +#ifndef STATIC_PIE_BOOTSTRAP
> if (l->l_ld == NULL)
> return;
> #endif
> @@ -111,21 +112,10 @@ elf_get_dynamic_info (struct link_map *l, bool check)
> if (info[DT_REL] != NULL)
> assert (info[DT_RELENT]->d_un.d_val == sizeof (ElfW(Rel)));
> #endif
> -#ifdef RTLD_BOOTSTRAP
> - if (check)
> - {
> - /* Only the bind now flags are allowed. */
> - assert (info[VERSYMIDX (DT_FLAGS_1)] == NULL
> - || (info[VERSYMIDX (DT_FLAGS_1)]->d_un.d_val & ~DF_1_NOW) == 0);
> - /* Flags must not be set for ld.so. */
> - assert (info[DT_FLAGS] == NULL
> - || (info[DT_FLAGS]->d_un.d_val & ~DF_BIND_NOW) == 0);
> -# ifdef STATIC_PIE_BOOTSTRAP
> - assert (info[DT_RUNPATH] == NULL);
> - assert (info[DT_RPATH] == NULL);
> -# endif
> - }
> -#else
> +#ifdef STATIC_PIE_BOOTSTRAP
> + assert (info[DT_RUNPATH] == NULL);
> + assert (info[DT_RPATH] == NULL);
> +#endif
> if (info[DT_FLAGS] != NULL)
> {
> /* Flags are used. Translate to the old form where available.
> @@ -163,7 +153,6 @@ elf_get_dynamic_info (struct link_map *l, bool check)
> if (info[DT_RUNPATH] != NULL)
> /* If both RUNPATH and RPATH are given, the latter is ignored. */
> info[DT_RPATH] = NULL;
> -#endif
> }
>
> #endif
> diff --git a/elf/rtld.c b/elf/rtld.c
> index fe5ee9ec4d..6cfb7cf672 100644
> --- a/elf/rtld.c
> +++ b/elf/rtld.c
> @@ -32,7 +32,6 @@
> #include <fpu_control.h>
> #include <hp-timing.h>
> #include <libc-lock.h>
> -#include "dynamic-link.h"
> #include <dl-librecon.h>
> #include <unsecvars.h>
> #include <dl-cache.h>
> @@ -50,9 +49,18 @@
> #include <dl-main.h>
> #include <gnu/lib-names.h>
> #include <dl-tunables.h>
> +#include <get-dynamic-info.h>
>
> #include <assert.h>
>
> +/* This #define produces dynamic linking inline functions for
> + bootstrap relocation instead of general-purpose relocation.
> + Since ld.so must not have any undefined symbols the result
> + is trivial: always the map of ld.so itself. */
> +#define RTLD_BOOTSTRAP
> +#define RESOLVE_MAP(map, scope, sym, version, flags) map
> +#include "dynamic-link.h"
> +
> /* Only enables rtld profiling for architectures which provides non generic
> hp-timing support. The generic support requires either syscall
> (clock_gettime), which will incur in extra overhead on loading time.
> @@ -507,15 +515,6 @@ _dl_start_final (void *arg, struct dl_start_final_info *info)
> # define bootstrap_map info.l
> #endif
>
> - /* This #define produces dynamic linking inline functions for
> - bootstrap relocation instead of general-purpose relocation.
> - Since ld.so must not have any undefined symbols the result
> - is trivial: always the map of ld.so itself. */
> -#define RTLD_BOOTSTRAP
> -#define RESOLVE_MAP(map, scope, sym, version, flags) map
> -#include "get-dynamic-info.h"
> -#include "dynamic-link.h"
> -
> static ElfW(Addr) __attribute_used__
> _dl_start (void *arg)
> {
> @@ -548,7 +547,7 @@ _dl_start (void *arg)
>
> /* Read our own dynamic section and fill in the info array. */
> bootstrap_map.l_ld = (void *) bootstrap_map.l_addr + elf_machine_dynamic ();
> - elf_get_dynamic_info (&bootstrap_map, true);
> + elf_get_dynamic_info (&bootstrap_map);
>
> #if NO_TLS_OFFSET != 0
> bootstrap_map.l_tls_offset = NO_TLS_OFFSET;
> @@ -1616,7 +1615,7 @@ dl_main (const ElfW(Phdr) *phdr,
> if (! rtld_is_main)
> {
> /* Extract the contents of the dynamic section for easy access. */
> - elf_get_dynamic_info (main_map, false);
> + elf_get_dynamic_info (main_map);
>
> /* If the main map is libc.so, update the base namespace to
> refer to this map. If libc.so is loaded later, this happens
> diff --git a/elf/setup-vdso.h b/elf/setup-vdso.h
> index 6fdffafcca..f44748bc98 100644
> --- a/elf/setup-vdso.h
> +++ b/elf/setup-vdso.h
> @@ -64,7 +64,7 @@ setup_vdso (struct link_map *main_map __attribute__ ((unused)),
> l->l_map_end += l->l_addr;
> l->l_text_end += l->l_addr;
> l->l_ld = (void *) ((ElfW(Addr)) l->l_ld + l->l_addr);
> - elf_get_dynamic_info (l, false);
> + elf_get_dynamic_info (l);
> _dl_setup_hash (l);
> l->l_relocated = 1;
>
> diff --git a/sysdeps/aarch64/dl-machine.h b/sysdeps/aarch64/dl-machine.h
> index e30e81c902..0d5ad218c1 100644
> --- a/sysdeps/aarch64/dl-machine.h
> +++ b/sysdeps/aarch64/dl-machine.h
> @@ -24,7 +24,9 @@
> #include <sysdep.h>
> #include <tls.h>
> #include <dl-tlsdesc.h>
> +#include <dl-static-tls.h>
> #include <dl-irel.h>
> +#include <dl-machine-rel.h>
> #include <cpu-features.c>
>
> /* Translate a processor specific dynamic tag to the index in l_info array. */
> @@ -190,10 +192,6 @@ _dl_start_user: \n\
>
> #define ELF_MACHINE_JMP_SLOT AARCH64_R(JUMP_SLOT)
>
> -/* AArch64 uses RELA not REL */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> #define DL_PLATFORM_INIT dl_platform_init ()
>
> static inline void __attribute__ ((unused))
> @@ -370,7 +368,7 @@ elf_machine_rela (struct link_map *map, struct r_scope_elem *scope[],
> }
> }
>
> -inline void
> +static inline void
> __attribute__ ((always_inline))
> elf_machine_rela_relative (ElfW(Addr) l_addr,
> const ElfW(Rela) *reloc,
> diff --git a/sysdeps/alpha/dl-machine.h b/sysdeps/alpha/dl-machine.h
> index 491541326f..7d6282e599 100644
> --- a/sysdeps/alpha/dl-machine.h
> +++ b/sysdeps/alpha/dl-machine.h
> @@ -25,6 +25,8 @@
> #define ELF_MACHINE_NAME "alpha"
>
> #include <string.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
>
> /* Mask identifying addresses reserved for the user program,
> @@ -240,10 +242,6 @@ $fixup_stack: \n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_ALPHA_JMP_SLOT
>
> -/* The alpha never uses Elf64_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> * _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/arc/dl-machine.h b/sysdeps/arc/dl-machine.h
> index 4b64ffec25..f843ed9bd6 100644
> --- a/sysdeps/arc/dl-machine.h
> +++ b/sysdeps/arc/dl-machine.h
> @@ -30,6 +30,8 @@
> #include <string.h>
> #include <link.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Dynamic Linking ABI for ARCv2 ISA.
>
> @@ -203,10 +205,6 @@ __start: \n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_ARC_JUMP_SLOT
>
> -/* ARC uses Rela relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Fixup a PLT entry to bounce directly to the function at VALUE. */
>
> static inline ElfW(Addr)
> @@ -318,7 +316,7 @@ elf_machine_rela (struct link_map *map, struct r_scope_elem *scope[],
> }
> }
>
> -inline void
> +static inline void
> __attribute__ ((always_inline))
> elf_machine_rela_relative (ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
> void *const reloc_addr_arg)
> @@ -327,7 +325,7 @@ elf_machine_rela_relative (ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
> *reloc_addr += l_addr;
> }
>
> -inline void
> +static inline void
> __attribute__ ((always_inline))
> elf_machine_lazy_rel (struct link_map *map, struct r_scope_elem *scope[],
> ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
> diff --git a/sysdeps/arm/dl-machine-rel.h b/sysdeps/arm/dl-machine-rel.h
> new file mode 100644
> index 0000000000..bec114706c
> --- /dev/null
> +++ b/sysdeps/arm/dl-machine-rel.h
> @@ -0,0 +1,31 @@
> +/* ELF dynamic relocation type supported by the architecture. ARM version.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> +#define ELF_MACHINE_NO_REL 0
> +
> +/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_PLT_REL 1
> +
> +#endif
> diff --git a/sysdeps/arm/dl-machine.h b/sysdeps/arm/dl-machine.h
> index dfa05eee44..c758bba169 100644
> --- a/sysdeps/arm/dl-machine.h
> +++ b/sysdeps/arm/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <tls.h>
> #include <dl-tlsdesc.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #ifndef CLEAR_CACHE
> # error CLEAR_CACHE definition required to handle TEXTREL
> @@ -231,10 +233,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_ARM_JUMP_SLOT
>
> -/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_PLT_REL 1
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> @@ -267,11 +265,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rel *reloc,
> #endif /* !dl_machine_h */
>
>
> -/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> -#define ELF_MACHINE_NO_REL 0
> -
> /* Names of the architecture-specific auditing callback functions. */
> #define ARCH_LA_PLTENTER arm_gnu_pltenter
> #define ARCH_LA_PLTEXIT arm_gnu_pltexit
> diff --git a/sysdeps/csky/dl-machine.h b/sysdeps/csky/dl-machine.h
> index ec22f87577..4dfd957877 100644
> --- a/sysdeps/csky/dl-machine.h
> +++ b/sysdeps/csky/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <sys/param.h>
> #include <sysdep.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero if ELF header is compatible with the running host. */
> static inline int
> @@ -172,10 +174,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_CKCORE_JUMP_SLOT
>
> -/* C-SKY never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/generic/dl-machine-rel.h b/sysdeps/generic/dl-machine-rel.h
> new file mode 100644
> index 0000000000..9167a1dffc
> --- /dev/null
> +++ b/sysdeps/generic/dl-machine-rel.h
> @@ -0,0 +1,27 @@
> +/* ELF dynamic relocation type supported by the architecture.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +/* Defined if the architecture supports Elf{32,64}_Rel relocations. */
> +#define ELF_MACHINE_NO_REL 1
> +/* Defined if the architecture supports Elf{32,64}_Rela relocations. */
> +#define ELF_MACHINE_NO_RELA 0
> +
> +#endif
> diff --git a/sysdeps/generic/dl-machine.h b/sysdeps/generic/dl-machine.h
> index 4a4ab4fc70..7da695d903 100644
> --- a/sysdeps/generic/dl-machine.h
> +++ b/sysdeps/generic/dl-machine.h
> @@ -20,6 +20,8 @@
>
> #include <string.h>
> #include <link.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> diff --git a/sysdeps/hppa/dl-machine.h b/sysdeps/hppa/dl-machine.h
> index e6a4575fce..7c5d0e9430 100644
> --- a/sysdeps/hppa/dl-machine.h
> +++ b/sysdeps/hppa/dl-machine.h
> @@ -29,6 +29,8 @@
> #include <dl-fptr.h>
> #include <abort-instr.h>
> #include <tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* These two definitions must match the definition of the stub in
> bfd/elf32-hppa.c (see plt_stub[]).
> @@ -509,10 +511,6 @@ asm ( \
> #define ELF_MACHINE_JMP_SLOT R_PARISC_IPLT
> #define ELF_MACHINE_SIZEOF_JMP_SLOT PLT_ENTRY_SIZE
>
> -/* We only use RELA. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Return the address of the entry point. */
> #define ELF_MACHINE_START_ADDRESS(map, start) \
> ({ \
> diff --git a/sysdeps/i386/dl-machine-rel.h b/sysdeps/i386/dl-machine-rel.h
> new file mode 100644
> index 0000000000..7ac46f78a6
> --- /dev/null
> +++ b/sysdeps/i386/dl-machine-rel.h
> @@ -0,0 +1,31 @@
> +/* ELF dynamic relocation type supported by the architecture. ARM version.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> +#define ELF_MACHINE_NO_REL 0
> +
> +/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_PLT_REL 1
> +
> +#endif
> diff --git a/sysdeps/i386/dl-machine.h b/sysdeps/i386/dl-machine.h
> index 78bed03eee..169bed50ff 100644
> --- a/sysdeps/i386/dl-machine.h
> +++ b/sysdeps/i386/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <sysdep.h>
> #include <tls.h>
> #include <dl-tlsdesc.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute__ ((unused))
> @@ -230,10 +232,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_386_JMP_SLOT
>
> -/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_PLT_REL 1
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> @@ -276,11 +274,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rel *reloc,
>
> #endif /* !dl_machine_h */
>
> -/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> -#define ELF_MACHINE_NO_REL 0
> -
> #ifdef RESOLVE_MAP
>
> /* Perform the relocation specified by RELOC and SYM (which is fully resolved).
> diff --git a/sysdeps/ia64/dl-machine.h b/sysdeps/ia64/dl-machine.h
> index 2217d0b556..c9608a51b0 100644
> --- a/sysdeps/ia64/dl-machine.h
> +++ b/sysdeps/ia64/dl-machine.h
> @@ -27,6 +27,8 @@
> #include <errno.h>
> #include <dl-fptr.h>
> #include <tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> @@ -319,10 +321,6 @@ elf_machine_runtime_setup (struct link_map *l, struct r_scope_elem *scope[],
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_IA64_IPLTLSB
>
> -/* According to the IA-64 specific documentation, Rela is always used. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Return the address of the entry point. */
> #define ELF_MACHINE_START_ADDRESS(map, start) \
> ({ \
> diff --git a/sysdeps/m68k/dl-machine.h b/sysdeps/m68k/dl-machine.h
> index 5e34c4784e..30323d62d4 100644
> --- a/sysdeps/m68k/dl-machine.h
> +++ b/sysdeps/m68k/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <sys/param.h>
> #include <sysdep.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -183,10 +185,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_68K_JMP_SLOT
>
> -/* The m68k never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> static inline Elf32_Addr
> elf_machine_fixup_plt (struct link_map *map, lookup_t t,
> const ElfW(Sym) *refsym, const ElfW(Sym) *sym,
> diff --git a/sysdeps/microblaze/dl-machine.h b/sysdeps/microblaze/dl-machine.h
> index 3fd4988e60..b8cc5a7fe6 100644
> --- a/sysdeps/microblaze/dl-machine.h
> +++ b/sysdeps/microblaze/dl-machine.h
> @@ -23,6 +23,8 @@
>
> #include <sys/param.h>
> #include <tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -169,10 +171,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_MICROBLAZE_JUMP_SLOT
>
> -/* The microblaze never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> static inline Elf32_Addr
> elf_machine_fixup_plt (struct link_map *map, lookup_t t,
> const ElfW(Sym) *refsym, const ElfW(Sym) *sym,
> diff --git a/sysdeps/mips/dl-machine-rel.h b/sysdeps/mips/dl-machine-rel.h
> new file mode 100644
> index 0000000000..ed39618041
> --- /dev/null
> +++ b/sysdeps/mips/dl-machine-rel.h
> @@ -0,0 +1,26 @@
> +/* ELF dynamic relocation type supported by the architecture. ARM version.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +#define ELF_MACHINE_PLT_REL 1
> +#define ELF_MACHINE_NO_REL 0
> +#define ELF_MACHINE_NO_RELA 0
> +
> +#endif
> diff --git a/sysdeps/mips/dl-machine.h b/sysdeps/mips/dl-machine.h
> index f8d1298b60..d7b8341b74 100644
> --- a/sysdeps/mips/dl-machine.h
> +++ b/sysdeps/mips/dl-machine.h
> @@ -32,6 +32,8 @@
> #include <sysdep.h>
> #include <sys/asm.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* The offset of gp from GOT might be system-dependent. It's set by
> ld. The same value is also */
> @@ -59,10 +61,6 @@
> ((((type) == ELF_MACHINE_JMP_SLOT) * ELF_RTYPE_CLASS_PLT) \
> | (((type) == R_MIPS_COPY) * ELF_RTYPE_CLASS_COPY))
>
> -#define ELF_MACHINE_PLT_REL 1
> -#define ELF_MACHINE_NO_REL 0
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> #define DT_MIPS(x) (DT_MIPS_##x - DT_LOPROC + DT_NUM)
> diff --git a/sysdeps/nios2/dl-machine.h b/sysdeps/nios2/dl-machine.h
> index 4de602b13d..430ca5d7ae 100644
> --- a/sysdeps/nios2/dl-machine.h
> +++ b/sysdeps/nios2/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <string.h>
> #include <link.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -200,10 +202,6 @@ _start:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_NIOS2_JUMP_SLOT
>
> -/* The Nios II never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Fixup a PLT entry to bounce directly to the function at VALUE. */
>
> static inline Elf32_Addr
> diff --git a/sysdeps/powerpc/powerpc32/dl-machine.h b/sysdeps/powerpc/powerpc32/dl-machine.h
> index cda012dc1b..8d062951ce 100644
> --- a/sysdeps/powerpc/powerpc32/dl-machine.h
> +++ b/sysdeps/powerpc/powerpc32/dl-machine.h
> @@ -25,6 +25,8 @@
> #include <dl-tls.h>
> #include <dl-irel.h>
> #include <hwcapinfo.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> @@ -145,10 +147,6 @@ __elf_preferred_address(struct link_map *loader, size_t maplength,
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_PPC_JMP_SLOT
>
> -/* The PowerPC never uses REL relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization function to initialize HWCAP/HWCAP2 and
> platform data so it can be copied into the TCB later. This is called
> very early in _dl_sysdep_start for dynamically linked binaries. */
> diff --git a/sysdeps/powerpc/powerpc64/dl-funcdesc.h b/sysdeps/powerpc/powerpc64/dl-funcdesc.h
> new file mode 100644
> index 0000000000..b2d1f76ce0
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/dl-funcdesc.h
> @@ -0,0 +1,34 @@
> +/* PowerPC ELFv1 function descriptor definition.
> + Copyright (C) 2009-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_FUNCDESC_H
> +#define _DL_FUNCDESC_H
> +
> +#if _CALL_ELF != 2
> +/* A PowerPC64 function descriptor. The .plt (procedure linkage
> + table) and .opd (official procedure descriptor) sections are
> + arrays of these. */
> +typedef struct
> +{
> + Elf64_Addr fd_func;
> + Elf64_Addr fd_toc;
> + Elf64_Addr fd_aux;
> +} Elf64_FuncDesc;
> +#endif
> +
> +#endif
> diff --git a/sysdeps/powerpc/powerpc64/dl-irel.h b/sysdeps/powerpc/powerpc64/dl-irel.h
> index 0e11b7ff64..aa9a2dca71 100644
> --- a/sysdeps/powerpc/powerpc64/dl-irel.h
> +++ b/sysdeps/powerpc/powerpc64/dl-irel.h
> @@ -23,7 +23,7 @@
> #include <stdio.h>
> #include <unistd.h>
> #include <ldsodefs.h>
> -#include <dl-machine.h>
> +#include <dl-funcdesc.h>
>
> #define ELF_MACHINE_IRELA 1
>
> diff --git a/sysdeps/powerpc/powerpc64/dl-machine.h b/sysdeps/powerpc/powerpc64/dl-machine.h
> index 3f92fbb369..3a4a21a412 100644
> --- a/sysdeps/powerpc/powerpc64/dl-machine.h
> +++ b/sysdeps/powerpc/powerpc64/dl-machine.h
> @@ -28,23 +28,14 @@
> #include <sysdep.h>
> #include <hwcapinfo.h>
> #include <cpu-features.c>
> +#include <dl-static-tls.h>
> +#include <dl-funcdesc.h>
> +#include <dl-machine-rel.h>
>
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> #define DT_PPC64(x) (DT_PPC64_##x - DT_LOPROC + DT_NUM)
>
> -#if _CALL_ELF != 2
> -/* A PowerPC64 function descriptor. The .plt (procedure linkage
> - table) and .opd (official procedure descriptor) sections are
> - arrays of these. */
> -typedef struct
> -{
> - Elf64_Addr fd_func;
> - Elf64_Addr fd_toc;
> - Elf64_Addr fd_aux;
> -} Elf64_FuncDesc;
> -#endif
> -
> #define ELF_MULT_MACHINES_SUPPORTED
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> @@ -292,10 +283,6 @@ BODY_PREFIX "_dl_start_user:\n" \
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_PPC64_JMP_SLOT
>
> -/* The PowerPC never uses REL relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization function to initialize HWCAP/HWCAP2 and
> platform data so it can be copied into the TCB later. This is called
> very early in _dl_sysdep_start for dynamically linked binaries. */
> diff --git a/sysdeps/riscv/dl-machine.h b/sysdeps/riscv/dl-machine.h
> index e7289d7ddc..ce2b3c3875 100644
> --- a/sysdeps/riscv/dl-machine.h
> +++ b/sysdeps/riscv/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <sys/asm.h>
> #include <dl-tls.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #ifndef _RTLD_PROLOGUE
> # define _RTLD_PROLOGUE(entry) \
> @@ -51,9 +53,6 @@
> || (__WORDSIZE == 64 && (type) == R_RISCV_TLS_TPREL64))) \
> | (ELF_RTYPE_CLASS_COPY * ((type) == R_RISCV_COPY)))
>
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute_used__
> elf_machine_matches_host (const ElfW(Ehdr) *ehdr)
> diff --git a/sysdeps/s390/s390-32/dl-machine.h b/sysdeps/s390/s390-32/dl-machine.h
> index cd16cc7e00..c1d9bb41db 100644
> --- a/sysdeps/s390/s390-32/dl-machine.h
> +++ b/sysdeps/s390/s390-32/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <link.h>
> #include <sysdeps/s390/dl-procinfo.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* This is an older, now obsolete value. */
> #define EM_S390_OLD 0xA390
> @@ -276,10 +278,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_390_JMP_SLOT
>
> -/* The S390 never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/s390/s390-64/dl-machine.h b/sysdeps/s390/s390-64/dl-machine.h
> index 0330a8d7de..d405f01a03 100644
> --- a/sysdeps/s390/s390-64/dl-machine.h
> +++ b/sysdeps/s390/s390-64/dl-machine.h
> @@ -27,6 +27,8 @@
> #include <link.h>
> #include <sysdeps/s390/dl-procinfo.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #define ELF_MACHINE_IRELATIVE R_390_IRELATIVE
>
> @@ -224,10 +226,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_390_JMP_SLOT
>
> -/* The 64 bit S/390 never uses Elf64_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/sh/dl-machine.h b/sysdeps/sh/dl-machine.h
> index 0c22dfd848..d14023e749 100644
> --- a/sysdeps/sh/dl-machine.h
> +++ b/sysdeps/sh/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <sys/param.h>
> #include <sysdep.h>
> #include <assert.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute__ ((unused))
> @@ -251,10 +253,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rela *reloc,
>
> #endif /* !dl_machine_h */
>
> -/* SH never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> #ifdef RESOLVE_MAP
>
> /* Perform the relocation specified by RELOC and SYM (which is fully resolved).
> diff --git a/sysdeps/sparc/sparc32/dl-machine.h b/sysdeps/sparc/sparc32/dl-machine.h
> index 6361cfae9e..78f53bc499 100644
> --- a/sysdeps/sparc/sparc32/dl-machine.h
> +++ b/sysdeps/sparc/sparc32/dl-machine.h
> @@ -28,6 +28,8 @@
> #include <tls.h>
> #include <dl-plt.h>
> #include <elf/dl-hwcaps.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -196,10 +198,6 @@ elf_machine_runtime_setup (struct link_map *l, struct r_scope_elem *scope[],
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_SPARC_JMP_SLOT
>
> -/* The SPARC never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Undo the sub %sp, 6*4, %sp; add %sp, 22*4, %o0 below to get at the
> value we want in __libc_stack_end. */
> #define DL_STACK_END(cookie) \
> diff --git a/sysdeps/sparc/sparc64/dl-machine.h b/sysdeps/sparc/sparc64/dl-machine.h
> index 3fd18c6e5e..3fa79d038f 100644
> --- a/sysdeps/sparc/sparc64/dl-machine.h
> +++ b/sysdeps/sparc/sparc64/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <ldsodefs.h>
> #include <sysdep.h>
> #include <dl-plt.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #define ELF64_R_TYPE_ID(info) ((info) & 0xff)
> #define ELF64_R_TYPE_DATA(info) ((info) >> 8)
> @@ -118,10 +120,6 @@ elf_machine_plt_value (struct link_map *map, const Elf64_Rela *reloc,
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_SPARC_JMP_SLOT
>
> -/* The SPARC never uses Elf64_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Set up the loaded object described by L so its unrelocated PLT
> entries will jump to the on-demand fixup code in dl-runtime.c. */
>
> diff --git a/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h b/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h
> index db388a022d..72b75d3beb 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h
> +++ b/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h
> @@ -24,7 +24,7 @@
> #include <sysdep-vdso.h>
>
> #if (defined(__PPC64__) || defined(__powerpc64__)) && _CALL_ELF != 2
> -# include <dl-machine.h>
> +# include <dl-funcdesc.h>
> /* The correct solution is for _dl_vdso_vsym to return the address of the OPD
> for the kernel VDSO function. That address would then be stored in the
> __vdso_* variables and returned as the result of the IFUNC resolver function.
> diff --git a/sysdeps/x86_64/dl-machine.h b/sysdeps/x86_64/dl-machine.h
> index 20afb0b4bc..b1a5297b66 100644
> --- a/sysdeps/x86_64/dl-machine.h
> +++ b/sysdeps/x86_64/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <sysdep.h>
> #include <tls.h>
> #include <dl-tlsdesc.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute__ ((unused))
> @@ -200,10 +202,6 @@ _dl_start_user:\n\
> // XXX This is a work-around for a broken linker. Remove!
> #define ELF_MACHINE_IRELATIVE R_X86_64_IRELATIVE
>
> -/* The x86-64 never uses Elf64_Rel/Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization function. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> --
> 2.30.2
>
The 10/13/2021 16:25, Adhemerval Zanella wrote:
> The 4af6982e4c fix does not fully handle RTLD_BOOTSTRAP usage on
> rtld.c due two issues:
>
> 1. RTLD_BOOTSTRAP is also used on dl-machine.h on various
> architectures and it changes the semantics of various machine
> relocation functions.
>
> 2. The elf_get_dynamic_info() change was done sideways, previously
> to 490e6c62aa get-dynamic-info.h was included by the first
> dynamic-link.h include *without* RTLD_BOOTSTRAP being defined.
> It means that the code within elf_get_dynamic_info() that uses
> RTLD_BOOTSTRAP is in fact unused.
>
> To fix 1. this patch now includes dynamic-link.h only once with
> RTLD_BOOTSTRAP defined. The ELF_DYNAMIC_RELOCATE call will now have
> the relocation fnctions with the expected semantics for the loader.
>
> And to fix 2. part of 4af6982e4c is reverted (the check argument
> elf_get_dynamic_info() is not required) and the RTLD_BOOTSTRAP
> pieces are removed.
>
> To reorganize the included the static TLS definition is moved to
> its own header to avoid a circular dependency (it is defined on
> dynamic-link.h and dl-machine.h requires it at same time other
> dynamic-link.h definition requires dl-machine.h defitions).
>
> Also ELF_MACHINE_NO_REL, ELF_MACHINE_NO_RELA, and ELF_MACHINE_PLT_REL
> are moved to its own header. Only ancient ABIs need special values
> (arm, i386, and mips), so a generic one is used as default.
>
> The powerpc Elf64_FuncDesc is also moved to its own header, since
> csu code required its definition (which would require either include
> elf/ folder or add a full path with elf/).
>
> Checked on x86_64, i686, aarch64, armhf, powerpc64, powerpc32,
> and powerpc64le.
i can confirm that aarch64 tests pass with this patch.
> ---
> elf/dl-load.c | 2 +-
> elf/dl-reloc-static-pie.c | 2 +-
> elf/dl-static-tls.h | 51 +++++++++++++++++++++
> elf/dynamic-link.h | 33 +------------
> elf/get-dynamic-info.h | 25 +++-------
> elf/rtld.c | 23 +++++-----
> elf/setup-vdso.h | 2 +-
> sysdeps/aarch64/dl-machine.h | 8 ++--
> sysdeps/alpha/dl-machine.h | 6 +--
> sysdeps/arc/dl-machine.h | 10 ++--
> sysdeps/arm/dl-machine-rel.h | 31 +++++++++++++
> sysdeps/arm/dl-machine.h | 11 +----
> sysdeps/csky/dl-machine.h | 6 +--
> sysdeps/generic/dl-machine-rel.h | 27 +++++++++++
> sysdeps/generic/dl-machine.h | 2 +
> sysdeps/hppa/dl-machine.h | 6 +--
> sysdeps/i386/dl-machine-rel.h | 31 +++++++++++++
> sysdeps/i386/dl-machine.h | 11 +----
> sysdeps/ia64/dl-machine.h | 6 +--
> sysdeps/m68k/dl-machine.h | 6 +--
> sysdeps/microblaze/dl-machine.h | 6 +--
> sysdeps/mips/dl-machine-rel.h | 26 +++++++++++
> sysdeps/mips/dl-machine.h | 6 +--
> sysdeps/nios2/dl-machine.h | 6 +--
> sysdeps/powerpc/powerpc32/dl-machine.h | 6 +--
> sysdeps/powerpc/powerpc64/dl-funcdesc.h | 34 ++++++++++++++
> sysdeps/powerpc/powerpc64/dl-irel.h | 2 +-
> sysdeps/powerpc/powerpc64/dl-machine.h | 19 ++------
> sysdeps/riscv/dl-machine.h | 5 +-
> sysdeps/s390/s390-32/dl-machine.h | 6 +--
> sysdeps/s390/s390-64/dl-machine.h | 6 +--
> sysdeps/sh/dl-machine.h | 6 +--
> sysdeps/sparc/sparc32/dl-machine.h | 6 +--
> sysdeps/sparc/sparc64/dl-machine.h | 6 +--
> sysdeps/unix/sysv/linux/powerpc/libc-vdso.h | 2 +-
> sysdeps/x86_64/dl-machine.h | 6 +--
> 36 files changed, 272 insertions(+), 175 deletions(-)
> create mode 100644 elf/dl-static-tls.h
> create mode 100644 sysdeps/arm/dl-machine-rel.h
> create mode 100644 sysdeps/generic/dl-machine-rel.h
> create mode 100644 sysdeps/i386/dl-machine-rel.h
> create mode 100644 sysdeps/mips/dl-machine-rel.h
> create mode 100644 sysdeps/powerpc/powerpc64/dl-funcdesc.h
>
> diff --git a/elf/dl-load.c b/elf/dl-load.c
> index 335c79fad5..18d3e8fe64 100644
> --- a/elf/dl-load.c
> +++ b/elf/dl-load.c
> @@ -1298,7 +1298,7 @@ _dl_map_object_from_fd (const char *name, const char *origname, int fd,
> else
> l->l_ld = (ElfW(Dyn) *) ((ElfW(Addr)) l->l_ld + l->l_addr);
>
> - elf_get_dynamic_info (l, false);
> + elf_get_dynamic_info (l);
>
> /* Make sure we are not dlopen'ing an object that has the
> DF_1_NOOPEN flag set, or a PIE object. */
> diff --git a/elf/dl-reloc-static-pie.c b/elf/dl-reloc-static-pie.c
> index 8a99c4995b..40a031f2e9 100644
> --- a/elf/dl-reloc-static-pie.c
> +++ b/elf/dl-reloc-static-pie.c
> @@ -52,7 +52,7 @@ _dl_relocate_static_pie (void)
> break;
> }
>
> - elf_get_dynamic_info (main_map, false);
> + elf_get_dynamic_info (main_map);
>
> # ifdef ELF_MACHINE_BEFORE_RTLD_RELOC
> ELF_MACHINE_BEFORE_RTLD_RELOC (main_map, main_map->l_info);
> diff --git a/elf/dl-static-tls.h b/elf/dl-static-tls.h
> new file mode 100644
> index 0000000000..730924fc01
> --- /dev/null
> +++ b/elf/dl-static-tls.h
> @@ -0,0 +1,51 @@
> +/* Inline functions for dynamic linking.
> + Copyright (C) 1995-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_STATIC_TLS_H
> +#define _DL_STATIC_TLS_H
> +
> +/* This macro is used as a callback from elf_machine_rel{a,} when a
> + static TLS reloc is about to be performed. Since (in dl-load.c) we
> + permit dynamic loading of objects that might use such relocs, we
> + have to check whether each use is actually doable. If the object
> + whose TLS segment the reference resolves to was allocated space in
> + the static TLS block at startup, then it's ok. Otherwise, we make
> + an attempt to allocate it in surplus space on the fly. If that
> + can't be done, we fall back to the error that DF_STATIC_TLS is
> + intended to produce. */
> +#define HAVE_STATIC_TLS(map, sym_map) \
> + (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET \
> + && ((sym_map)->l_tls_offset \
> + != FORCED_DYNAMIC_TLS_OFFSET), 1))
> +
> +#define CHECK_STATIC_TLS(map, sym_map) \
> + do { \
> + if (!HAVE_STATIC_TLS (map, sym_map)) \
> + _dl_allocate_static_tls (sym_map); \
> + } while (0)
> +
> +#define TRY_STATIC_TLS(map, sym_map) \
> + (__builtin_expect ((sym_map)->l_tls_offset \
> + != FORCED_DYNAMIC_TLS_OFFSET, 1) \
> + && (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET, 1) \
> + || _dl_try_allocate_static_tls (sym_map, true) == 0))
> +
> +int _dl_try_allocate_static_tls (struct link_map *map, bool optional)
> + attribute_hidden;
> +
> +#endif
> diff --git a/elf/dynamic-link.h b/elf/dynamic-link.h
> index 21cdfc88bb..ac4cc70dea 100644
> --- a/elf/dynamic-link.h
> +++ b/elf/dynamic-link.h
> @@ -16,35 +16,7 @@
> License along with the GNU C Library; if not, see
> <https://www.gnu.org/licenses/>. */
>
> -/* This macro is used as a callback from elf_machine_rel{a,} when a
> - static TLS reloc is about to be performed. Since (in dl-load.c) we
> - permit dynamic loading of objects that might use such relocs, we
> - have to check whether each use is actually doable. If the object
> - whose TLS segment the reference resolves to was allocated space in
> - the static TLS block at startup, then it's ok. Otherwise, we make
> - an attempt to allocate it in surplus space on the fly. If that
> - can't be done, we fall back to the error that DF_STATIC_TLS is
> - intended to produce. */
> -#define HAVE_STATIC_TLS(map, sym_map) \
> - (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET \
> - && ((sym_map)->l_tls_offset \
> - != FORCED_DYNAMIC_TLS_OFFSET), 1))
> -
> -#define CHECK_STATIC_TLS(map, sym_map) \
> - do { \
> - if (!HAVE_STATIC_TLS (map, sym_map)) \
> - _dl_allocate_static_tls (sym_map); \
> - } while (0)
> -
> -#define TRY_STATIC_TLS(map, sym_map) \
> - (__builtin_expect ((sym_map)->l_tls_offset \
> - != FORCED_DYNAMIC_TLS_OFFSET, 1) \
> - && (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET, 1) \
> - || _dl_try_allocate_static_tls (sym_map, true) == 0))
> -
> -int _dl_try_allocate_static_tls (struct link_map *map, bool optional)
> - attribute_hidden;
> -
> +#include <dl-machine.h>
> #include <elf.h>
>
> #ifdef RESOLVE_MAP
> @@ -91,9 +63,6 @@ elf_machine_lazy_rel (struct link_map *map, struct r_scope_elem *scope[],
> # endif
> #endif
>
> -#include <dl-machine.h>
> -
> -
> #ifdef RESOLVE_MAP
>
> # if defined RTLD_BOOTSTRAP || defined STATIC_PIE_BOOTSTRAP
> diff --git a/elf/get-dynamic-info.h b/elf/get-dynamic-info.h
> index d169099fbc..1ac0663d1f 100644
> --- a/elf/get-dynamic-info.h
> +++ b/elf/get-dynamic-info.h
> @@ -22,10 +22,11 @@
> #define _GET_DYNAMIC_INFO_H
>
> #include <assert.h>
> +#include <dl-machine-rel.h>
> #include <libc-diag.h>
>
> static inline void __attribute__ ((unused, always_inline))
> -elf_get_dynamic_info (struct link_map *l, bool check)
> +elf_get_dynamic_info (struct link_map *l)
> {
> #if __ELF_NATIVE_CLASS == 32
> typedef Elf32_Word d_tag_utype;
> @@ -33,7 +34,7 @@ elf_get_dynamic_info (struct link_map *l, bool check)
> typedef Elf64_Xword d_tag_utype;
> #endif
>
> -#if !defined RTLD_BOOTSTRAP && !defined STATIC_PIE_BOOTSTRAP
> +#ifndef STATIC_PIE_BOOTSTRAP
> if (l->l_ld == NULL)
> return;
> #endif
> @@ -111,21 +112,10 @@ elf_get_dynamic_info (struct link_map *l, bool check)
> if (info[DT_REL] != NULL)
> assert (info[DT_RELENT]->d_un.d_val == sizeof (ElfW(Rel)));
> #endif
> -#ifdef RTLD_BOOTSTRAP
> - if (check)
> - {
> - /* Only the bind now flags are allowed. */
> - assert (info[VERSYMIDX (DT_FLAGS_1)] == NULL
> - || (info[VERSYMIDX (DT_FLAGS_1)]->d_un.d_val & ~DF_1_NOW) == 0);
> - /* Flags must not be set for ld.so. */
> - assert (info[DT_FLAGS] == NULL
> - || (info[DT_FLAGS]->d_un.d_val & ~DF_BIND_NOW) == 0);
> -# ifdef STATIC_PIE_BOOTSTRAP
> - assert (info[DT_RUNPATH] == NULL);
> - assert (info[DT_RPATH] == NULL);
> -# endif
> - }
> -#else
> +#ifdef STATIC_PIE_BOOTSTRAP
> + assert (info[DT_RUNPATH] == NULL);
> + assert (info[DT_RPATH] == NULL);
> +#endif
> if (info[DT_FLAGS] != NULL)
> {
> /* Flags are used. Translate to the old form where available.
> @@ -163,7 +153,6 @@ elf_get_dynamic_info (struct link_map *l, bool check)
> if (info[DT_RUNPATH] != NULL)
> /* If both RUNPATH and RPATH are given, the latter is ignored. */
> info[DT_RPATH] = NULL;
> -#endif
> }
>
> #endif
> diff --git a/elf/rtld.c b/elf/rtld.c
> index fe5ee9ec4d..6cfb7cf672 100644
> --- a/elf/rtld.c
> +++ b/elf/rtld.c
> @@ -32,7 +32,6 @@
> #include <fpu_control.h>
> #include <hp-timing.h>
> #include <libc-lock.h>
> -#include "dynamic-link.h"
> #include <dl-librecon.h>
> #include <unsecvars.h>
> #include <dl-cache.h>
> @@ -50,9 +49,18 @@
> #include <dl-main.h>
> #include <gnu/lib-names.h>
> #include <dl-tunables.h>
> +#include <get-dynamic-info.h>
>
> #include <assert.h>
>
> +/* This #define produces dynamic linking inline functions for
> + bootstrap relocation instead of general-purpose relocation.
> + Since ld.so must not have any undefined symbols the result
> + is trivial: always the map of ld.so itself. */
> +#define RTLD_BOOTSTRAP
> +#define RESOLVE_MAP(map, scope, sym, version, flags) map
> +#include "dynamic-link.h"
> +
> /* Only enables rtld profiling for architectures which provides non generic
> hp-timing support. The generic support requires either syscall
> (clock_gettime), which will incur in extra overhead on loading time.
> @@ -507,15 +515,6 @@ _dl_start_final (void *arg, struct dl_start_final_info *info)
> # define bootstrap_map info.l
> #endif
>
> - /* This #define produces dynamic linking inline functions for
> - bootstrap relocation instead of general-purpose relocation.
> - Since ld.so must not have any undefined symbols the result
> - is trivial: always the map of ld.so itself. */
> -#define RTLD_BOOTSTRAP
> -#define RESOLVE_MAP(map, scope, sym, version, flags) map
> -#include "get-dynamic-info.h"
> -#include "dynamic-link.h"
> -
> static ElfW(Addr) __attribute_used__
> _dl_start (void *arg)
> {
> @@ -548,7 +547,7 @@ _dl_start (void *arg)
>
> /* Read our own dynamic section and fill in the info array. */
> bootstrap_map.l_ld = (void *) bootstrap_map.l_addr + elf_machine_dynamic ();
> - elf_get_dynamic_info (&bootstrap_map, true);
> + elf_get_dynamic_info (&bootstrap_map);
>
> #if NO_TLS_OFFSET != 0
> bootstrap_map.l_tls_offset = NO_TLS_OFFSET;
> @@ -1616,7 +1615,7 @@ dl_main (const ElfW(Phdr) *phdr,
> if (! rtld_is_main)
> {
> /* Extract the contents of the dynamic section for easy access. */
> - elf_get_dynamic_info (main_map, false);
> + elf_get_dynamic_info (main_map);
>
> /* If the main map is libc.so, update the base namespace to
> refer to this map. If libc.so is loaded later, this happens
> diff --git a/elf/setup-vdso.h b/elf/setup-vdso.h
> index 6fdffafcca..f44748bc98 100644
> --- a/elf/setup-vdso.h
> +++ b/elf/setup-vdso.h
> @@ -64,7 +64,7 @@ setup_vdso (struct link_map *main_map __attribute__ ((unused)),
> l->l_map_end += l->l_addr;
> l->l_text_end += l->l_addr;
> l->l_ld = (void *) ((ElfW(Addr)) l->l_ld + l->l_addr);
> - elf_get_dynamic_info (l, false);
> + elf_get_dynamic_info (l);
> _dl_setup_hash (l);
> l->l_relocated = 1;
>
> diff --git a/sysdeps/aarch64/dl-machine.h b/sysdeps/aarch64/dl-machine.h
> index e30e81c902..0d5ad218c1 100644
> --- a/sysdeps/aarch64/dl-machine.h
> +++ b/sysdeps/aarch64/dl-machine.h
> @@ -24,7 +24,9 @@
> #include <sysdep.h>
> #include <tls.h>
> #include <dl-tlsdesc.h>
> +#include <dl-static-tls.h>
> #include <dl-irel.h>
> +#include <dl-machine-rel.h>
> #include <cpu-features.c>
>
> /* Translate a processor specific dynamic tag to the index in l_info array. */
> @@ -190,10 +192,6 @@ _dl_start_user: \n\
>
> #define ELF_MACHINE_JMP_SLOT AARCH64_R(JUMP_SLOT)
>
> -/* AArch64 uses RELA not REL */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> #define DL_PLATFORM_INIT dl_platform_init ()
>
> static inline void __attribute__ ((unused))
> @@ -370,7 +368,7 @@ elf_machine_rela (struct link_map *map, struct r_scope_elem *scope[],
> }
> }
>
> -inline void
> +static inline void
> __attribute__ ((always_inline))
> elf_machine_rela_relative (ElfW(Addr) l_addr,
> const ElfW(Rela) *reloc,
> diff --git a/sysdeps/alpha/dl-machine.h b/sysdeps/alpha/dl-machine.h
> index 491541326f..7d6282e599 100644
> --- a/sysdeps/alpha/dl-machine.h
> +++ b/sysdeps/alpha/dl-machine.h
> @@ -25,6 +25,8 @@
> #define ELF_MACHINE_NAME "alpha"
>
> #include <string.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
>
> /* Mask identifying addresses reserved for the user program,
> @@ -240,10 +242,6 @@ $fixup_stack: \n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_ALPHA_JMP_SLOT
>
> -/* The alpha never uses Elf64_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> * _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/arc/dl-machine.h b/sysdeps/arc/dl-machine.h
> index 4b64ffec25..f843ed9bd6 100644
> --- a/sysdeps/arc/dl-machine.h
> +++ b/sysdeps/arc/dl-machine.h
> @@ -30,6 +30,8 @@
> #include <string.h>
> #include <link.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Dynamic Linking ABI for ARCv2 ISA.
>
> @@ -203,10 +205,6 @@ __start: \n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_ARC_JUMP_SLOT
>
> -/* ARC uses Rela relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Fixup a PLT entry to bounce directly to the function at VALUE. */
>
> static inline ElfW(Addr)
> @@ -318,7 +316,7 @@ elf_machine_rela (struct link_map *map, struct r_scope_elem *scope[],
> }
> }
>
> -inline void
> +static inline void
> __attribute__ ((always_inline))
> elf_machine_rela_relative (ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
> void *const reloc_addr_arg)
> @@ -327,7 +325,7 @@ elf_machine_rela_relative (ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
> *reloc_addr += l_addr;
> }
>
> -inline void
> +static inline void
> __attribute__ ((always_inline))
> elf_machine_lazy_rel (struct link_map *map, struct r_scope_elem *scope[],
> ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
> diff --git a/sysdeps/arm/dl-machine-rel.h b/sysdeps/arm/dl-machine-rel.h
> new file mode 100644
> index 0000000000..bec114706c
> --- /dev/null
> +++ b/sysdeps/arm/dl-machine-rel.h
> @@ -0,0 +1,31 @@
> +/* ELF dynamic relocation type supported by the architecture. ARM version.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> +#define ELF_MACHINE_NO_REL 0
> +
> +/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_PLT_REL 1
> +
> +#endif
> diff --git a/sysdeps/arm/dl-machine.h b/sysdeps/arm/dl-machine.h
> index dfa05eee44..c758bba169 100644
> --- a/sysdeps/arm/dl-machine.h
> +++ b/sysdeps/arm/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <tls.h>
> #include <dl-tlsdesc.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #ifndef CLEAR_CACHE
> # error CLEAR_CACHE definition required to handle TEXTREL
> @@ -231,10 +233,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_ARM_JUMP_SLOT
>
> -/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_PLT_REL 1
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> @@ -267,11 +265,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rel *reloc,
> #endif /* !dl_machine_h */
>
>
> -/* ARM never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> -#define ELF_MACHINE_NO_REL 0
> -
> /* Names of the architecture-specific auditing callback functions. */
> #define ARCH_LA_PLTENTER arm_gnu_pltenter
> #define ARCH_LA_PLTEXIT arm_gnu_pltexit
> diff --git a/sysdeps/csky/dl-machine.h b/sysdeps/csky/dl-machine.h
> index ec22f87577..4dfd957877 100644
> --- a/sysdeps/csky/dl-machine.h
> +++ b/sysdeps/csky/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <sys/param.h>
> #include <sysdep.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero if ELF header is compatible with the running host. */
> static inline int
> @@ -172,10 +174,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_CKCORE_JUMP_SLOT
>
> -/* C-SKY never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/generic/dl-machine-rel.h b/sysdeps/generic/dl-machine-rel.h
> new file mode 100644
> index 0000000000..9167a1dffc
> --- /dev/null
> +++ b/sysdeps/generic/dl-machine-rel.h
> @@ -0,0 +1,27 @@
> +/* ELF dynamic relocation type supported by the architecture.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +/* Defined if the architecture supports Elf{32,64}_Rel relocations. */
> +#define ELF_MACHINE_NO_REL 1
> +/* Defined if the architecture supports Elf{32,64}_Rela relocations. */
> +#define ELF_MACHINE_NO_RELA 0
> +
> +#endif
> diff --git a/sysdeps/generic/dl-machine.h b/sysdeps/generic/dl-machine.h
> index 4a4ab4fc70..7da695d903 100644
> --- a/sysdeps/generic/dl-machine.h
> +++ b/sysdeps/generic/dl-machine.h
> @@ -20,6 +20,8 @@
>
> #include <string.h>
> #include <link.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> diff --git a/sysdeps/hppa/dl-machine.h b/sysdeps/hppa/dl-machine.h
> index e6a4575fce..7c5d0e9430 100644
> --- a/sysdeps/hppa/dl-machine.h
> +++ b/sysdeps/hppa/dl-machine.h
> @@ -29,6 +29,8 @@
> #include <dl-fptr.h>
> #include <abort-instr.h>
> #include <tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* These two definitions must match the definition of the stub in
> bfd/elf32-hppa.c (see plt_stub[]).
> @@ -509,10 +511,6 @@ asm ( \
> #define ELF_MACHINE_JMP_SLOT R_PARISC_IPLT
> #define ELF_MACHINE_SIZEOF_JMP_SLOT PLT_ENTRY_SIZE
>
> -/* We only use RELA. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Return the address of the entry point. */
> #define ELF_MACHINE_START_ADDRESS(map, start) \
> ({ \
> diff --git a/sysdeps/i386/dl-machine-rel.h b/sysdeps/i386/dl-machine-rel.h
> new file mode 100644
> index 0000000000..7ac46f78a6
> --- /dev/null
> +++ b/sysdeps/i386/dl-machine-rel.h
> @@ -0,0 +1,31 @@
> +/* ELF dynamic relocation type supported by the architecture. ARM version.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> +#define ELF_MACHINE_NO_REL 0
> +
> +/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> + Prelinked libraries may use Elf32_Rela though. */
> +#define ELF_MACHINE_PLT_REL 1
> +
> +#endif
> diff --git a/sysdeps/i386/dl-machine.h b/sysdeps/i386/dl-machine.h
> index 78bed03eee..169bed50ff 100644
> --- a/sysdeps/i386/dl-machine.h
> +++ b/sysdeps/i386/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <sysdep.h>
> #include <tls.h>
> #include <dl-tlsdesc.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute__ ((unused))
> @@ -230,10 +232,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_386_JMP_SLOT
>
> -/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_PLT_REL 1
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> @@ -276,11 +274,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rel *reloc,
>
> #endif /* !dl_machine_h */
>
> -/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
> - Prelinked libraries may use Elf32_Rela though. */
> -#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
> -#define ELF_MACHINE_NO_REL 0
> -
> #ifdef RESOLVE_MAP
>
> /* Perform the relocation specified by RELOC and SYM (which is fully resolved).
> diff --git a/sysdeps/ia64/dl-machine.h b/sysdeps/ia64/dl-machine.h
> index 2217d0b556..c9608a51b0 100644
> --- a/sysdeps/ia64/dl-machine.h
> +++ b/sysdeps/ia64/dl-machine.h
> @@ -27,6 +27,8 @@
> #include <errno.h>
> #include <dl-fptr.h>
> #include <tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> @@ -319,10 +321,6 @@ elf_machine_runtime_setup (struct link_map *l, struct r_scope_elem *scope[],
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_IA64_IPLTLSB
>
> -/* According to the IA-64 specific documentation, Rela is always used. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Return the address of the entry point. */
> #define ELF_MACHINE_START_ADDRESS(map, start) \
> ({ \
> diff --git a/sysdeps/m68k/dl-machine.h b/sysdeps/m68k/dl-machine.h
> index 5e34c4784e..30323d62d4 100644
> --- a/sysdeps/m68k/dl-machine.h
> +++ b/sysdeps/m68k/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <sys/param.h>
> #include <sysdep.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -183,10 +185,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_68K_JMP_SLOT
>
> -/* The m68k never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> static inline Elf32_Addr
> elf_machine_fixup_plt (struct link_map *map, lookup_t t,
> const ElfW(Sym) *refsym, const ElfW(Sym) *sym,
> diff --git a/sysdeps/microblaze/dl-machine.h b/sysdeps/microblaze/dl-machine.h
> index 3fd4988e60..b8cc5a7fe6 100644
> --- a/sysdeps/microblaze/dl-machine.h
> +++ b/sysdeps/microblaze/dl-machine.h
> @@ -23,6 +23,8 @@
>
> #include <sys/param.h>
> #include <tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -169,10 +171,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_MICROBLAZE_JUMP_SLOT
>
> -/* The microblaze never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> static inline Elf32_Addr
> elf_machine_fixup_plt (struct link_map *map, lookup_t t,
> const ElfW(Sym) *refsym, const ElfW(Sym) *sym,
> diff --git a/sysdeps/mips/dl-machine-rel.h b/sysdeps/mips/dl-machine-rel.h
> new file mode 100644
> index 0000000000..ed39618041
> --- /dev/null
> +++ b/sysdeps/mips/dl-machine-rel.h
> @@ -0,0 +1,26 @@
> +/* ELF dynamic relocation type supported by the architecture. ARM version.
> + Copyright (C) 2001-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_MACHINE_REL_H
> +#define _DL_MACHINE_REL_H
> +
> +#define ELF_MACHINE_PLT_REL 1
> +#define ELF_MACHINE_NO_REL 0
> +#define ELF_MACHINE_NO_RELA 0
> +
> +#endif
> diff --git a/sysdeps/mips/dl-machine.h b/sysdeps/mips/dl-machine.h
> index f8d1298b60..d7b8341b74 100644
> --- a/sysdeps/mips/dl-machine.h
> +++ b/sysdeps/mips/dl-machine.h
> @@ -32,6 +32,8 @@
> #include <sysdep.h>
> #include <sys/asm.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* The offset of gp from GOT might be system-dependent. It's set by
> ld. The same value is also */
> @@ -59,10 +61,6 @@
> ((((type) == ELF_MACHINE_JMP_SLOT) * ELF_RTYPE_CLASS_PLT) \
> | (((type) == R_MIPS_COPY) * ELF_RTYPE_CLASS_COPY))
>
> -#define ELF_MACHINE_PLT_REL 1
> -#define ELF_MACHINE_NO_REL 0
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> #define DT_MIPS(x) (DT_MIPS_##x - DT_LOPROC + DT_NUM)
> diff --git a/sysdeps/nios2/dl-machine.h b/sysdeps/nios2/dl-machine.h
> index 4de602b13d..430ca5d7ae 100644
> --- a/sysdeps/nios2/dl-machine.h
> +++ b/sysdeps/nios2/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <string.h>
> #include <link.h>
> #include <dl-tls.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -200,10 +202,6 @@ _start:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_NIOS2_JUMP_SLOT
>
> -/* The Nios II never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Fixup a PLT entry to bounce directly to the function at VALUE. */
>
> static inline Elf32_Addr
> diff --git a/sysdeps/powerpc/powerpc32/dl-machine.h b/sysdeps/powerpc/powerpc32/dl-machine.h
> index cda012dc1b..8d062951ce 100644
> --- a/sysdeps/powerpc/powerpc32/dl-machine.h
> +++ b/sysdeps/powerpc/powerpc32/dl-machine.h
> @@ -25,6 +25,8 @@
> #include <dl-tls.h>
> #include <dl-irel.h>
> #include <hwcapinfo.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> @@ -145,10 +147,6 @@ __elf_preferred_address(struct link_map *loader, size_t maplength,
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_PPC_JMP_SLOT
>
> -/* The PowerPC never uses REL relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization function to initialize HWCAP/HWCAP2 and
> platform data so it can be copied into the TCB later. This is called
> very early in _dl_sysdep_start for dynamically linked binaries. */
> diff --git a/sysdeps/powerpc/powerpc64/dl-funcdesc.h b/sysdeps/powerpc/powerpc64/dl-funcdesc.h
> new file mode 100644
> index 0000000000..b2d1f76ce0
> --- /dev/null
> +++ b/sysdeps/powerpc/powerpc64/dl-funcdesc.h
> @@ -0,0 +1,34 @@
> +/* PowerPC ELFv1 function descriptor definition.
> + Copyright (C) 2009-2021 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <https://www.gnu.org/licenses/>. */
> +
> +#ifndef _DL_FUNCDESC_H
> +#define _DL_FUNCDESC_H
> +
> +#if _CALL_ELF != 2
> +/* A PowerPC64 function descriptor. The .plt (procedure linkage
> + table) and .opd (official procedure descriptor) sections are
> + arrays of these. */
> +typedef struct
> +{
> + Elf64_Addr fd_func;
> + Elf64_Addr fd_toc;
> + Elf64_Addr fd_aux;
> +} Elf64_FuncDesc;
> +#endif
> +
> +#endif
> diff --git a/sysdeps/powerpc/powerpc64/dl-irel.h b/sysdeps/powerpc/powerpc64/dl-irel.h
> index 0e11b7ff64..aa9a2dca71 100644
> --- a/sysdeps/powerpc/powerpc64/dl-irel.h
> +++ b/sysdeps/powerpc/powerpc64/dl-irel.h
> @@ -23,7 +23,7 @@
> #include <stdio.h>
> #include <unistd.h>
> #include <ldsodefs.h>
> -#include <dl-machine.h>
> +#include <dl-funcdesc.h>
>
> #define ELF_MACHINE_IRELA 1
>
> diff --git a/sysdeps/powerpc/powerpc64/dl-machine.h b/sysdeps/powerpc/powerpc64/dl-machine.h
> index 3f92fbb369..3a4a21a412 100644
> --- a/sysdeps/powerpc/powerpc64/dl-machine.h
> +++ b/sysdeps/powerpc/powerpc64/dl-machine.h
> @@ -28,23 +28,14 @@
> #include <sysdep.h>
> #include <hwcapinfo.h>
> #include <cpu-features.c>
> +#include <dl-static-tls.h>
> +#include <dl-funcdesc.h>
> +#include <dl-machine-rel.h>
>
> /* Translate a processor specific dynamic tag to the index
> in l_info array. */
> #define DT_PPC64(x) (DT_PPC64_##x - DT_LOPROC + DT_NUM)
>
> -#if _CALL_ELF != 2
> -/* A PowerPC64 function descriptor. The .plt (procedure linkage
> - table) and .opd (official procedure descriptor) sections are
> - arrays of these. */
> -typedef struct
> -{
> - Elf64_Addr fd_func;
> - Elf64_Addr fd_toc;
> - Elf64_Addr fd_aux;
> -} Elf64_FuncDesc;
> -#endif
> -
> #define ELF_MULT_MACHINES_SUPPORTED
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> @@ -292,10 +283,6 @@ BODY_PREFIX "_dl_start_user:\n" \
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_PPC64_JMP_SLOT
>
> -/* The PowerPC never uses REL relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization function to initialize HWCAP/HWCAP2 and
> platform data so it can be copied into the TCB later. This is called
> very early in _dl_sysdep_start for dynamically linked binaries. */
> diff --git a/sysdeps/riscv/dl-machine.h b/sysdeps/riscv/dl-machine.h
> index e7289d7ddc..ce2b3c3875 100644
> --- a/sysdeps/riscv/dl-machine.h
> +++ b/sysdeps/riscv/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <sys/asm.h>
> #include <dl-tls.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #ifndef _RTLD_PROLOGUE
> # define _RTLD_PROLOGUE(entry) \
> @@ -51,9 +53,6 @@
> || (__WORDSIZE == 64 && (type) == R_RISCV_TLS_TPREL64))) \
> | (ELF_RTYPE_CLASS_COPY * ((type) == R_RISCV_COPY)))
>
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute_used__
> elf_machine_matches_host (const ElfW(Ehdr) *ehdr)
> diff --git a/sysdeps/s390/s390-32/dl-machine.h b/sysdeps/s390/s390-32/dl-machine.h
> index cd16cc7e00..c1d9bb41db 100644
> --- a/sysdeps/s390/s390-32/dl-machine.h
> +++ b/sysdeps/s390/s390-32/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <link.h>
> #include <sysdeps/s390/dl-procinfo.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* This is an older, now obsolete value. */
> #define EM_S390_OLD 0xA390
> @@ -276,10 +278,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_390_JMP_SLOT
>
> -/* The S390 never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/s390/s390-64/dl-machine.h b/sysdeps/s390/s390-64/dl-machine.h
> index 0330a8d7de..d405f01a03 100644
> --- a/sysdeps/s390/s390-64/dl-machine.h
> +++ b/sysdeps/s390/s390-64/dl-machine.h
> @@ -27,6 +27,8 @@
> #include <link.h>
> #include <sysdeps/s390/dl-procinfo.h>
> #include <dl-irel.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #define ELF_MACHINE_IRELATIVE R_390_IRELATIVE
>
> @@ -224,10 +226,6 @@ _dl_start_user:\n\
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_390_JMP_SLOT
>
> -/* The 64 bit S/390 never uses Elf64_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization functions. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> diff --git a/sysdeps/sh/dl-machine.h b/sysdeps/sh/dl-machine.h
> index 0c22dfd848..d14023e749 100644
> --- a/sysdeps/sh/dl-machine.h
> +++ b/sysdeps/sh/dl-machine.h
> @@ -24,6 +24,8 @@
> #include <sys/param.h>
> #include <sysdep.h>
> #include <assert.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute__ ((unused))
> @@ -251,10 +253,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rela *reloc,
>
> #endif /* !dl_machine_h */
>
> -/* SH never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> #ifdef RESOLVE_MAP
>
> /* Perform the relocation specified by RELOC and SYM (which is fully resolved).
> diff --git a/sysdeps/sparc/sparc32/dl-machine.h b/sysdeps/sparc/sparc32/dl-machine.h
> index 6361cfae9e..78f53bc499 100644
> --- a/sysdeps/sparc/sparc32/dl-machine.h
> +++ b/sysdeps/sparc/sparc32/dl-machine.h
> @@ -28,6 +28,8 @@
> #include <tls.h>
> #include <dl-plt.h>
> #include <elf/dl-hwcaps.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int
> @@ -196,10 +198,6 @@ elf_machine_runtime_setup (struct link_map *l, struct r_scope_elem *scope[],
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_SPARC_JMP_SLOT
>
> -/* The SPARC never uses Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Undo the sub %sp, 6*4, %sp; add %sp, 22*4, %o0 below to get at the
> value we want in __libc_stack_end. */
> #define DL_STACK_END(cookie) \
> diff --git a/sysdeps/sparc/sparc64/dl-machine.h b/sysdeps/sparc/sparc64/dl-machine.h
> index 3fd18c6e5e..3fa79d038f 100644
> --- a/sysdeps/sparc/sparc64/dl-machine.h
> +++ b/sysdeps/sparc/sparc64/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <ldsodefs.h>
> #include <sysdep.h>
> #include <dl-plt.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> #define ELF64_R_TYPE_ID(info) ((info) & 0xff)
> #define ELF64_R_TYPE_DATA(info) ((info) >> 8)
> @@ -118,10 +120,6 @@ elf_machine_plt_value (struct link_map *map, const Elf64_Rela *reloc,
> /* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
> #define ELF_MACHINE_JMP_SLOT R_SPARC_JMP_SLOT
>
> -/* The SPARC never uses Elf64_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* Set up the loaded object described by L so its unrelocated PLT
> entries will jump to the on-demand fixup code in dl-runtime.c. */
>
> diff --git a/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h b/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h
> index db388a022d..72b75d3beb 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h
> +++ b/sysdeps/unix/sysv/linux/powerpc/libc-vdso.h
> @@ -24,7 +24,7 @@
> #include <sysdep-vdso.h>
>
> #if (defined(__PPC64__) || defined(__powerpc64__)) && _CALL_ELF != 2
> -# include <dl-machine.h>
> +# include <dl-funcdesc.h>
> /* The correct solution is for _dl_vdso_vsym to return the address of the OPD
> for the kernel VDSO function. That address would then be stored in the
> __vdso_* variables and returned as the result of the IFUNC resolver function.
> diff --git a/sysdeps/x86_64/dl-machine.h b/sysdeps/x86_64/dl-machine.h
> index 20afb0b4bc..b1a5297b66 100644
> --- a/sysdeps/x86_64/dl-machine.h
> +++ b/sysdeps/x86_64/dl-machine.h
> @@ -26,6 +26,8 @@
> #include <sysdep.h>
> #include <tls.h>
> #include <dl-tlsdesc.h>
> +#include <dl-static-tls.h>
> +#include <dl-machine-rel.h>
>
> /* Return nonzero iff ELF header is compatible with the running host. */
> static inline int __attribute__ ((unused))
> @@ -200,10 +202,6 @@ _dl_start_user:\n\
> // XXX This is a work-around for a broken linker. Remove!
> #define ELF_MACHINE_IRELATIVE R_X86_64_IRELATIVE
>
> -/* The x86-64 never uses Elf64_Rel/Elf32_Rel relocations. */
> -#define ELF_MACHINE_NO_REL 1
> -#define ELF_MACHINE_NO_RELA 0
> -
> /* We define an initialization function. This is called very early in
> _dl_sysdep_start. */
> #define DL_PLATFORM_INIT dl_platform_init ()
> --
> 2.30.2
>
The 10/13/2021 16:25, Adhemerval Zanella wrote:
> The 4af6982e4c fix does not fully handle RTLD_BOOTSTRAP usage on
> rtld.c due two issues:
>
> 1. RTLD_BOOTSTRAP is also used on dl-machine.h on various
> architectures and it changes the semantics of various machine
> relocation functions.
>
> 2. The elf_get_dynamic_info() change was done sideways, previously
> to 490e6c62aa get-dynamic-info.h was included by the first
> dynamic-link.h include *without* RTLD_BOOTSTRAP being defined.
> It means that the code within elf_get_dynamic_info() that uses
> RTLD_BOOTSTRAP is in fact unused.
>
> To fix 1. this patch now includes dynamic-link.h only once with
> RTLD_BOOTSTRAP defined. The ELF_DYNAMIC_RELOCATE call will now have
> the relocation fnctions with the expected semantics for the loader.
>
> And to fix 2. part of 4af6982e4c is reverted (the check argument
> elf_get_dynamic_info() is not required) and the RTLD_BOOTSTRAP
> pieces are removed.
>
> To reorganize the included the static TLS definition is moved to
typo: includes?
> its own header to avoid a circular dependency (it is defined on
> dynamic-link.h and dl-machine.h requires it at same time other
> dynamic-link.h definition requires dl-machine.h defitions).
>
> Also ELF_MACHINE_NO_REL, ELF_MACHINE_NO_RELA, and ELF_MACHINE_PLT_REL
> are moved to its own header. Only ancient ABIs need special values
> (arm, i386, and mips), so a generic one is used as default.
>
> The powerpc Elf64_FuncDesc is also moved to its own header, since
> csu code required its definition (which would require either include
> elf/ folder or add a full path with elf/).
>
> Checked on x86_64, i686, aarch64, armhf, powerpc64, powerpc32,
> and powerpc64le.
i now reviewed the generic and aarch64 specific changes
and they look good.
Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>
> ---
> elf/dl-load.c | 2 +-
> elf/dl-reloc-static-pie.c | 2 +-
> elf/dl-static-tls.h | 51 +++++++++++++++++++++
> elf/dynamic-link.h | 33 +------------
> elf/get-dynamic-info.h | 25 +++-------
> elf/rtld.c | 23 +++++-----
> elf/setup-vdso.h | 2 +-
> sysdeps/aarch64/dl-machine.h | 8 ++--
> sysdeps/alpha/dl-machine.h | 6 +--
> sysdeps/arc/dl-machine.h | 10 ++--
> sysdeps/arm/dl-machine-rel.h | 31 +++++++++++++
> sysdeps/arm/dl-machine.h | 11 +----
> sysdeps/csky/dl-machine.h | 6 +--
> sysdeps/generic/dl-machine-rel.h | 27 +++++++++++
> sysdeps/generic/dl-machine.h | 2 +
> sysdeps/hppa/dl-machine.h | 6 +--
> sysdeps/i386/dl-machine-rel.h | 31 +++++++++++++
> sysdeps/i386/dl-machine.h | 11 +----
> sysdeps/ia64/dl-machine.h | 6 +--
> sysdeps/m68k/dl-machine.h | 6 +--
> sysdeps/microblaze/dl-machine.h | 6 +--
> sysdeps/mips/dl-machine-rel.h | 26 +++++++++++
> sysdeps/mips/dl-machine.h | 6 +--
> sysdeps/nios2/dl-machine.h | 6 +--
> sysdeps/powerpc/powerpc32/dl-machine.h | 6 +--
> sysdeps/powerpc/powerpc64/dl-funcdesc.h | 34 ++++++++++++++
> sysdeps/powerpc/powerpc64/dl-irel.h | 2 +-
> sysdeps/powerpc/powerpc64/dl-machine.h | 19 ++------
> sysdeps/riscv/dl-machine.h | 5 +-
> sysdeps/s390/s390-32/dl-machine.h | 6 +--
> sysdeps/s390/s390-64/dl-machine.h | 6 +--
> sysdeps/sh/dl-machine.h | 6 +--
> sysdeps/sparc/sparc32/dl-machine.h | 6 +--
> sysdeps/sparc/sparc64/dl-machine.h | 6 +--
> sysdeps/unix/sysv/linux/powerpc/libc-vdso.h | 2 +-
> sysdeps/x86_64/dl-machine.h | 6 +--
> 36 files changed, 272 insertions(+), 175 deletions(-)
> create mode 100644 elf/dl-static-tls.h
> create mode 100644 sysdeps/arm/dl-machine-rel.h
> create mode 100644 sysdeps/generic/dl-machine-rel.h
> create mode 100644 sysdeps/i386/dl-machine-rel.h
> create mode 100644 sysdeps/mips/dl-machine-rel.h
> create mode 100644 sysdeps/powerpc/powerpc64/dl-funcdesc.h
On 14/10/2021 13:42, Szabolcs Nagy wrote:
> The 10/13/2021 16:25, Adhemerval Zanella wrote:
>> The 4af6982e4c fix does not fully handle RTLD_BOOTSTRAP usage on
>> rtld.c due two issues:
>>
>> 1. RTLD_BOOTSTRAP is also used on dl-machine.h on various
>> architectures and it changes the semantics of various machine
>> relocation functions.
>>
>> 2. The elf_get_dynamic_info() change was done sideways, previously
>> to 490e6c62aa get-dynamic-info.h was included by the first
>> dynamic-link.h include *without* RTLD_BOOTSTRAP being defined.
>> It means that the code within elf_get_dynamic_info() that uses
>> RTLD_BOOTSTRAP is in fact unused.
>>
>> To fix 1. this patch now includes dynamic-link.h only once with
>> RTLD_BOOTSTRAP defined. The ELF_DYNAMIC_RELOCATE call will now have
>> the relocation fnctions with the expected semantics for the loader.
>>
>> And to fix 2. part of 4af6982e4c is reverted (the check argument
>> elf_get_dynamic_info() is not required) and the RTLD_BOOTSTRAP
>> pieces are removed.
>>
>> To reorganize the included the static TLS definition is moved to
>
> typo: includes?
Ack.
>
>> its own header to avoid a circular dependency (it is defined on
>> dynamic-link.h and dl-machine.h requires it at same time other
>> dynamic-link.h definition requires dl-machine.h defitions).
>>
>> Also ELF_MACHINE_NO_REL, ELF_MACHINE_NO_RELA, and ELF_MACHINE_PLT_REL
>> are moved to its own header. Only ancient ABIs need special values
>> (arm, i386, and mips), so a generic one is used as default.
>>
>> The powerpc Elf64_FuncDesc is also moved to its own header, since
>> csu code required its definition (which would require either include
>> elf/ folder or add a full path with elf/).
>>
>> Checked on x86_64, i686, aarch64, armhf, powerpc64, powerpc32,
>> and powerpc64le.
>
> i now reviewed the generic and aarch64 specific changes
> and they look good.
>
> Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>
Thank you, I will commit this shortly.
>
>> ---
>> elf/dl-load.c | 2 +-
>> elf/dl-reloc-static-pie.c | 2 +-
>> elf/dl-static-tls.h | 51 +++++++++++++++++++++
>> elf/dynamic-link.h | 33 +------------
>> elf/get-dynamic-info.h | 25 +++-------
>> elf/rtld.c | 23 +++++-----
>> elf/setup-vdso.h | 2 +-
>> sysdeps/aarch64/dl-machine.h | 8 ++--
>> sysdeps/alpha/dl-machine.h | 6 +--
>> sysdeps/arc/dl-machine.h | 10 ++--
>> sysdeps/arm/dl-machine-rel.h | 31 +++++++++++++
>> sysdeps/arm/dl-machine.h | 11 +----
>> sysdeps/csky/dl-machine.h | 6 +--
>> sysdeps/generic/dl-machine-rel.h | 27 +++++++++++
>> sysdeps/generic/dl-machine.h | 2 +
>> sysdeps/hppa/dl-machine.h | 6 +--
>> sysdeps/i386/dl-machine-rel.h | 31 +++++++++++++
>> sysdeps/i386/dl-machine.h | 11 +----
>> sysdeps/ia64/dl-machine.h | 6 +--
>> sysdeps/m68k/dl-machine.h | 6 +--
>> sysdeps/microblaze/dl-machine.h | 6 +--
>> sysdeps/mips/dl-machine-rel.h | 26 +++++++++++
>> sysdeps/mips/dl-machine.h | 6 +--
>> sysdeps/nios2/dl-machine.h | 6 +--
>> sysdeps/powerpc/powerpc32/dl-machine.h | 6 +--
>> sysdeps/powerpc/powerpc64/dl-funcdesc.h | 34 ++++++++++++++
>> sysdeps/powerpc/powerpc64/dl-irel.h | 2 +-
>> sysdeps/powerpc/powerpc64/dl-machine.h | 19 ++------
>> sysdeps/riscv/dl-machine.h | 5 +-
>> sysdeps/s390/s390-32/dl-machine.h | 6 +--
>> sysdeps/s390/s390-64/dl-machine.h | 6 +--
>> sysdeps/sh/dl-machine.h | 6 +--
>> sysdeps/sparc/sparc32/dl-machine.h | 6 +--
>> sysdeps/sparc/sparc64/dl-machine.h | 6 +--
>> sysdeps/unix/sysv/linux/powerpc/libc-vdso.h | 2 +-
>> sysdeps/x86_64/dl-machine.h | 6 +--
>> 36 files changed, 272 insertions(+), 175 deletions(-)
>> create mode 100644 elf/dl-static-tls.h
>> create mode 100644 sysdeps/arm/dl-machine-rel.h
>> create mode 100644 sysdeps/generic/dl-machine-rel.h
>> create mode 100644 sysdeps/i386/dl-machine-rel.h
>> create mode 100644 sysdeps/mips/dl-machine-rel.h
>> create mode 100644 sysdeps/powerpc/powerpc64/dl-funcdesc.h
Now the powerpc crash has reappeared.
Andreas.
On 15/10/2021 08:23, Andreas Schwab wrote:
> Now the powerpc crash has reappeared.
>
> Andreas.
>
I just make check on gcc203 using gcc 10.2.1 and ld 2.35.
And I also checked if RTLD_BOOTSTRAP is being correctly defined
for dl-machine.h and do-rel.h by comparing 349b0441da with
9d3c9a046a pre-processed rtld.c and the only difference is the
nested function removal.
So I am out of ideas of what might be happening in your
environment.
On Okt 15 2021, Adhemerval Zanella wrote:
> I just make check on gcc203 using gcc 10.2.1 and ld 2.35.
Like this?
Generating locale C.UTF-8: this might take a while...
Generating locale en_US.UTF-8: this might take a while...
/bin/bash tst-iconv_prog.sh /home/schwab/glibc/Build ' env' \
'GCONV_PATH=/home/schwab/glibc/Build/iconvdata LOCPATH=/home/schwab/glibc/Build/localedata LC_ALL=C' > /home/schwab/glibc/Build/iconv/tst-iconv_prog.out; \
../scripts/evaluate-test.sh iconv/tst-iconv_prog $? false false > /home/schwab/glibc/Build/iconv/tst-iconv_prog.test-result
Segmentation fault
Charmap: "UTF-8" Inputfile: "C" Outputdir: "C.UTF-8" failed
/bin/sh: 4: cannot create /home/schwab/glibc/Build/localedata/C.UTF-8/LC_CTYPE.test-result: Directory nonexistent
make[2]: *** [../gen-locales.mk:17: /home/schwab/glibc/Build/localedata/C.UTF-8/LC_CTYPE] Error 2
make[2]: *** Waiting for unfinished jobs....
Segmentation fault
Charmap: "UTF-8" Inputfile: "en_US" Outputdir: "en_US.UTF-8" failed
/bin/sh: 4: cannot create /home/schwab/glibc/Build/localedata/en_US.UTF-8/LC_CTYPE.test-result: Directory nonexistent
make[2]: *** [../gen-locales.mk:17: /home/schwab/glibc/Build/localedata/en_US.UTF-8/LC_CTYPE] Error 2
tst-iconv_prog.sh: line 222: 1434924 Done echo -en "$twobyte"
1434925 Segmentation fault | timeout -k 4 3 $PROG $c -f $from -t "$to" &> /dev/null
make[2]: Leaving directory '/home/schwab/glibc/iconv'
make[1]: *** [Makefile:483: iconv/tests] Error 2
make[1]: Leaving directory '/home/schwab/glibc'
make: *** [Makefile:9: check] Error 2
Andreas.
On 15/10/2021 09:46, Andreas Schwab wrote:
> On Okt 15 2021, Adhemerval Zanella wrote:
>
>> I just make check on gcc203 using gcc 10.2.1 and ld 2.35.
>
> Like this?
>
> Generating locale C.UTF-8: this might take a while...
> Generating locale en_US.UTF-8: this might take a while...
> /bin/bash tst-iconv_prog.sh /home/schwab/glibc/Build ' env' \
> 'GCONV_PATH=/home/schwab/glibc/Build/iconvdata LOCPATH=/home/schwab/glibc/Build/localedata LC_ALL=C' > /home/schwab/glibc/Build/iconv/tst-iconv_prog.out; \
> ../scripts/evaluate-test.sh iconv/tst-iconv_prog $? false false > /home/schwab/glibc/Build/iconv/tst-iconv_prog.test-result
> Segmentation fault
> Charmap: "UTF-8" Inputfile: "C" Outputdir: "C.UTF-8" failed
> /bin/sh: 4: cannot create /home/schwab/glibc/Build/localedata/C.UTF-8/LC_CTYPE.test-result: Directory nonexistent
> make[2]: *** [../gen-locales.mk:17: /home/schwab/glibc/Build/localedata/C.UTF-8/LC_CTYPE] Error 2
> make[2]: *** Waiting for unfinished jobs....
> Segmentation fault
> Charmap: "UTF-8" Inputfile: "en_US" Outputdir: "en_US.UTF-8" failed
> /bin/sh: 4: cannot create /home/schwab/glibc/Build/localedata/en_US.UTF-8/LC_CTYPE.test-result: Directory nonexistent
> make[2]: *** [../gen-locales.mk:17: /home/schwab/glibc/Build/localedata/en_US.UTF-8/LC_CTYPE] Error 2
> tst-iconv_prog.sh: line 222: 1434924 Done echo -en "$twobyte"
> 1434925 Segmentation fault | timeout -k 4 3 $PROG $c -f $from -t "$to" &> /dev/null
> make[2]: Leaving directory '/home/schwab/glibc/iconv'
> make[1]: *** [Makefile:483: iconv/tests] Error 2
> make[1]: Leaving directory '/home/schwab/glibc'
> make: *** [Makefile:9: check] Error 2
>
> Andreas.
>
Yes:
$ uname -a
Linux gcc203 5.10.0-8-powerpc64 #1 SMP Debian 5.10.46-4 (2021-08-03) ppc64 GNU/Linux
$ file /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/elf/ld.so
/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/elf/ld.so: ELF 32-bit MSB shared object, PowerPC or cisco 4500, version 1 (SYSV), dynamically linked, with debug_info, not stripped
$ env GCONV_PATH=/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/iconvdata LOCPATH=/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/localedata LC_ALL=C I18NPATH=../localedata /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/elf/ld.so.1 --library-path /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/math:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/elf:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/dlfcn:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/nss:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/nis:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/rt:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/resolv:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/mathvec:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/support:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/crypt:/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/nptl /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/locale/localedef --quiet -c -f UTF-8 -i C /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/localedata/C.UTF-8; echo $?
0
$ make test t=iconv/tst-iconv_prog
make -r PARALLELMFLAGS="" -C /home/azanella/projects/glibc/glibc-git objdir=`pwd` test
make[1]: Entering directory '/home/azanella/projects/glibc/glibc-git'
make subdir=iconv -C iconv/ ..=../ /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/iconv/tst-iconv_prog.out
make[2]: Entering directory '/home/azanella/projects/glibc/glibc-git/iconv'
/bin/bash tst-iconv_prog.sh /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4 ' env' \
'GCONV_PATH=/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/iconvdata LOCPATH=/home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/localedata LC_ALL=C' > /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/iconv/tst-iconv_prog.out; \
../scripts/evaluate-test.sh iconv/tst-iconv_prog $? false false > /home/azanella/projects/glibc/build/powerpc-linux-gnu-power4/iconv/tst-iconv_prog.test-result
make[2]: Leaving directory '/home/azanella/projects/glibc/glibc-git/iconv'
PASS: iconv/tst-iconv_prog
original exit status 0
OK: from: "ANSI_X3.110", to: "UTF-8//TRANSLIT//IGNORE", input "\x00\x23", flags "-c"
OK: from: "ARMSCII-8", to: "UTF-8//TRANSLIT//IGNORE", input "\x00\xa1", flags "-c"
OK: from: "ASMO_449", to: "UTF-8//TRANSLIT//IGNORE", input "\x00\xa1", flags "-c"
OK: from: "BIG5", to: "UTF-8//TRANSLIT//IGNORE", input "\x00\x81", flags "-c"
[...]
PASS: from: "INVALID", to: "INVALID", input "\x00\x00", flags "", return code 1
PASS: from: "INVALID", to: "UTF-8", input "\x00\x00", flags "", return code 1
PASS: from: "UTF-8", to: "INVALID", input "\x00\x00", flags "", return code 1
make[1]: Leaving directory '/home/azanella/projects/glibc/glibc-git'
$ echo $?
0
That's why I am trying to understand why it is failing on your environment.
On Okt 15 2021, Adhemerval Zanella wrote:
> That's why I am trying to understand why it is failing on your environment.
Please take a look.
Andreas.
On 15/10/2021 10:31, Andreas Schwab wrote:
> On Okt 15 2021, Adhemerval Zanella wrote:
>
>> That's why I am trying to understand why it is failing on your environment.
>
> Please take a look.
>
> Andreas.
>
Sigh... It is the --enable-bind-now option which trigger the original issue
and it seems I cycled back where elf_get_dynamic_info() *does* have different
semantic depending whether RTLD_BOOTSTRAP is defined.
This fixes powerpc, I will test on x86_64, aarch64, and powerpc64 with and
without --enable-bind-now and --enable-static-pie.
I hope now with explicit argument it makes clear the function semantic.
diff --git a/elf/dl-load.c b/elf/dl-load.c
index 18d3e8fe64..a1f1682188 100644
--- a/elf/dl-load.c
+++ b/elf/dl-load.c
@@ -1298,7 +1298,7 @@ _dl_map_object_from_fd (const char *name, const char *origname, int fd,
else
l->l_ld = (ElfW(Dyn) *) ((ElfW(Addr)) l->l_ld + l->l_addr);
- elf_get_dynamic_info (l);
+ elf_get_dynamic_info (l, false, false);
/* Make sure we are not dlopen'ing an object that has the
DF_1_NOOPEN flag set, or a PIE object. */
diff --git a/elf/dl-reloc-static-pie.c b/elf/dl-reloc-static-pie.c
index 40a031f2e9..5b85df8a2e 100644
--- a/elf/dl-reloc-static-pie.c
+++ b/elf/dl-reloc-static-pie.c
@@ -25,7 +25,6 @@
#include <dl-machine.h>
-#define STATIC_PIE_BOOTSTRAP
#define RESOLVE_MAP(map, scope, sym, version, flags) map
#include "dynamic-link.h"
#include "get-dynamic-info.h"
@@ -52,7 +51,7 @@ _dl_relocate_static_pie (void)
break;
}
- elf_get_dynamic_info (main_map);
+ elf_get_dynamic_info (main_map, false, true);
# ifdef ELF_MACHINE_BEFORE_RTLD_RELOC
ELF_MACHINE_BEFORE_RTLD_RELOC (main_map, main_map->l_info);
diff --git a/elf/get-dynamic-info.h b/elf/get-dynamic-info.h
index 1ac0663d1f..f63e07dc6d 100644
--- a/elf/get-dynamic-info.h
+++ b/elf/get-dynamic-info.h
@@ -26,7 +26,8 @@
#include <libc-diag.h>
static inline void __attribute__ ((unused, always_inline))
-elf_get_dynamic_info (struct link_map *l)
+elf_get_dynamic_info (struct link_map *l, bool bootstrap,
+ bool static_pie_bootstrap)
{
#if __ELF_NATIVE_CLASS == 32
typedef Elf32_Word d_tag_utype;
@@ -35,7 +36,7 @@ elf_get_dynamic_info (struct link_map *l)
#endif
#ifndef STATIC_PIE_BOOTSTRAP
- if (l->l_ld == NULL)
+ if (!bootstrap && l->l_ld == NULL)
return;
#endif
@@ -112,47 +113,63 @@ elf_get_dynamic_info (struct link_map *l)
if (info[DT_REL] != NULL)
assert (info[DT_RELENT]->d_un.d_val == sizeof (ElfW(Rel)));
#endif
-#ifdef STATIC_PIE_BOOTSTRAP
- assert (info[DT_RUNPATH] == NULL);
- assert (info[DT_RPATH] == NULL);
-#endif
- if (info[DT_FLAGS] != NULL)
+ if (bootstrap || static_pie_bootstrap)
{
- /* Flags are used. Translate to the old form where available.
- Since these l_info entries are only tested for NULL pointers it
- is ok if they point to the DT_FLAGS entry. */
- l->l_flags = info[DT_FLAGS]->d_un.d_val;
-
- if (l->l_flags & DF_SYMBOLIC)
- info[DT_SYMBOLIC] = info[DT_FLAGS];
- if (l->l_flags & DF_TEXTREL)
- info[DT_TEXTREL] = info[DT_FLAGS];
- if (l->l_flags & DF_BIND_NOW)
- info[DT_BIND_NOW] = info[DT_FLAGS];
+ assert (info[DT_RUNPATH] == NULL);
+ assert (info[DT_RPATH] == NULL);
}
- if (info[VERSYMIDX (DT_FLAGS_1)] != NULL)
+ if (bootstrap)
{
- l->l_flags_1 = info[VERSYMIDX (DT_FLAGS_1)]->d_un.d_val;
- if (l->l_flags_1 & DF_1_NODELETE)
- l->l_nodelete_pending = true;
-
- /* Only DT_1_SUPPORTED_MASK bits are supported, and we would like
- to assert this, but we can't. Users have been setting
- unsupported DF_1_* flags for a long time and glibc has ignored
- them. Therefore to avoid breaking existing applications the
- best we can do is add a warning during debugging with the
- intent of notifying the user of the problem. */
- if (__builtin_expect (GLRO(dl_debug_mask) & DL_DEBUG_FILES, 0)
- && l->l_flags_1 & ~DT_1_SUPPORTED_MASK)
- _dl_debug_printf ("\nWARNING: Unsupported flag value(s) of 0x%x in DT_FLAGS_1.\n",
- l->l_flags_1 & ~DT_1_SUPPORTED_MASK);
-
- if (l->l_flags_1 & DF_1_NOW)
- info[DT_BIND_NOW] = info[VERSYMIDX (DT_FLAGS_1)];
+ /* Only the bind now flags are allowed. */
+ assert (info[VERSYMIDX (DT_FLAGS_1)] == NULL
+ || (info[VERSYMIDX (DT_FLAGS_1)]->d_un.d_val & ~DF_1_NOW) == 0);
+ /* Flags must not be set for ld.so. */
+ assert (info[DT_FLAGS] == NULL
+ || (info[DT_FLAGS]->d_un.d_val & ~DF_BIND_NOW) == 0);
}
- if (info[DT_RUNPATH] != NULL)
- /* If both RUNPATH and RPATH are given, the latter is ignored. */
- info[DT_RPATH] = NULL;
+ else
+ {
+ if (info[DT_FLAGS] != NULL)
+ {
+ /* Flags are used. Translate to the old form where available.
+ Since these l_info entries are only tested for NULL pointers it
+ is ok if they point to the DT_FLAGS entry. */
+ l->l_flags = info[DT_FLAGS]->d_un.d_val;
+
+ if (l->l_flags & DF_SYMBOLIC)
+ info[DT_SYMBOLIC] = info[DT_FLAGS];
+ if (l->l_flags & DF_TEXTREL)
+ info[DT_TEXTREL] = info[DT_FLAGS];
+ if (l->l_flags & DF_BIND_NOW)
+ info[DT_BIND_NOW] = info[DT_FLAGS];
+ }
+
+ if (info[VERSYMIDX (DT_FLAGS_1)] != NULL)
+ {
+ l->l_flags_1 = info[VERSYMIDX (DT_FLAGS_1)]->d_un.d_val;
+ if (l->l_flags_1 & DF_1_NODELETE)
+ l->l_nodelete_pending = true;
+
+ /* Only DT_1_SUPPORTED_MASK bits are supported, and we would like
+ to assert this, but we can't. Users have been setting
+ unsupported DF_1_* flags for a long time and glibc has ignored
+ them. Therefore to avoid breaking existing applications the
+ best we can do is add a warning during debugging with the
+ intent of notifying the user of the problem. */
+ if (__builtin_expect (GLRO(dl_debug_mask) & DL_DEBUG_FILES, 0)
+ && l->l_flags_1 & ~DT_1_SUPPORTED_MASK)
+ _dl_debug_printf ("\nWARNING: Unsupported flag value(s) of 0x%x "
+ "in DT_FLAGS_1.\n",
+ l->l_flags_1 & ~DT_1_SUPPORTED_MASK);
+
+ if (l->l_flags_1 & DF_1_NOW)
+ info[DT_BIND_NOW] = info[VERSYMIDX (DT_FLAGS_1)];
+ }
+
+ if (info[DT_RUNPATH] != NULL)
+ /* If both RUNPATH and RPATH are given, the latter is ignored. */
+ info[DT_RPATH] = NULL;
+ }
}
#endif
diff --git a/elf/rtld.c b/elf/rtld.c
index 6cfb7cf672..c66a1d00bc 100644
--- a/elf/rtld.c
+++ b/elf/rtld.c
@@ -547,7 +547,7 @@ _dl_start (void *arg)
/* Read our own dynamic section and fill in the info array. */
bootstrap_map.l_ld = (void *) bootstrap_map.l_addr + elf_machine_dynamic ();
- elf_get_dynamic_info (&bootstrap_map);
+ elf_get_dynamic_info (&bootstrap_map, true, false);
#if NO_TLS_OFFSET != 0
bootstrap_map.l_tls_offset = NO_TLS_OFFSET;
@@ -1615,7 +1615,7 @@ dl_main (const ElfW(Phdr) *phdr,
if (! rtld_is_main)
{
/* Extract the contents of the dynamic section for easy access. */
- elf_get_dynamic_info (main_map);
+ elf_get_dynamic_info (main_map, false, false);
/* If the main map is libc.so, update the base namespace to
refer to this map. If libc.so is loaded later, this happens
diff --git a/elf/setup-vdso.h b/elf/setup-vdso.h
index f44748bc98..3f20578046 100644
--- a/elf/setup-vdso.h
+++ b/elf/setup-vdso.h
@@ -64,7 +64,7 @@ setup_vdso (struct link_map *main_map __attribute__ ((unused)),
l->l_map_end += l->l_addr;
l->l_text_end += l->l_addr;
l->l_ld = (void *) ((ElfW(Addr)) l->l_ld + l->l_addr);
- elf_get_dynamic_info (l);
+ elf_get_dynamic_info (l, false, false);
_dl_setup_hash (l);
l->l_relocated = 1;
@@ -1298,7 +1298,7 @@ _dl_map_object_from_fd (const char *name, const char *origname, int fd,
else
l->l_ld = (ElfW(Dyn) *) ((ElfW(Addr)) l->l_ld + l->l_addr);
- elf_get_dynamic_info (l, false);
+ elf_get_dynamic_info (l);
/* Make sure we are not dlopen'ing an object that has the
DF_1_NOOPEN flag set, or a PIE object. */
@@ -52,7 +52,7 @@ _dl_relocate_static_pie (void)
break;
}
- elf_get_dynamic_info (main_map, false);
+ elf_get_dynamic_info (main_map);
# ifdef ELF_MACHINE_BEFORE_RTLD_RELOC
ELF_MACHINE_BEFORE_RTLD_RELOC (main_map, main_map->l_info);
new file mode 100644
@@ -0,0 +1,51 @@
+/* Inline functions for dynamic linking.
+ Copyright (C) 1995-2021 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _DL_STATIC_TLS_H
+#define _DL_STATIC_TLS_H
+
+/* This macro is used as a callback from elf_machine_rel{a,} when a
+ static TLS reloc is about to be performed. Since (in dl-load.c) we
+ permit dynamic loading of objects that might use such relocs, we
+ have to check whether each use is actually doable. If the object
+ whose TLS segment the reference resolves to was allocated space in
+ the static TLS block at startup, then it's ok. Otherwise, we make
+ an attempt to allocate it in surplus space on the fly. If that
+ can't be done, we fall back to the error that DF_STATIC_TLS is
+ intended to produce. */
+#define HAVE_STATIC_TLS(map, sym_map) \
+ (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET \
+ && ((sym_map)->l_tls_offset \
+ != FORCED_DYNAMIC_TLS_OFFSET), 1))
+
+#define CHECK_STATIC_TLS(map, sym_map) \
+ do { \
+ if (!HAVE_STATIC_TLS (map, sym_map)) \
+ _dl_allocate_static_tls (sym_map); \
+ } while (0)
+
+#define TRY_STATIC_TLS(map, sym_map) \
+ (__builtin_expect ((sym_map)->l_tls_offset \
+ != FORCED_DYNAMIC_TLS_OFFSET, 1) \
+ && (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET, 1) \
+ || _dl_try_allocate_static_tls (sym_map, true) == 0))
+
+int _dl_try_allocate_static_tls (struct link_map *map, bool optional)
+ attribute_hidden;
+
+#endif
@@ -16,35 +16,7 @@
License along with the GNU C Library; if not, see
<https://www.gnu.org/licenses/>. */
-/* This macro is used as a callback from elf_machine_rel{a,} when a
- static TLS reloc is about to be performed. Since (in dl-load.c) we
- permit dynamic loading of objects that might use such relocs, we
- have to check whether each use is actually doable. If the object
- whose TLS segment the reference resolves to was allocated space in
- the static TLS block at startup, then it's ok. Otherwise, we make
- an attempt to allocate it in surplus space on the fly. If that
- can't be done, we fall back to the error that DF_STATIC_TLS is
- intended to produce. */
-#define HAVE_STATIC_TLS(map, sym_map) \
- (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET \
- && ((sym_map)->l_tls_offset \
- != FORCED_DYNAMIC_TLS_OFFSET), 1))
-
-#define CHECK_STATIC_TLS(map, sym_map) \
- do { \
- if (!HAVE_STATIC_TLS (map, sym_map)) \
- _dl_allocate_static_tls (sym_map); \
- } while (0)
-
-#define TRY_STATIC_TLS(map, sym_map) \
- (__builtin_expect ((sym_map)->l_tls_offset \
- != FORCED_DYNAMIC_TLS_OFFSET, 1) \
- && (__builtin_expect ((sym_map)->l_tls_offset != NO_TLS_OFFSET, 1) \
- || _dl_try_allocate_static_tls (sym_map, true) == 0))
-
-int _dl_try_allocate_static_tls (struct link_map *map, bool optional)
- attribute_hidden;
-
+#include <dl-machine.h>
#include <elf.h>
#ifdef RESOLVE_MAP
@@ -91,9 +63,6 @@ elf_machine_lazy_rel (struct link_map *map, struct r_scope_elem *scope[],
# endif
#endif
-#include <dl-machine.h>
-
-
#ifdef RESOLVE_MAP
# if defined RTLD_BOOTSTRAP || defined STATIC_PIE_BOOTSTRAP
@@ -22,10 +22,11 @@
#define _GET_DYNAMIC_INFO_H
#include <assert.h>
+#include <dl-machine-rel.h>
#include <libc-diag.h>
static inline void __attribute__ ((unused, always_inline))
-elf_get_dynamic_info (struct link_map *l, bool check)
+elf_get_dynamic_info (struct link_map *l)
{
#if __ELF_NATIVE_CLASS == 32
typedef Elf32_Word d_tag_utype;
@@ -33,7 +34,7 @@ elf_get_dynamic_info (struct link_map *l, bool check)
typedef Elf64_Xword d_tag_utype;
#endif
-#if !defined RTLD_BOOTSTRAP && !defined STATIC_PIE_BOOTSTRAP
+#ifndef STATIC_PIE_BOOTSTRAP
if (l->l_ld == NULL)
return;
#endif
@@ -111,21 +112,10 @@ elf_get_dynamic_info (struct link_map *l, bool check)
if (info[DT_REL] != NULL)
assert (info[DT_RELENT]->d_un.d_val == sizeof (ElfW(Rel)));
#endif
-#ifdef RTLD_BOOTSTRAP
- if (check)
- {
- /* Only the bind now flags are allowed. */
- assert (info[VERSYMIDX (DT_FLAGS_1)] == NULL
- || (info[VERSYMIDX (DT_FLAGS_1)]->d_un.d_val & ~DF_1_NOW) == 0);
- /* Flags must not be set for ld.so. */
- assert (info[DT_FLAGS] == NULL
- || (info[DT_FLAGS]->d_un.d_val & ~DF_BIND_NOW) == 0);
-# ifdef STATIC_PIE_BOOTSTRAP
- assert (info[DT_RUNPATH] == NULL);
- assert (info[DT_RPATH] == NULL);
-# endif
- }
-#else
+#ifdef STATIC_PIE_BOOTSTRAP
+ assert (info[DT_RUNPATH] == NULL);
+ assert (info[DT_RPATH] == NULL);
+#endif
if (info[DT_FLAGS] != NULL)
{
/* Flags are used. Translate to the old form where available.
@@ -163,7 +153,6 @@ elf_get_dynamic_info (struct link_map *l, bool check)
if (info[DT_RUNPATH] != NULL)
/* If both RUNPATH and RPATH are given, the latter is ignored. */
info[DT_RPATH] = NULL;
-#endif
}
#endif
@@ -32,7 +32,6 @@
#include <fpu_control.h>
#include <hp-timing.h>
#include <libc-lock.h>
-#include "dynamic-link.h"
#include <dl-librecon.h>
#include <unsecvars.h>
#include <dl-cache.h>
@@ -50,9 +49,18 @@
#include <dl-main.h>
#include <gnu/lib-names.h>
#include <dl-tunables.h>
+#include <get-dynamic-info.h>
#include <assert.h>
+/* This #define produces dynamic linking inline functions for
+ bootstrap relocation instead of general-purpose relocation.
+ Since ld.so must not have any undefined symbols the result
+ is trivial: always the map of ld.so itself. */
+#define RTLD_BOOTSTRAP
+#define RESOLVE_MAP(map, scope, sym, version, flags) map
+#include "dynamic-link.h"
+
/* Only enables rtld profiling for architectures which provides non generic
hp-timing support. The generic support requires either syscall
(clock_gettime), which will incur in extra overhead on loading time.
@@ -507,15 +515,6 @@ _dl_start_final (void *arg, struct dl_start_final_info *info)
# define bootstrap_map info.l
#endif
- /* This #define produces dynamic linking inline functions for
- bootstrap relocation instead of general-purpose relocation.
- Since ld.so must not have any undefined symbols the result
- is trivial: always the map of ld.so itself. */
-#define RTLD_BOOTSTRAP
-#define RESOLVE_MAP(map, scope, sym, version, flags) map
-#include "get-dynamic-info.h"
-#include "dynamic-link.h"
-
static ElfW(Addr) __attribute_used__
_dl_start (void *arg)
{
@@ -548,7 +547,7 @@ _dl_start (void *arg)
/* Read our own dynamic section and fill in the info array. */
bootstrap_map.l_ld = (void *) bootstrap_map.l_addr + elf_machine_dynamic ();
- elf_get_dynamic_info (&bootstrap_map, true);
+ elf_get_dynamic_info (&bootstrap_map);
#if NO_TLS_OFFSET != 0
bootstrap_map.l_tls_offset = NO_TLS_OFFSET;
@@ -1616,7 +1615,7 @@ dl_main (const ElfW(Phdr) *phdr,
if (! rtld_is_main)
{
/* Extract the contents of the dynamic section for easy access. */
- elf_get_dynamic_info (main_map, false);
+ elf_get_dynamic_info (main_map);
/* If the main map is libc.so, update the base namespace to
refer to this map. If libc.so is loaded later, this happens
@@ -64,7 +64,7 @@ setup_vdso (struct link_map *main_map __attribute__ ((unused)),
l->l_map_end += l->l_addr;
l->l_text_end += l->l_addr;
l->l_ld = (void *) ((ElfW(Addr)) l->l_ld + l->l_addr);
- elf_get_dynamic_info (l, false);
+ elf_get_dynamic_info (l);
_dl_setup_hash (l);
l->l_relocated = 1;
@@ -24,7 +24,9 @@
#include <sysdep.h>
#include <tls.h>
#include <dl-tlsdesc.h>
+#include <dl-static-tls.h>
#include <dl-irel.h>
+#include <dl-machine-rel.h>
#include <cpu-features.c>
/* Translate a processor specific dynamic tag to the index in l_info array. */
@@ -190,10 +192,6 @@ _dl_start_user: \n\
#define ELF_MACHINE_JMP_SLOT AARCH64_R(JUMP_SLOT)
-/* AArch64 uses RELA not REL */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
#define DL_PLATFORM_INIT dl_platform_init ()
static inline void __attribute__ ((unused))
@@ -370,7 +368,7 @@ elf_machine_rela (struct link_map *map, struct r_scope_elem *scope[],
}
}
-inline void
+static inline void
__attribute__ ((always_inline))
elf_machine_rela_relative (ElfW(Addr) l_addr,
const ElfW(Rela) *reloc,
@@ -25,6 +25,8 @@
#define ELF_MACHINE_NAME "alpha"
#include <string.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Mask identifying addresses reserved for the user program,
@@ -240,10 +242,6 @@ $fixup_stack: \n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_ALPHA_JMP_SLOT
-/* The alpha never uses Elf64_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization functions. This is called very early in
* _dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()
@@ -30,6 +30,8 @@
#include <string.h>
#include <link.h>
#include <dl-tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Dynamic Linking ABI for ARCv2 ISA.
@@ -203,10 +205,6 @@ __start: \n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_ARC_JUMP_SLOT
-/* ARC uses Rela relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Fixup a PLT entry to bounce directly to the function at VALUE. */
static inline ElfW(Addr)
@@ -318,7 +316,7 @@ elf_machine_rela (struct link_map *map, struct r_scope_elem *scope[],
}
}
-inline void
+static inline void
__attribute__ ((always_inline))
elf_machine_rela_relative (ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
void *const reloc_addr_arg)
@@ -327,7 +325,7 @@ elf_machine_rela_relative (ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
*reloc_addr += l_addr;
}
-inline void
+static inline void
__attribute__ ((always_inline))
elf_machine_lazy_rel (struct link_map *map, struct r_scope_elem *scope[],
ElfW(Addr) l_addr, const ElfW(Rela) *reloc,
new file mode 100644
@@ -0,0 +1,31 @@
+/* ELF dynamic relocation type supported by the architecture. ARM version.
+ Copyright (C) 2001-2021 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _DL_MACHINE_REL_H
+#define _DL_MACHINE_REL_H
+
+/* ARM never uses Elf32_Rela relocations for the dynamic linker.
+ Prelinked libraries may use Elf32_Rela though. */
+#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
+#define ELF_MACHINE_NO_REL 0
+
+/* ARM never uses Elf32_Rela relocations for the dynamic linker.
+ Prelinked libraries may use Elf32_Rela though. */
+#define ELF_MACHINE_PLT_REL 1
+
+#endif
@@ -26,6 +26,8 @@
#include <tls.h>
#include <dl-tlsdesc.h>
#include <dl-irel.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
#ifndef CLEAR_CACHE
# error CLEAR_CACHE definition required to handle TEXTREL
@@ -231,10 +233,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_ARM_JUMP_SLOT
-/* ARM never uses Elf32_Rela relocations for the dynamic linker.
- Prelinked libraries may use Elf32_Rela though. */
-#define ELF_MACHINE_PLT_REL 1
-
/* We define an initialization functions. This is called very early in
_dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()
@@ -267,11 +265,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rel *reloc,
#endif /* !dl_machine_h */
-/* ARM never uses Elf32_Rela relocations for the dynamic linker.
- Prelinked libraries may use Elf32_Rela though. */
-#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
-#define ELF_MACHINE_NO_REL 0
-
/* Names of the architecture-specific auditing callback functions. */
#define ARCH_LA_PLTENTER arm_gnu_pltenter
#define ARCH_LA_PLTEXIT arm_gnu_pltexit
@@ -24,6 +24,8 @@
#include <sys/param.h>
#include <sysdep.h>
#include <dl-tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero if ELF header is compatible with the running host. */
static inline int
@@ -172,10 +174,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_CKCORE_JUMP_SLOT
-/* C-SKY never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization functions. This is called very early in
_dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()
new file mode 100644
@@ -0,0 +1,27 @@
+/* ELF dynamic relocation type supported by the architecture.
+ Copyright (C) 2001-2021 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _DL_MACHINE_REL_H
+#define _DL_MACHINE_REL_H
+
+/* Defined if the architecture supports Elf{32,64}_Rel relocations. */
+#define ELF_MACHINE_NO_REL 1
+/* Defined if the architecture supports Elf{32,64}_Rela relocations. */
+#define ELF_MACHINE_NO_RELA 0
+
+#endif
@@ -20,6 +20,8 @@
#include <string.h>
#include <link.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
@@ -29,6 +29,8 @@
#include <dl-fptr.h>
#include <abort-instr.h>
#include <tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* These two definitions must match the definition of the stub in
bfd/elf32-hppa.c (see plt_stub[]).
@@ -509,10 +511,6 @@ asm ( \
#define ELF_MACHINE_JMP_SLOT R_PARISC_IPLT
#define ELF_MACHINE_SIZEOF_JMP_SLOT PLT_ENTRY_SIZE
-/* We only use RELA. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Return the address of the entry point. */
#define ELF_MACHINE_START_ADDRESS(map, start) \
({ \
new file mode 100644
@@ -0,0 +1,31 @@
+/* ELF dynamic relocation type supported by the architecture. ARM version.
+ Copyright (C) 2001-2021 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _DL_MACHINE_REL_H
+#define _DL_MACHINE_REL_H
+
+/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
+ Prelinked libraries may use Elf32_Rela though. */
+#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
+#define ELF_MACHINE_NO_REL 0
+
+/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
+ Prelinked libraries may use Elf32_Rela though. */
+#define ELF_MACHINE_PLT_REL 1
+
+#endif
@@ -26,6 +26,8 @@
#include <sysdep.h>
#include <tls.h>
#include <dl-tlsdesc.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int __attribute__ ((unused))
@@ -230,10 +232,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_386_JMP_SLOT
-/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
- Prelinked libraries may use Elf32_Rela though. */
-#define ELF_MACHINE_PLT_REL 1
-
/* We define an initialization functions. This is called very early in
_dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()
@@ -276,11 +274,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rel *reloc,
#endif /* !dl_machine_h */
-/* The i386 never uses Elf32_Rela relocations for the dynamic linker.
- Prelinked libraries may use Elf32_Rela though. */
-#define ELF_MACHINE_NO_RELA defined RTLD_BOOTSTRAP
-#define ELF_MACHINE_NO_REL 0
-
#ifdef RESOLVE_MAP
/* Perform the relocation specified by RELOC and SYM (which is fully resolved).
@@ -27,6 +27,8 @@
#include <errno.h>
#include <dl-fptr.h>
#include <tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Translate a processor specific dynamic tag to the index
in l_info array. */
@@ -319,10 +321,6 @@ elf_machine_runtime_setup (struct link_map *l, struct r_scope_elem *scope[],
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_IA64_IPLTLSB
-/* According to the IA-64 specific documentation, Rela is always used. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Return the address of the entry point. */
#define ELF_MACHINE_START_ADDRESS(map, start) \
({ \
@@ -24,6 +24,8 @@
#include <sys/param.h>
#include <sysdep.h>
#include <dl-tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int
@@ -183,10 +185,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_68K_JMP_SLOT
-/* The m68k never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
static inline Elf32_Addr
elf_machine_fixup_plt (struct link_map *map, lookup_t t,
const ElfW(Sym) *refsym, const ElfW(Sym) *sym,
@@ -23,6 +23,8 @@
#include <sys/param.h>
#include <tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int
@@ -169,10 +171,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_MICROBLAZE_JUMP_SLOT
-/* The microblaze never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
static inline Elf32_Addr
elf_machine_fixup_plt (struct link_map *map, lookup_t t,
const ElfW(Sym) *refsym, const ElfW(Sym) *sym,
new file mode 100644
@@ -0,0 +1,26 @@
+/* ELF dynamic relocation type supported by the architecture. ARM version.
+ Copyright (C) 2001-2021 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _DL_MACHINE_REL_H
+#define _DL_MACHINE_REL_H
+
+#define ELF_MACHINE_PLT_REL 1
+#define ELF_MACHINE_NO_REL 0
+#define ELF_MACHINE_NO_RELA 0
+
+#endif
@@ -32,6 +32,8 @@
#include <sysdep.h>
#include <sys/asm.h>
#include <dl-tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* The offset of gp from GOT might be system-dependent. It's set by
ld. The same value is also */
@@ -59,10 +61,6 @@
((((type) == ELF_MACHINE_JMP_SLOT) * ELF_RTYPE_CLASS_PLT) \
| (((type) == R_MIPS_COPY) * ELF_RTYPE_CLASS_COPY))
-#define ELF_MACHINE_PLT_REL 1
-#define ELF_MACHINE_NO_REL 0
-#define ELF_MACHINE_NO_RELA 0
-
/* Translate a processor specific dynamic tag to the index
in l_info array. */
#define DT_MIPS(x) (DT_MIPS_##x - DT_LOPROC + DT_NUM)
@@ -24,6 +24,8 @@
#include <string.h>
#include <link.h>
#include <dl-tls.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int
@@ -200,10 +202,6 @@ _start:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_NIOS2_JUMP_SLOT
-/* The Nios II never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Fixup a PLT entry to bounce directly to the function at VALUE. */
static inline Elf32_Addr
@@ -25,6 +25,8 @@
#include <dl-tls.h>
#include <dl-irel.h>
#include <hwcapinfo.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Translate a processor specific dynamic tag to the index
in l_info array. */
@@ -145,10 +147,6 @@ __elf_preferred_address(struct link_map *loader, size_t maplength,
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_PPC_JMP_SLOT
-/* The PowerPC never uses REL relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization function to initialize HWCAP/HWCAP2 and
platform data so it can be copied into the TCB later. This is called
very early in _dl_sysdep_start for dynamically linked binaries. */
new file mode 100644
@@ -0,0 +1,34 @@
+/* PowerPC ELFv1 function descriptor definition.
+ Copyright (C) 2009-2021 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _DL_FUNCDESC_H
+#define _DL_FUNCDESC_H
+
+#if _CALL_ELF != 2
+/* A PowerPC64 function descriptor. The .plt (procedure linkage
+ table) and .opd (official procedure descriptor) sections are
+ arrays of these. */
+typedef struct
+{
+ Elf64_Addr fd_func;
+ Elf64_Addr fd_toc;
+ Elf64_Addr fd_aux;
+} Elf64_FuncDesc;
+#endif
+
+#endif
@@ -23,7 +23,7 @@
#include <stdio.h>
#include <unistd.h>
#include <ldsodefs.h>
-#include <dl-machine.h>
+#include <dl-funcdesc.h>
#define ELF_MACHINE_IRELA 1
@@ -28,23 +28,14 @@
#include <sysdep.h>
#include <hwcapinfo.h>
#include <cpu-features.c>
+#include <dl-static-tls.h>
+#include <dl-funcdesc.h>
+#include <dl-machine-rel.h>
/* Translate a processor specific dynamic tag to the index
in l_info array. */
#define DT_PPC64(x) (DT_PPC64_##x - DT_LOPROC + DT_NUM)
-#if _CALL_ELF != 2
-/* A PowerPC64 function descriptor. The .plt (procedure linkage
- table) and .opd (official procedure descriptor) sections are
- arrays of these. */
-typedef struct
-{
- Elf64_Addr fd_func;
- Elf64_Addr fd_toc;
- Elf64_Addr fd_aux;
-} Elf64_FuncDesc;
-#endif
-
#define ELF_MULT_MACHINES_SUPPORTED
/* Return nonzero iff ELF header is compatible with the running host. */
@@ -292,10 +283,6 @@ BODY_PREFIX "_dl_start_user:\n" \
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_PPC64_JMP_SLOT
-/* The PowerPC never uses REL relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization function to initialize HWCAP/HWCAP2 and
platform data so it can be copied into the TCB later. This is called
very early in _dl_sysdep_start for dynamically linked binaries. */
@@ -26,6 +26,8 @@
#include <sys/asm.h>
#include <dl-tls.h>
#include <dl-irel.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
#ifndef _RTLD_PROLOGUE
# define _RTLD_PROLOGUE(entry) \
@@ -51,9 +53,6 @@
|| (__WORDSIZE == 64 && (type) == R_RISCV_TLS_TPREL64))) \
| (ELF_RTYPE_CLASS_COPY * ((type) == R_RISCV_COPY)))
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int __attribute_used__
elf_machine_matches_host (const ElfW(Ehdr) *ehdr)
@@ -26,6 +26,8 @@
#include <link.h>
#include <sysdeps/s390/dl-procinfo.h>
#include <dl-irel.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* This is an older, now obsolete value. */
#define EM_S390_OLD 0xA390
@@ -276,10 +278,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_390_JMP_SLOT
-/* The S390 never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization functions. This is called very early in
_dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()
@@ -27,6 +27,8 @@
#include <link.h>
#include <sysdeps/s390/dl-procinfo.h>
#include <dl-irel.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
#define ELF_MACHINE_IRELATIVE R_390_IRELATIVE
@@ -224,10 +226,6 @@ _dl_start_user:\n\
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_390_JMP_SLOT
-/* The 64 bit S/390 never uses Elf64_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization functions. This is called very early in
_dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()
@@ -24,6 +24,8 @@
#include <sys/param.h>
#include <sysdep.h>
#include <assert.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int __attribute__ ((unused))
@@ -251,10 +253,6 @@ elf_machine_plt_value (struct link_map *map, const Elf32_Rela *reloc,
#endif /* !dl_machine_h */
-/* SH never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
#ifdef RESOLVE_MAP
/* Perform the relocation specified by RELOC and SYM (which is fully resolved).
@@ -28,6 +28,8 @@
#include <tls.h>
#include <dl-plt.h>
#include <elf/dl-hwcaps.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int
@@ -196,10 +198,6 @@ elf_machine_runtime_setup (struct link_map *l, struct r_scope_elem *scope[],
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_SPARC_JMP_SLOT
-/* The SPARC never uses Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Undo the sub %sp, 6*4, %sp; add %sp, 22*4, %o0 below to get at the
value we want in __libc_stack_end. */
#define DL_STACK_END(cookie) \
@@ -26,6 +26,8 @@
#include <ldsodefs.h>
#include <sysdep.h>
#include <dl-plt.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
#define ELF64_R_TYPE_ID(info) ((info) & 0xff)
#define ELF64_R_TYPE_DATA(info) ((info) >> 8)
@@ -118,10 +120,6 @@ elf_machine_plt_value (struct link_map *map, const Elf64_Rela *reloc,
/* A reloc type used for ld.so cmdline arg lookups to reject PLT entries. */
#define ELF_MACHINE_JMP_SLOT R_SPARC_JMP_SLOT
-/* The SPARC never uses Elf64_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* Set up the loaded object described by L so its unrelocated PLT
entries will jump to the on-demand fixup code in dl-runtime.c. */
@@ -24,7 +24,7 @@
#include <sysdep-vdso.h>
#if (defined(__PPC64__) || defined(__powerpc64__)) && _CALL_ELF != 2
-# include <dl-machine.h>
+# include <dl-funcdesc.h>
/* The correct solution is for _dl_vdso_vsym to return the address of the OPD
for the kernel VDSO function. That address would then be stored in the
__vdso_* variables and returned as the result of the IFUNC resolver function.
@@ -26,6 +26,8 @@
#include <sysdep.h>
#include <tls.h>
#include <dl-tlsdesc.h>
+#include <dl-static-tls.h>
+#include <dl-machine-rel.h>
/* Return nonzero iff ELF header is compatible with the running host. */
static inline int __attribute__ ((unused))
@@ -200,10 +202,6 @@ _dl_start_user:\n\
// XXX This is a work-around for a broken linker. Remove!
#define ELF_MACHINE_IRELATIVE R_X86_64_IRELATIVE
-/* The x86-64 never uses Elf64_Rel/Elf32_Rel relocations. */
-#define ELF_MACHINE_NO_REL 1
-#define ELF_MACHINE_NO_RELA 0
-
/* We define an initialization function. This is called very early in
_dl_sysdep_start. */
#define DL_PLATFORM_INIT dl_platform_init ()