powerpc/8xx: Fix warning in hw_breakpoint_handler()
authorRussell Currey <ruscur@russell.cc>
Mon, 24 Oct 2022 04:13:46 +0000 (15:13 +1100)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 24 Nov 2022 12:31:49 +0000 (23:31 +1100)
In hw_breakpoint_handler(), ea is set by wp_get_instr_detail() except
for 8xx, leading the variable to be passed uninitialised to
wp_check_constraints().  This is safe as wp_check_constraints() returns
early without using ea, so just set it to make the compiler happy.

Signed-off-by: Russell Currey <ruscur@russell.cc>
Reviewed-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20221024041346.103608-1-ruscur@russell.cc
arch/powerpc/kernel/hw_breakpoint.c

index 8db1a15..e1b4e70 100644 (file)
@@ -646,7 +646,7 @@ int hw_breakpoint_handler(struct die_args *args)
        ppc_inst_t instr = ppc_inst(0);
        int type = 0;
        int size = 0;
-       unsigned long ea;
+       unsigned long ea = 0;
 
        /* Disable breakpoints during exception handling */
        hw_breakpoint_disable();