On Mon, Jul 17, 2006 at 05:23:41PM -0700, Andrew Morton wrote: > On Mon, 17 Jul 2006 19:10:59 -0400 > Horms <horms@xxxxxxxxxxxx> wrote: > > > On Tue, Jul 18, 2006 at 12:27:51AM +0200, Andi Kleen wrote: > > > On Monday 17 July 2006 18:17, Horms wrote: > > > ... > > > 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. > > It's kind-of already implemented, via pause_on_oops. Perhaps doing > something like > > if (panic_on_oops) > pause_on_oops = max(pause_on_oops, 5*HZ); > > would be sufficient. Thanks, that may well be sufficient. And I assume that it is nicely out of the arch-dependant code in die(). I will poke around a bit more. -- 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
![]() |