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 | e5e547c71f72722ca5fdd8ee67cf75f99ee586cf (patch) | |
| tree | a04c85fa85a5e6b3ab7d02c7fed05437928ad36a /src/pkg/runtime | |
| parent | 14e8885c3237eae03b125316cccea3973264f8d0 (diff) | |
| download | go-e5e547c71f72722ca5fdd8ee67cf75f99ee586cf.tar.xz | |
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 #8153.
LGTM=minux
R=golang-codereviews, minux
CC=golang-codereviews, iant, r
https://golang.org/cl/118670043
Diffstat (limited to 'src/pkg/runtime')
| -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 757c1c39a3..778d95a28b 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"); + } } } |