diff options
| author | Austin Clements <austin@google.com> | 2017-04-06 14:32:37 -0400 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2017-04-21 14:34:10 +0000 |
| commit | bb6309cd63b35a81a8527efaad58847a83039947 (patch) | |
| tree | 9c4e67937daf7addc7be023232e5b69526118a4b /src/runtime/sys_linux_ppc64x.s | |
| parent | fc7b83d192e9d0a70f4fd3f0692b54ee21b70312 (diff) | |
| download | go-bb6309cd63b35a81a8527efaad58847a83039947.tar.gz go-bb6309cd63b35a81a8527efaad58847a83039947.zip | |
runtime: inform arena placement using sbrk(0)
On 32-bit architectures (or if we fail to map a 64-bit-style arena),
we try to map the heap arena just above the end of the process image.
While we can accept any address, using lower addresses is preferable
because lower addresses cause us to map less of the heap bitmap.
However, if a program is linked against C code that has global
constructors, those constructors may call brk/sbrk to allocate memory
(e.g., many C malloc implementations do this for small allocations).
The brk also starts just above the process image, so this may adjust
the brk past the beginning of where we want to put the heap arena. In
this case, the kernel will pick a different address for the arena and
it will usually be very high (at least, as these things go in a 32-bit
address space).
Fix this by consulting the current value of the brk and using this in
addition to the end of the process image to compute the initial arena
placement.
This is implemented only on Linux currently, since we have no evidence
that it's an issue on any other OSes.
Fixes #19831.
Change-Id: Id64b45d08d8c91e4f50d92d0339146250b04f2f8
Reviewed-on: https://go-review.googlesource.com/39810
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/runtime/sys_linux_ppc64x.s')
| -rw-r--r-- | src/runtime/sys_linux_ppc64x.s | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/runtime/sys_linux_ppc64x.s b/src/runtime/sys_linux_ppc64x.s index ef7dab21b7..2b2aa61d06 100644 --- a/src/runtime/sys_linux_ppc64x.s +++ b/src/runtime/sys_linux_ppc64x.s @@ -21,6 +21,7 @@ #define SYS_close 6 #define SYS_getpid 20 #define SYS_kill 37 +#define SYS_brk 45 #define SYS_fcntl 55 #define SYS_gettimeofday 78 #define SYS_select 82 // always return -ENOSYS @@ -422,3 +423,11 @@ TEXT runtime·closeonexec(SB),NOSPLIT|NOFRAME,$0 MOVD $1, R5 // FD_CLOEXEC SYSCALL $SYS_fcntl RET + +// func sbrk0() uintptr +TEXT runtime·sbrk0(SB),NOSPLIT|NOFRAME,$0 + // Implemented as brk(NULL). + MOVD $0, R3 + SYSCALL $SYS_brk + MOVD R3, ret+0(FP) + RET |