RFR: 8057710: Refactor G1 heap region default sizes

Stefan Johansson stefan.johansson at oracle.com
Mon Sep 8 08:13:35 UTC 2014


Thanks Mikael for the review.

Stefan

On 2014-09-08 09:51, Mikael Gerdin wrote:
> 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