[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20060718191345.GE20141@verge.net.au>
Date: Tue, 18 Jul 2006 15:13:48 -0400
From: Horms <horms@...ge.net.au>
To: Andrew Morton <akpm@...l.org>
Cc: ak@...e.de, rmk@....linux.org.uk, tony.luck@...el.com,
paulus@...ba.org, anton@...ba.org, chris@...kel.net,
linux-ia64@...r.kernel.org, linuxppc-dev@...abs.org,
discuss@...-64.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] panic_on_oops: remove ssleep()
On Mon, Jul 17, 2006 at 05:23:41PM -0700, Andrew Morton wrote:
> On Mon, 17 Jul 2006 19:10:59 -0400
> Horms <horms@...ge.net.au> 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/
-
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/
Powered by blists - more mailing lists