diff options
| author | Russ Cox <rsc@golang.org> | 2014-08-06 14:50:09 -0400 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2014-08-06 14:50:09 -0400 |
| commit | 75fb1ca6bd4703784106379e72fc1a527ced7816 (patch) | |
| tree | 7502a140443040cf5e803e456f1ee358562214c8 /src | |
| parent | fa1b7e0aa2db2ac548824595fe7b88f455d2bcfa (diff) | |
| download | go-75fb1ca6bd4703784106379e72fc1a527ced7816.tar.gz | |
runtime: turn off 'unexpected return pc' print on arm traceback
It can happen legitimately if a profiling signal arrives at just the wrong moment.
It's harmless.
Fixes issue 8153.
LGTM=minux
R=golang-codereviews, minux
CC=golang-codereviews, iant, r
https://codereview.appspot.com/118670043
Diffstat (limited to 'src')
| -rw-r--r-- | src/pkg/runtime/traceback_arm.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/pkg/runtime/traceback_arm.c b/src/pkg/runtime/traceback_arm.c index 757c1c39a..778d95a28 100644 --- a/src/pkg/runtime/traceback_arm.c +++ b/src/pkg/runtime/traceback_arm.c @@ -128,9 +128,14 @@ runtime·gentraceback(uintptr pc0, uintptr sp0, uintptr lr0, G *gp, int32 skip, frame.lr = *(uintptr*)frame.sp; flr = runtime·findfunc(frame.lr); if(flr == nil) { - runtime·printf("runtime: unexpected return pc for %s called from %p\n", runtime·funcname(f), frame.lr); - if(callback != nil) + // This happens if you get a profiling interrupt at just the wrong time. + // In that context it is okay to stop early. + // But if callback is set, we're doing a garbage collection and must + // get everything, so crash loudly. + if(callback != nil) { + runtime·printf("runtime: unexpected return pc for %s called from %p\n", runtime·funcname(f), frame.lr); runtime·throw("unknown caller pc"); + } } } |