mirror of
https://github.com/golang/go.git
synced 2025-12-08 06:10:04 +00:00
runtime: clean up system calls during cgo callback init
During a cgocallback, the runtime calls needm to get an m. The calls made during needm cannot themselves assume that there is an m or a g (which is attached to the m). In the old days of making direct system calls, the only thing you had to do for such functions was mark them //go:nosplit, to avoid the use of g in the stack split prologue. But now, on operating systems that make system calls through shared libraries and use code that saves state in the g or m before doing so, it's not safe to assume g exists. In fact, it is not even safe to call getg(), because it might fault deferencing the TLS storage to find the g pointer (that storage may not be initialized yet, at least on Windows, and perhaps on other systems in the future). The specific routines that are problematic are usleep and osyield, which are called during lock contention in lockextra, called from needm. All this is rather subtle and hidden, so in addition to fixing the problem on Windows, this CL makes the fact of not running on a g much clearer by introducing variants usleep_no_g and osyield_no_g whose names should make clear that there is no g. And then we can remove the various sketchy getg() == nil checks in the existing routines. As part of this cleanup, this CL also deletes onosstack on Windows. onosstack is from back when the runtime was implemented in C. It predates systemstack but does essentially the same thing. Instead of having two different copies of this code, we can use systemstack consistently. This way we need not port onosstack to each architecture. This CL is part of a stack adding windows/arm64 support (#36439), intended to land in the Go 1.17 cycle. This CL is, however, not windows/arm64-specific. It is cleanup meant to make the port (and future ports) easier. Change-Id: I3352de1fd0a3c26267c6e209063e6e86abd26187 Reviewed-on: https://go-review.googlesource.com/c/go/+/288793 Trust: Russ Cox <rsc@golang.org> Trust: Jason A. Donenfeld <Jason@zx2c4.com> Reviewed-by: Cherry Zhang <cherryyz@google.com> Reviewed-by: Jason A. Donenfeld <Jason@zx2c4.com>
This commit is contained in:
parent
e7ee3c1fa8
commit
0d94f989d1
30 changed files with 256 additions and 258 deletions
|
|
@ -521,6 +521,11 @@ func sysconf(name int32) int64 {
|
|||
|
||||
func usleep1(usec uint32)
|
||||
|
||||
//go:nosplit
|
||||
func usleep_no_g(µs uint32) {
|
||||
usleep1(µs)
|
||||
}
|
||||
|
||||
//go:nosplit
|
||||
func usleep(µs uint32) {
|
||||
usleep1(µs)
|
||||
|
|
@ -569,18 +574,15 @@ func setNonblock(fd int32) {
|
|||
func osyield1()
|
||||
|
||||
//go:nosplit
|
||||
func osyield() {
|
||||
_g_ := getg()
|
||||
|
||||
// Check the validity of m because we might be called in cgo callback
|
||||
// path early enough where there isn't a m available yet.
|
||||
if _g_ != nil && _g_.m != nil {
|
||||
sysvicall0(&libc_sched_yield)
|
||||
return
|
||||
}
|
||||
func osyield_no_g() {
|
||||
osyield1()
|
||||
}
|
||||
|
||||
//go:nosplit
|
||||
func osyield() {
|
||||
sysvicall0(&libc_sched_yield)
|
||||
}
|
||||
|
||||
//go:linkname executablePath os.executablePath
|
||||
var executablePath string
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue