diff options
author | Linas Vepstas <linas@austin.ibm.com> | 2007-11-19 20:28:15 -0500 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2007-11-20 00:10:09 -0500 |
commit | a2b51812a4dc5db09ab4d4638d4d8ed456e2457e (patch) | |
tree | c61eeb12f3c3164208468cf8837711f843a32792 /arch/powerpc/kernel | |
parent | fbe481756df57673b6acbcd2e139d0d2658f2188 (diff) |
[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 'arch/powerpc/kernel')
-rw-r--r-- | arch/powerpc/kernel/rtas.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/powerpc/kernel/rtas.c b/arch/powerpc/kernel/rtas.c index 52e95c2158c0..053cac19f714 100644 --- a/arch/powerpc/kernel/rtas.c +++ b/arch/powerpc/kernel/rtas.c | |||
@@ -638,18 +638,18 @@ void rtas_halt(void) | |||
638 | /* Must be in the RMO region, so we place it here */ | 638 | /* Must be in the RMO region, so we place it here */ |
639 | static char rtas_os_term_buf[2048]; | 639 | static char rtas_os_term_buf[2048]; |
640 | 640 | ||
641 | void rtas_os_term(char *str) | 641 | void rtas_panic_msg(char *str) |
642 | { | 642 | { |
643 | int status; | 643 | snprintf(rtas_os_term_buf, 2048, "OS panic: %s", str); |
644 | } | ||
644 | 645 | ||
645 | if (panic_timeout) | 646 | void rtas_os_term(void) |
646 | return; | 647 | { |
648 | int status; | ||
647 | 649 | ||
648 | if (RTAS_UNKNOWN_SERVICE == rtas_token("ibm,os-term")) | 650 | if (RTAS_UNKNOWN_SERVICE == rtas_token("ibm,os-term")) |
649 | return; | 651 | return; |
650 | 652 | ||
651 | snprintf(rtas_os_term_buf, 2048, "OS panic: %s", str); | ||
652 | |||
653 | do { | 653 | do { |
654 | status = rtas_call(rtas_token("ibm,os-term"), 1, 1, NULL, | 654 | status = rtas_call(rtas_token("ibm,os-term"), 1, 1, NULL, |
655 | __pa(rtas_os_term_buf)); | 655 | __pa(rtas_os_term_buf)); |