Thread (177 messages) 177 messages, 28 authors, 2022-05-24

Re: [PATCH 17/30] tracing: Improve panic/die notifiers

From: "Guilherme G. Piccoli" <gpiccoli@igalia.com>
Date: 2022-04-29 14:46:17
Also in: kexec, linux-alpha, linux-edac, linux-hyperv, linux-leds, linux-mips, linux-pm, linux-remoteproc, linux-s390, linux-tegra, linuxppc-dev, lkml, netdev, rcu, sparclinux, xen-devel

On 29/04/2022 10:56, Steven Rostedt wrote:
[...]
No. The fallthrough keyword is only needed when there's code between case
labels. As it is very common to list multiple cases for the same code path.
That is:

	case DIE_OOPS:
 	case PANIC_NOTIFIER:
 		do_dump = 1;
 		break;

Does not need a fall through label, as there's no code between the DIE_OOPS
and the PANIC_NOTIFIER. But if you had:

	case DIE_OOPS:
		x = true;
 	case PANIC_NOTIFIER:
 		do_dump = 1;
 		break;

Then you do.

-- Steve
Thanks a bunch for the clarification, changed that for V2 =)

_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help