diff options
author | Joel Sing <joel@sing.id.au> | 2020-04-16 15:33:53 +1000 |
---|---|---|
committer | Joel Sing <joel@sing.id.au> | 2020-04-16 15:31:20 +0000 |
commit | 4eaf855155acb2da69adbab7728c1a59b9315d41 (patch) | |
tree | 6e5ea821a812d487465b7449f3e94eb221467f57 /src/runtime/signal_unix.go | |
parent | 2a029b3f26169be7c89cb2cdcc3db4b5d097a8b8 (diff) | |
download | go-4eaf855155acb2da69adbab7728c1a59b9315d41.tar.gz go-4eaf855155acb2da69adbab7728c1a59b9315d41.zip |
runtime: clean up now unused pushCallSupported
All platforms now support pushCall, hence remove the now unnecessary
pushCallSupported flag/guard.
Change-Id: I99e4be73839da68a742f3c239bae9ce2f8764624
Reviewed-on: https://go-review.googlesource.com/c/go/+/228497
Run-TryBot: Joel Sing <joel@sing.id.au>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'src/runtime/signal_unix.go')
-rw-r--r-- | src/runtime/signal_unix.go | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/runtime/signal_unix.go b/src/runtime/signal_unix.go index 5dbfbaf4b1..c33f88b046 100644 --- a/src/runtime/signal_unix.go +++ b/src/runtime/signal_unix.go @@ -336,7 +336,7 @@ func doSigPreempt(gp *g, ctxt *sigctxt) { atomic.Store(&gp.m.signalPending, 0) } -const preemptMSupported = pushCallSupported +const preemptMSupported = true // preemptM sends a preemption request to mp. This request may be // handled asynchronously and may be coalesced with other requests to @@ -345,11 +345,6 @@ const preemptMSupported = pushCallSupported // safe-point, it will preempt the goroutine. It always atomically // increments mp.preemptGen after handling a preemption request. func preemptM(mp *m) { - if !pushCallSupported { - // This architecture doesn't support ctxt.pushCall - // yet, so doSigPreempt won't work. - return - } if GOOS == "darwin" && GOARCH == "arm64" && !iscgo { // On darwin, we use libc calls, and cgo is required on ARM64 // so we have TLS set up to save/restore G during C calls. If cgo is |