]> Cypherpunks repositories - gostls13.git/commitdiff
Revert "runtime: process ptr bitmaps one word at a time"
authorKeith Randall <khr@google.com>
Tue, 9 Aug 2022 15:44:16 +0000 (15:44 +0000)
committerKeith Randall <khr@google.com>
Tue, 9 Aug 2022 16:09:24 +0000 (16:09 +0000)
This reverts commit c3833a55433f4b2981253f64444fe5c3d1bc910a.

Reason for revert: Bug somewhere in this code, causing wasm and maybe linux/386 to fail.

Change-Id: I05f7cfa467598ca0c2c84fd4f752cc4ef117cc51
Reviewed-on: https://go-review.googlesource.com/c/go/+/422394
Run-TryBot: Keith Randall <khr@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Than McIntosh <thanm@google.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
src/cmd/compile/internal/reflectdata/reflect.go
src/reflect/type.go
src/runtime/mbitmap.go

index 5833c71f4c5347940ef086f5bfdbbb6d221881f5..83d66eab421f5f5b62229bde28bb1f80c45c5079 100644 (file)
@@ -1549,11 +1549,7 @@ func dgcsym(t *types.Type, write bool) (lsym *obj.LSym, useGCProg bool, ptrdata
 
 // dgcptrmask emits and returns the symbol containing a pointer mask for type t.
 func dgcptrmask(t *types.Type, write bool) *obj.LSym {
-       // Bytes we need for the ptrmask.
-       n := (types.PtrDataSize(t)/int64(types.PtrSize) + 7) / 8
-       // Runtime wants ptrmasks padded to a multiple of uintptr in size.
-       n = (n + int64(types.PtrSize) - 1) &^ (int64(types.PtrSize) - 1)
-       ptrmask := make([]byte, n)
+       ptrmask := make([]byte, (types.PtrDataSize(t)/int64(types.PtrSize)+7)/8)
        fillptrmask(t, ptrmask)
        p := fmt.Sprintf("runtime.gcbits.%x", ptrmask)
 
index cb657905d0b9fce44dd7c6dd5d35350281edab64..f9365c473461373d00a1854405b59239a3396e6b 100644 (file)
@@ -2271,10 +2271,7 @@ func bucketOf(ktyp, etyp *rtype) *rtype {
 
        if ktyp.ptrdata != 0 || etyp.ptrdata != 0 {
                nptr := (bucketSize*(1+ktyp.size+etyp.size) + goarch.PtrSize) / goarch.PtrSize
-               n := (nptr + 7) / 8
-               // Runtime needs pointer masks to be a multiple of uintptr in size.
-               n = (n + goarch.PtrSize - 1) &^ (goarch.PtrSize - 1)
-               mask := make([]byte, n)
+               mask := make([]byte, (nptr+7)/8)
                base := bucketSize / goarch.PtrSize
 
                if ktyp.ptrdata != 0 {
@@ -2980,10 +2977,7 @@ func ArrayOf(length int, elem Type) Type {
                // Element is small with pointer mask; array is still small.
                // Create direct pointer mask by turning each 1 bit in elem
                // into length 1 bits in larger mask.
-               n := (array.ptrdata/goarch.PtrSize + 7) / 8
-               // Runtime needs pointer masks to be a multiple of uintptr in size.
-               n = (n + goarch.PtrSize - 1) &^ (goarch.PtrSize - 1)
-               mask := make([]byte, n)
+               mask := make([]byte, (array.ptrdata/goarch.PtrSize+7)/8)
                emitGCMask(mask, 0, typ, array.len)
                array.gcdata = &mask[0]
 
@@ -3152,13 +3146,8 @@ type bitVector struct {
 
 // append a bit to the bitmap.
 func (bv *bitVector) append(bit uint8) {
-       if bv.n%(8*goarch.PtrSize) == 0 {
-               // Runtime needs pointer masks to be a multiple of uintptr in size.
-               // Since reflect passes bv.data directly to the runtime as a pointer mask,
-               // we append a full uintptr of zeros at a time.
-               for i := 0; i < goarch.PtrSize; i++ {
-                       bv.data = append(bv.data, 0)
-               }
+       if bv.n%8 == 0 {
+               bv.data = append(bv.data, 0)
        }
        bv.data[bv.n/8] |= bit << (bv.n % 8)
        bv.n++
index d45494992627611583583c61f089acc9b76e37da..1c7ae8a68e39db053efe62f7a429a1399cd986ae 100644 (file)
@@ -893,19 +893,6 @@ func (h writeHeapBits) flush(addr, size uintptr) {
        }
 }
 
-// Read the bytes starting at the aligned pointer p into a uintptr.
-// Read is little-endian.
-func readUintptr(p *byte) uintptr {
-       x := *(*uintptr)(unsafe.Pointer(p))
-       if goarch.BigEndian {
-               if goarch.PtrSize == 8 {
-                       return uintptr(sys.Bswap64(uint64(x)))
-               }
-               return uintptr(sys.Bswap32(uint32(x)))
-       }
-       return x
-}
-
 // heapBitsSetType records that the new allocation [x, x+size)
 // holds in [x, x+dataSize) one or more values of type typ.
 // (The number of values is given by dataSize / typ.size.)
@@ -930,7 +917,7 @@ func readUintptr(p *byte) uintptr {
 // machines, callers must execute a store/store (publication) barrier
 // between calling this function and making the object reachable.
 func heapBitsSetType(x, size, dataSize uintptr, typ *_type) {
-       const doubleCheck = false // slow but helpful; enable to test modifications to this code
+       const doubleCheck = true // slow but helpful; enable to test modifications to this code
 
        if doubleCheck && dataSize%typ.size != 0 {
                throw("heapBitsSetType: dataSize not a multiple of typ.size")
@@ -1008,65 +995,19 @@ func heapBitsSetType(x, size, dataSize uintptr, typ *_type) {
        // objects with scalar tails, all but the last tail does have to
        // be initialized, because there is no way to say "skip forward".
 
-       ptrs := typ.ptrdata / goarch.PtrSize
-       if typ.size == dataSize { // Single element
-               if ptrs <= ptrBits { // Single small element
-                       m := readUintptr(typ.gcdata)
-                       h = h.write(m, ptrs)
-               } else { // Single large element
-                       p := typ.gcdata
-                       for {
-                               h = h.write(readUintptr(p), ptrBits)
-                               p = addb(p, ptrBits/8)
-                               ptrs -= ptrBits
-                               if ptrs <= ptrBits {
-                                       break
-                               }
-                       }
-                       m := readUintptr(p)
-                       h = h.write(m, ptrs)
+       for i := uintptr(0); true; i += typ.size {
+               p := typ.gcdata
+               var j uintptr
+               for j = 0; j+8*goarch.PtrSize < typ.ptrdata; j += 8 * goarch.PtrSize {
+                       h = h.write(uintptr(*p), 8)
+                       p = add1(p)
                }
-       } else { // Repeated element
-               words := typ.size / goarch.PtrSize // total words, including scalar tail
-               if words <= ptrBits {              // Repeated small element
-                       n := dataSize / typ.size
-                       m := readUintptr(typ.gcdata)
-                       // Make larger unit to repeat
-                       for words <= ptrBits/2 {
-                               if n&1 != 0 {
-                                       h = h.write(m, words)
-                               }
-                               n /= 2
-                               m |= m << words
-                               ptrs += words
-                               words *= 2
-                               if n == 1 {
-                                       break
-                               }
-                       }
-                       for n > 1 {
-                               h = h.write(m, words)
-                               n--
-                       }
-                       h = h.write(m, ptrs)
-               } else { // Repeated large element
-                       for i := uintptr(0); true; i += typ.size {
-                               p := typ.gcdata
-                               j := ptrs
-                               for j > ptrBits {
-                                       h = h.write(readUintptr(p), ptrBits)
-                                       p = addb(p, ptrBits/8)
-                                       j -= ptrBits
-                               }
-                               m := readUintptr(p)
-                               h = h.write(m, j)
-                               if i+typ.size == dataSize {
-                                       break // don't need the trailing nonptr bits on the last element.
-                               }
-                               // Pad with zeros to the start of the next element.
-                               h = h.pad(typ.size - typ.ptrdata)
-                       }
+               h = h.write(uintptr(*p), (typ.ptrdata-j)/goarch.PtrSize)
+               if i+typ.size == dataSize {
+                       break // don't need the trailing nonptr bits on the last element.
                }
+               // Pad with zeros to the start of the next element.
+               h = h.pad(typ.size - typ.ptrdata)
        }
        h.flush(x, size)