diff options
| author | Austin Clements <austin@google.com> | 2018-02-20 11:59:02 -0500 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2018-02-21 20:32:36 +0000 |
| commit | ea8d7a370d66550d587414cc0cab650f35400f94 (patch) | |
| tree | 4eb1a643759747684701012db1b6fd7d556e2dea /src/runtime/stack.go | |
| parent | ed1959c6e6f6321383efd52dc250480a4aab40f6 (diff) | |
| download | go-ea8d7a370d66550d587414cc0cab650f35400f94.tar.xz | |
runtime: clarify address space limit constants and comments
Now that we support the full non-contiguous virtual address space of
amd64 hardware, some of the comments and constants related to this are
out of date.
This renames memLimitBits to heapAddrBits because 1<<memLimitBits is
no longer the limit of the address space and rewrites the comment to
focus first on hardware limits (which span OSes) and then discuss
kernel limits.
Second, this eliminates the memLimit constant because there's no
longer a meaningful "highest possible heap pointer value" on amd64.
Updates #23862.
Change-Id: I44b32033d2deb6b69248fb8dda14fc0e65c47f11
Reviewed-on: https://go-review.googlesource.com/95498
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Rick Hudson <rlh@golang.org>
Diffstat (limited to 'src/runtime/stack.go')
| -rw-r--r-- | src/runtime/stack.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/stack.go b/src/runtime/stack.go index 029bff5af4..b5dda0d9e6 100644 --- a/src/runtime/stack.go +++ b/src/runtime/stack.go @@ -144,7 +144,7 @@ var stackpoolmu mutex // Global pool of large stack spans. var stackLarge struct { lock mutex - free [memLimitBits - pageShift]mSpanList // free lists by log_2(s.npages) + free [heapAddrBits - pageShift]mSpanList // free lists by log_2(s.npages) } func stackinit() { |