From 611eabd489ce4eab3a70c410f363774bbcbbd3e9 Mon Sep 17 00:00:00 2001 From: Bobby Bingham Date: Sun, 16 Mar 2014 16:17:28 -0500 Subject: [PATCH] superh: fix dynamic linking of __fpscr_values Applications ended up with copy relocations for this array, which resulted in libc's references to this array pointing to the application's copy. The dynamic linker, however, can require this array before the application is relocated, and therefore before the application's copy of this array is initialized. This resulted in garbage being loaded into FPSCR before executing main, which violated the ABI. We fix this by putting the array in crt1 and making the libc copy private. This prevents libc's reference to the array from pointing to an uninitialized copy in the application. --- arch/sh/crt_arch.h | 3 +++ arch/sh/src/__fpsrc_values.c | 5 ++++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/arch/sh/crt_arch.h b/arch/sh/crt_arch.h index a69d1c9f..5fd39fc6 100644 --- a/arch/sh/crt_arch.h +++ b/arch/sh/crt_arch.h @@ -7,3 +7,6 @@ _start: \n\ bsr __cstart \n\ nop \n\ "); + +/* used by gcc for switching the FPU between single and double precision */ +const unsigned long __fpscr_values[2] = { 0, 0x80000 }; diff --git a/arch/sh/src/__fpsrc_values.c b/arch/sh/src/__fpsrc_values.c index 56056617..64b458f9 100644 --- a/arch/sh/src/__fpsrc_values.c +++ b/arch/sh/src/__fpsrc_values.c @@ -1,2 +1,5 @@ +#include "libc.h" + /* used by gcc for switching the FPU between single and double precision */ -const unsigned long __fpscr_values[2] = { 0, 0x80000 }; +const unsigned long __fpscr_values[2] ATTR_LIBC_VISIBILITY = { 0, 0x80000 }; + -- 2.25.1