RFR: 8081607: Change default GC for server configurations to G1 (original) (raw)
Stefan Johansson stefan.johansson at oracle.com
Tue Jun 23 08:24:19 UTC 2015
- Previous message: RFR: 8081607: Change default GC for server configurations to G1
- Next message: RFR: 8081607: Change default GC for server configurations to G1
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Thanks Per,
Will change this before pushing.
Stefan
On 2015-06-23 10:23, Per Liden wrote:
Looks good, but could we remove the negation to make it easier to read, i.e. turn:
#if !defined(JAVASEEMBEDDED) // Select G1 for non-embedded builds only. FLAGSETERGO(bool, UseG1GC, true); #else FLAGSETERGO(bool, UseParallelGC, true); #endif into: #if defined(JAVASEEMBEDDED) FLAGSETERGO(bool, UseParallelGC, true); #else FLAGSETERGO(bool, UseG1GC, true); #endif (No need to send out a new webrev for that) cheers, /Per On 2015-06-23 10:07, Stefan Johansson wrote: Hi,
Please review this change to make G1 the default GC: https://bugs.openjdk.java.net/browse/JDK-8081607 JEP: http://openjdk.java.net/jeps/248 Webrev: http://cr.openjdk.java.net/~sjohanss/8081607/hotspot.00/ Summary: Changing the the default GC for server configurations to G1 (but not on embedded). Also updated test to reflect the change. Testing: * JPRT * Ad-hoc RBT runs, some issues found and fixed. Thanks, Stefan
- Previous message: RFR: 8081607: Change default GC for server configurations to G1
- Next message: RFR: 8081607: Change default GC for server configurations to G1
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]