diff options
| author | David Crawshaw <crawshaw@golang.org> | 2017-09-24 10:48:18 -0400 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@golang.org> | 2018-02-14 15:27:11 +0000 |
| commit | b03f1d1a7e1c84d1fa7e4bcf8c705dfeb1a4565d (patch) | |
| tree | 68f4eb7a60998a8895ed3d11d35b06f670ccf007 /src/runtime/proc.go | |
| parent | ebb77aa867b4dddd63fc397a907ac93020480723 (diff) | |
| download | go-b03f1d1a7e1c84d1fa7e4bcf8c705dfeb1a4565d.tar.xz | |
runtime: remove extraneous stackPreempt setting
The stackguard is set to stackPreempt earlier in reentersyscall, and
as it comes with throwsplit = true there's no way for the stackguard
to be set to anything else by the end of reentersyscall.
Change-Id: I4e942005b22ac784c52398c74093ac887fc8ec24
Reviewed-on: https://go-review.googlesource.com/65673
Run-TryBot: David Crawshaw <crawshaw@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
Diffstat (limited to 'src/runtime/proc.go')
| -rw-r--r-- | src/runtime/proc.go | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/runtime/proc.go b/src/runtime/proc.go index 2e958f7fc5..f20e77eee5 100644 --- a/src/runtime/proc.go +++ b/src/runtime/proc.go @@ -2824,10 +2824,6 @@ func reentersyscall(pc, sp uintptr) { save(pc, sp) } - // Goroutines must not split stacks in Gsyscall status (it would corrupt g->sched). - // We set _StackGuard to StackPreempt so that first split stack check calls morestack. - // Morestack detects this case and throws. - _g_.stackguard0 = stackPreempt _g_.m.locks-- } |