Re: [PATCH] i386: Do backtrace fallback too

From: Dave Jones
Date: Sat Jul 29 2006 - 03:52:30 EST


On Sat, Jul 29, 2006 at 03:00:06AM +0000, Linux Kernel wrote:
> commit c97d20a6c51067a38f53680d9609b4cf2867d077
> tree 59867ac01d1b752ba7e520e33f9f84cade6d024e
> parent b783fd925cdd56d24d164e5bdcb072f2a67aedf4
> author Andi Kleen <ak@xxxxxxx> Fri, 28 Jul 2006 14:44:57 +0200
> committer Linus Torvalds <torvalds@xxxxxxxxxxx> Sat, 29 Jul 2006 09:28:00 -0700
>
> [PATCH] i386: Do backtrace fallback too
>
> Similar patch to earlier x86-64 patch. When the dwarf2 unwinder fails
> dump the left over stack with the old unwinder.
>
> Also some clarifications in the headers.
>
> Signed-off-by: Andi Kleen <ak@xxxxxxx>
> Signed-off-by: Linus Torvalds <torvalds@xxxxxxxx>
>
> arch/i386/kernel/traps.c | 17 ++++++++++++++---
> 1 files changed, 14 insertions(+), 3 deletions(-)

Hmm, this breaks the build for me..

arch/i386/kernel/traps.c: In function 'show_trace_log_lvl':
arch/i386/kernel/traps.c:195: error: invalid type argument of '->'
arch/i386/kernel/traps.c:198: error: invalid type argument of '->'
arch/i386/kernel/traps.c:199: error: invalid type argument of '->'
make[1]: *** [arch/i386/kernel/traps.o] Error 1

(The line numbers are different to mainline due to some unrelated
patches, they point to the UNW_PC/UNW_SP usages),


Also, shouldn't this..

print_symbol("DWARF2 unwinder stuck at %s\n",
UNW_PC(info.regs));

be using %p ?

Dave

--
http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/