Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 19 Mar 2018 14:28:20 +0000 (UTC)
From:      Marius Strobl <marius@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   svn commit: r331205 - in stable/11: lib/libc/sparc64/sys libexec/rtld-elf libexec/rtld-elf/aarch64 libexec/rtld-elf/amd64 libexec/rtld-elf/arm libexec/rtld-elf/i386 libexec/rtld-elf/mips libexec/rt...
Message-ID:  <201803191428.w2JESKXm027512@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: marius
Date: Mon Mar 19 14:28:20 2018
New Revision: 331205
URL: https://svnweb.freebsd.org/changeset/base/331205

Log:
  MFC: r328834
  
  o Let rtld(1) set up psABI user trap handlers prior to executing the
    objects' init functions instead of doing the setup via a constructor
    in libc as the init functions may already depend on these handlers
    to be in place. This gets us rid of:
    - the undefined order in which libc constructors as __guard_setup()
      and jemalloc_constructor() are executed WRT __sparc_utrap_setup(),
    - the requirement to link libc last so __sparc_utrap_setup() gets
      called prior to constructors in other libraries (see r122883).
    For static binaries, crt1.o still sets up the user trap handlers.
  o Move misplaced prototypes for MD functions in to the MD prototype
    section of rtld.h.
  o Sprinkle nitems().

Modified:
  stable/11/lib/libc/sparc64/sys/__sparc_utrap_setup.c
  stable/11/libexec/rtld-elf/aarch64/reloc.c
  stable/11/libexec/rtld-elf/amd64/reloc.c
  stable/11/libexec/rtld-elf/arm/reloc.c
  stable/11/libexec/rtld-elf/i386/reloc.c
  stable/11/libexec/rtld-elf/mips/reloc.c
  stable/11/libexec/rtld-elf/powerpc/reloc.c
  stable/11/libexec/rtld-elf/powerpc64/reloc.c
  stable/11/libexec/rtld-elf/riscv/reloc.c
  stable/11/libexec/rtld-elf/rtld.c
  stable/11/libexec/rtld-elf/rtld.h
  stable/11/libexec/rtld-elf/sparc64/reloc.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/lib/libc/sparc64/sys/__sparc_utrap_setup.c
==============================================================================
--- stable/11/lib/libc/sparc64/sys/__sparc_utrap_setup.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/lib/libc/sparc64/sys/__sparc_utrap_setup.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -29,13 +29,11 @@
 #include <sys/cdefs.h>
 __FBSDID("$FreeBSD$");
 
-#include <sys/types.h>
+#include <sys/param.h>
 
 #include <machine/utrap.h>
 #include <machine/sysarch.h>
 
-#include <stdlib.h>
-
 #include "__sparc_utrap_private.h"
 
 static const struct sparc_utrap_args ua[] = {
@@ -47,10 +45,10 @@ static const struct sparc_utrap_args ua[] = {
 };
 
 static const struct sparc_utrap_install_args uia[] = {
-	{ sizeof (ua) / sizeof (*ua), ua }
+	{ nitems(ua), ua }
 };
 
-void __sparc_utrap_setup(void) __attribute__((constructor));
+void __sparc_utrap_setup(void);
 
 void
 __sparc_utrap_setup(void)

Modified: stable/11/libexec/rtld-elf/aarch64/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/aarch64/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/aarch64/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -306,6 +306,13 @@ reloc_jmpslot(Elf_Addr *where, Elf_Addr target, const 
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
+}
+
+void
+pre_init(void)
+{
+
 }
 
 /*

Modified: stable/11/libexec/rtld-elf/amd64/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/amd64/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/amd64/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -485,6 +485,12 @@ ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] 
 }
 
 void
+pre_init(void)
+{
+
+}
+
+void
 allocate_initial_tls(Obj_Entry *objs)
 {
     /*

Modified: stable/11/libexec/rtld-elf/arm/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/arm/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/arm/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -481,6 +481,13 @@ reloc_jmpslot(Elf_Addr *where, Elf_Addr target, const 
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
+}
+
+void
+pre_init(void)
+{
+
 }
 
 void

Modified: stable/11/libexec/rtld-elf/i386/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/i386/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/i386/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -471,6 +471,12 @@ ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] 
 }
 
 void
+pre_init(void)
+{
+
+}
+
+void
 allocate_initial_tls(Obj_Entry *objs)
 {
     void* tls;

Modified: stable/11/libexec/rtld-elf/mips/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/mips/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/mips/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -628,6 +628,13 @@ reloc_jmpslot(Elf_Addr *where, Elf_Addr target, const 
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
+}
+
+void
+pre_init(void)
+{
+
 }
 
 void

Modified: stable/11/libexec/rtld-elf/powerpc/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/powerpc/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/powerpc/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -626,6 +626,13 @@ init_pltgot(Obj_Entry *obj)
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
+}
+
+void
+pre_init(void)
+{
+
 }
 
 void

Modified: stable/11/libexec/rtld-elf/powerpc64/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/powerpc64/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/powerpc64/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -531,6 +531,13 @@ init_pltgot(Obj_Entry *obj)
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
+}
+
+void
+pre_init(void)
+{
+
 }
 
 void

Modified: stable/11/libexec/rtld-elf/riscv/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/riscv/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/riscv/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -368,6 +368,13 @@ reloc_non_plt(Obj_Entry *obj, Obj_Entry *obj_rtld, int
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
+}
+
+void
+pre_init(void)
+{
+
 }
 
 void

Modified: stable/11/libexec/rtld-elf/rtld.c
==============================================================================
--- stable/11/libexec/rtld-elf/rtld.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/rtld.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -753,6 +753,12 @@ _rtld(Elf_Addr *sp, func_ptr_type *exit_proc, Obj_Entr
 	    obj_main->fini_array = (Elf_Addr)NULL;
     }
 
+    /*
+     * Execute MD initializers required before we call the objects'
+     * init functions.
+     */
+    pre_init();
+
     wlock_acquire(rtld_bind_lock, &lockstate);
     if (obj_main->crt_no_init)
 	preinit_main();

Modified: stable/11/libexec/rtld-elf/rtld.h
==============================================================================
--- stable/11/libexec/rtld-elf/rtld.h	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/rtld.h	Mon Mar 19 14:28:20 2018	(r331205)
@@ -371,8 +371,6 @@ void dump_Elf_Rela(Obj_Entry *, const Elf_Rela *, u_lo
 unsigned long elf_hash(const char *);
 const Elf_Sym *find_symdef(unsigned long, const Obj_Entry *,
   const Obj_Entry **, int, SymCache *, struct Struct_RtldLockState *);
-void ifunc_init(Elf_Auxinfo[__min_size(AT_COUNT)]);
-void init_pltgot(Obj_Entry *);
 void lockdflt_init(void);
 void digest_notes(Obj_Entry *, Elf_Addr, Elf_Addr);
 Obj_Entry *globallist_curr(const Obj_Entry *obj);
@@ -402,6 +400,9 @@ int reloc_plt(Obj_Entry *);
 int reloc_jmpslots(Obj_Entry *, int flags, struct Struct_RtldLockState *);
 int reloc_iresolve(Obj_Entry *, struct Struct_RtldLockState *);
 int reloc_gnu_ifunc(Obj_Entry *, int flags, struct Struct_RtldLockState *);
+void ifunc_init(Elf_Auxinfo[__min_size(AT_COUNT)]);
+void pre_init(void);
+void init_pltgot(Obj_Entry *);
 void allocate_initial_tls(Obj_Entry *);
 
 #endif /* } */

Modified: stable/11/libexec/rtld-elf/sparc64/reloc.c
==============================================================================
--- stable/11/libexec/rtld-elf/sparc64/reloc.c	Mon Mar 19 13:51:33 2018	(r331204)
+++ stable/11/libexec/rtld-elf/sparc64/reloc.c	Mon Mar 19 14:28:20 2018	(r331205)
@@ -365,8 +365,7 @@ reloc_nonplt_object(Obj_Entry *obj, const Elf_Rela *re
 	 * Note: R_SPARC_TLS_TPOFF64 must be the numerically largest
 	 * relocation type.
 	 */
-	if (type >= sizeof(reloc_target_bitmask) /
-	    sizeof(*reloc_target_bitmask)) {
+	if (type >= nitems(reloc_target_bitmask)) {
 		_rtld_error("%s: Unsupported relocation type %d in non-PLT "
 		    "object\n", obj->path, type);
 		return (-1);
@@ -791,8 +790,18 @@ reloc_jmpslot(Elf_Addr *wherep, Elf_Addr target, const
 void
 ifunc_init(Elf_Auxinfo aux_info[__min_size(AT_COUNT)] __unused)
 {
+
 }
 
+extern void __sparc_utrap_setup(void);
+
+void
+pre_init(void)
+{
+
+	__sparc_utrap_setup();
+}
+
 /*
  * Install rtld function call into this PLT slot.
  */
@@ -821,6 +830,7 @@ init_pltgot(Obj_Entry *obj)
 static void
 install_plt(Elf_Word *pltgot, Elf_Addr proc)
 {
+
 	pltgot[0] = SAVE;
 	flush(pltgot, 0);
 	pltgot[1] = SETHI_l0 | HIVAL(proc, 42);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201803191428.w2JESKXm027512>