diff options
| author | Austin Clements <austin@google.com> | 2017-02-09 14:03:49 -0500 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2017-02-14 15:52:54 +0000 |
| commit | d089a6c7187f1ff85277515405ec6c641588a7ff (patch) | |
| tree | 2eb4d16038ec3ade478a83249cc879898efd5b8b /src/runtime/trace.go | |
| parent | c5ebcd2c8ac6c5bdf85ec0a346974efd4b0cbe49 (diff) | |
| download | go-d089a6c7187f1ff85277515405ec6c641588a7ff.tar.gz go-d089a6c7187f1ff85277515405ec6c641588a7ff.zip | |
runtime: remove stack barriers
Now that we don't rescan stacks, stack barriers are unnecessary. This
removes all of the code and structures supporting them as well as
tests that were specifically for stack barriers.
Updates #17503.
Change-Id: Ia29221730e0f2bbe7beab4fa757f31a032d9690c
Reviewed-on: https://go-review.googlesource.com/36620
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/trace.go')
| -rw-r--r-- | src/runtime/trace.go | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/runtime/trace.go b/src/runtime/trace.go index 10cf4711fc..fa5e422b0c 100644 --- a/src/runtime/trace.go +++ b/src/runtime/trace.go @@ -572,12 +572,7 @@ func traceStackID(mp *m, buf []uintptr, skip int) uint64 { nstk = callers(skip+1, buf[:]) } else if gp != nil { gp = mp.curg - // This may happen when tracing a system call, - // so we must lock the stack. - if gcTryLockStackBarriers(gp) { - nstk = gcallers(gp, skip, buf[:]) - gcUnlockStackBarriers(gp) - } + nstk = gcallers(gp, skip, buf[:]) } if nstk > 0 { nstk-- // skip runtime.goexit |