]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: rename "arena index" to "arena map"
authorAustin Clements <austin@google.com>
Thu, 22 Feb 2018 17:35:30 +0000 (12:35 -0500)
committerAustin Clements <austin@google.com>
Fri, 23 Feb 2018 21:59:48 +0000 (21:59 +0000)
There are too many places where I want to talk about "indexing into
the arena index". Make this less awkward and ambiguous by calling it
the "arena map" instead.

Change-Id: I726b0667bb2139dbc006175a0ec09a871cdf73f9
Reviewed-on: https://go-review.googlesource.com/96777
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Rick Hudson <rlh@golang.org>
src/runtime/malloc.go
src/runtime/mbitmap.go
src/runtime/mheap.go

index 6c36e1eea36bce13c97f25a7c4ff4c63f73a8db8..6f78455c8b90744be7246e315d5b906c32d47add 100644 (file)
 // themselves allocated off-heap.
 //
 // Since arenas are aligned, the address space can be viewed as a
-// series of arena frames. The arena index (mheap_.arenas) maps from
+// series of arena frames. The arena map (mheap_.arenas) maps from
 // arena frame number to *heapArena, or nil for parts of the address
 // space not backed by the Go heap. Since arenas are large, the arena
 // index is just a single-level mapping.
 //
-// The arena index covers the entire possible address space, allowing
+// The arena map covers the entire possible address space, allowing
 // the Go heap to use any part of the address space. The allocator
 // attempts to keep arenas contiguous so that large spans (and hence
 // large objects) can cross arenas.
@@ -203,10 +203,10 @@ const (
        // mips32 only has access to the low 2GB of virtual memory, so
        // we further limit it to 31 bits.
        //
-       // The size of the arena index is proportional to
+       // The size of the arena map is proportional to
        // 1<<heapAddrBits, so it's important that this not be too
        // large. 48 bits is about the threshold; above that we would
-       // need to go to a two level arena index.
+       // need to go to a two level arena map.
        heapAddrBits = _64bit*48 + (1-_64bit)*(32-(sys.GoarchMips+sys.GoarchMipsle))
 
        // maxAlloc is the maximum size of an allocation. On 64-bit,
@@ -227,7 +227,7 @@ const (
        pagesPerArena = heapArenaBytes / pageSize
 
        // arenaBaseOffset is the pointer value that corresponds to
-       // index 0 in the heap arena index.
+       // index 0 in the heap arena map.
        //
        // On amd64, the address space is 48 bits, sign extended to 64
        // bits. This offset lets us handle "negative" addresses (or
@@ -323,10 +323,10 @@ func mallocinit() {
                throw("bad system page size")
        }
 
-       // Map the arena index. Most of this will never be written to,
+       // Map the arena map. Most of this will never be written to,
        mheap_.arenas = (*[(1 << heapAddrBits) / heapArenaBytes]*heapArena)(persistentalloc(unsafe.Sizeof(*mheap_.arenas), sys.PtrSize, nil))
        if mheap_.arenas == nil {
-               throw("failed to allocate arena index")
+               throw("failed to allocate arena map")
        }
 
        // Initialize the heap.
index 1b17740f4959ec6c71c370b7eb35e03181f672e9..85d79c685b73d6943dd22a2324ab2e3ed0d711f5 100644 (file)
@@ -146,7 +146,7 @@ type heapBits struct {
 }
 
 // Make the compiler check that heapBits.arena is large enough to hold
-// the maximum arena index.
+// the maximum arena frame number.
 var _ = heapBits{arena: (1<<heapAddrBits)/heapArenaBytes - 1}
 
 // markBits provides access to the mark bit for an object in the heap.
index b529f086f4390c8a4ab08da51814307a7c2a3f4e..3460c54d72c9ecdb6d2d3fab339b43e1403523f6 100644 (file)
@@ -96,14 +96,14 @@ type mheap struct {
        nlargefree  uint64                  // number of frees for large objects (>maxsmallsize)
        nsmallfree  [_NumSizeClasses]uint64 // number of frees for small objects (<=maxsmallsize)
 
-       // arenas is the heap arena index.
+       // arenas is the heap arena map.
        // arenas[(va+arenaBaseOffset)/heapArenaBytes] points to the
        // metadata for the heap arena containing va.
        //
        // Use arenaIndex to compute indexes into this array.
        //
        // For regions of the address space that are not backed by the
-       // Go heap, the arena index contains nil.
+       // Go heap, the arena map contains nil.
        //
        // Modifications are protected by mheap_.lock. Reads can be
        // performed without locking; however, a given entry can