diff options
| author | Eric W. Biederman <ebiederm@xmission.com> | 2017-08-08 21:52:28 +0300 |
|---|---|---|
| committer | Eric W. Biederman <ebiederm@xmission.com> | 2017-08-18 01:31:27 +0300 |
| commit | 20229305afdc0e49b3af2204ab848d6bddcb9fc3 (patch) | |
| tree | 23414a4c94b65dd04740e4bf39dd16b593223162 /arch/mips/kernel | |
| parent | cc731525f26af85a1c3537da41e0abd1d35e0bdb (diff) | |
| download | linux-20229305afdc0e49b3af2204ab848d6bddcb9fc3.tar.xz | |
mips/signal: In force_fcr31_sig return in the impossible case
In a recent discussion Maciej Rozycki reported that this case is
impossible.
Handle the impossible case by just returning instead of trying to
handle it. This makes static analysis simpler as it means nothing
needs to consider the impossible case after the return statement.
As the code no longer has to deal with this case remove FPE_FIXME from
the mips siginfo.h
Cc: "Maciej W. Rozycki" <macro@imgtec.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Link: http://lkml.kernel.org/r/20170718140651.15973-4-ebiederm@xmission.com
Ref: ea1b75cf9138 ("signal/mips: Document a conflict with SI_USER with SIGFPE")
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'arch/mips/kernel')
| -rw-r--r-- | arch/mips/kernel/traps.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/mips/kernel/traps.c b/arch/mips/kernel/traps.c index 6c9cca9c5341..2bf414993347 100644 --- a/arch/mips/kernel/traps.c +++ b/arch/mips/kernel/traps.c @@ -735,7 +735,7 @@ void force_fcr31_sig(unsigned long fcr31, void __user *fault_addr, else if (fcr31 & FPU_CSR_INE_X) si.si_code = FPE_FLTRES; else - si.si_code = FPE_FIXME; + return; /* Broken hardware? */ force_sig_info(SIGFPE, &si, tsk); } |