# HG changeset patch # User brutisso # Date 1372319999 -7200 # Node ID 752f5cbc2ba69f17765ce1f8fa0bd860c82d6a28 # Parent 8adb4bc92f18373624fcb04f4b72722ac684578d 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 8adb4bc92f18 -r 752f5cbc2ba6 hotspot/src/share/vm/runtime/arguments.cpp --- a/hotspot/src/share/vm/runtime/arguments.cpp Wed Jun 26 16:58:37 2013 +0200 +++ b/hotspot/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); } }