RFR: 8179268: Factor out AdaptiveSizePolicy from top-level interfaces CollectorPolicy and CollectedHeap
per.liden at oracle.com
Mon Jul 10 14:54:04 UTC 2017
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)
AdaptiveSizePolicyOutput::print() is actually called from
runtime/java.cpp also, so it's used outside of ParallelGC. I'm fine with
moving it, but we should have the proper #includes in java.cpp.
(Your patch doesn't actually build in its current form. I suspect you're
using precompiled headers which have a tendency to hide a lot of errors
caused by missing includes)
> Tested by running hotspot_gc jtreg tests without regressions.
258 void cleared_all_soft_refs();
Please declare this virtual too (that's the best we can do to signal
intent until we have C++11/override)
Please remove "this->" to match the super-call style used in other
places in this file.
Btw, I can sponsor the patch if you want.
More information about the hotspot-gc-dev