diff options
| author | Linas Vepstas <linas@austin.ibm.com> | 2007-11-20 02:28:15 +0100 |
|---|---|---|
| committer | Paul Mackerras <paulus@samba.org> | 2007-11-20 06:10:09 +0100 |
| commit | a2b51812a4dc5db09ab4d4638d4d8ed456e2457e (patch) | |
| tree | c61eeb12f3c3164208468cf8837711f843a32792 /include | |
| parent | [POWERPC] vdso: Fixes for cache block sizes (diff) | |
| download | linux-a2b51812a4dc5db09ab4d4638d4d8ed456e2457e.tar.xz linux-a2b51812a4dc5db09ab4d4638d4d8ed456e2457e.zip | |
[POWERPC] Fix RTAS os-term usage on kernel panic
The rtas_os_term() routine was being called at the wrong time.
The actual rtas call "os-term" will not ever return, and so
calling it from the panic notifier is too early. Instead,
call it from the machine_reset() call.
This splits the rtas_os_term() routine into two: one part to capture
the kernel panic message, invoked during the panic notifier, and
another part that is invoked during machine_reset().
Prior to this patch, the os-term call was never being made,
because panic_timeout was always non-zero. Calling os-term
helps keep the hypervisor happy! We have to keep the hypervisor
happy to avoid service, dump and error reporting problems.
Signed-off-by: Linas Vepstas <linas@austin.ibm.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/asm-powerpc/rtas.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/asm-powerpc/rtas.h b/include/asm-powerpc/rtas.h index 8eaa7b28d9d0..87db8728e82d 100644 --- a/include/asm-powerpc/rtas.h +++ b/include/asm-powerpc/rtas.h @@ -164,7 +164,8 @@ extern int rtas_call(int token, int, int, int *, ...); extern void rtas_restart(char *cmd); extern void rtas_power_off(void); extern void rtas_halt(void); -extern void rtas_os_term(char *str); +extern void rtas_panic_msg(char *str); +extern void rtas_os_term(void); extern int rtas_get_sensor(int sensor, int index, int *state); extern int rtas_get_power_level(int powerdomain, int *level); extern int rtas_set_power_level(int powerdomain, int level, int *setlevel); |