RFR: 8057710: Refactor G1 heap region default sizes

Mikael Gerdin mikael.gerdin at oracle.com
Mon Sep 8 07:51:46 UTC 2014


Hi Stefan,

On Friday 05 September 2014 18.32.05 Stefan Johansson wrote:
> Hi,
> 
> Please review this small change for RFE:
> https://bugs.openjdk.java.net/browse/JDK-8057710
> 
> Webrev:
> http://cr.openjdk.java.net/~sjohanss/8057710/webrev.00/

Looks good.
/Mikael

> 
> Summary:
> Before this patch the min and max size for G1 heap regions were defined
> by defines and this patch changes this to instead use static const
> within a class and then have getter for the values.
> 
> Testing:
> * Builds through JPRT
> * Manually verified that sizes stay the same
> 
> Thanks,
> Stefan




More information about the hotspot-gc-dev mailing list