[parisc-linux] Changing hppa default on panic to "reboot after 180 seconds"?

[Carlos O'Donell]

On Sat, May 01, 2004 at 06:45:54PM -0400, Carlos O'Donell wrote:
> On Sat, May 01, 2004 at 06:33:43PM -0400, Carlos O'Donell wrote:
> > 
> > parisc-linux,
> > 
> > I do a lot of my work remotely on older parisc boxes, and I usually slam
> > the kernel into a panic. Previously I had my entire cluster of 32+ boxes
> > running, so I could crash 32 times before having to go into the office.
> > Lately I've been a little more power conservative.
> > 
> > I remember having lots more fun on my PowerPC box because it would
> > reboot after 180 seconds after a panic.
> > 
> > Would anyone be opposed to adding 'panic_timeout = 180' in our
> > arch/parisc/kernel/setup.c:setup_arch()? :)
> 
> Ignore this please... I missed the __setup("panic=", panic_setup) line.

So how about this hookup?

Index: arch/parisc/kernel/traps.c
===================================================================
RCS file: /var/cvs/linux-2.6/arch/parisc/kernel/traps.c,v
retrieving revision 1.13
diff -u -p -r1.13 traps.c
--- a/arch/parisc/kernel/traps.c	28 Apr 2004 19:12:10 -0000	1.13
+++ b/arch/parisc/kernel/traps.c	1 May 2004 22:54:34 -0000
@@ -404,9 +404,9 @@ void transfer_pim_to_trap_frame(struct p
 
 
 /*
- * This routine handles page faults.  It determines the address,
- * and the problem, and then passes it off to one of the appropriate
- * routines.
+ * This routine is called as a last resort when everything else
+ * has gone clearly wrong. We get called for faults in kernel space,
+ * and HPMC's.
  */
 void parisc_terminate(char *msg, struct pt_regs *regs, int code, unsigned long offset)
 {
@@ -452,9 +452,8 @@ void parisc_terminate(char *msg, struct 
 	 * system will shut down immediately right here. */
 	pdc_soft_power_button(0);
 	
-	/* Gutter the processor! */
-	for(;;)
-	    ;
+	/* Call kernel panic() so reboot timeouts work properly */
+	panic(msg);
 }
 
 void handle_interruption(int code, struct pt_regs *regs)