summaryrefslogtreecommitdiff
path: root/doc
diff options
context:
space:
mode:
authorRuss Cox <rsc@golang.org>2014-10-29 15:14:24 -0400
committerRuss Cox <rsc@golang.org>2014-10-29 15:14:24 -0400
commit166ba836a61c401a93d2064899743ce46c8440aa (patch)
tree53f2bab22571e7606db89fb1264be4c62e173783 /doc
parentdc2d730ec3dc9e56ae8da19f3125bb3c3cb9c992 (diff)
downloadgo-166ba836a61c401a93d2064899743ce46c8440aa.tar.gz
runtime: fix line number in first stack frame in printed stack trace
Originally traceback was only used for printing the stack when an unexpected signal came in. In that case, the initial PC is taken from the signal and should be used unaltered. For the callers, the PC is the return address, which might be on the line after the call; we subtract 1 to get to the CALL instruction. Traceback is now used for a variety of things, and for almost all of those the initial PC is a return address, whether from getcallerpc, or gp->sched.pc, or gp->syscallpc. In those cases, we need to subtract 1 from this initial PC, but the traceback code had a hard rule "never subtract 1 from the initial PC", left over from the signal handling days. Change gentraceback to take a flag that specifies whether we are tracing a trap. Change traceback to default to "starting with a return PC", which is the overwhelmingly common case. Add tracebacktrap, like traceback but starting with a trap PC. Use tracebacktrap in signal handlers. Fixes issue 7690. LGTM=iant, r R=r, iant CC=golang-codereviews https://codereview.appspot.com/167810044
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions