]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.14] runtime: set g to gsignal before adjustSignalStack
authorCherry Zhang <cherryyz@google.com>
Mon, 15 Jun 2020 20:20:53 +0000 (16:20 -0400)
committerIan Lance Taylor <iant@golang.org>
Thu, 15 Oct 2020 18:53:06 +0000 (18:53 +0000)
commitfa44af7df1a96235fca2ec10c3129a4d2711ee28
tree6189dc385a92c1a7056acec8842a3a8ade410823
parentb5a3989dac97270b89cfce250cbb42695647d5cb
[release-branch.go1.14] runtime: set g to gsignal before adjustSignalStack

When a signal is received, the runtime probes whether an
alternate signal stack is set, if so, adjust gsignal's stack to
point to the alternate signal stack. This is done in
adjustSignalStack, which calls sigaltstack "syscall", which is a
libc call on darwin through asmcgocall. asmcgocall decides
whether to do stack switch based on whether we're running on g0
stack, gsignal stack, or regular g stack. If g is not set to
gsignal, asmcgocall may make wrong decision. Set g first.
adjustSignalStack is recursively nosplit, so it is okay that
temporarily gsignal.stack doesn't match the stack we're running
on.

Updates #39079.
Fixes #41991.

Change-Id: I59b2c5dc08c3c951f1098fff038bf2e06d7ca055
Reviewed-on: https://go-review.googlesource.com/c/go/+/238020
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
(cherry picked from commit d286e61b6787fe2b55bf0ec8a814962ebda8d202)
Reviewed-on: https://go-review.googlesource.com/c/go/+/262557
Trust: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
src/runtime/signal_unix.go