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/mbitmap.go | |
| parent | ed1959c6e6f6321383efd52dc250480a4aab40f6 (diff) | |
| download | go-ea8d7a370d66550d587414cc0cab650f35400f94.tar.gz go-ea8d7a370d66550d587414cc0cab650f35400f94.zip | |
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/mbitmap.go')
| -rw-r--r-- | src/runtime/mbitmap.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/mbitmap.go b/src/runtime/mbitmap.go index baae3d911b..1b17740f49 100644 --- a/src/runtime/mbitmap.go +++ b/src/runtime/mbitmap.go @@ -147,7 +147,7 @@ type heapBits struct { // Make the compiler check that heapBits.arena is large enough to hold // the maximum arena index. -var _ = heapBits{arena: memLimit / heapArenaBytes} +var _ = heapBits{arena: (1<<heapAddrBits)/heapArenaBytes - 1} // markBits provides access to the mark bit for an object in the heap. // bytep points to the byte holding the mark bit. |