CC: [email protected] CC: [email protected] TO: Guo Ren <[email protected]> CC: Palmer Dabbelt <[email protected]> CC: "Patrick Stählin" <[email protected]> CC: Pekka Enberg <[email protected]>
From: kernel test robot <[email protected]> arch/riscv/kernel/probes/kprobes.c:260:3-6: WARNING: Use BUG_ON instead of if condition followed by BUG. Please make sure the condition has no side effects (see conditional BUG_ON definition in include/asm-generic/bug.h) Use BUG_ON instead of a if condition followed by BUG. Semantic patch information: This makes an effort to find cases where BUG() follows an if condition on an expression and replaces the if condition and BUG() with a BUG_ON having the conditional expression of the if statement as argument. Generated by: scripts/coccinelle/misc/bugon.cocci Fixes: c22b0bcb1dd0 ("riscv: Add kprobes supported") CC: Guo Ren <[email protected]> Reported-by: kernel test robot <[email protected]> Signed-off-by: kernel test robot <[email protected]> --- tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 5695e51619745d4fe3ec2506a2f0cd982c5e27a4 commit: c22b0bcb1dd024cb9caad9230e3a387d8b061df5 riscv: Add kprobes supported :::::: branch date: 3 hours ago :::::: commit date: 6 weeks ago Please take the patch only if it's a positive warning. Thanks! kprobes.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/arch/riscv/kernel/probes/kprobes.c +++ b/arch/riscv/kernel/probes/kprobes.c @@ -256,8 +256,7 @@ int __kprobes kprobe_fault_handler(struc * normal page fault. */ regs->epc = (unsigned long) cur->addr; - if (!instruction_pointer(regs)) - BUG(); + BUG_ON(!instruction_pointer(regs)); if (kcb->kprobe_status == KPROBE_REENTER) restore_previous_kprobe(kcb); _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
