RFR: 8179268: Factor out AdaptiveSizePolicy from top-level interfaces CollectorPolicy and CollectedHeap

Mikael Gerdin mikael.gerdin at oracle.com
Wed Jul 5 11:12:20 UTC 2017


Hi Roman,

On 2017-07-04 20:47, Roman Kennke wrote:
> AdaptiveSizePolicy is not used/called from outside the GCs, and not all
> GCs need them. It makes sense to remove it from the CollectedHeap and
> CollectorPolicy interfaces and move them down to the actual subclasses
> that used them.
> 
> I moved AdaptiveSizePolicyOutput to parallelScavengeHeap.hpp, it's only
> used/implemented in the parallel GC. Also, I made this class AllStatic
> (was StackObj)
> 
> Tested by running hotspot_gc jtreg tests without regressions.
> 
> http://cr.openjdk.java.net/~rkennke/8179268/webrev.00/

Please correct me if I'm wrong here but it looks like all the non-G1 
collectors set the _should_clear_all_soft_refs based on 
gc_overhead_limit_near.
Perhaps the ClearedAllSoftRefs scoped object could be modified to only 
work with GenCollectorPolicy derived policies (which include parallel 
*shrugs*) and G1 should just stop worrying about _all_soft_refs_clear.
Looking closer, I can't even find G1 code looking at that member so 
maybe it, too, should be moved to GenCollectorPolicy?

What do you think?
/Mikael

> <http://cr.openjdk.java.net/%7Erkennke/8179268/webrev.00/>
> 
> Roman
> 



More information about the hotspot-gc-dev mailing list