RFR (S): 8234179: Move HeapRegion::_recorded_rs_length/_predicted_elapsed_time_ms into G1CollectionSet

Leo Korinth leo.korinth at oracle.com
Fri Nov 22 16:37:52 UTC 2019


On 22/11/2019 10:57, Thomas Schatzl wrote:
> Hi all,
> 
>    can I have reviews for this change that moves two members 
> (_recorded_rs_length/_predicted_elapsed_time_ms) of HeapRegion into 
> G1CollectionSet that is the only user of these members?
> 
> This also allows more efficient allocation of this information in the 
> future, as these are only needed for eden regions.

Looks good in general, the array _inc_collection_set_stats should be 
initialized though.

/Leo
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8234179
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8234179/webrev/
> Testing:
> hs-tier1-5
> 
> Thanks,
>    Thomas



More information about the hotspot-gc-dev mailing list