From 234c58467c3709bafdd3ffa6ac73655e1dfd9ddb Mon Sep 17 00:00:00 2001 From: Rich Felker Date: Sat, 12 Sep 2015 02:55:28 +0000 Subject: [PATCH] make sh clone asm fdpic-compatible clone calls back to a function pointer provided by the caller, which will actually be a pointer to a function descriptor on fdpic. the obvious solution is to have a separate version of clone for fdpic, but I have taken a simpler approach to go around the problem. instead of calling the pointed-to function from asm, a direct call is made to an internal C function which then calls the pointed-to function. this lets the C compiler generate the appropriate calling convention for an indirect call with no need for ABI-specific assembly. --- arch/sh/src/__shcall.c | 5 +++++ src/thread/sh/clone.s | 12 +++++++++--- 2 files changed, 14 insertions(+), 3 deletions(-) create mode 100644 arch/sh/src/__shcall.c diff --git a/arch/sh/src/__shcall.c b/arch/sh/src/__shcall.c new file mode 100644 index 00000000..dfe80a7f --- /dev/null +++ b/arch/sh/src/__shcall.c @@ -0,0 +1,5 @@ +__attribute__((__visibility__("hidden"))) +int __shcall(void *arg, int (*func)(void *)) +{ + return func(arg); +} diff --git a/src/thread/sh/clone.s b/src/thread/sh/clone.s index f8ad8451..aa4d0dfd 100644 --- a/src/thread/sh/clone.s +++ b/src/thread/sh/clone.s @@ -33,10 +33,12 @@ __clone: nop 1: ! we are the child, call fn(arg) - jsr @r1 - mov r2, r4 + mov.l 1f, r0 + mov r1, r5 + bsrf r0 + mov r2, r4 - mov #1, r3 ! __NR_exit +2: mov #1, r3 ! __NR_exit mov r0, r4 trapa #31 @@ -45,3 +47,7 @@ __clone: or r0, r0 or r0, r0 or r0, r0 + +.align 2 +.hidden __shcall +1: .long __shcall@PCREL+(.-2b) -- GitLab