[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] alpha-linux-user: Implement signals.
From: |
Riku Voipio |
Subject: |
Re: [Qemu-devel] [PATCH] alpha-linux-user: Implement signals. |
Date: |
Wed, 10 Feb 2010 12:04:29 +0000 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Tue, Feb 09, 2010 at 10:46:32AM -0800, Richard Henderson wrote:
> Ping?
The linux-user side of the patch seems fine, but the target-alpha
code doesn't apply. Either something has changed upstream or this
depends on one of your other alpha patches. If the latter case, I
don't mind if this is applied together with the rest of alpha
patches.
> r~
>
> On 01/04/2010 03:17 PM, Richard Henderson wrote:
>> Move userland PALcode handling into linux-user main loop so that
>> we can send signals from there. This also makes alpha_palcode.c
>> system-level only, so don't build it for userland. Add defines
>> for GENTRAP PALcall mapping to signals.
>>
>> Signed-off-by: Richard Henderson<address@hidden>
>> ---
>> Makefile.target | 3 +-
>> hw/alpha_palcode.c | 81 +-----------
>> linux-user/alpha/target_signal.h | 27 ++++
>> linux-user/main.c | 137 ++++++++++++++++----
>> linux-user/signal.c | 267
>> ++++++++++++++++++++++++++++++++++++++
>> linux-user/syscall.c | 61 ++++++++-
>> linux-user/syscall_defs.h | 23 +++-
>> target-alpha/cpu.h | 4 +-
>> target-alpha/translate.c | 3 +-
>> 9 files changed, 489 insertions(+), 117 deletions(-)
>>
>> diff --git a/Makefile.target b/Makefile.target
>> index 7c1f30c..0ecfe76 100644
>> --- a/Makefile.target
>> +++ b/Makefile.target
>> @@ -47,7 +47,6 @@ libobj-$(CONFIG_NOSOFTFLOAT) += fpu/softfloat-native.o
>> libobj-y += op_helper.o helper.o
>> libobj-$(CONFIG_NEED_MMU) += mmu.o
>> libobj-$(TARGET_ARM) += neon_helper.o iwmmxt_helper.o
>> -libobj-$(TARGET_ALPHA) += alpha_palcode.o
>>
>> # NOTE: the disassembler code is only needed for debugging
>> libobj-y += disas.o
>> @@ -295,6 +294,8 @@ obj-m68k-y += m68k-semi.o dummy_m68k.o
>>
>> obj-s390x-y = s390-virtio-bus.o s390-virtio.o
>>
>> +obj-alpha-y = alpha_palcode.o
>> +
>> main.o vl.o: QEMU_CFLAGS+=$(GPROF_CFLAGS)
>>
>> vl.o: QEMU_CFLAGS+=$(SDL_CFLAGS)
>> diff --git a/hw/alpha_palcode.c b/hw/alpha_palcode.c
>> index 843bd14..c1220ad 100644
>> --- a/hw/alpha_palcode.c
>> +++ b/hw/alpha_palcode.c
>> @@ -21,11 +21,9 @@
>> #include<stdlib.h>
>> #include<stdio.h>
>>
>> -#include "qemu.h"
>> #include "cpu.h"
>> #include "exec-all.h"
>>
>> -#if !defined (CONFIG_USER_ONLY)
>> /* Shared handlers */
>> static void pal_reset (CPUState *env);
>> /* Console handlers */
>> @@ -997,12 +995,9 @@ int cpu_ppc_handle_mmu_fault (CPUState *env, uint32_t
>> address, int rw,
>> uint64_t physical, page_size, end;
>> int prot, zbits, ret;
>>
>> -#if defined(CONFIG_USER_ONLY)
>> - ret = 2;
>> -#else
>> - ret = virtual_to_physical(env,&physical,&zbits,&prot,
>> - address, mmu_idx, rw);
>> -#endif
>> + ret = virtual_to_physical(env,&physical,&zbits,&prot,
>> + address, mmu_idx, rw);
>> +
>> switch (ret) {
>> case 0:
>> /* No fault */
>> @@ -1050,73 +1045,3 @@ int cpu_ppc_handle_mmu_fault (CPUState *env, uint32_t
>> address, int rw,
>> return ret;
>> }
>> #endif
>> -
>> -#else /* !defined (CONFIG_USER_ONLY) */
>> -void pal_init (CPUState *env)
>> -{
>> -}
>> -
>> -void call_pal (CPUState *env, int palcode)
>> -{
>> - target_long ret;
>> -
>> - switch (palcode) {
>> - case 0x80:
>> - /* BPT */
>> - qemu_log("BPT\n");
>> - /* FIXME: Sends SIGTRAP, si_code=TRAP_BRKPT. */
>> - exit(1);
>> - case 0x81:
>> - /* BUGCHK */
>> - qemu_log("BUGCHK\n");
>> - /* FIXME: Sends SIGTRAP, si_code=SI_FAULT. */
>> - exit(1);
>> - case 0x83:
>> - /* CALLSYS */
>> - qemu_log("CALLSYS n " TARGET_FMT_ld "\n", env->ir[0]);
>> - ret = do_syscall(env, env->ir[IR_V0], env->ir[IR_A0],
>> env->ir[IR_A1],
>> - env->ir[IR_A2], env->ir[IR_A3], env->ir[IR_A4],
>> - env->ir[IR_A5]);
>> - if (ret>= 0) {
>> - env->ir[IR_A3] = 0;
>> - env->ir[IR_V0] = ret;
>> - } else {
>> - env->ir[IR_A3] = 1;
>> - env->ir[IR_V0] = -ret;
>> - }
>> - break;
>> - case 0x86:
>> - /* IMB */
>> - qemu_log("IMB\n");
>> - /* ??? We can probably elide the code using page_unprotect that is
>> - checking for self-modifying code. Instead we could simply call
>> - tb_flush here. Until we work out the changes required to turn
>> - off the extra write protection, this can be a no-op. */
>> - break;
>> - case 0x9E:
>> - /* RDUNIQUE */
>> - qemu_log("RDUNIQUE: " TARGET_FMT_lx "\n", env->unique);
>> - /* Handled in the translator for usermode. */
>> - abort();
>> - case 0x9F:
>> - /* WRUNIQUE */
>> - qemu_log("WRUNIQUE: " TARGET_FMT_lx "\n", env->ir[IR_A0]);
>> - /* Handled in the translator for usermode. */
>> - abort();
>> - case 0xAA:
>> - /* GENTRAP */
>> - qemu_log("GENTRAP: " TARGET_FMT_lx "\n", env->ir[IR_A0]);
>> - /* FIXME: This is supposed to send a signal:
>> - SIGFPE:
>> - GEN_INTOVF, GEN_INTDIV, GEN_FLTOVF, GEN_FLTDIV,
>> - GEN_FLTUND, GEN_FLTINV, GEN_FLTINE, GEN_ROPRAND
>> - SIGTRAP:
>> - others
>> - with various settings of si_code. */
>> - exit(1);
>> - default:
>> - qemu_log("%s: unhandled palcode %02x\n", __func__, palcode);
>> - exit(1);
>> - }
>> -}
>> -#endif
>> diff --git a/linux-user/alpha/target_signal.h
>> b/linux-user/alpha/target_signal.h
>> index 2382ffd..cb86402 100644
>> --- a/linux-user/alpha/target_signal.h
>> +++ b/linux-user/alpha/target_signal.h
>> @@ -26,4 +26,31 @@ static inline abi_ulong
>> get_sp_from_cpustate(CPUAlphaState *state)
>> return state->ir[IR_SP];
>> }
>>
>> +/* From<asm/gentrap.h>. */
>> +#define TARGET_GEN_INTOVF -1 /* integer overflow */
>> +#define TARGET_GEN_INTDIV -2 /* integer division by zero */
>> +#define TARGET_GEN_FLTOVF -3 /* fp overflow */
>> +#define TARGET_GEN_FLTDIV -4 /* fp division by zero */
>> +#define TARGET_GEN_FLTUND -5 /* fp underflow */
>> +#define TARGET_GEN_FLTINV -6 /* invalid fp operand */
>> +#define TARGET_GEN_FLTINE -7 /* inexact fp operand */
>> +#define TARGET_GEN_DECOVF -8 /* decimal overflow (for COBOL??) */
>> +#define TARGET_GEN_DECDIV -9 /* decimal division by zero */
>> +#define TARGET_GEN_DECINV -10 /* invalid decimal operand */
>> +#define TARGET_GEN_ROPRAND -11 /* reserved operand */
>> +#define TARGET_GEN_ASSERTERR -12 /* assertion error */
>> +#define TARGET_GEN_NULPTRERR -13 /* null pointer error */
>> +#define TARGET_GEN_STKOVF -14 /* stack overflow */
>> +#define TARGET_GEN_STRLENERR -15 /* string length error */
>> +#define TARGET_GEN_SUBSTRERR -16 /* substring error */
>> +#define TARGET_GEN_RANGERR -17 /* range error */
>> +#define TARGET_GEN_SUBRNG -18
>> +#define TARGET_GEN_SUBRNG1 -19
>> +#define TARGET_GEN_SUBRNG2 -20
>> +#define TARGET_GEN_SUBRNG3 -21
>> +#define TARGET_GEN_SUBRNG4 -22
>> +#define TARGET_GEN_SUBRNG5 -23
>> +#define TARGET_GEN_SUBRNG6 -24
>> +#define TARGET_GEN_SUBRNG7 -25
>> +
>> #endif /* TARGET_SIGNAL_H */
>> diff --git a/linux-user/main.c b/linux-user/main.c
>> index 91e5009..bddfb45 100644
>> --- a/linux-user/main.c
>> +++ b/linux-user/main.c
>> @@ -2351,6 +2351,7 @@ void cpu_loop (CPUState *env)
>> {
>> int trapnr;
>> target_siginfo_t info;
>> + abi_long sysret;
>>
>> while (1) {
>> trapnr = cpu_alpha_exec (env);
>> @@ -2365,16 +2366,22 @@ void cpu_loop (CPUState *env)
>> exit(1);
>> break;
>> case EXCP_ARITH:
>> - fprintf(stderr, "Arithmetic trap.\n");
>> - exit(1);
>> + info.si_signo = TARGET_SIGFPE;
>> + info.si_errno = 0;
>> + info.si_code = TARGET_FPE_FLTINV;
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> break;
>> case EXCP_HW_INTERRUPT:
>> fprintf(stderr, "External interrupt. Exit\n");
>> exit(1);
>> break;
>> case EXCP_DFAULT:
>> - fprintf(stderr, "MMU data fault\n");
>> - exit(1);
>> + info.si_signo = TARGET_SIGSEGV;
>> + info.si_errno = 0;
>> + info.si_code = 0; /* ??? SEGV_MAPERR vs SEGV_ACCERR. */
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> break;
>> case EXCP_DTB_MISS_PAL:
>> fprintf(stderr, "MMU data TLB miss in PALcode\n");
>> @@ -2393,36 +2400,116 @@ void cpu_loop (CPUState *env)
>> exit(1);
>> break;
>> case EXCP_UNALIGN:
>> - fprintf(stderr, "Unaligned access\n");
>> - exit(1);
>> + info.si_signo = TARGET_SIGBUS;
>> + info.si_errno = 0;
>> + info.si_code = TARGET_BUS_ADRALN;
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> break;
>> case EXCP_OPCDEC:
>> - fprintf(stderr, "Invalid instruction\n");
>> - exit(1);
>> + do_sigill:
>> + info.si_signo = TARGET_SIGILL;
>> + info.si_errno = 0;
>> + info.si_code = TARGET_ILL_ILLOPC;
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> break;
>> case EXCP_FEN:
>> - fprintf(stderr, "Floating-point not allowed\n");
>> - exit(1);
>> + /* No-op. Linux simply re-enables the FPU. */
>> break;
>> case EXCP_CALL_PAL ... (EXCP_CALL_PALP - 1):
>> - call_pal(env, (trapnr>> 6) | 0x80);
>> + switch ((trapnr>> 6) | 0x80) {
>> + case 0x80:
>> + /* BPT */
>> + info.si_signo = TARGET_SIGTRAP;
>> + info.si_errno = 0;
>> + info.si_code = TARGET_TRAP_BRKPT;
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> + break;
>> + case 0x81:
>> + /* BUGCHK */
>> + info.si_signo = TARGET_SIGTRAP;
>> + info.si_errno = 0;
>> + info.si_code = 0;
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> + break;
>> + case 0x83:
>> + /* CALLSYS */
>> + trapnr = env->ir[IR_V0];
>> + sysret = do_syscall(env, trapnr,
>> + env->ir[IR_A0], env->ir[IR_A1],
>> + env->ir[IR_A2], env->ir[IR_A3],
>> + env->ir[IR_A4], env->ir[IR_A5]);
>> + if (trapnr != TARGET_NR_sigreturn
>> +&& trapnr != TARGET_NR_rt_sigreturn) {
>> + env->ir[IR_V0] = (sysret< 0 ? -sysret : sysret);
>> + env->ir[IR_A3] = (sysret< 0);
>> + }
>> + break;
>> + case 0x86:
>> + /* IMB */
>> + /* ??? We can probably elide the code using page_unprotect
>> + that is checking for self-modifying code. Instead we
>> + could simply call tb_flush here. Until we work out the
>> + changes required to turn off the extra write protection,
>> + this can be a no-op. */
>> + break;
>> + case 0x9E:
>> + /* RDUNIQUE */
>> + /* Handled in the translator for usermode. */
>> + abort();
>> + case 0x9F:
>> + /* WRUNIQUE */
>> + /* Handled in the translator for usermode. */
>> + abort();
>> + case 0xAA:
>> + /* GENTRAP */
>> + info.si_signo = TARGET_SIGFPE;
>> + switch (env->ir[IR_A0]) {
>> + case TARGET_GEN_INTOVF:
>> + info.si_code = TARGET_FPE_INTOVF;
>> + break;
>> + case TARGET_GEN_INTDIV:
>> + info.si_code = TARGET_FPE_INTDIV;
>> + break;
>> + case TARGET_GEN_FLTOVF:
>> + info.si_code = TARGET_FPE_FLTOVF;
>> + break;
>> + case TARGET_GEN_FLTUND:
>> + info.si_code = TARGET_FPE_FLTUND;
>> + break;
>> + case TARGET_GEN_FLTINV:
>> + info.si_code = TARGET_FPE_FLTINV;
>> + break;
>> + case TARGET_GEN_FLTINE:
>> + info.si_code = TARGET_FPE_FLTRES;
>> + break;
>> + case TARGET_GEN_ROPRAND:
>> + info.si_code = 0;
>> + break;
>> + default:
>> + info.si_signo = TARGET_SIGTRAP;
>> + info.si_code = 0;
>> + break;
>> + }
>> + info.si_errno = 0;
>> + info._sifields._sigfault._addr = env->pc;
>> + queue_signal(env, info.si_signo,&info);
>> + break;
>> + default:
>> + goto do_sigill;
>> + }
>> break;
>> case EXCP_CALL_PALP ... (EXCP_CALL_PALE - 1):
>> - fprintf(stderr, "Privileged call to PALcode\n");
>> - exit(1);
>> - break;
>> + goto do_sigill;
>> case EXCP_DEBUG:
>> - {
>> - int sig;
>> -
>> - sig = gdb_handlesig (env, TARGET_SIGTRAP);
>> - if (sig)
>> - {
>> - info.si_signo = sig;
>> - info.si_errno = 0;
>> - info.si_code = TARGET_TRAP_BRKPT;
>> - queue_signal(env, info.si_signo,&info);
>> - }
>> + info.si_signo = gdb_handlesig (env, TARGET_SIGTRAP);
>> + if (info.si_signo) {
>> + info.si_errno = 0;
>> + info.si_code = TARGET_TRAP_BRKPT;
>> + queue_signal(env, info.si_signo,&info);
>> }
>> break;
>> default:
>> diff --git a/linux-user/signal.c b/linux-user/signal.c
>> index b0faf2e..e9abb1a 100644
>> --- a/linux-user/signal.c
>> +++ b/linux-user/signal.c
>> @@ -4410,6 +4410,273 @@ badframe:
>> return 0;
>> }
>>
>> +#elif defined(TARGET_ALPHA)
>> +
>> +struct target_sigcontext {
>> + abi_long sc_onstack;
>> + abi_long sc_mask;
>> + abi_long sc_pc;
>> + abi_long sc_ps;
>> + abi_long sc_regs[32];
>> + abi_long sc_ownedfp;
>> + abi_long sc_fpregs[32];
>> + abi_ulong sc_fpcr;
>> + abi_ulong sc_fp_control;
>> + abi_ulong sc_reserved1;
>> + abi_ulong sc_reserved2;
>> + abi_ulong sc_ssize;
>> + abi_ulong sc_sbase;
>> + abi_ulong sc_traparg_a0;
>> + abi_ulong sc_traparg_a1;
>> + abi_ulong sc_traparg_a2;
>> + abi_ulong sc_fp_trap_pc;
>> + abi_ulong sc_fp_trigger_sum;
>> + abi_ulong sc_fp_trigger_inst;
>> +};
>> +
>> +struct target_ucontext {
>> + abi_ulong uc_flags;
>> + abi_ulong uc_link;
>> + abi_ulong uc_osf_sigmask;
>> + target_stack_t uc_stack;
>> + struct target_sigcontext uc_mcontext;
>> + target_sigset_t uc_sigmask;
>> +};
>> +
>> +struct target_sigframe {
>> + struct target_sigcontext sc;
>> + unsigned int retcode[3];
>> +};
>> +
>> +struct target_rt_sigframe {
>> + target_siginfo_t info;
>> + struct target_ucontext uc;
>> + unsigned int retcode[3];
>> +};
>> +
>> +#define INSN_MOV_R30_R16 0x47fe0410
>> +#define INSN_LDI_R0 0x201f0000
>> +#define INSN_CALLSYS 0x00000083
>> +
>> +static int setup_sigcontext(struct target_sigcontext *sc, CPUState *env,
>> + abi_ulong frame_addr, target_sigset_t *set)
>> +{
>> + int i, err = 0;
>> +
>> + err |= __put_user(on_sig_stack(frame_addr),&sc->sc_onstack);
>> + err |= __put_user(set->sig[0],&sc->sc_mask);
>> + err |= __put_user(env->pc,&sc->sc_pc);
>> + err |= __put_user(8,&sc->sc_ps);
>> +
>> + for (i = 0; i< 31; ++i) {
>> + err |= __put_user(env->ir[i],&sc->sc_regs[i]);
>> + }
>> + err |= __put_user(0,&sc->sc_regs[31]);
>> +
>> + for (i = 0; i< 31; ++i) {
>> + err |= __put_user(env->fir[i],&sc->sc_fpregs[i]);
>> + }
>> + err |= __put_user(0,&sc->sc_fpregs[31]);
>> + err |= __put_user(cpu_alpha_load_fpcr(env),&sc->sc_fpcr);
>> +
>> + err |= __put_user(0,&sc->sc_traparg_a0); /* FIXME */
>> + err |= __put_user(0,&sc->sc_traparg_a1); /* FIXME */
>> + err |= __put_user(0,&sc->sc_traparg_a2); /* FIXME */
>> +
>> + return err;
>> +}
>> +
>> +static int restore_sigcontext(CPUState *env, struct target_sigcontext *sc)
>> +{
>> + uint64_t fpcr;
>> + int i, err = 0;
>> +
>> + err |= __get_user(env->pc,&sc->sc_pc);
>> +
>> + for (i = 0; i< 31; ++i) {
>> + err |= __get_user(env->ir[i],&sc->sc_regs[i]);
>> + }
>> + for (i = 0; i< 31; ++i) {
>> + err |= __get_user(env->fir[i],&sc->sc_fpregs[i]);
>> + }
>> +
>> + err |= __get_user(fpcr,&sc->sc_fpcr);
>> + cpu_alpha_store_fpcr(env, fpcr);
>> +
>> + return err;
>> +}
>> +
>> +static inline abi_ulong get_sigframe(struct target_sigaction *sa,
>> + CPUState *env, unsigned long framesize)
>> +{
>> + abi_ulong sp = env->ir[IR_SP];
>> +
>> + /* This is the X/Open sanctioned signal stack switching. */
>> + if ((sa->sa_flags& TARGET_SA_ONSTACK) != 0&& !sas_ss_flags(sp)) {
>> + sp = target_sigaltstack_used.ss_sp +
>> target_sigaltstack_used.ss_size;
>> + }
>> + return (sp - framesize)& -32;
>> +}
>> +
>> +static void setup_frame(int sig, struct target_sigaction *ka,
>> + target_sigset_t *set, CPUState *env)
>> +{
>> + abi_ulong frame_addr, r26;
>> + struct target_sigframe *frame;
>> + int err = 0;
>> +
>> + frame_addr = get_sigframe(ka, env, sizeof(*frame));
>> + if (!lock_user_struct(VERIFY_WRITE, frame, frame_addr, 0)) {
>> + goto give_sigsegv;
>> + }
>> +
>> + err |= setup_sigcontext(&frame->sc, env, frame_addr, set);
>> +
>> + if (ka->sa_restorer) {
>> + r26 = ka->sa_restorer;
>> + } else {
>> + err |= __put_user(INSN_MOV_R30_R16,&frame->retcode[0]);
>> + err |= __put_user(INSN_LDI_R0 + TARGET_NR_sigreturn,
>> +&frame->retcode[1]);
>> + err |= __put_user(INSN_CALLSYS,&frame->retcode[2]);
>> + /* imb() */
>> + r26 = frame_addr;
>> + }
>> +
>> + unlock_user_struct(frame, frame_addr, 1);
>> +
>> + if (err) {
>> + give_sigsegv:
>> + if (sig == TARGET_SIGSEGV) {
>> + ka->_sa_handler = TARGET_SIG_DFL;
>> + }
>> + force_sig(TARGET_SIGSEGV);
>> + }
>> +
>> + env->ir[IR_RA] = r26;
>> + env->ir[IR_PV] = env->pc = ka->_sa_handler;
>> + env->ir[IR_A0] = sig;
>> + env->ir[IR_A1] = 0;
>> + env->ir[IR_A2] = frame_addr + offsetof(struct target_sigframe, sc);
>> + env->ir[IR_SP] = frame_addr;
>> +}
>> +
>> +static void setup_rt_frame(int sig, struct target_sigaction *ka,
>> + target_siginfo_t *info,
>> + target_sigset_t *set, CPUState *env)
>> +{
>> + abi_ulong frame_addr, r26;
>> + struct target_rt_sigframe *frame;
>> + int i, err = 0;
>> +
>> + frame_addr = get_sigframe(ka, env, sizeof(*frame));
>> + if (!lock_user_struct(VERIFY_WRITE, frame, frame_addr, 0)) {
>> + goto give_sigsegv;
>> + }
>> +
>> + err |= copy_siginfo_to_user(&frame->info, info);
>> +
>> + err |= __put_user(0,&frame->uc.uc_flags);
>> + err |= __put_user(0,&frame->uc.uc_link);
>> + err |= __put_user(set->sig[0],&frame->uc.uc_osf_sigmask);
>> + err |= __put_user(target_sigaltstack_used.ss_sp,
>> +&frame->uc.uc_stack.ss_sp);
>> + err |= __put_user(sas_ss_flags(env->ir[IR_SP]),
>> +&frame->uc.uc_stack.ss_flags);
>> + err |= __put_user(target_sigaltstack_used.ss_size,
>> +&frame->uc.uc_stack.ss_size);
>> + err |= setup_sigcontext(&frame->uc.uc_mcontext, env, frame_addr, set);
>> + for (i = 0; i< TARGET_NSIG_WORDS; ++i) {
>> + err |= __put_user(set->sig[i],&frame->uc.uc_sigmask.sig[i]);
>> + }
>> +
>> + if (ka->sa_restorer) {
>> + r26 = ka->sa_restorer;
>> + } else {
>> + err |= __put_user(INSN_MOV_R30_R16,&frame->retcode[0]);
>> + err |= __put_user(INSN_LDI_R0 + TARGET_NR_rt_sigreturn,
>> +&frame->retcode[1]);
>> + err |= __put_user(INSN_CALLSYS,&frame->retcode[2]);
>> + /* imb(); */
>> + r26 = frame_addr;
>> + }
>> +
>> + if (err) {
>> + give_sigsegv:
>> + if (sig == TARGET_SIGSEGV) {
>> + ka->_sa_handler = TARGET_SIG_DFL;
>> + }
>> + force_sig(TARGET_SIGSEGV);
>> + }
>> +
>> + env->ir[IR_RA] = r26;
>> + env->ir[IR_PV] = env->pc = ka->_sa_handler;
>> + env->ir[IR_A0] = sig;
>> + env->ir[IR_A1] = frame_addr + offsetof(struct target_rt_sigframe, info);
>> + env->ir[IR_A2] = frame_addr + offsetof(struct target_rt_sigframe, uc);
>> + env->ir[IR_SP] = frame_addr;
>> +}
>> +
>> +long do_sigreturn(CPUState *env)
>> +{
>> + struct target_sigcontext *sc;
>> + abi_ulong sc_addr = env->ir[IR_A0];
>> + target_sigset_t target_set;
>> + sigset_t set;
>> +
>> + if (!lock_user_struct(VERIFY_READ, sc, sc_addr, 1)) {
>> + goto badframe;
>> + }
>> +
>> + target_sigemptyset(&target_set);
>> + if (__get_user(target_set.sig[0],&sc->sc_mask)) {
>> + goto badframe;
>> + }
>> +
>> + target_to_host_sigset_internal(&set,&target_set);
>> + sigprocmask(SIG_SETMASK,&set, NULL);
>> +
>> + if (restore_sigcontext(env, sc)) {
>> + goto badframe;
>> + }
>> + unlock_user_struct(sc, sc_addr, 0);
>> + return env->ir[IR_V0];
>> +
>> + badframe:
>> + unlock_user_struct(sc, sc_addr, 0);
>> + force_sig(TARGET_SIGSEGV);
>> +}
>> +
>> +long do_rt_sigreturn(CPUState *env)
>> +{
>> + abi_ulong frame_addr = env->ir[IR_A0];
>> + struct target_rt_sigframe *frame;
>> + sigset_t set;
>> +
>> + if (!lock_user_struct(VERIFY_READ, frame, frame_addr, 1)) {
>> + goto badframe;
>> + }
>> + target_to_host_sigset(&set,&frame->uc.uc_sigmask);
>> + sigprocmask(SIG_SETMASK,&set, NULL);
>> +
>> + if (restore_sigcontext(env,&frame->uc.uc_mcontext)) {
>> + goto badframe;
>> + }
>> + if (do_sigaltstack(frame_addr + offsetof(struct target_rt_sigframe,
>> + uc.uc_stack),
>> + 0, env->ir[IR_SP]) == -EFAULT) {
>> + goto badframe;
>> + }
>> +
>> + unlock_user_struct(frame, frame_addr, 0);
>> + return env->ir[IR_V0];
>> +
>> +
>> + badframe:
>> + unlock_user_struct(frame, frame_addr, 0);
>> + force_sig(TARGET_SIGSEGV);
>> +}
>> +
>> #else
>>
>> static void setup_frame(int sig, struct target_sigaction *ka,
>> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
>> index 1acf1f5..65c15be 100644
>> --- a/linux-user/syscall.c
>> +++ b/linux-user/syscall.c
>> @@ -4774,20 +4774,18 @@ abi_long do_syscall(void *cpu_env, int num, abi_long
>> arg1,
>> #ifdef TARGET_NR_sigaction
>> case TARGET_NR_sigaction:
>> {
>> -#if !defined(TARGET_MIPS)
>> +#if defined(TARGET_ALPHA)
>> + struct target_sigaction act, oact, *pact = 0;
>> struct target_old_sigaction *old_act;
>> - struct target_sigaction act, oact, *pact;
>> if (arg2) {
>> if (!lock_user_struct(VERIFY_READ, old_act, arg2, 1))
>> goto efault;
>> act._sa_handler = old_act->_sa_handler;
>> target_siginitset(&act.sa_mask, old_act->sa_mask);
>> act.sa_flags = old_act->sa_flags;
>> - act.sa_restorer = old_act->sa_restorer;
>> + act.sa_restorer = 0;
>> unlock_user_struct(old_act, arg2, 0);
>> pact =&act;
>> - } else {
>> - pact = NULL;
>> }
>> ret = get_errno(do_sigaction(arg1, pact,&oact));
>> if (!is_error(ret)&& arg3) {
>> @@ -4796,10 +4794,9 @@ abi_long do_syscall(void *cpu_env, int num, abi_long
>> arg1,
>> old_act->_sa_handler = oact._sa_handler;
>> old_act->sa_mask = oact.sa_mask.sig[0];
>> old_act->sa_flags = oact.sa_flags;
>> - old_act->sa_restorer = oact.sa_restorer;
>> unlock_user_struct(old_act, arg3, 1);
>> }
>> -#else
>> +#elif defined(TARGET_MIPS)
>> struct target_sigaction act, oact, *pact, *old_act;
>>
>> if (arg2) {
>> @@ -4827,12 +4824,61 @@ abi_long do_syscall(void *cpu_env, int num, abi_long
>> arg1,
>> old_act->sa_mask.sig[3] = 0;
>> unlock_user_struct(old_act, arg3, 1);
>> }
>> +#else
>> + struct target_old_sigaction *old_act;
>> + struct target_sigaction act, oact, *pact;
>> + if (arg2) {
>> + if (!lock_user_struct(VERIFY_READ, old_act, arg2, 1))
>> + goto efault;
>> + act._sa_handler = old_act->_sa_handler;
>> + target_siginitset(&act.sa_mask, old_act->sa_mask);
>> + act.sa_flags = old_act->sa_flags;
>> + act.sa_restorer = old_act->sa_restorer;
>> + unlock_user_struct(old_act, arg2, 0);
>> + pact =&act;
>> + } else {
>> + pact = NULL;
>> + }
>> + ret = get_errno(do_sigaction(arg1, pact,&oact));
>> + if (!is_error(ret)&& arg3) {
>> + if (!lock_user_struct(VERIFY_WRITE, old_act, arg3, 0))
>> + goto efault;
>> + old_act->_sa_handler = oact._sa_handler;
>> + old_act->sa_mask = oact.sa_mask.sig[0];
>> + old_act->sa_flags = oact.sa_flags;
>> + old_act->sa_restorer = oact.sa_restorer;
>> + unlock_user_struct(old_act, arg3, 1);
>> + }
>> #endif
>> }
>> break;
>> #endif
>> case TARGET_NR_rt_sigaction:
>> {
>> +#if defined(TARGET_ALPHA)
>> + struct target_sigaction act, oact, *pact = 0;
>> + struct target_rt_sigaction *rt_act;
>> + /* ??? arg4 == sizeof(sigset_t). */
>> + if (arg2) {
>> + if (!lock_user_struct(VERIFY_READ, rt_act, arg2, 1))
>> + goto efault;
>> + act._sa_handler = rt_act->_sa_handler;
>> + act.sa_mask = rt_act->sa_mask;
>> + act.sa_flags = rt_act->sa_flags;
>> + act.sa_restorer = arg5;
>> + unlock_user_struct(rt_act, arg2, 0);
>> + pact =&act;
>> + }
>> + ret = get_errno(do_sigaction(arg1, pact,&oact));
>> + if (!is_error(ret)&& arg3) {
>> + if (!lock_user_struct(VERIFY_WRITE, rt_act, arg3, 0))
>> + goto efault;
>> + rt_act->_sa_handler = oact._sa_handler;
>> + rt_act->sa_mask = oact.sa_mask;
>> + rt_act->sa_flags = oact.sa_flags;
>> + unlock_user_struct(rt_act, arg3, 1);
>> + }
>> +#else
>> struct target_sigaction *act;
>> struct target_sigaction *oact;
>>
>> @@ -4854,6 +4900,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long
>> arg1,
>> unlock_user_struct(act, arg2, 0);
>> if (oact)
>> unlock_user_struct(oact, arg3, 1);
>> +#endif
>> }
>> break;
>> #ifdef TARGET_NR_sgetmask /* not on alpha */
>> diff --git a/linux-user/syscall_defs.h b/linux-user/syscall_defs.h
>> index 2d45753..63c2bc3 100644
>> --- a/linux-user/syscall_defs.h
>> +++ b/linux-user/syscall_defs.h
>> @@ -472,8 +472,28 @@ int do_sigaction(int sig, const struct target_sigaction
>> *act,
>>
>> #endif
>>
>> -#if defined(TARGET_MIPS)
>> +#if defined(TARGET_ALPHA)
>> +struct target_old_sigaction {
>> + abi_ulong _sa_handler;
>> + abi_ulong sa_mask;
>> + abi_ulong sa_flags;
>> +};
>> +
>> +struct target_rt_sigaction {
>> + abi_ulong _sa_handler;
>> + abi_ulong sa_flags;
>> + target_sigset_t sa_mask;
>> +};
>>
>> +/* This is the struct used inside the kernel. The ka_restorer
>> + field comes from the 5th argument to sys_rt_sigaction. */
>> +struct target_sigaction {
>> + abi_ulong _sa_handler;
>> + abi_ulong sa_flags;
>> + target_sigset_t sa_mask;
>> + abi_ulong sa_restorer;
>> +};
>> +#elif defined(TARGET_MIPS)
>> struct target_sigaction {
>> uint32_t sa_flags;
>> #if defined(TARGET_ABI_MIPSN32)
>> @@ -483,7 +503,6 @@ struct target_sigaction {
>> #endif
>> target_sigset_t sa_mask;
>> };
>> -
>> #else
>> struct target_old_sigaction {
>> abi_ulong _sa_handler;
>> diff --git a/target-alpha/cpu.h b/target-alpha/cpu.h
>> index eda1b4a..617f55c 100644
>> --- a/target-alpha/cpu.h
>> +++ b/target-alpha/cpu.h
>> @@ -511,11 +511,9 @@ uint64_t cpu_alpha_load_fpcr (CPUState *env);
>> void cpu_alpha_store_fpcr (CPUState *env, uint64_t val);
>> int cpu_alpha_mfpr (CPUState *env, int iprn, uint64_t *valp);
>> int cpu_alpha_mtpr (CPUState *env, int iprn, uint64_t val, uint64_t
>> *oldvalp);
>> -void pal_init (CPUState *env);
>> #if !defined (CONFIG_USER_ONLY)
>> +void pal_init (CPUState *env);
>> void call_pal (CPUState *env);
>> -#else
>> -void call_pal (CPUState *env, int palcode);
>> #endif
>>
>> static inline void cpu_pc_from_tb(CPUState *env, TranslationBlock *tb)
>> diff --git a/target-alpha/translate.c b/target-alpha/translate.c
>> index a11e5ed..719b423 100644
>> --- a/target-alpha/translate.c
>> +++ b/target-alpha/translate.c
>> @@ -3094,8 +3094,9 @@ CPUAlphaState * cpu_alpha_init (const char *cpu_model)
>> env->ps |= 1<< 3;
>> cpu_alpha_store_fpcr(env, (FPCR_INVD | FPCR_DZED | FPCR_OVFD
>> | FPCR_UNFD | FPCR_INED | FPCR_DNOD));
>> -#endif
>> +#else
>> pal_init(env);
>> +#endif
>>
>> /* Initialize IPR */
>> #if defined (CONFIG_USER_ONLY)
>
>