From: Warner Losh <imp@bsdimp.com>
To: qemu-devel@nongnu.org
Cc: Peter Maydell <peter.maydell@linaro.org>,
Stacey Son <sson@FreeBSD.org>, Kyle Evans <kevans@freebsd.org>,
Richard Henderson <richard.henderson@linaro.org>,
def@FreeBSD.org, jrtc27@FreeBSD.org, Warner Losh <imp@bsdimp.com>
Subject: [PULL 37/40] bsd-user/signal.c: do_sigaltstack
Date: Mon, 31 Jan 2022 12:56:33 -0700 [thread overview]
Message-ID: <20220131195636.31991-38-imp@bsdimp.com> (raw)
In-Reply-To: <20220131195636.31991-1-imp@bsdimp.com>
Implement the meat of the sigaltstack(2) system call with do_sigaltstack.
With that, all the stubbed out routines are complete, so remove
now-incorrect comment.
Signed-off-by: Stacey Son <sson@FreeBSD.org>
Signed-off-by: Kyle Evans <kevans@freebsd.org>
Signed-off-by: Warner Losh <imp@bsdimp.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
---
bsd-user/signal.c | 72 +++++++++++++++++++++++++++++++++++++++++++----
1 file changed, 67 insertions(+), 5 deletions(-)
diff --git a/bsd-user/signal.c b/bsd-user/signal.c
index 5c94bd02e38..ad22ba9d90d 100644
--- a/bsd-user/signal.c
+++ b/bsd-user/signal.c
@@ -25,11 +25,6 @@
#include "hw/core/tcg-cpu-ops.h"
#include "host-signal.h"
-/*
- * Stubbed out routines until we merge signal support from bsd-user
- * fork.
- */
-
static struct target_sigaction sigact_table[TARGET_NSIG];
static void host_signal_handler(int host_sig, siginfo_t *info, void *puc);
static void target_to_host_sigset_internal(sigset_t *d,
@@ -573,6 +568,73 @@ static void host_signal_handler(int host_sig, siginfo_t *info, void *puc)
cpu_exit(thread_cpu);
}
+/* do_sigaltstack() returns target values and errnos. */
+/* compare to kern/kern_sig.c sys_sigaltstack() and kern_sigaltstack() */
+abi_long do_sigaltstack(abi_ulong uss_addr, abi_ulong uoss_addr, abi_ulong sp)
+{
+ TaskState *ts = (TaskState *)thread_cpu->opaque;
+ int ret;
+ target_stack_t oss;
+
+ if (uoss_addr) {
+ /* Save current signal stack params */
+ oss.ss_sp = tswapl(ts->sigaltstack_used.ss_sp);
+ oss.ss_size = tswapl(ts->sigaltstack_used.ss_size);
+ oss.ss_flags = tswapl(sas_ss_flags(ts, sp));
+ }
+
+ if (uss_addr) {
+ target_stack_t *uss;
+ target_stack_t ss;
+ size_t minstacksize = TARGET_MINSIGSTKSZ;
+
+ ret = -TARGET_EFAULT;
+ if (!lock_user_struct(VERIFY_READ, uss, uss_addr, 1)) {
+ goto out;
+ }
+ __get_user(ss.ss_sp, &uss->ss_sp);
+ __get_user(ss.ss_size, &uss->ss_size);
+ __get_user(ss.ss_flags, &uss->ss_flags);
+ unlock_user_struct(uss, uss_addr, 0);
+
+ ret = -TARGET_EPERM;
+ if (on_sig_stack(ts, sp)) {
+ goto out;
+ }
+
+ ret = -TARGET_EINVAL;
+ if (ss.ss_flags != TARGET_SS_DISABLE
+ && ss.ss_flags != TARGET_SS_ONSTACK
+ && ss.ss_flags != 0) {
+ goto out;
+ }
+
+ if (ss.ss_flags == TARGET_SS_DISABLE) {
+ ss.ss_size = 0;
+ ss.ss_sp = 0;
+ } else {
+ ret = -TARGET_ENOMEM;
+ if (ss.ss_size < minstacksize) {
+ goto out;
+ }
+ }
+
+ ts->sigaltstack_used.ss_sp = ss.ss_sp;
+ ts->sigaltstack_used.ss_size = ss.ss_size;
+ }
+
+ if (uoss_addr) {
+ ret = -TARGET_EFAULT;
+ if (copy_to_user(uoss_addr, &oss, sizeof(oss))) {
+ goto out;
+ }
+ }
+
+ ret = 0;
+out:
+ return ret;
+}
+
/* do_sigaction() return host values and errnos */
int do_sigaction(int sig, const struct target_sigaction *act,
struct target_sigaction *oact)
--
2.33.1
next prev parent reply other threads:[~2022-01-31 20:19 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-31 19:55 [PULL 00/40] Bsd user arm 2022q1 patches Warner Losh
2022-01-31 19:55 ` [PULL 01/40] bsd-user: Complete FreeBSD siginfo Warner Losh
2022-01-31 19:55 ` [PULL 02/40] bsd-user: Create setup_sigframe_arch to setup sigframe context Warner Losh
2022-01-31 19:55 ` [PULL 03/40] bsd-user/arm/signal.c: Implement setup_sigframe_arch for arm Warner Losh
2022-01-31 19:56 ` [PULL 04/40] bsd-user/arm/signal.c: get_mcontext should zero vfp data Warner Losh
2022-01-31 19:56 ` [PULL 05/40] bsd-user: Remove vestiges of signal queueing code Warner Losh
2022-01-31 19:56 ` [PULL 06/40] bsd-user: Bring in docs from linux-user for signal_pending Warner Losh
2022-01-31 19:56 ` [PULL 07/40] bsd-user/arm/target_arch_cpu.h: Move EXCP_ATOMIC to match linux-user Warner Losh
2022-01-31 19:56 ` [PULL 08/40] bsd-user/signal.c: implement force_sig_fault Warner Losh
2022-01-31 19:56 ` [PULL 09/40] bsd-user/signal-common.h: Move signal functions prototypes to here Warner Losh
2022-01-31 19:56 ` [PULL 10/40] bsd-user/signal.c: Implement cpu_loop_exit_sigsegv Warner Losh
2022-01-31 19:56 ` [PULL 11/40] bsd-user/signal.c: implement cpu_loop_exit_sigbus Warner Losh
2022-01-31 19:56 ` [PULL 12/40] bsd-user/arm/arget_arch_cpu.h: Move EXCP_DEBUG and EXCP_BKPT together Warner Losh
2022-01-31 19:56 ` [PULL 13/40] bsd-user/arm/target_arch_cpu.h: Correct code pointer Warner Losh
2022-01-31 19:56 ` [PULL 14/40] bsd-user/arm/target_arch_cpu.h: Use force_sig_fault for EXCP_UDEF Warner Losh
2022-01-31 19:56 ` [PULL 15/40] bsd-user/arm/target_arch_cpu.h: Implement data faults Warner Losh
2022-01-31 19:56 ` [PULL 16/40] bsd-user/signal.c: implement abstract target / host signal translation Warner Losh
2022-01-31 19:56 ` [PULL 17/40] bsd-user/signal.c: Implement signal_init() Warner Losh
2022-01-31 19:56 ` [PULL 18/40] bsd-user/signal.c: Add si_type argument to queue_signal Warner Losh
2022-01-31 19:56 ` [PULL 19/40] bsd-user/host/arm/host-signal.h: Implement host_signal_* Warner Losh
2022-01-31 19:56 ` [PULL 20/40] bsd-user/host/i386/host-signal.h: " Warner Losh
2022-01-31 19:56 ` [PULL 21/40] bsd-user/host/x86_64/host-signal.h: " Warner Losh
2022-01-31 19:56 ` [PULL 22/40] bsd-user: Add host signals to the build Warner Losh
2022-01-31 19:56 ` [PULL 23/40] bsd-user: Add trace events for bsd-user Warner Losh
2022-01-31 19:56 ` [PULL 24/40] bsd-user/signal.c: host_to_target_siginfo_noswap Warner Losh
2022-01-31 19:56 ` [PULL 25/40] bsd-user/signal.c: Implement rewind_if_in_safe_syscall Warner Losh
2022-01-31 19:56 ` [PULL 26/40] bsd-user/signal.c: Implement host_signal_handler Warner Losh
2022-01-31 19:56 ` [PULL 27/40] bsd-user/strace.c: print_taken_signal Warner Losh
2022-01-31 19:56 ` [PULL 28/40] bsd-user/signal.c: Implement dump_core_and_abort Warner Losh
2022-01-31 19:56 ` [PULL 29/40] bsd-user/signal.c: Fill in queue_signal Warner Losh
2022-01-31 19:56 ` [PULL 30/40] bsd-user/signal.c: sigset manipulation routines Warner Losh
2022-01-31 19:56 ` [PULL 31/40] bsd-user/signal.c: setup_frame Warner Losh
2022-01-31 19:56 ` [PULL 32/40] bsd-user/signal.c: handle_pending_signal Warner Losh
2022-01-31 19:56 ` [PULL 33/40] bsd-user/signal.c: tswap_siginfo Warner Losh
2022-01-31 19:56 ` [PULL 34/40] bsd-user/signal.c: process_pending_signals Warner Losh
2022-01-31 19:56 ` [PULL 35/40] bsd-user/signal.c: implement do_sigreturn Warner Losh
2022-01-31 19:56 ` [PULL 36/40] bsd-user/signal.c: implement do_sigaction Warner Losh
2022-01-31 19:56 ` Warner Losh [this message]
2022-01-31 19:56 ` [PULL 38/40] MAINTAINERS: Add tests/vm/*bsd to the list to get reviews on Warner Losh
2022-01-31 19:56 ` [PULL 39/40] bsd-user: Rename arg name for target_cpu_reset to env Warner Losh
2022-01-31 19:56 ` [PULL 40/40] bsd-user/freebsd/target_os_ucontext.h: Prefer env as arg name for CPUArchState args Warner Losh
2022-02-01 16:31 ` [PULL 00/40] Bsd user arm 2022q1 patches Peter Maydell
2022-02-01 21:03 ` Warner Losh
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=20220131195636.31991-38-imp@bsdimp.com \
--to=imp@bsdimp.com \
--cc=def@FreeBSD.org \
--cc=jrtc27@FreeBSD.org \
--cc=kevans@freebsd.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=sson@FreeBSD.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.