Re: [PATCH] defer printks in irqs

[Tim Bird <tim.bird@am.sony.com>]

Andrew Morton wrote:
>> On Thu, 24 Jan 2008 15:23:03 -0800 Tim Bird <tim.bird@am.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
=============================