diff options
| author | Ian Lance Taylor <iant@golang.org> | 2017-02-03 21:13:30 -0800 |
|---|---|---|
| committer | Ian Lance Taylor <iant@golang.org> | 2017-02-08 04:14:41 +0000 |
| commit | 87ad863f359de3760578acb7f7a4d7e333c9cdc8 (patch) | |
| tree | b586db826b070743c97ba29bd4ddf70cb1ac57c0 /src/runtime/runtime2.go | |
| parent | 14c2849c3ebe498971413ee5e8c9780fabc8578e (diff) | |
| download | go-87ad863f359de3760578acb7f7a4d7e333c9cdc8.tar.xz | |
runtime: use atomic ops for fwdSig, make sigtable immutable
The fwdSig array is accessed by the signal handler, which may run in
parallel with other threads manipulating it via the os/signal package.
Use atomic accesses to ensure that there are no problems.
Move the _SigHandling flag out of the sigtable array. This makes sigtable
immutable and safe to read from the signal handler.
Change-Id: Icfa407518c4ebe1da38580920ced764898dfc9ad
Reviewed-on: https://go-review.googlesource.com/36321
Run-TryBot: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/runtime/runtime2.go')
| -rw-r--r-- | src/runtime/runtime2.go | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/runtime/runtime2.go b/src/runtime/runtime2.go index 3b649761c9..61c8bd91b9 100644 --- a/src/runtime/runtime2.go +++ b/src/runtime/runtime2.go @@ -610,7 +610,6 @@ const ( _SigThrow // if signal.Notify doesn't take it, exit loudly _SigPanic // if the signal is from the kernel, panic _SigDefault // if the signal isn't explicitly requested, don't monitor it - _SigHandling // our signal handler is registered _SigGoExit // cause all runtime procs to exit (only used on Plan 9). _SigSetStack // add SA_ONSTACK to libc handler _SigUnblock // unblocked in minit |