# HG changeset patch # User brutisso # Date 1372319999 -7200 # Node ID c92b74c62d97803cf445933555cb5a5da6d7f202 # Parent 71963b3f802a04381b881f19589ac06cfb39a8c2 8017483: G1 tests fail with native OOME on Solaris x86 after HeapBaseMinAddress has been increased Summary: Set HeapBaseMinAddress as default rather than ergo Reviewed-by: stefank, jmasa, kvn diff -r 71963b3f802a -r c92b74c62d97 src/share/vm/runtime/arguments.cpp --- a/src/share/vm/runtime/arguments.cpp Wed Jun 26 16:58:37 2013 +0200 +++ b/src/share/vm/runtime/arguments.cpp Thu Jun 27 09:59:59 2013 +0200 @@ -1571,7 +1571,9 @@ // By default HeapBaseMinAddress is 2G on all platforms except Solaris x86. // G1 currently needs a lot of C-heap, so on Solaris we have to give G1 // some extra space for the C-heap compared to other collectors. - FLAG_SET_ERGO(uintx, HeapBaseMinAddress, 1*G); + // Use FLAG_SET_DEFAULT here rather than FLAG_SET_ERGO to make sure that + // code that checks for default values work correctly. + FLAG_SET_DEFAULT(HeapBaseMinAddress, 1*G); } }