Skip to content

Commit

Permalink
[release-branch.go1.21] runtime: increase g0 stack size in non-cgo case
Browse files Browse the repository at this point in the history
Currently, for non-cgo programs, the g0 stack size is 8 KiB on
most platforms. With PGO which could cause aggressive inlining in
the runtime, the runtime stack frames are larger and could
overflow the 8 KiB g0 stack. Increase it to 16 KiB. This is only
one per OS thread, so it shouldn't increase memory use much.

Updates golang#62120.
Updates golang#62489.
Fixes golang#62537.

Change-Id: I565b154517021f1fd849424dafc3f0f26a755cac
Reviewed-on: https://go-review.googlesource.com/c/go/+/526995
Reviewed-by: Michael Pratt <mpratt@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
(cherry picked from commit c6d550a)
Reviewed-on: https://go-review.googlesource.com/c/go/+/527055
  • Loading branch information
cherrymui authored and bradfitz committed Sep 25, 2023
1 parent 99dbeeb commit 6951745
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/runtime/proc.go
Expand Up @@ -1543,7 +1543,7 @@ func mstart0() {
// but is somewhat arbitrary.
size := gp.stack.hi
if size == 0 {
size = 8192 * sys.StackGuardMultiplier
size = 16384 * sys.StackGuardMultiplier
}
gp.stack.hi = uintptr(noescape(unsafe.Pointer(&size)))
gp.stack.lo = gp.stack.hi - size + 1024
Expand Down Expand Up @@ -1939,7 +1939,7 @@ func allocm(pp *p, fn func(), id int64) *m {
if iscgo || mStackIsSystemAllocated() {
mp.g0 = malg(-1)
} else {
mp.g0 = malg(8192 * sys.StackGuardMultiplier)
mp.g0 = malg(16384 * sys.StackGuardMultiplier)
}
mp.g0.m = mp

Expand Down

0 comments on commit 6951745

Please sign in to comment.