RFR (L) 8034246: remove CMS and ParNew adaptive size policy code

John Coomes John.Coomes at oracle.com
Wed May 7 04:38:11 UTC 2014


Mikael Gerdin (mikael.gerdin at oracle.com) wrote:
> John,
> 
> On Monday 05 May 2014 09.14.40 John Coomes wrote:
> > I'd appreciate reviews of this change to delete unused adaptive size
> > policy code for cms and parnew.  It's big in terms of line count, but
> > most comes from the deletion of 6 files.
> > 
> > http://cr.openjdk.java.net/~jcoomes/8034246-rm-cms-asp/
> 
> Good to get rid of code that hasn't been working for a while.
> 
> In src/share/vm/memory/collectorPolicy.cpp the include statement:
> #include "gc_implementation/shared/adaptiveSizePolicy.hpp"
> is already present a few lines up.

Yes; fixed.

> CMSCollector::size_policy and ConcurrentMarkSweepGeneration::size_policy are 
> basically identical now, do you want to fix this now or leave this as just a 
> code removal change? (personally, I'm fine either way)

The ConcurrentMarkSweepGeneration one is no longer used, so I removed
it.  Updated webrev is at the same location.

> Consider the change reviewed nevertheless.

Thanks!

-John



More information about the hotspot-gc-dev mailing list