]> Cypherpunks repositories - gostls13.git/commit
cmd/internal/ld, runtime: unify stack reservation in PE header and runtime
authorShenghou Ma <minux@golang.org>
Wed, 29 Apr 2015 03:02:49 +0000 (23:02 -0400)
committerMinux Ma <minux@golang.org>
Wed, 29 Apr 2015 03:27:10 +0000 (03:27 +0000)
commitda11a9dda32e3588250b29862baa5cea25b8b67b
tree22c2391da57e4039d41eb9fcdd9f4d07b8adb095
parentedac5d91445c6d29b2a83d5396654d6038564463
cmd/internal/ld, runtime: unify stack reservation in PE header and runtime

With 128KB stack reservation, on 32-bit Windows, the maximum number
threads is ~9000.

The original 65535-byte stack commit is causing problem on Windows
XP where it makes the stack reservation to be 1MB despite the fact
that the runtime specified 128KB.

While we're at here, also fix the extra spacings in the unable to
create more OS thread error message: println will insert a space
between each argument.

See #9457 for more information.

Change-Id: I3a82f7d9717d3d55211b6eb1c34b00b0eaad83ed
Reviewed-on: https://go-review.googlesource.com/2237
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
Run-TryBot: Minux Ma <minux@golang.org>
src/cmd/internal/ld/pe.go
src/runtime/os1_windows.go