Petr Mladek 42a0bb3f71 printk/nmi: generic solution for safe printk in NMI ... printk() takes some locks and could not be used a safe way in NMI context. The chance of a deadlock is real especially when printing stacks from all CPUs. This particular problem has been addressed on x86 by the commit a9edc88093 ("x86/nmi: Perform a safe NMI stack trace on all CPUs"). The patchset brings two big advantages. First, it makes the NMI backtraces safe on all architectures for free. Second, it makes all NMI messages almost safe on all architectures (the temporary buffer is limited. We still should keep the number of messages in NMI context at minimum). Note that there already are several messages printed in NMI context: WARN_ON(in_nmi()), BUG_ON(in_nmi()), anything being printed out from MCE handlers. These are not easy to avoid. This patch reuses most of the code and makes it generic. It is useful for all messages and architectures that support NMI. The alternative printk_func is set when entering and is reseted when leaving NMI context. It queues IRQ work to copy the messages into the main ring buffer in a safe context. __printk_nmi_flush() copies all available messages and reset the buffer. Then we could use a simple cmpxchg operations to get synchronized with writers. There is also used a spinlock to get synchronized with other flushers. We do not longer use seq_buf because it depends on external lock. It would be hard to make all supported operations safe for a lockless use. It would be confusing and error prone to make only some operations safe. The code is put into separate printk/nmi.c as suggested by Steven Rostedt. It needs a per-CPU buffer and is compiled only on architectures that call nmi_enter(). This is achieved by the new HAVE_NMI Kconfig flag. The are MN10300 and Xtensa architectures. We need to clean up NMI handling there first. Let's do it separately. The patch is heavily based on the draft from Peter Zijlstra, see https://lkml.org/lkml/2015/6/10/327 [arnd@arndb.de: printk-nmi: use %zu format string for size_t] [akpm@linux-foundation.org: min_t->min - all types are size_t here] Signed-off-by: Petr Mladek <pmladek@suse.com> Suggested-by: Peter Zijlstra <peterz@infradead.org> Suggested-by: Steven Rostedt <rostedt@goodmis.org> Cc: Jan Kara <jack@suse.cz> Acked-by: Russell King <rmk+kernel@arm.linux.org.uk> [arm part] Cc: Daniel Thompson <daniel.thompson@linaro.org> Cc: Jiri Kosina <jkosina@suse.com> Cc: Ingo Molnar <mingo@redhat.com> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Ralf Baechle <ralf@linux-mips.org> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com> Cc: David Miller <davem@davemloft.net> Cc: Daniel Thompson <daniel.thompson@linaro.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>		2016-05-20 17:58:30 -07:00
..
boot
configs	bf609: enable soft switch gpio driver by default	2015-04-23 21:34:29 +08:00
include	exit_thread: remove empty bodies	2016-05-20 17:58:30 -07:00
kernel	arch, ftrace: for KASAN put hard/soft IRQ entries into separate sections	2016-03-25 16:37:42 -07:00
lib	Blackfin: comment spelling s/divsor/divisor/	2016-04-18 12:51:57 +02:00
mach-bf518
mach-bf527	Input: ad7879 - move header to platform_data directory	2016-03-08 10:51:10 -08:00
mach-bf533	arch/blackfin/mach-bf533/boards/stamp.c: add linux/delay.h	2015-01-08 15:10:52 -08:00
mach-bf537	Linux 4.5	2016-03-15 16:54:45 -07:00
mach-bf538	Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input	2016-03-17 21:51:52 -07:00
mach-bf548	gpio: Include linux/gpio.h instead of asm/gpio.h	2016-02-16 00:20:03 +01:00
mach-bf561	MTD updates for v4.5:	2016-01-13 11:25:54 -08:00
mach-bf609	blackfin: Fix misspelling of 'register' in comment.	2016-04-18 12:45:52 +02:00
mach-common	This is the bulk of GPIO changes for kernel v4.6:	2016-03-17 21:05:32 -07:00
mm
oprofile
Clear_BSD.txt
Kconfig	printk/nmi: generic solution for safe printk in NMI	2016-05-20 17:58:30 -07:00
Kconfig.debug
Makefile