Muchun Song eb9036b4cf printk: fix deadlock when kernel panic ... commit 8a8109f303 upstream. printk_safe_flush_on_panic() caused the following deadlock on our server: CPU0: CPU1: panic rcu_dump_cpu_stacks kdump_nmi_shootdown_cpus nmi_trigger_cpumask_backtrace register_nmi_handler(crash_nmi_callback) printk_safe_flush __printk_safe_flush raw_spin_lock_irqsave(&read_lock) // send NMI to other processors apic_send_IPI_allbutself(NMI_VECTOR) // NMI interrupt, dead loop crash_nmi_callback printk_safe_flush_on_panic printk_safe_flush __printk_safe_flush // deadlock raw_spin_lock_irqsave(&read_lock) DEADLOCK: read_lock is taken on CPU1 and will never get released. It happens when panic() stops a CPU by NMI while it has been in the middle of printk_safe_flush(). Handle the lock the same way as logbuf_lock. The printk_safe buffers are flushed only when both locks can be safely taken. It can avoid the deadlock _in this particular case_ at expense of losing contents of printk_safe buffers. Note: It would actually be safe to re-init the locks when all CPUs were stopped by NMI. But it would require passing this information from arch-specific code. It is not worth the complexity. Especially because logbuf_lock and printk_safe buffers have been obsoleted by the lockless ring buffer. Fixes: cf9b1106c8 ("printk/nmi: flush NMI messages on the system panic") Signed-off-by: Muchun Song <songmuchun@bytedance.com> Reviewed-by: Petr Mladek <pmladek@suse.com> Cc: <stable@vger.kernel.org> Acked-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com> Signed-off-by: Petr Mladek <pmladek@suse.com> Link: https://lore.kernel.org/r/20210210034823.64867-1-songmuchun@bytedance.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2021-03-04 11:38:41 +01:00
..
Makefile	printk: add lockless ringbuffer	2020-07-10 08:48:19 +02:00
braille.c	printk: Replace strncmp() with str_has_prefix()	2019-08-16 09:54:08 +02:00
braille.h	License cleanup: add SPDX GPL-2.0 license identifier to files with no license	2017-11-02 11:10:55 +01:00
console_cmdline.h	printk: Fix preferred console selection with multiple matches	2020-02-18 09:34:42 +01:00
internal.h	printk: move dictionary keys to dev_printk_info	2020-09-22 11:27:48 +02:00
printk.c	printk: avoid prb_first_valid_seq() where possible	2021-03-04 11:38:07 +01:00
printk_ringbuffer.c	printk: ringbuffer: fix line counting	2021-01-27 11:55:08 +01:00
printk_ringbuffer.h	printk: avoid and/or handle record truncation	2020-09-30 13:30:28 +02:00
printk_safe.c	printk: fix deadlock when kernel panic	2021-03-04 11:38:41 +01:00