qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-24 12:00:14 -07:00
Code Releases Activity

runtime: fix crash on Plan 9

Browse Source 
Since CL 46037, the runtime is crashing after calling
exitThread on Plan 9.

The exitThread function shouldn't be called on
Plan 9, because the system manages thread stacks.

Fixes #22221.

Change-Id: I5d61c9660a87dc27e4cfcb3ca3ddcb4b752f2397
Reviewed-on: https://go-review.googlesource.com/70190
Run-TryBot: David du Colombier <0intro@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
This commit is contained in:
David du Colombier 2017-10-12 01:39:46 +02:00 committed by Austin Clements
parent a509cae90d
commit 926373ea79
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/runtime/proc.go
View File

@ -1170,10 +1170,10 @@ func mstart() {
mstart1(0) mstart1(0)
// Exit this thread. // Exit this thread.
if GOOS == "windows" || GOOS == "solaris" { if GOOS == "windows" || GOOS == "solaris" || GOOS == "plan9" {
// Windows and Solaris always system-allocate the // Window, Solaris and Plan 9 always system-allocate
// stack, but put it in _g_.stack before mstart, so // the stack, but put it in _g_.stack before mstart,
// the logic above hasn't set osStack yet. // so the logic above hasn't set osStack yet.
osStack = true osStack = true
} }
mexit(osStack) mexit(osStack)