[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v15 5/5] linux-user: Handle EXCP_FPE properly fo
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v15 5/5] linux-user: Handle EXCP_FPE properly for MIPS |
Date: |
Fri, 28 Jun 2019 11:06:22 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
Le 28/06/2019 à 10:54, Aleksandar Markovic a écrit :
> From: Aleksandar Markovic <address@hidden>
>
> Handle EXCP_FPE properly for MIPS in cpu loop.
>
> Note that a vast majority of FP instructions are not affected by
> the absence of the code in this patch, as they use alternative code
> paths for handling floating point exceptions (see, for example,
> invocations of update_fcr31()) - they rely on softfloat library for
> keeping track on exceptions that needs to be raised. However, there
> are few MIPS FP instructions (an example is CTC1) that use function
> do_raise_exception() directly, and they need the case that is added
> in this patch to propagate the FPE exception as designed.
>
> The code is based on kernel's function force_fcr31_sig() in
> arch/mips/kernel.traps.c.
>
> Reported-by: Yunqiang Su <address@hidden>
> Signed-off-by: Aleksandar Markovic <address@hidden>
> ---
> linux-user/mips/cpu_loop.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/linux-user/mips/cpu_loop.c b/linux-user/mips/cpu_loop.c
> index 43ba267..0ba894f 100644
> --- a/linux-user/mips/cpu_loop.c
> +++ b/linux-user/mips/cpu_loop.c
> @@ -540,6 +540,23 @@ done_syscall:
> info.si_code = TARGET_ILL_ILLOPC;
> queue_signal(env, info.si_signo, QEMU_SI_FAULT, &info);
> break;
> + case EXCP_FPE:
> + info.si_signo = TARGET_SIGFPE;
> + info.si_errno = 0;
> + info.si_code = TARGET_FPE_FLTUNK;
> + if (GET_FP_CAUSE(env->active_fpu.fcr31) & FP_INVALID) {
> + info.si_code = TARGET_FPE_FLTINV;
> + } else if (GET_FP_CAUSE(env->active_fpu.fcr31) & FP_DIV0) {
> + info.si_code = TARGET_FPE_FLTDIV;
> + } else if (GET_FP_CAUSE(env->active_fpu.fcr31) & FP_OVERFLOW) {
> + info.si_code = TARGET_FPE_FLTOVF;
> + } else if (GET_FP_CAUSE(env->active_fpu.fcr31) & FP_UNDERFLOW) {
> + info.si_code = TARGET_FPE_FLTUND;
> + } else if (GET_FP_CAUSE(env->active_fpu.fcr31) & FP_INEXACT) {
> + info.si_code = TARGET_FPE_FLTRES;
> + }
> + queue_signal(env, info.si_signo, QEMU_SI_FAULT, &info);
> + break;
> /* The code below was inspired by the MIPS Linux kernel trap
> * handling code in arch/mips/kernel/traps.c.
> */
>
Reviewed-by: Laurent Vivier <address@hidden>
- [Qemu-devel] [PATCH v15 0/5] linux-user: A set of miscellaneous patches, Aleksandar Markovic, 2019/06/28
- [Qemu-devel] [PATCH v15 1/5] linux-user: Add support for translation of statx() syscall, Aleksandar Markovic, 2019/06/28
- [Qemu-devel] [PATCH v15 4/5] linux-user: Introduce TARGET_HAVE_ARCH_STRUCT_FLOCK, Aleksandar Markovic, 2019/06/28
- [Qemu-devel] [PATCH v15 5/5] linux-user: Handle EXCP_FPE properly for MIPS, Aleksandar Markovic, 2019/06/28
- Re: [Qemu-devel] [PATCH v15 5/5] linux-user: Handle EXCP_FPE properly for MIPS,
Laurent Vivier <=
- [Qemu-devel] [PATCH v15 3/5] linux-user: Fix target_flock structure for MIPS O64 ABI, Aleksandar Markovic, 2019/06/28
- [Qemu-devel] [PATCH v15 2/5] linux-user: Add support for strace for statx() syscall, Aleksandar Markovic, 2019/06/28
- Re: [Qemu-devel] [PATCH v15 0/5] linux-user: A set of miscellaneous patches, no-reply, 2019/06/28
- Re: [Qemu-devel] [PATCH v15 0/5] linux-user: A set of miscellaneous patches, no-reply, 2019/06/28
Re: [Qemu-devel] [PATCH v15 0/5] linux-user: A set of miscellaneous patches, no-reply, 2019/06/28