Changeset e3038b4 in mainline for kernel/arch/ia64/src/interrupt.c

Timestamp:

2010-06-28T22:45:51Z (15 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

49eb681

Parents:

05e3cb8 (diff), e4a4b44 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge unified panic architecture (Phase 1).

Note that this is still work in progress as there are the following sharp edges:

• imprecise detection of read/write accesses on some architectures
• missing or imperfect capability to print stack traces on some architectures
• istate_t on some architectures may contain too little valuable information
• basically all trap frames need to be reorganized to look like a normal stack frame on the stack trace so that there are no missing frames
• panic_common() could print more information about the current context such as the task name, thread name, ASID etc.
• functions that call panic_*() should be protected against inlining to avoid missing or confusing stack frames in the stack trace

File:

• kernel/arch/ia64/src/interrupt.c (modified) (4 diffs)

kernel/arch/ia64/src/interrupt.c

@@ -133 +133 @@
 }
 
-static void dump_interrupted_context(istate_t *istate)
-{
-        putchar('\n');
-        printf("Interrupted context dump:\n");
+void istate_decode(istate_t *istate)
+{
         printf("ar.bsp=%p\tar.bspstore=%p\n", istate->ar_bsp,
             istate->ar_bspstore);
@@ -183 +181 @@
         
         fault_if_from_uspace(istate, "General Exception (%s).", desc);
-        
-        dump_interrupted_context(istate);
-        panic("General Exception (%s).", desc);
+        panic_badtrap(istate, vector, "General Exception (%s).", desc);
 }
 
@@ -195 +191 @@
         fault_if_from_uspace(istate, "Interruption: %#hx (%s).",
             (uint16_t) vector, vector_to_string(vector));
-        dump_interrupted_context(istate);
-        panic("Interruption: %#hx (%s).", (uint16_t) vector,
-            vector_to_string(vector));
+        panic_badtrap(istate, vector, "Interruption: %#hx (%s).",
+            (uint16_t) vector, vector_to_string(vector));
 #endif
 }
@@ -226 +221 @@
         fault_if_from_uspace(istate, "Interruption: %#hx (%s).",
             (uint16_t) vector, vector_to_string(vector));
-        dump_interrupted_context(istate);
-        panic("Interruption: %#hx (%s).", (uint16_t) vector,
-            vector_to_string(vector));
+        panic_badtrap(istate, vector, "Interruption: %#hx (%s).",
+            (uint16_t) vector, vector_to_string(vector));
 }