RFR: 8027643 - Merge GenCollectorPolicy and TwoGenerationCollectorPolicy

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Fri May 2 00:26:02 UTC 2014


Hi Stefan,

Thanks for the review!

Yes, the _generations array is going away.
/Jesper


Stefan Johansson skrev 30/4/14 13:29:
> Hi Jesper,
>
> The change looks good.
>
> Do the further cleanups you mention include getting rid of the _generations
> array in GenCollectorPolicy and instead having a GenerationSpec for young and
> one for old? If not, I think that is something that should be added in upcoming
> one, since the policy no longer supports anything but 2 generations.
>
> Cheers,
> Stefan
>
> On 2014-04-30 00:28, Jesper Wilhelmsson wrote:
>> Hi,
>>
>> This is another step in cleaning up the collector policy code. It is the first
>> of a few related patches. This change merges the GenCollectorPolicy class with
>> TwoGenerationalCollectorPolicy.
>>
>> I have deliberately refrained from doing any other cleanups that is made
>> possible by this change. That will come in later changes. That makes this
>> change really easy to review :-)
>>
>> Webrev: http://cr.openjdk.java.net/~jwilhelm/8027643/webrev/
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8027643
>>
>> Thanks!
>> /Jesper
>



More information about the hotspot-gc-dev mailing list