summaryrefslogtreecommitdiffstats
path: root/arch/sh
diff options
context:
space:
mode:
authorEric W. Biederman <ebiederm@xmission.com>2018-05-29 09:40:11 -0500
committerEric W. Biederman <ebiederm@xmission.com>2018-05-29 09:59:21 -0500
commit26da35010c6d6ce317d511c8186585bdd8ab6629 (patch)
tree6c4eae125c289bae0873a4132355890c29b85aff /arch/sh
parent0bb0a1149e6fb6d8cfdcbcb2dc54d9a7b37db718 (diff)
downloadop-kernel-dev-26da35010c6d6ce317d511c8186585bdd8ab6629.zip
op-kernel-dev-26da35010c6d6ce317d511c8186585bdd8ab6629.tar.gz
signal/sh: Stop gcc warning about an impossible case in do_divide_error
Geert Uytterhoeven <geert@linux-m68k.org> reported: > HOSTLD scripts/mod/modpost > CC arch/sh/kernel/traps_32.o > arch/sh/kernel/traps_32.c: In function 'do_divide_error': > arch/sh/kernel/traps_32.c:606:17: error: 'code' may be used uninitialized in this function [-Werror=uninitialized] > cc1: all warnings being treated as errors It is clear from inspection that do_divide_error is only called with TRAP_DIVZERO_ERROR or TRAP_DIVOVF_ERROR, as that is the way set_exception_table_vec is called. So let gcc know the other cases should not be considered by returning in all other cases. This removes the warning and let's the code continue to build. Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Fixes: c65626c0cd4d ("signal/sh: Use force_sig_fault where appropriate") Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'arch/sh')
-rw-r--r--arch/sh/kernel/traps_32.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/arch/sh/kernel/traps_32.c b/arch/sh/kernel/traps_32.c
index 660a4bc..60709ad 100644
--- a/arch/sh/kernel/traps_32.c
+++ b/arch/sh/kernel/traps_32.c
@@ -602,6 +602,9 @@ asmlinkage void do_divide_error(unsigned long r4)
case TRAP_DIVOVF_ERROR:
code = FPE_INTOVF;
break;
+ default:
+ /* Let gcc know unhandled cases don't make it past here */
+ return;
}
force_sig_fault(SIGFPE, code, NULL, current);
}
OpenPOWER on IntegriCloud