Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751670AbbEYMry (ORCPT ); Mon, 25 May 2015 08:47:54 -0400 Received: from cantor2.suse.de ([195.135.220.15]:53645 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751484AbbEYMrI (ORCPT ); Mon, 25 May 2015 08:47:08 -0400 From: Petr Mladek To: Andrew Morton Cc: Frederic Weisbecker , Steven Rostedt , Dave Anderson , "Paul E. McKenney" , Kay Sievers , Jiri Kosina , Michal Hocko , Jan Kara , linux-kernel@vger.kernel.org, Wang Long , peifeiyue@huawei.com, dzickus@redhat.com, morgan.wang@huawei.com, sasha.levin@oracle.com, Petr Mladek Subject: [PATCH 08/10] printk: Detect scheduler messages in vprintk_format_and_analyze() Date: Mon, 25 May 2015 14:46:31 +0200 Message-Id: <1432557993-20458-9-git-send-email-pmladek@suse.cz> X-Mailer: git-send-email 1.8.5.6 In-Reply-To: <1432557993-20458-1-git-send-email-pmladek@suse.cz> References: <1432557993-20458-1-git-send-email-pmladek@suse.cz> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2749 Lines: 81 The special hack for messages printed by the scheduler belongs to the new vprintk_format_and_analyze(). This patch just shuffles the code. There is no change in the functionality. Signed-off-by: Petr Mladek --- kernel/printk/printk.c | 23 ++++++++++++++--------- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index 7d009144f97f..6e53b6f60ca3 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -1747,15 +1747,16 @@ static int vprintk_delayed_warnings(void) * @dict: pointer to the message dictionary * @facility: syslog facility * @level: syslog level; might be replaced by the one found in the fomatted text + * @in_sched: is set to true when the message is printed by the scheduler * @ftext: pointer to the formatted text after the syslog prefix * @ftext_len: length of the formatted text without the syslog prefix * * This function modifies the global textbuf and therefore it must be called * under lockbuf_lock! */ -static enum log_flags vprinkt_format_and_analyze(const char *fmt, va_list args, +static enum log_flags vprintk_format_and_analyze(const char *fmt, va_list args, const char *dict, int facility, - int *level, + int *level, bool *in_sched, char **ftext, size_t *ftext_len) { @@ -1775,6 +1776,15 @@ static enum log_flags vprinkt_format_and_analyze(const char *fmt, va_list args, lflags |= LOG_NEWLINE; } + /* + * Messages printed by the scheduler must not wake up the console. + * They are detected via a special @level, see printk_deferred(). + */ + if (*level == LOGLEVEL_SCHED) { + *level = LOGLEVEL_DEFAULT; + *in_sched = true; + } + /* strip kernel syslog prefix and extract log level or control flags */ if (facility == 0) { int kern_level = printk_get_level(text); @@ -1824,11 +1834,6 @@ asmlinkage int vprintk_emit(int facility, int level, int printed_len = 0; bool in_sched = false; - if (level == LOGLEVEL_SCHED) { - level = LOGLEVEL_DEFAULT; - in_sched = true; - } - boot_delay_msec(level); printk_delay(); @@ -1880,8 +1885,8 @@ asmlinkage int vprintk_emit(int facility, int level, printed_len += vprintk_delayed_warnings(); - lflags = vprinkt_format_and_analyze(fmt, args, dict, facility, - &level, + lflags = vprintk_format_and_analyze(fmt, args, dict, facility, + &level, &in_sched, &text, &text_len); if (!(lflags & LOG_NEWLINE)) { -- 1.8.5.6 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/