Re: [debug patch] printk: Add a printk killswitch to robustify NMI watchdog messages
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [debug patch] printk: Add a printk killswitch to robustify NMI watchdog messages
- From: Ingo Molnar <mingo@xxxxxxx>
- Date: Mon, 6 Jun 2011 17:08:08 +0200
- Cc: Arne Jansen <lists@xxxxxxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, mingo@xxxxxxxxxx, hpa@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, efault@xxxxxx, npiggin@xxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, frank.rowand@xxxxxxxxxxx, tglx@xxxxxxxxxxxxx, linux-tip-commits@xxxxxxxxxxxxxxx
- In-reply-to: <20110606150409.GE30348@xxxxxxx>
- User-agent: Mutt/1.5.20 (2009-08-17)
* Ingo Molnar <mingo@xxxxxxx> wrote:
> The real fix might be to remove the lockdep_off()/on() call from
> printk(), that looks actively evil ... we had to hack through
> several layers of side-effects before we found the real bug - so
> it's not like the off()/on() made things more robust!
The other obvious fix would be to *remove* the blasted wakeup from
printk(). It's a serious debugging robustness violation and it's not
like the wakeup is super important latency-wise.
We *already* have a timer tick driven klogd wakeup poll routine. So i
doubt we'd have many problems from not doing wakeups from printk().
Opinions?
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]