linux-kernel - Re: [PATCH] defer printks in irqs

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] [day] [month] [year] [list]

Date:	Mon, 28 Jan 2008 13:14:05 -0800
From:	Tim Bird <tim.bird@...sony.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
CC:	a.p.zijlstra@...llo.nl, rostedt@...dmis.org, dwalker@...sta.com,
	linux-kernel@...r.kernel.org, mingo@...e.hu,
	torvalds@...ux-foundation.org, hch@...radead.org,
	mathieu.desnoyers@...ymtl.ca, ghaskins@...ell.com,
	acme@...stprotocols.net, tglx@...utronix.de, sam@...nborg.org,
	fche@...hat.com, jan.kiszka@...mens.com, johnstul@...ibm.com,
	arjan@...radead.org, srostedt@...hat.com
Subject: Re: [PATCH] defer printks in irqs

Andrew Morton wrote:
>> On Thu, 24 Jan 2008 15:23:03 -0800 Tim Bird <tim.bird@...sony.com> wrote:
>> Peter Zijlstra wrote:
>>> I suspect these features reduce the chance a crash messages makes it out
>>> onto the console, but fail to spot any of the copious text mention this
>>> critical issue.
>> Anything not in interrupts would not be deferred.
>> The following code is meant to avoid deferring in oops as well.
>> I'm not sure if that covers all crash cases or not.
>> If you know of other cases, and ways to detect them, please advise.
>>
> 
> Yes, using oops_in_progress seems an appropriate fix for that.
> 
> I assume that another downside is that printk-from-interrupt will now get
> jumbled up with an interrupted pritk-from-non-interrupt.

No.  The order of printk messages (whether printk'ed
from interrupt context or not) should not be affected.
This patch should only affect the timing of when the
messages are output to console.

Note that an oops will flush any buffered messages
ahead of the oops messages themselves.

Some messages, not in oops, that were printk'ed in
interrupt context, may still be in the log buffer
but not have made it to console if the machine hangs
hard.  Because of this (IMO slight) downgrade in
debugging utility, I chose to make the changes
configurable.

> Please use checkpatch.

Will do from now on!  Thanks!
  -- Tim


=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================

--
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/