diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2021-04-28 12:33:57 -0700 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2021-05-15 21:44:38 +0200 |
| commit | e6f960fcbe0d2ecd72d7f7d10a2ed510701db35e (patch) | |
| tree | e8f94630d25c58f60598fe919fc81e2a805cb094 /linux-user/s390x | |
| parent | 915c69dc029e0fe509f8e4870993977068b66eaf (diff) | |
| download | qemu-e6f960fcbe0d2ecd72d7f7d10a2ed510701db35e.zip qemu-e6f960fcbe0d2ecd72d7f7d10a2ed510701db35e.tar.gz qemu-e6f960fcbe0d2ecd72d7f7d10a2ed510701db35e.tar.bz2 | |
linux-user/s390x: Remove restore_sigregs return value
The function cannot fail.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: David Hildenbrand <david@redhat.com>
Message-Id: <20210428193408.233706-5-richard.henderson@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'linux-user/s390x')
| -rw-r--r-- | linux-user/s390x/signal.c | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/linux-user/s390x/signal.c b/linux-user/s390x/signal.c index 1dfca71..e455a98 100644 --- a/linux-user/s390x/signal.c +++ b/linux-user/s390x/signal.c @@ -230,10 +230,8 @@ give_sigsegv: force_sigsegv(sig); } -static int -restore_sigregs(CPUS390XState *env, target_sigregs *sc) +static void restore_sigregs(CPUS390XState *env, target_sigregs *sc) { - int err = 0; int i; for (i = 0; i < 16; i++) { @@ -251,8 +249,6 @@ restore_sigregs(CPUS390XState *env, target_sigregs *sc) for (i = 0; i < 16; i++) { __get_user(*get_freg(env, i), &sc->fpregs.fprs[i]); } - - return err; } long do_sigreturn(CPUS390XState *env) @@ -271,9 +267,7 @@ long do_sigreturn(CPUS390XState *env) target_to_host_sigset_internal(&set, &target_set); set_sigmask(&set); /* ~_BLOCKABLE? */ - if (restore_sigregs(env, &frame->sregs)) { - goto badframe; - } + restore_sigregs(env, &frame->sregs); unlock_user_struct(frame, frame_addr, 0); return -TARGET_QEMU_ESIGRETURN; @@ -297,9 +291,7 @@ long do_rt_sigreturn(CPUS390XState *env) set_sigmask(&set); /* ~_BLOCKABLE? */ - if (restore_sigregs(env, &frame->uc.tuc_mcontext)) { - goto badframe; - } + restore_sigregs(env, &frame->uc.tuc_mcontext); target_restore_altstack(&frame->uc.tuc_stack, env); |