From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-arm@nongnu.org, alex.bennee@linaro.org, laurent@vivier.eu
Subject: [PATCH v2 05/23] linux-user/arm: Implement setup_sigtramp
Date: Fri, 18 Jun 2021 12:29:33 -0700 [thread overview]
Message-ID: <20210618192951.125651-6-richard.henderson@linaro.org> (raw)
In-Reply-To: <20210618192951.125651-1-richard.henderson@linaro.org>
ARM is more complicated than the others, in that we also
have trampolines for using SA_RESTORER with FDPIC, and
we need to create trampolines for both ARM and Thumb modes.
Cc: qemu-arm@nongnu.org
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
linux-user/arm/target_signal.h | 2 +
linux-user/arm/signal.c | 170 +++++++++++++++++++--------------
2 files changed, 100 insertions(+), 72 deletions(-)
diff --git a/linux-user/arm/target_signal.h b/linux-user/arm/target_signal.h
index 0998dd6dfa..1e7fb0cecb 100644
--- a/linux-user/arm/target_signal.h
+++ b/linux-user/arm/target_signal.h
@@ -22,4 +22,6 @@ typedef struct target_sigaltstack {
#include "../generic/signal.h"
#define TARGET_ARCH_HAS_SETUP_FRAME
+#define TARGET_ARCH_HAS_SIGTRAMP_PAGE 1
+
#endif /* ARM_TARGET_SIGNAL_H */
diff --git a/linux-user/arm/signal.c b/linux-user/arm/signal.c
index 2d30345fc2..63bdd59ab9 100644
--- a/linux-user/arm/signal.c
+++ b/linux-user/arm/signal.c
@@ -101,13 +101,12 @@ struct sigframe_v1
{
struct target_sigcontext sc;
abi_ulong extramask[TARGET_NSIG_WORDS-1];
- abi_ulong retcode[4];
};
struct sigframe_v2
{
struct target_ucontext_v2 uc;
- abi_ulong retcode[4];
+ abi_ulong fdpic_ret;
};
struct rt_sigframe_v1
@@ -116,49 +115,20 @@ struct rt_sigframe_v1
abi_ulong puc;
struct target_siginfo info;
struct target_ucontext_v1 uc;
- abi_ulong retcode[4];
};
struct rt_sigframe_v2
{
struct target_siginfo info;
struct target_ucontext_v2 uc;
- abi_ulong retcode[4];
+ abi_ulong fdpic_ret;
};
/*
- * For ARM syscalls, we encode the syscall number into the instruction.
+ * Stubs needed to make sure the FD register (r9) contains the right value.
+ * There are 4 of them, each consuming 8 bytes.
*/
-#define SWI_SYS_SIGRETURN (0xef000000|(TARGET_NR_sigreturn + ARM_SYSCALL_BASE))
-#define SWI_SYS_RT_SIGRETURN (0xef000000|(TARGET_NR_rt_sigreturn + ARM_SYSCALL_BASE))
-
-/*
- * For Thumb syscalls, we pass the syscall number via r7. We therefore
- * need two 16-bit instructions.
- */
-#define SWI_THUMB_SIGRETURN (0xdf00 << 16 | 0x2700 | (TARGET_NR_sigreturn))
-#define SWI_THUMB_RT_SIGRETURN (0xdf00 << 16 | 0x2700 | (TARGET_NR_rt_sigreturn))
-
-static const abi_ulong retcodes[4] = {
- SWI_SYS_SIGRETURN, SWI_THUMB_SIGRETURN,
- SWI_SYS_RT_SIGRETURN, SWI_THUMB_RT_SIGRETURN
-};
-
-/*
- * Stub needed to make sure the FD register (r9) contains the right
- * value.
- */
-static const unsigned long sigreturn_fdpic_codes[3] = {
- 0xe59fc004, /* ldr r12, [pc, #4] to read function descriptor */
- 0xe59c9004, /* ldr r9, [r12, #4] to setup GOT */
- 0xe59cf000 /* ldr pc, [r12] to jump into restorer */
-};
-
-static const unsigned long sigreturn_fdpic_thumb_codes[3] = {
- 0xc008f8df, /* ldr r12, [pc, #8] to read function descriptor */
- 0x9004f8dc, /* ldr r9, [r12, #4] to setup GOT */
- 0xf000f8dc /* ldr pc, [r12] to jump into restorer */
-};
+static abi_ulong sigreturn_fdpic_tramp;
static inline int valid_user_regs(CPUARMState *regs)
{
@@ -219,13 +189,12 @@ get_sigframe(struct target_sigaction *ka, CPUARMState *regs, int framesize)
static int
setup_return(CPUARMState *env, struct target_sigaction *ka,
- abi_ulong *rc, abi_ulong frame_addr, int usig, abi_ulong rc_addr)
+ abi_ulong frame_addr, int usig)
{
abi_ulong handler = 0;
abi_ulong handler_fdpic_GOT = 0;
abi_ulong retcode;
-
- int thumb;
+ int thumb, retcode_idx;
int is_fdpic = info_is_fdpic(((TaskState *)thread_cpu->opaque)->info);
if (is_fdpic) {
@@ -243,6 +212,7 @@ setup_return(CPUARMState *env, struct target_sigaction *ka,
}
thumb = handler & 1;
+ retcode_idx = thumb + (ka->sa_flags & TARGET_SA_SIGINFO ? 2 : 0);
uint32_t cpsr = cpsr_read(env);
@@ -260,37 +230,24 @@ setup_return(CPUARMState *env, struct target_sigaction *ka,
if (ka->sa_flags & TARGET_SA_RESTORER) {
if (is_fdpic) {
- /* For FDPIC we ensure that the restorer is called with a
- * correct r9 value. For that we need to write code on
- * the stack that sets r9 and jumps back to restorer
- * value.
+ /*
+ * For FDPIC we ensure that the restorer is called with a
+ * correct r9 value. For that we use a special trampoline
+ * that reads the function descriptor from the frame,
+ * sets r9 and jumps back to restorer value.
*/
- if (thumb) {
- __put_user(sigreturn_fdpic_thumb_codes[0], rc);
- __put_user(sigreturn_fdpic_thumb_codes[1], rc + 1);
- __put_user(sigreturn_fdpic_thumb_codes[2], rc + 2);
- __put_user((abi_ulong)ka->sa_restorer, rc + 3);
- } else {
- __put_user(sigreturn_fdpic_codes[0], rc);
- __put_user(sigreturn_fdpic_codes[1], rc + 1);
- __put_user(sigreturn_fdpic_codes[2], rc + 2);
- __put_user((abi_ulong)ka->sa_restorer, rc + 3);
- }
-
- retcode = rc_addr + thumb;
+ abi_ulong fd_ofs = (retcode_idx & 2
+ ? offsetof(struct rt_sigframe_v2, fdpic_ret)
+ : offsetof(struct sigframe_v2, fdpic_ret));
+ put_user_ual(ka->sa_restorer, frame_addr + fd_ofs);
+ /* Each trampoline variant consumes 8-byte slot. */
+ retcode = sigreturn_fdpic_tramp + retcode_idx * 8 + thumb;
} else {
retcode = ka->sa_restorer;
}
} else {
- unsigned int idx = thumb;
-
- if (ka->sa_flags & TARGET_SA_SIGINFO) {
- idx += 2;
- }
-
- __put_user(retcodes[idx], rc);
-
- retcode = rc_addr + thumb;
+ /* Each trampoline variant consumes one 4-byte slot. */
+ retcode = default_sigreturn + retcode_idx * 4 + thumb;
}
env->regs[0] = usig;
@@ -394,8 +351,7 @@ static void setup_frame_v1(int usig, struct target_sigaction *ka,
__put_user(set->sig[i], &frame->extramask[i - 1]);
}
- if (setup_return(regs, ka, frame->retcode, frame_addr, usig,
- frame_addr + offsetof(struct sigframe_v1, retcode))) {
+ if (setup_return(regs, ka, frame_addr, usig)) {
goto sigsegv;
}
@@ -419,8 +375,7 @@ static void setup_frame_v2(int usig, struct target_sigaction *ka,
setup_sigframe_v2(&frame->uc, set, regs);
- if (setup_return(regs, ka, frame->retcode, frame_addr, usig,
- frame_addr + offsetof(struct sigframe_v2, retcode))) {
+ if (setup_return(regs, ka, frame_addr, usig)) {
goto sigsegv;
}
@@ -475,8 +430,7 @@ static void setup_rt_frame_v1(int usig, struct target_sigaction *ka,
__put_user(set->sig[i], &frame->uc.tuc_sigmask.sig[i]);
}
- if (setup_return(env, ka, frame->retcode, frame_addr, usig,
- frame_addr + offsetof(struct rt_sigframe_v1, retcode))) {
+ if (setup_return(env, ka, frame_addr, usig)) {
goto sigsegv;
}
@@ -509,8 +463,7 @@ static void setup_rt_frame_v2(int usig, struct target_sigaction *ka,
setup_sigframe_v2(&frame->uc, set, env);
- if (setup_return(env, ka, frame->retcode, frame_addr, usig,
- frame_addr + offsetof(struct rt_sigframe_v2, retcode))) {
+ if (setup_return(env, ka, frame_addr, usig)) {
goto sigsegv;
}
@@ -841,3 +794,76 @@ long do_rt_sigreturn(CPUARMState *env)
return do_rt_sigreturn_v1(env);
}
}
+
+void setup_sigtramp(abi_ulong sigtramp_page)
+{
+ enum {
+ /* For ARM, we encode the syscall number into the instruction. */
+ SWI_SYS_SIGRETURN =
+ 0xef000000 | (TARGET_NR_sigreturn + ARM_SYSCALL_BASE),
+ SWI_SYS_RT_SIGRETURN =
+ 0xef000000 | (TARGET_NR_rt_sigreturn + ARM_SYSCALL_BASE),
+
+ /*
+ * For Thumb , we pass the syscall number via r7.
+ * We therefore need two 16-bit instructions.
+ */
+ SWI_THUMB_SIGRETURN =
+ 0xdf00 << 16 | 0x2700 | TARGET_NR_sigreturn,
+ SWI_THUMB_RT_SIGRETURN =
+ 0xdf00 << 16 | 0x2700 | TARGET_NR_rt_sigreturn,
+
+ SIGFRAME_FDPIC_OFS = offsetof(struct sigframe_v2, fdpic_ret),
+ RT_SIGFRAME_FDPIC_OFS = offsetof(struct rt_sigframe_v2, fdpic_ret),
+ };
+
+ uint32_t total_size = 4 * 4 + 2 * 8;
+ uint32_t *tramp = lock_user(VERIFY_WRITE, sigtramp_page, total_size, 0);
+ uint32_t i = 0;
+
+ assert(tramp != NULL);
+
+ default_sigreturn = sigtramp_page;
+ __put_user(SWI_SYS_SIGRETURN, &tramp[i++]);
+ __put_user(SWI_THUMB_SIGRETURN, &tramp[i++]);
+ __put_user(SWI_SYS_RT_SIGRETURN, &tramp[i++]);
+ __put_user(SWI_THUMB_RT_SIGRETURN, &tramp[i++]);
+
+ /*
+ * FDPIC require trampolines to call sa_restorer.
+ *
+ * ARM versions use:
+ * ldr r9, [sp, #ofs]
+ * ldmia r9, {r9, pc}
+ *
+ * Thumb versions use:
+ * ldrd r9, r10, [sp, #ofs]
+ * bx r10
+ * nop
+ */
+ sigreturn_fdpic_tramp = sigtramp_page + i * 4;
+
+ /* ARM sigframe */
+ QEMU_BUILD_BUG_ON(SIGFRAME_FDPIC_OFS > 0xfff);
+ __put_user(0xe59d9000 | SIGFRAME_FDPIC_OFS, &tramp[i++]);
+ __put_user(0xe8998200, &tramp[i++]);
+
+ /* Thumb sigframe */
+ QEMU_BUILD_BUG_ON(SIGFRAME_FDPIC_OFS > 0xff << 2);
+ QEMU_BUILD_BUG_ON(SIGFRAME_FDPIC_OFS & 3);
+ __put_user(0x9a00e9dd | (SIGFRAME_FDPIC_OFS << 14), &tramp[i++]);
+ __put_user(0x46c04750, &tramp[i++]);
+
+ /* ARM rt_sigframe */
+ QEMU_BUILD_BUG_ON(RT_SIGFRAME_FDPIC_OFS > 0xfff);
+ __put_user(0xe59d9000 | RT_SIGFRAME_FDPIC_OFS, &tramp[i++]);
+ __put_user(0xe8998200, &tramp[i++]);
+
+ /* Thumb rt_sigframe */
+ QEMU_BUILD_BUG_ON(RT_SIGFRAME_FDPIC_OFS > 0xff << 2);
+ QEMU_BUILD_BUG_ON(RT_SIGFRAME_FDPIC_OFS & 3);
+ __put_user(0x9a00e9dd | (RT_SIGFRAME_FDPIC_OFS << 14), &tramp[i++]);
+ __put_user(0x46c04750, &tramp[i++]);
+
+ unlock_user(tramp, sigtramp_page, total_size);
+}
--
2.25.1
next prev parent reply other threads:[~2021-06-18 19:34 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-18 19:29 [PATCH v2 00/23] linux-user: Move signal trampolines to new page Richard Henderson
2021-06-18 19:29 ` [PATCH v2 01/23] linux-user: Add infrastructure for a signal trampoline page Richard Henderson
2021-06-19 9:33 ` Philippe Mathieu-Daudé
2021-06-29 13:30 ` Peter Maydell
2021-06-18 19:29 ` [PATCH v2 02/23] linux-user/aarch64: Implement setup_sigtramp Richard Henderson
2021-06-29 13:36 ` Peter Maydell
2021-07-01 19:27 ` Richard Henderson
2021-06-18 19:29 ` [PATCH v2 03/23] linux-user/arm: Split out v2_frame Richard Henderson
2021-06-29 13:53 ` Peter Maydell
2021-06-29 14:30 ` Richard Henderson
2021-06-18 19:29 ` [PATCH v2 04/23] linux-user/arm: Force v2 frames for fdpic Richard Henderson
2021-06-29 13:54 ` Peter Maydell
2021-06-18 19:29 ` Richard Henderson [this message]
2021-06-29 14:09 ` [PATCH v2 05/23] linux-user/arm: Implement setup_sigtramp Peter Maydell
2021-06-29 18:32 ` Richard Henderson
2021-06-18 19:29 ` [PATCH v2 06/23] linux-user/alpha: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 07/23] linux-user/cris: " Richard Henderson
2021-06-19 9:33 ` Philippe Mathieu-Daudé
2021-06-19 12:55 ` Richard Henderson
2021-06-19 14:17 ` Philippe Mathieu-Daudé
2021-06-18 19:29 ` [PATCH v2 08/23] linux-user/hexagon: " Richard Henderson
2021-06-19 9:31 ` Philippe Mathieu-Daudé
2021-06-18 19:29 ` [PATCH v2 09/23] linux-user/hppa: Document non-use of setup_sigtramp Richard Henderson
2021-06-18 19:29 ` [PATCH v2 10/23] linux-user/i386: Implement setup_sigtramp Richard Henderson
2021-06-29 14:40 ` Peter Maydell
2021-06-29 18:30 ` Richard Henderson
2021-06-18 19:29 ` [PATCH v2 11/23] linux-user/m68k: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 12/23] linux-user/microblaze: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 13/23] linux-user/mips: Tidy install_sigtramp Richard Henderson
2021-06-19 9:29 ` Philippe Mathieu-Daudé
2021-06-18 19:29 ` [PATCH v2 14/23] linux-user/mips: Implement setup_sigtramp Richard Henderson
2021-06-18 19:29 ` [PATCH v2 15/23] linux-user/nios2: Document non-use of setup_sigtramp Richard Henderson
2021-06-19 9:35 ` Philippe Mathieu-Daudé
2021-06-18 19:29 ` [PATCH v2 16/23] linux-user/openrisc: Implement setup_sigtramp Richard Henderson
2021-06-18 19:29 ` [PATCH v2 17/23] linux-user/ppc: " Richard Henderson
2021-06-29 14:52 ` Peter Maydell
2021-06-18 19:29 ` [PATCH v2 18/23] linux-user/riscv: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 19/23] linux-user/s390x: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 20/23] linux-user/sh4: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 21/23] linux-user/sparc: " Richard Henderson
2021-07-02 9:05 ` Philippe Mathieu-Daudé
2021-06-18 19:29 ` [PATCH v2 22/23] linux-user/xtensa: " Richard Henderson
2021-06-18 19:29 ` [PATCH v2 23/23] linux-user: Remove default for TARGET_ARCH_HAS_SIGTRAMP_PAGE Richard Henderson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210618192951.125651-6-richard.henderson@linaro.org \
--to=richard.henderson@linaro.org \
--cc=alex.bennee@linaro.org \
--cc=laurent@vivier.eu \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).