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/lfstack_64bit.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/lfstack_64bit.go')
| -rw-r--r-- | src/runtime/lfstack_64bit.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/lfstack_64bit.go b/src/runtime/lfstack_64bit.go index 19d8045203..9524d651a1 100644 --- a/src/runtime/lfstack_64bit.go +++ b/src/runtime/lfstack_64bit.go @@ -11,7 +11,7 @@ import "unsafe" const ( // addrBits is the number of bits needed to represent a virtual address. // - // See memLimitBits for a table of address space sizes on + // See heapAddrBits for a table of address space sizes on // various architectures. 48 bits is enough for all // architectures except s390x. // |