Commit b03f1d1a authored by David Crawshaw's avatar David Crawshaw Committed by Brad Fitzpatrick

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: 's avatarAustin Clements <austin@google.com>
parent ebb77aa8
...@@ -2824,10 +2824,6 @@ func reentersyscall(pc, sp uintptr) { ...@@ -2824,10 +2824,6 @@ func reentersyscall(pc, sp uintptr) {
save(pc, sp) 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-- _g_.m.locks--
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment