Hi,
I forgot signed-off-by
Signed-off-by: Akio Takebe <takebe_akio@xxxxxxxxxxxxxx>
Best Regards,
Akio Takebe
>Hi, Alex
>
>>On Fri, 2007-03-23 at 15:00 +0900, Akio Takebe wrote:
>>> Hi,
>>>
>>> I found the bug of panic_domain().
>>> When we compile xen with crash_debug=y, debugger_trap_immediate() and
>>> debugger_trap_fatal() is not nop.
>>> So if xen call panic_domain() to crash guest,
>>> xen call debugger routine, then hangup system.
>>>
>>> domain_crash_synchronous() has __FILE__, __LINE__ macros.
>>> So I remove panic_domain() and replace it with printk() and
>>> domain_crash_synchronous() as x86 do.
>>
>> Wouldn't it work just as well to convert panic_domain() to a macro
>>and remove the debugger_trap_immediate() and debugger_trap_fatal()
>>calls? The abstraction of panic_domain() is easier to use than
>>requiring someone to call all the relevant functions manually. Thanks,
>>
>Yes, I used also show_registers() if panic_domain has regs arguments.
>(e.g. the following part.)
>
>diff -r 2216a45bf058 xen/arch/ia64/vmx/vlsapic.c
>--- a/xen/arch/ia64/vmx/vlsapic.c Tue Mar 20 15:19:38 2007 -0600
>+++ b/xen/arch/ia64/vmx/vlsapic.c Fri Mar 23 14:41:48 2007 +0900
>@@ -480,8 +480,11 @@ int vmx_check_pending_irq(VCPU *vcpu)
> mask = irq_masked(vcpu, h_pending, h_inservice);
> if ( vpsr.i && IRQ_NO_MASKED == mask ) {
> isr = vpsr.val & IA64_PSR_RI;
>- if ( !vpsr.ic )
>- panic_domain(regs,"Interrupt when IC=0\n");
>+ if ( !vpsr.ic ){
>+ printk("Interrupt when IC=0\n");
>+ show_registers(regs);
>+ domain_crash_synchronous();
>+ }
> update_vhpi(vcpu, h_pending);
> vmx_reflect_interruption(0, isr, 0, 12, regs); // EXT IRQ
> } else if (mask == IRQ_MASKED_BY_INSVC) {
>
>I think panic_domain() don't need debugger_trap_immediate()
>and debugger_trap_fatal().
>So what I really want to do is removing them.
>
>BTW, as you said, panic_domain() is easier to use than
>domain_crash_synchronous() macro.
>I make new patch.
>We can know the caller function by CallTrace (without __FILE__, __LINE__),
>so I only remove the debug routines.
>Is the attached new patch better?
>
>Best Regards,
>
>Akio Takebe
_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
|
Home