linux-kernel - Re: printk: preempt_disable with long time resulting in softlockup/RCU stall issues

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 8 Jul 2015 14:00:17 -0400
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Joe Perches <joe@...ches.com>
Cc:	"Luis R. Rodriguez" <mcgrof@...e.com>,
	Gavin Hu <gavin.hu.2010@...il.com>,
	Petr Mladek <pmladek@...e.com>, linux-kernel@...r.kernel.org,
	Alex Elder <elder@...aro.org>,
	Peter Hurley <peter@...leysoftware.com>,
	Tejun Heo <tj@...nel.org>, cxie4@...vell.com, cldu@...vell.com,
	xjian@...vell.com, fswu@...vell.com, Jan Kara <jack@...e.cz>
Subject: Re: printk: preempt_disable with long time resulting in
 softlockup/RCU stall issues

On Wed, 08 Jul 2015 10:35:29 -0700
Joe Perches <joe@...ches.com> wrote:

> On Wed, 2015-07-08 at 19:04 +0200, Luis R. Rodriguez wrote:
> > On Wed, Jul 08, 2015 at 05:08:35PM +0800, Gavin Hu wrote:
> > > Hi,
> > > 
> > > Yes. We should disable the printk_limit feature when panic to avoid missing
> > > messages.
> > 
> > Sounds like you have been looking into it and have a good idea of what you
> > want to do, why not try it and send some RFC patches ?
> > 
> > While at it, then we could consider doing different things depending on the
> > message type. KERN_EMERG would disable preemption, whereas KERN_INFO may not be
> > so critical to require it.
> 
> That might be a bit difficult to implement with complete correctness
> given KERN_EMERG use and continuation lines.
> 

Or just have the current context determine what to do. If printk() was
called with preemption or interrupts disabled, it flushes the full
buffer before returning, otherwise it allows the writes to console be
preempted.

That would be great for -rt, as currently we don't even allow printk()
to write out if called from a interrupt disabled context, unless the
console it writes to is marked as preempt safe (serial console is
unfortunately not).

-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/