diff options
author | Jason Wessel <jason.wessel@windriver.com> | 2010-05-20 22:04:26 -0400 |
---|---|---|
committer | Jason Wessel <jason.wessel@windriver.com> | 2010-05-20 22:04:26 -0400 |
commit | 5dd11d5d47d248850c58292513f0e164ba98b01e (patch) | |
tree | fd3c5f27dd40d6a483483de3644ca060a5cafd75 | |
parent | ba797b28131b1f1367b662936ea370239d603cff (diff) |
mips,kgdb: kdb low level trap catch and stack trace
The only way the debugger can handle a trap in inside rcu_lock,
notify_die, or atomic_notifier_call_chain without a recursive fault is
to have a low level "first opportunity handler" do_trap_or_bp() handler.
Generally this will be something the vast majority of folks will not
need, but for those who need it, it is added as a kernel .config
option called KGDB_LOW_LEVEL_TRAP.
Also added was a die notification for oops such that kdb can catch an
oops for analysis.
There appeared to be no obvious way to pass the struct pt_regs from
the original exception back to the stack back tracer, so a special
case was added to show_stack() for when kdb is active because you
generally desire to generally look at the back trace of the original
exception.
Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Acked-by: Ralf Baechle <ralf@linux-mips.org>
-rw-r--r-- | arch/mips/include/asm/kgdb.h | 2 | ||||
-rw-r--r-- | arch/mips/kernel/kgdb.c | 22 | ||||
-rw-r--r-- | arch/mips/kernel/traps.c | 13 | ||||
-rw-r--r-- | lib/Kconfig.kgdb | 2 |
4 files changed, 37 insertions, 2 deletions
diff --git a/arch/mips/include/asm/kgdb.h b/arch/mips/include/asm/kgdb.h index 48223b09396c..19002d605ac4 100644 --- a/arch/mips/include/asm/kgdb.h +++ b/arch/mips/include/asm/kgdb.h | |||
@@ -38,6 +38,8 @@ extern int kgdb_early_setup; | |||
38 | extern void *saved_vectors[32]; | 38 | extern void *saved_vectors[32]; |
39 | extern void handle_exception(struct pt_regs *regs); | 39 | extern void handle_exception(struct pt_regs *regs); |
40 | extern void breakinst(void); | 40 | extern void breakinst(void); |
41 | extern int kgdb_ll_trap(int cmd, const char *str, | ||
42 | struct pt_regs *regs, long err, int trap, int sig); | ||
41 | 43 | ||
42 | #endif /* __KERNEL__ */ | 44 | #endif /* __KERNEL__ */ |
43 | 45 | ||
diff --git a/arch/mips/kernel/kgdb.c b/arch/mips/kernel/kgdb.c index 6ed4c83c869b..9b78ff6e9b84 100644 --- a/arch/mips/kernel/kgdb.c +++ b/arch/mips/kernel/kgdb.c | |||
@@ -203,7 +203,7 @@ static int kgdb_mips_notify(struct notifier_block *self, unsigned long cmd, | |||
203 | if (atomic_read(&kgdb_active) != -1) | 203 | if (atomic_read(&kgdb_active) != -1) |
204 | kgdb_nmicallback(smp_processor_id(), regs); | 204 | kgdb_nmicallback(smp_processor_id(), regs); |
205 | 205 | ||
206 | if (kgdb_handle_exception(trap, compute_signal(trap), 0, regs)) | 206 | if (kgdb_handle_exception(trap, compute_signal(trap), cmd, regs)) |
207 | return NOTIFY_DONE; | 207 | return NOTIFY_DONE; |
208 | 208 | ||
209 | if (atomic_read(&kgdb_setting_breakpoint)) | 209 | if (atomic_read(&kgdb_setting_breakpoint)) |
@@ -217,6 +217,26 @@ static int kgdb_mips_notify(struct notifier_block *self, unsigned long cmd, | |||
217 | return NOTIFY_STOP; | 217 | return NOTIFY_STOP; |
218 | } | 218 | } |
219 | 219 | ||
220 | #ifdef CONFIG_KGDB_LOW_LEVEL_TRAP | ||
221 | int kgdb_ll_trap(int cmd, const char *str, | ||
222 | struct pt_regs *regs, long err, int trap, int sig) | ||
223 | { | ||
224 | struct die_args args = { | ||
225 | .regs = regs, | ||
226 | .str = str, | ||
227 | .err = err, | ||
228 | .trapnr = trap, | ||
229 | .signr = sig, | ||
230 | |||
231 | }; | ||
232 | |||
233 | if (!kgdb_io_module_registered) | ||
234 | return NOTIFY_DONE; | ||
235 | |||
236 | return kgdb_mips_notify(NULL, cmd, &args); | ||
237 | } | ||
238 | #endif /* CONFIG_KGDB_LOW_LEVEL_TRAP */ | ||
239 | |||
220 | static struct notifier_block kgdb_notifier = { | 240 | static struct notifier_block kgdb_notifier = { |
221 | .notifier_call = kgdb_mips_notify, | 241 | .notifier_call = kgdb_mips_notify, |
222 | }; | 242 | }; |
diff --git a/arch/mips/kernel/traps.c b/arch/mips/kernel/traps.c index d612c6dcb746..950bde8813fc 100644 --- a/arch/mips/kernel/traps.c +++ b/arch/mips/kernel/traps.c | |||
@@ -26,6 +26,7 @@ | |||
26 | #include <linux/kgdb.h> | 26 | #include <linux/kgdb.h> |
27 | #include <linux/kdebug.h> | 27 | #include <linux/kdebug.h> |
28 | #include <linux/notifier.h> | 28 | #include <linux/notifier.h> |
29 | #include <linux/kdb.h> | ||
29 | 30 | ||
30 | #include <asm/bootinfo.h> | 31 | #include <asm/bootinfo.h> |
31 | #include <asm/branch.h> | 32 | #include <asm/branch.h> |
@@ -185,6 +186,11 @@ void show_stack(struct task_struct *task, unsigned long *sp) | |||
185 | regs.regs[29] = task->thread.reg29; | 186 | regs.regs[29] = task->thread.reg29; |
186 | regs.regs[31] = 0; | 187 | regs.regs[31] = 0; |
187 | regs.cp0_epc = task->thread.reg31; | 188 | regs.cp0_epc = task->thread.reg31; |
189 | #ifdef CONFIG_KGDB_KDB | ||
190 | } else if (atomic_read(&kgdb_active) != -1 && | ||
191 | kdb_current_regs) { | ||
192 | memcpy(®s, kdb_current_regs, sizeof(regs)); | ||
193 | #endif /* CONFIG_KGDB_KDB */ | ||
188 | } else { | 194 | } else { |
189 | prepare_frametrace(®s); | 195 | prepare_frametrace(®s); |
190 | } | 196 | } |
@@ -360,6 +366,8 @@ void __noreturn die(const char * str, struct pt_regs * regs) | |||
360 | unsigned long dvpret = dvpe(); | 366 | unsigned long dvpret = dvpe(); |
361 | #endif /* CONFIG_MIPS_MT_SMTC */ | 367 | #endif /* CONFIG_MIPS_MT_SMTC */ |
362 | 368 | ||
369 | notify_die(DIE_OOPS, str, (struct pt_regs *)regs, SIGSEGV, 0, 0); | ||
370 | |||
363 | console_verbose(); | 371 | console_verbose(); |
364 | spin_lock_irq(&die_lock); | 372 | spin_lock_irq(&die_lock); |
365 | bust_spinlocks(1); | 373 | bust_spinlocks(1); |
@@ -704,6 +712,11 @@ static void do_trap_or_bp(struct pt_regs *regs, unsigned int code, | |||
704 | siginfo_t info; | 712 | siginfo_t info; |
705 | char b[40]; | 713 | char b[40]; |
706 | 714 | ||
715 | #ifdef CONFIG_KGDB_LOW_LEVEL_TRAP | ||
716 | if (kgdb_ll_trap(DIE_TRAP, str, regs, code, 0, 0) == NOTIFY_STOP) | ||
717 | return; | ||
718 | #endif /* CONFIG_KGDB_LOW_LEVEL_TRAP */ | ||
719 | |||
707 | if (notify_die(DIE_TRAP, str, regs, code, 0, 0) == NOTIFY_STOP) | 720 | if (notify_die(DIE_TRAP, str, regs, code, 0, 0) == NOTIFY_STOP) |
708 | return; | 721 | return; |
709 | 722 | ||
diff --git a/lib/Kconfig.kgdb b/lib/Kconfig.kgdb index c56ccb4ad292..43cb93fa2651 100644 --- a/lib/Kconfig.kgdb +++ b/lib/Kconfig.kgdb | |||
@@ -59,7 +59,7 @@ config KGDB_TESTS_BOOT_STRING | |||
59 | 59 | ||
60 | config KGDB_LOW_LEVEL_TRAP | 60 | config KGDB_LOW_LEVEL_TRAP |
61 | bool "KGDB: Allow debugging with traps in notifiers" | 61 | bool "KGDB: Allow debugging with traps in notifiers" |
62 | depends on X86 | 62 | depends on X86 || MIPS |
63 | default n | 63 | default n |
64 | help | 64 | help |
65 | This will add an extra call back to kgdb for the breakpoint | 65 | This will add an extra call back to kgdb for the breakpoint |