Request for review (xs) - 7112997

John Coomes John.Coomes at oracle.com
Mon Nov 28 22:17:04 UTC 2011


Jon Masamitsu (jon.masamitsu at oracle.com) wrote:
> Bengt,
> 
> Thanks for the review.  I've made your suggested changes and
> also removed some additional dead code (unused mark_and_follow()
> methods) suggested by another reviewer.  New webrev is at
> 
> http://cr.openjdk.java.net/~jmasa/7112997/webrev.01

Looks good to me.

-John

> On 11/27/11 23:38, Bengt Rutisson wrote:
> >
> > Jon,
> >
> > I think you removed the only calls to 
> > ParCompactionManager::should_verify_only() and 
> > ParCompactionManager::should_reset_only(). Can you remove those 
> > methods as well? That will propagate to removing the enum values 
> > VerifyUpdate and ResetObjects in ParCompactionManager::Action.
> >
> > Other than that it looks good.
> >
> > Thanks for doing cleanups like this one!
> > Bengt
> >
> > On 2011-11-23 20:24, Jon Masamitsu wrote:
> >> 7112997: Remove obsolete code ResetObjectsClosure and 
> >> VerifyUpdateClosure
> >> Summary: Remove obsolete code.
> >>
> >> This was debugging code that used  the MarkSweep code to check
> >> the UseParallelOldGC  compaction.   It has been broken for sometime
> >> and there are no prospects for fixing it.
> >>
> >> http://cr.openjdk.java.net/~jmasa/7112997/webrev.00/
> >>
> >> Thanks.
> >>
> >



More information about the hotspot-gc-dev mailing list