motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-04 00:03:54 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

linux-user/s390x: Remove restore_sigregs return value

Browse source 
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>
This commit is contained in:
Richard Henderson 2021-04-28 12:33:57 -07:00 • committed by Laurent Vivier
parent 915c69dc02
commit e6f960fcbe
1 changed files with 3 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

14
linux-user/s390x/signal.c
View file

@ -230,10 +230,8 @@ give_sigsegv:
force_sigsegv(sig); force_sigsegv(sig);
} }
static int static void restore_sigregs(CPUS390XState *env, target_sigregs *sc)
restore_sigregs(CPUS390XState *env, target_sigregs *sc)
{ {
int err = 0;
int i; int i;
for (i = 0; i < 16; i++) { for (i = 0; i < 16; i++) {
@ -251,8 +249,6 @@ restore_sigregs(CPUS390XState *env, target_sigregs *sc)
for (i = 0; i < 16; i++) { for (i = 0; i < 16; i++) {
__get_user(*get_freg(env, i), &sc->fpregs.fprs[i]); __get_user(*get_freg(env, i), &sc->fpregs.fprs[i]);
} }
return err;
} }
long do_sigreturn(CPUS390XState *env) long do_sigreturn(CPUS390XState *env)
@ -271,9 +267,7 @@ long do_sigreturn(CPUS390XState *env)
target_to_host_sigset_internal(&set, &target_set); target_to_host_sigset_internal(&set, &target_set);
set_sigmask(&set); /* ~_BLOCKABLE? */ set_sigmask(&set); /* ~_BLOCKABLE? */
if (restore_sigregs(env, &frame->sregs)) { restore_sigregs(env, &frame->sregs);
goto badframe;
}
unlock_user_struct(frame, frame_addr, 0); unlock_user_struct(frame, frame_addr, 0);
return -TARGET_QEMU_ESIGRETURN; return -TARGET_QEMU_ESIGRETURN;
@ -297,9 +291,7 @@ long do_rt_sigreturn(CPUS390XState *env)
set_sigmask(&set); /* ~_BLOCKABLE? */ set_sigmask(&set); /* ~_BLOCKABLE? */
if (restore_sigregs(env, &frame->uc.tuc_mcontext)) { restore_sigregs(env, &frame->uc.tuc_mcontext);
goto badframe;
}
target_restore_altstack(&frame->uc.tuc_stack, env); target_restore_altstack(&frame->uc.tuc_stack, env);