diff options
author | Björn-Egil Dahlberg <[email protected]> | 2010-07-21 15:42:42 +0200 |
---|---|---|
committer | Björn-Egil Dahlberg <[email protected]> | 2010-07-22 18:00:21 +0200 |
commit | cc28713df8cdcc83d9df38e114d51708ea633ed8 (patch) | |
tree | 54f1cbad504f964fb46e3fb6569c52a7eea9fa1f | |
parent | 9267b2dc792c0a7632a0dcdc0a49510eb13e465e (diff) | |
download | otp-cc28713df8cdcc83d9df38e114d51708ea633ed8.tar.gz otp-cc28713df8cdcc83d9df38e114d51708ea633ed8.tar.bz2 otp-cc28713df8cdcc83d9df38e114d51708ea633ed8.zip |
Fix wrapping in next vheap calculation
OTP #8730
-rw-r--r-- | erts/emulator/beam/erl_gc.c | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/erts/emulator/beam/erl_gc.c b/erts/emulator/beam/erl_gc.c index e9411389a0..30b29c646d 100644 --- a/erts/emulator/beam/erl_gc.c +++ b/erts/emulator/beam/erl_gc.c @@ -1996,18 +1996,27 @@ do_next_vheap_size(Uint vheap, Uint vheap_sz) { * * ---------------------- */ + Uint vheap_max = heap_sizes[num_heap_sizes - 1]; + if (vheap_sz == vheap_max) { + return vheap_sz; + } - if (vheap > (Uint) (vheap_sz*3/4)) { + if ((Uint) vheap/3 > (Uint) (vheap_sz/4)) { + Uint new_vheap_sz = vheap_sz; - while(vheap > (Uint) (vheap_sz*3/4)) { - vheap_sz = vheap_sz*2; + while((Uint) vheap/3 > (Uint) (vheap_sz/4)) { + new_vheap_sz = (vheap_sz << 1); + if (new_vheap_sz < vheap_sz || new_vheap_sz > vheap_max ) { + return vheap_max; + } + vheap_sz = new_vheap_sz; } return erts_next_heap_size(vheap_sz, 0); } if (vheap < (Uint) (vheap_sz/4)) { - return erts_next_heap_size((Uint) (vheap_sz / 2), 0); + return erts_next_heap_size((Uint) (vheap_sz >> 1), 0); } return vheap_sz; |