Re: Re: [PATCH] kexec: framework and i386 (Take XIV)

[Akio Takebe <takebe_akio@jp.fujitsu.com>]

Hi, Horms

I tested the following patch with Horms kexec patch.

My tests is:
  push NMI bottun after loading kdump kernel.
  
The results is:
  OK, I could get vmcore

diff -r b688d4a68a3e xen/arch/x86/traps.c
--- a/xen/arch/x86/traps.c      Tue Aug 22 14:59:16 2006 +0100
+++ b/xen/arch/x86/traps.c      Tue Sep 05 06:37:49 2006 +0900
@@ -105,6 +105,8 @@ static int debug_stack_lines = 20;
 static int debug_stack_lines = 20;
 integer_param("debug_stack_lines", debug_stack_lines);
 
+extern void crash_kexec(struct cpu_user_regs *regs);
+
 #ifdef CONFIG_X86_32
 #define stack_words_per_line 8
 #define ESP_BEFORE_EXCEPTION(regs) ((unsigned long *)&regs->esp)
@@ -1611,8 +1613,10 @@ asmlinkage void do_nmi(struct cpu_user_r
             mem_parity_error(regs);
         else if ( reason & 0x40 )
             io_check_error(regs);
-        else if ( !nmi_watchdog )
+        else if ( !nmi_watchdog ){
+            crash_kexec(NULL);
             unknown_nmi_error((unsigned char)(reason&0xff));
+        }
     }
 }
 


Best Regards,

Akio Takebe

>On Fri, Sep 01, 2006 at 05:45:59PM +0900, Akio Takebe wrote:
>> >Hi, Horms
>> >
>> >>That seems like a good idea to me. Though I think you are missing { }.
>> >>Can you test to see if this works?
>> >Oops, You're right. But I think unknown_nmi_error() is not called,
>> >because crash_kexec() is called before that.
>> Sorry.
>> In the only case of CONFIG_KEXEC=y, the above is right.
>
>Yes, I think that is the case. I will put your patch into the kexec
>series, as I think that it is a worthy addition.
>
>-- 
>Horms
>  H: http://www.vergenet.net/~horms/
>  W: http://www.valinux.co.jp/en/
>
>
>_______________________________________________
>Xen-devel mailing list
>Xen-devel@lists.xensource.com
>http://lists.xensource.com/xen-devel