>From be116ae18f15f0d2d05ddf0b53eaac184943d312 Mon Sep 17 00:00:00 2001 From: Jan Kara Date: Mon, 21 Dec 2015 14:26:13 +0100 Subject: [PATCH 2/2] printk: Skip messages on oops When there are too many messages in the kernel printk buffer it can take very long to print them to console (especially when using slow serial console). This is undesirable during oops so when we encounter oops and there are more than 100 messages to print, print just the newest 100 messages and then the oops message. Signed-off-by: Jan Kara --- kernel/printk/printk.c | 34 +++++++++++++++++++++++++++++++++- 1 file changed, 33 insertions(+), 1 deletion(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index d455d1bd0d2c..fc67ab70e9c7 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -262,6 +262,9 @@ static u64 console_seq; static u32 console_idx; static enum log_flags console_prev; +/* current record sequence when oops happened */ +static u64 oops_start_seq; + /* the next printk record to read after the last 'clear' command */ static u64 clear_seq; static u32 clear_idx; @@ -1783,6 +1786,8 @@ asmlinkage int vprintk_emit(int facility, int level, NULL, 0, recursion_msg, strlen(recursion_msg)); } + if (oops_in_progress && !sync_print && !oops_start_seq) + oops_start_seq = log_next_seq; /* * The printf needs to come first; we need the syslog @@ -2292,6 +2297,12 @@ out: raw_spin_unlock_irqrestore(&logbuf_lock, flags); } +/* + * When oops happens and there are more messages to be printed in the printk + * buffer that this, skip some mesages and print only this many newest messages. + */ +#define PRINT_MSGS_BEFORE_OOPS 100 + /** * console_unlock - unlock the console system * @@ -2348,7 +2359,28 @@ again: seen_seq = log_next_seq; } - if (console_seq < log_first_seq) { + /* + * If oops happened and there are more than + * PRINT_MSGS_BEFORE_OOPS messages pending before oops message, + * skip them to make oops appear faster. + */ + if (oops_start_seq && + console_seq + PRINT_MSGS_BEFORE_OOPS < oops_start_seq) { + len = sprintf(text, + "** %u printk messages dropped due to oops ** ", + (unsigned)(oops_start_seq - console_seq - + PRINT_MSGS_BEFORE_OOPS)); + if (console_seq < log_first_seq) { + console_seq = log_first_seq; + console_idx = log_first_idx; + } + while (console_seq < + oops_start_seq - PRINT_MSGS_BEFORE_OOPS) { + console_idx = log_next(console_idx); + console_seq++; + } + console_prev = 0; + } else if (console_seq < log_first_seq) { len = sprintf(text, "** %u printk messages dropped ** ", (unsigned)(log_first_seq - console_seq)); -- 2.6.2