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
--- 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);
}
}