diff options
| author | Austin Clements <austin@google.com> | 2023-04-20 16:09:48 +0000 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2023-04-20 16:19:35 +0000 |
| commit | d11ff3f08155b7614485d9b555e97f7a9555ede5 (patch) | |
| tree | 6dbf61a63363f0ab8deb5074c550e10ff1666036 /src/runtime/stack.go | |
| parent | df777cfa15caccbec74b9f72d19af317e2870138 (diff) | |
| download | go-d11ff3f08155b7614485d9b555e97f7a9555ede5.tar.xz | |
Revert "runtime, cmd: rationalize StackLimit and StackGuard"
This reverts commit CL 486380.
Submitted out of order and breaks bootstrap.
Change-Id: I67bd225094b5c9713b97f70feba04d2c99b7da76
Reviewed-on: https://go-review.googlesource.com/c/go/+/486916
Reviewed-by: David Chase <drchase@google.com>
TryBot-Bypass: Austin Clements <austin@google.com>
Diffstat (limited to 'src/runtime/stack.go')
| -rw-r--r-- | src/runtime/stack.go | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/src/runtime/stack.go b/src/runtime/stack.go index 708a6ee2e5..39dbed5114 100644 --- a/src/runtime/stack.go +++ b/src/runtime/stack.go @@ -85,18 +85,19 @@ const ( _FixedStack6 = _FixedStack5 | (_FixedStack5 >> 16) _FixedStack = _FixedStack6 + 1 - // stackNosplit is the maximum number of bytes that a chain of NOSPLIT - // functions can use. - // This arithmetic must match that in cmd/internal/objabi/stack.go:StackNosplit. - stackNosplit = abi.StackNosplitBase * sys.StackGuardMultiplier - // The stack guard is a pointer this many bytes above the // bottom of the stack. // - // The guard leaves enough room for a stackNosplit chain of NOSPLIT calls - // plus one stackSmall frame plus stackSystem bytes for the OS. + // The guard leaves enough room for one _StackSmall frame plus + // a _StackLimit chain of NOSPLIT calls plus _StackSystem + // bytes for the OS. + // This arithmetic must match that in cmd/internal/objabi/stack.go:StackLimit. + _StackGuard = 928*sys.StackGuardMultiplier + _StackSystem + + // The maximum number of bytes that a chain of NOSPLIT + // functions can use. // This arithmetic must match that in cmd/internal/objabi/stack.go:StackLimit. - _StackGuard = stackNosplit + _StackSystem + abi.StackSmall + _StackLimit = _StackGuard - _StackSystem - abi.StackSmall ) const ( @@ -1210,7 +1211,7 @@ func shrinkstack(gp *g) { // down to the SP plus the stack guard space that ensures // there's room for nosplit functions. avail := gp.stack.hi - gp.stack.lo - if used := gp.stack.hi - gp.sched.sp + stackNosplit; used >= avail/4 { + if used := gp.stack.hi - gp.sched.sp + _StackLimit; used >= avail/4 { return } |
