aboutsummaryrefslogtreecommitdiff
path: root/linux-user
diff options
context:
space:
mode:
authorGiuseppe Musacchio <thatlemon@gmail.com>2019-10-01 18:39:52 +0200
committerPalmer Dabbelt <palmer@sifive.com>2019-10-28 07:47:27 -0700
commit2921343b3df93e4848034ec615c01ee221212c3a (patch)
tree10146679519bfe9eade69487326f6aa6fb0a2c27 /linux-user
parentc3a28b5d04f3b177befa1235e95e7bb09023f859 (diff)
downloadqemu-2921343b3df93e4848034ec615c01ee221212c3a.zip
qemu-2921343b3df93e4848034ec615c01ee221212c3a.tar.gz
qemu-2921343b3df93e4848034ec615c01ee221212c3a.tar.bz2
linux-user/riscv: Propagate fault address
The CPU loop tagged all the queued signals as QEMU_SI_KILL while it was filling the `_sigfault` part of `siginfo`: this caused QEMU to copy the wrong fields over to the userspace program. Make sure the fault address recorded by the MMU is is stored in the CPU environment structure. In case of memory faults store the exception address into `siginfo`. Signed-off-by: Giuseppe Musacchio <thatlemon@gmail.com> Reviewed-by: Palmer Dabbelt <palmer@sifive.com> Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
Diffstat (limited to 'linux-user')
-rw-r--r--linux-user/riscv/cpu_loop.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/linux-user/riscv/cpu_loop.c b/linux-user/riscv/cpu_loop.c
index 12aa3c0..aa9e437 100644
--- a/linux-user/riscv/cpu_loop.c
+++ b/linux-user/riscv/cpu_loop.c
@@ -89,6 +89,7 @@ void cpu_loop(CPURISCVState *env)
case RISCV_EXCP_STORE_PAGE_FAULT:
signum = TARGET_SIGSEGV;
sigcode = TARGET_SEGV_MAPERR;
+ sigaddr = env->badaddr;
break;
case EXCP_DEBUG:
gdbstep:
@@ -108,7 +109,7 @@ void cpu_loop(CPURISCVState *env)
.si_code = sigcode,
._sifields._sigfault._addr = sigaddr
};
- queue_signal(env, info.si_signo, QEMU_SI_KILL, &info);
+ queue_signal(env, info.si_signo, QEMU_SI_FAULT, &info);
}
process_pending_signals(env);