]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: drop _SigUnblock for SIGSYS on Linux
authorMikio Hara <mikioh.mikioh@gmail.com>
Thu, 28 Apr 2016 00:35:19 +0000 (09:35 +0900)
committerMikio Hara <mikioh.mikioh@gmail.com>
Thu, 28 Apr 2016 21:48:44 +0000 (21:48 +0000)
The _SigUnblock flag was appended to SIGSYS slot of runtime signal table
for Linux in https://go-review.googlesource.com/22202, but there is
still no concrete opinion on whether SIGSYS must be an unblocked signal
for runtime.

This change removes _SigUnblock flag from SIGSYS on Linux for
consistency in runtime signal handling and adds a reference to #15204 to
runtime signal table for FreeBSD.

Updates #15204.

Change-Id: I42992b1d852c2ab5dd37d6dbb481dba46929f665
Reviewed-on: https://go-review.googlesource.com/22537
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/runtime/signal_freebsd.go
src/runtime/sigtab_linux_generic.go
src/runtime/sigtab_linux_mips64x.go

index c8b09e92d9ac55b73735265abeb57f7e0479ae91..c4cb68720eb6ae3a4e84c2ca3693c9343fd8c007 100644 (file)
@@ -24,7 +24,7 @@ var sigtable = [...]sigTabT{
        /* 9 */ {0, "SIGKILL: kill"},
        /* 10 */ {_SigPanic + _SigUnblock, "SIGBUS: bus error"},
        /* 11 */ {_SigPanic + _SigUnblock, "SIGSEGV: segmentation violation"},
-       /* 12 */ {_SigNotify, "SIGSYS: bad system call"},
+       /* 12 */ {_SigNotify, "SIGSYS: bad system call"}, // see golang.org/issues/15204
        /* 13 */ {_SigNotify, "SIGPIPE: write to broken pipe"},
        /* 14 */ {_SigNotify, "SIGALRM: alarm clock"},
        /* 15 */ {_SigNotify + _SigKill, "SIGTERM: termination"},
index e97497f18c6c885a94c2aba8ed7174e01d859789..ea36bf3645254c4f76b7a1b6e7aee97ec05cc262 100644 (file)
@@ -45,7 +45,7 @@ var sigtable = [...]sigTabT{
        /* 28 */ {_SigNotify, "SIGWINCH: window size change"},
        /* 29 */ {_SigNotify, "SIGIO: i/o now possible"},
        /* 30 */ {_SigNotify, "SIGPWR: power failure restart"},
-       /* 31 */ {_SigThrow + _SigUnblock, "SIGSYS: bad system call"},
+       /* 31 */ {_SigThrow, "SIGSYS: bad system call"},
        /* 32 */ {_SigSetStack + _SigUnblock, "signal 32"}, /* SIGCANCEL; see issue 6997 */
        /* 33 */ {_SigSetStack + _SigUnblock, "signal 33"}, /* SIGSETXID; see issues 3871, 9400, 12498 */
        /* 34 */ {_SigNotify, "signal 34"},
index f7d81811bacd9b685f048c75f20506d1649e3cbf..201fe3deeb73513a4fd394c9968b6b7bae38af7f 100644 (file)
@@ -25,7 +25,7 @@ var sigtable = [...]sigTabT{
        /* 9 */ {0, "SIGKILL: kill"},
        /* 10 */ {_SigPanic + _SigUnblock, "SIGBUS: bus error"},
        /* 11 */ {_SigPanic + _SigUnblock, "SIGSEGV: segmentation violation"},
-       /* 12 */ {_SigThrow + _SigUnblock, "SIGSYS: bad system call"},
+       /* 12 */ {_SigThrow, "SIGSYS: bad system call"},
        /* 13 */ {_SigNotify, "SIGPIPE: write to broken pipe"},
        /* 14 */ {_SigNotify, "SIGALRM: alarm clock"},
        /* 15 */ {_SigNotify + _SigKill, "SIGTERM: termination"},