introduce new symbol-lookup-free rcrt1/dlstart stage chaining

previously, the call into stage 2 was made by looking up the symbol
name "__dls2" (which was chosen short to be easy to look up) from the
dynamic symbol table. this was no problem for the dynamic linker,
since it always exports all its symbols. in the case of the static pie
entry point, however, the dynamic symbol table does not contain the
necessary symbol unless -rdynamic/-E was used when linking. this
linking requirement is a major obstacle both to practical use of
static-pie as a nommu binary format (since it greatly enlarges the
file) and to upstream toolchain support for static-pie (adding -E to
default linking specs is not reasonable).

this patch replaces the runtime symbolic lookup with a link-time
lookup via an inline asm fragment, which reloc.h is responsible for
providing. in this initial commit, the asm is provided only for i386,
and the old lookup code is left in place as a fallback for archs that
have not yet transitioned.

modifying crt_arch.h to pass the stage-2 function pointer as an
argument was considered as an alternative, but such an approach would
not be compatible with fdpic, where it's impossible to compute
function pointers without already having performed relocations. it was
also deemed desirable to keep crt_arch.h as simple/minimal as
possible.

in principle, archs with pc-relative or got-relative addressing of
static variables could instead load the stage-2 function pointer from
a static volatile object. that does not work for fdpic, and is not
safe against reordering on mips-like archs that use got slots even for
static functions, but it's a valid on i386 and many others, and could
provide a reasonable default implementation in the future.

diff --git a/arch/i386/reloc.h b/arch/i386/reloc.h
index b52ef4024efce8e0af1dfcecaecb25a5fd8ea437..032f454bda5f2898f841f9bcb925976dd602166c 100644 (file)
--- a/arch/i386/reloc.h
+++ b/arch/i386/reloc.h
@@ -14,3 +14,10 @@
 
 #define CRTJMP(pc,sp) __asm__ __volatile__( \
        "mov %1,%%esp ; jmp *%0" : : "r"(pc), "r"(sp) : "memory" )
+
+#define GETFUNCSYM(fp, sym, got) __asm__ ( \
+       ".hidden " #sym "\n" \
+       "       call 1f\n" \
+       "1:     addl $" #sym "-.,(%%esp)\n" \
+       "       pop %0" \
+       : "=r"(*fp) : : "memory" )

diff --git a/src/ldso/dlstart.c b/src/ldso/dlstart.c
index 3aaa200fbbb5c55df7e1e5179ab1471df99c0d5e..eb919ab44b16a76d0118f43a8645b4f9fb8be813 100644 (file)
--- a/src/ldso/dlstart.c
+++ b/src/ldso/dlstart.c
@@ -74,6 +74,7 @@ void _dlstart_c(size_t *sp, size_t *dynv)
                *rel_addr = (size_t)base + rel[2];
        }
 
+#ifndef GETFUNCSYM
        const char *strings = (void *)(base + dyn[DT_STRTAB]);
        const Sym *syms = (void *)(base + dyn[DT_SYMTAB]);
 
@@ -85,6 +86,11 @@ void _dlstart_c(size_t *sp, size_t *dynv)
                        break;
        }
        ((stage2_func)(base + syms[i].st_value))(base, sp);
+#else
+       stage2_func dls2;
+       GETFUNCSYM(&dls2, __dls2, base+dyn[DT_PLTGOT]);
+       dls2(base, sp);
+#endif
 }
 
 #endif