Hi, Horms and Ian
Thank you for your reply, Horms.
I forgot Signed-off-by of the patch.
Signed-off-by: Horms <horms@xxxxxxxxxxxx>
Signed-off-by: Akio Takebe <takebe_akio@xxxxxxxxxxxxxx>
Is the Signed-off-by OK, Horms?
Best Regards,
Akio Takebe
>[ Ian Campbell added to CC list ]
>
>On Tue, Sep 05, 2006 at 06:45:35AM +0900, Akio Takebe wrote:
>> 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
>
>
>Hi Takebe-san,
>
>this patch seems ok to me, but it seems that it never went into the
>tree. Ian, what are your thoughts on it?
>
>> 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 *)®s->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
_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
|