On Tue, Jul 18, 2006 at 12:27:51AM +0200, Andi Kleen wrote: > On Monday 17 July 2006 18:17, Horms wrote: > > This patch is part of an effort to unify the panic_on_oops behaviour > > across all architectures that implement it. > > > > It was pointed out to me by Andi Kleen that if an oops has occured > > in interrupt context, then calling sleep() in the oops path will only cause > > a panic, and that it would be really better for it not to be in the path at > > all. > > > > This patch removes the ssleep() call and reworks the console message > > accordinly. I have a slght concern that the resulting console message is > > too long, feedback welcome. > > Keeping the delay might be actually useful so that you can see the panic > before system reboots when reboot on panic is enabled. I would just use a loop > of mdelays(1) with touch_nmi_watchdog/touch_softirq_watchdog()s > inbetween. Ok, I will look into making that happen. I agree that the pause is quite useful. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - : send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |