Changeset e3038b4 in mainline for kernel/generic/include/panic.h

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/generic/include/panic.h (modified) (2 diffs)

kernel/generic/include/panic.h

@@ -1 +1 @@
 /*
- * Copyright (c) 2001-2004 Jakub Jermar
+ * Copyright (c) 2010 Jakub Jermar
  * All rights reserved.
  *
@@ -37 +37 @@
 
 #include <typedefs.h>
-#include <stacktrace.h>
-#include <print.h>
 
-#ifdef CONFIG_DEBUG
+#define panic(fmt, ...) \
+        panic_common(PANIC_OTHER, NULL, 0, 0, fmt, ##__VA_ARGS__)
 
-#define panic(format, ...) \
-        do { \
-                silent = false; \
-                printf("Kernel panic in %s() at %s:%u\n", \
-                    __func__, __FILE__, __LINE__); \
-                stack_trace(); \
-                panic_printf("Panic message: " format "\n", \
-                    ##__VA_ARGS__);\
-        } while (0)
+#define panic_assert(fmt, ...) \
+        panic_common(PANIC_ASSERT, NULL, 0, 0, fmt, ##__VA_ARGS__)
 
-#else /* CONFIG_DEBUG */
+#define panic_badtrap(istate, n, fmt, ...) \
+        panic_common(PANIC_BADTRAP, istate, 0, n, fmt, ##__VA_ARGS__)
 
-#define panic(format, ...) \
-        do { \
-                silent = false; \
-                panic_printf("Kernel panic: " format "\n", ##__VA_ARGS__); \
-                stack_trace(); \
-        } while (0)
+#define panic_memtrap(istate, access, addr, fmt, ...) \
+        panic_common(PANIC_MEMTRAP, istate, access, addr, fmt, ##__VA_ARGS__)
 
-#endif /* CONFIG_DEBUG */
+typedef enum {
+        PANIC_OTHER,
+        PANIC_ASSERT,
+        PANIC_BADTRAP,
+        PANIC_MEMTRAP
+} panic_category_t;
+
+struct istate;
 
 extern bool silent;
 
-extern void panic_printf(const char *fmt, ...) __attribute__((noreturn));
+extern void panic_common(panic_category_t, struct istate *, int,
+    uintptr_t, const char *, ...) __attribute__ ((noreturn));
 
 #endif