]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: remove remnants of signal stack workaround
authorHeisenberg <lziqiang1@gmail.com>
Mon, 31 Aug 2020 11:22:36 +0000 (19:22 +0800)
committerAustin Clements <austin@google.com>
Tue, 1 Sep 2020 13:23:55 +0000 (13:23 +0000)
Updates #35979

Change-Id: Ic3a6e1b5e9d544979a3c8d909a36a55efa3b9c9d
Reviewed-on: https://go-review.googlesource.com/c/go/+/251757
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
src/runtime/os_linux.go

index 9702920bcffd446db0507ef6b7125ef337e391d4..371db73502915d92d360b41713854e9fe1df7512 100644 (file)
@@ -5,7 +5,6 @@
 package runtime
 
 import (
-       "runtime/internal/atomic"
        "runtime/internal/sys"
        "unsafe"
 )
@@ -476,21 +475,7 @@ func rt_sigaction(sig uintptr, new, old *sigactiont, size uintptr) int32
 func getpid() int
 func tgkill(tgid, tid, sig int)
 
-// touchStackBeforeSignal stores an errno value. If non-zero, it means
-// that we should touch the signal stack before sending a signal.
-// This is used on systems that have a bug when the signal stack must
-// be faulted in.  See #35777 and #37436.
-//
-// This is accessed atomically as it is set and read in different threads.
-//
-// TODO(austin): Remove this after Go 1.15 when we remove the
-// mlockGsignal workaround.
-var touchStackBeforeSignal uint32
-
 // signalM sends a signal to mp.
 func signalM(mp *m, sig int) {
-       if atomic.Load(&touchStackBeforeSignal) != 0 {
-               atomic.Cas((*uint32)(unsafe.Pointer(mp.gsignal.stack.hi-4)), 0, 0)
-       }
        tgkill(getpid(), int(mp.procid), sig)
 }