diff options
author | Elias Naur <elias.naur@gmail.com> | 2015-11-17 11:41:06 +0100 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2015-11-24 17:16:47 +0000 |
commit | a7383fc4670947ffa513aae4ce2c8917d753da26 (patch) | |
tree | 9c2bff480ac2e430e4174043c11be55acb3505a0 /src/runtime/os_darwin.go | |
parent | 0ae57c3b0b95310dd86ac0e0251048136d5a2d99 (diff) | |
download | go-a7383fc4670947ffa513aae4ce2c8917d753da26.tar.gz go-a7383fc4670947ffa513aae4ce2c8917d753da26.zip |
runtime: use a proper type, sigset, for m.sigmask
Replace the cross platform but unsafe [4]uintptr type with a OS
specific type, sigset. Most OSes already define sigset, and this
change defines a suitable sigset for the OSes that don't (darwin,
openbsd). The OSes that don't use m.sigmask (windows, plan9, nacl)
now defines sigset as the empty type, struct{}.
The gain is strongly typed access to m.sigmask, saving a dynamic
size sanity check and unsafe.Pointer casting. Also, some storage is
saved for each M, since [4]uinptr was conservative for most OSes.
The cost is that OSes that don't need m.sigmask has to define sigset.
completes ./all.bash with GOOS linux, on amd64
completes ./make.bash with GOOSes openbsd, android, plan9, windows,
darwin, solaris, netbsd, freebsd, dragonfly, all amd64.
With GOOS=nacl ./make.bash failed with a seemingly unrelated error.
[Replay of CL 16942 by Elias Naur.]
Change-Id: I98f144d626033ae5318576115ed635415ac71b2c
Reviewed-on: https://go-review.googlesource.com/17033
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Russ Cox <rsc@golang.org>
Diffstat (limited to 'src/runtime/os_darwin.go')
-rw-r--r-- | src/runtime/os_darwin.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/os_darwin.go b/src/runtime/os_darwin.go index 7a70639b02..75a6eebb70 100644 --- a/src/runtime/os_darwin.go +++ b/src/runtime/os_darwin.go @@ -24,7 +24,7 @@ func mach_thread_self() uint32 func sysctl(mib *uint32, miblen uint32, out *byte, size *uintptr, dst *byte, ndst uintptr) int32 //go:noescape -func sigprocmask(how uint32, new, old *uint32) +func sigprocmask(how uint32, new, old *sigset) //go:noescape func sigaction(mode uint32, new, old *sigactiont) |