]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: define darwin/arm64's address space as 33 bits
authorMichael Anthony Knyszek <mknyszek@google.com>
Wed, 6 Nov 2019 23:56:03 +0000 (23:56 +0000)
committerMichael Knyszek <mknyszek@google.com>
Thu, 7 Nov 2019 02:33:31 +0000 (02:33 +0000)
On iOS, the address space is not 48 bits as one might believe, since
it's arm64 hardware. In fact, all pointers are truncated to 33 bits, and
the OS only gives applications access to the range [1<<32, 2<<32).

While today this has no effect on the Go runtime, future changes which
care about address space size need this to be correct.

Updates #35112.

Change-Id: Id518a2298080f7e3d31cf7d909506a37748cc49a
Reviewed-on: https://go-review.googlesource.com/c/go/+/205758
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
src/runtime/malloc.go

index 854609220d3406beda7fce0084b85b58471cffad..1f82dbd12421b1f0834a7f839264f4a46ec09789 100644 (file)
@@ -206,8 +206,16 @@ const (
        // mips32 only has access to the low 2GB of virtual memory, so
        // we further limit it to 31 bits.
        //
+       // On darwin/arm64, although 64-bit pointers are presumably
+       // available, pointers are truncated to 33 bits. Furthermore,
+       // only the top 4 GiB of the address space are actually available
+       // to the application, but we allow the whole 33 bits anyway for
+       // simplicity.
+       // TODO(mknyszek): Consider limiting it to 32 bits and using
+       // arenaBaseOffset to offset into the top 4 GiB.
+       //
        // WebAssembly currently has a limit of 4GB linear memory.
-       heapAddrBits = (_64bit*(1-sys.GoarchWasm)*(1-sys.GoosAix))*48 + (1-_64bit+sys.GoarchWasm)*(32-(sys.GoarchMips+sys.GoarchMipsle)) + 60*sys.GoosAix
+       heapAddrBits = (_64bit*(1-sys.GoarchWasm)*(1-sys.GoosAix)*(1-sys.GoosDarwin*sys.GoarchArm64))*48 + (1-_64bit+sys.GoarchWasm)*(32-(sys.GoarchMips+sys.GoarchMipsle)) + 60*sys.GoosAix + 33*sys.GoosDarwin*sys.GoarchArm64
 
        // maxAlloc is the maximum size of an allocation. On 64-bit,
        // it's theoretically possible to allocate 1<<heapAddrBits bytes. On