On Tue, 27 Oct 2009 10:36:53 +1100, Keith Owens wrote:
>On Mon, 26 Oct 2009 18:15:22 -0400,
>Takao Indoh <indou.takao@xxxxxxxxxxxxxx> wrote:
>>As Intel manual says, it seems that pmsa_iip has a value of ip register
>>where INIT interrupted when psr.ic is 0. Ok, I'll make a patch so that
>>pmsa_iip is used irrespective of a value of psr.ic.
>>
>>Please let me know if I need to confirm something else before I make a
>>patch.
>
>That looks good. The only problem I can think of is an MCA/INIT
>arriving while code like SAVE_MIN or SAVE_REST is executing. Back
>tracing at that point using pmsa_iip is going to be a problem, you have
>no idea what state the registers or stack are in.
I inserted another debug code into IA64_NATIVE_DO_SAVE_MIN to send
INIT during SAVE_MIN.
diff -Nurp a/arch/ia64/kernel/minstate.h b/arch/ia64/kernel/minstate.h
--- a/arch/ia64/kernel/minstate.h 2009-10-22 12:28:07.000000000 -0400
+++ b/arch/ia64/kernel/minstate.h 2009-10-27 12:42:02.000000000 -0400
@@ -55,6 +55,14 @@
mov r21=ar.fpsr; /* M */ \
__COVER; /* B;; (or nothing) */ \
;; \
+ movl r24=debug_flag; \
+ ;; \
+ ld8 r23=[r24]; \
+ ;; \
+ cmp.eq p2,p3=0,r23; \
+(p2) br.sptk 2f; \
+1: br.sptk 1b; \
+2: \
adds r16=IA64_TASK_THREAD_ON_USTACK_OFFSET,r16; \
;; \
ld1 r17=[r16]; /* load current->thread.on_ustack flag */ \
As a result, backtrace in the case using xip works, and backtrace in the
case using iip does NOT work, as you said. Hm, it seems that I have to
reconsider how to fix.
Thanks,
Takao Indoh
--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|