RFR (XS) JDK-8164482,[REDO] G1 does not implement millis_since_last_gc which is needed by RMI GC

Thomas Schatzl thomas.schatzl at oracle.com
Mon Sep 5 11:25:36 UTC 2016


Hi,

On Mon, 2016-09-05 at 13:17 +0200, Thomas Schatzl wrote:
> Hi again,
> 
> On Mon, 2016-09-05 at 13:13 +0200, Thomas Schatzl wrote:
> > 
> > Hi,
> > 
> > On Fri, 2016-09-02 at 09:40 -0400, Joseph Provino wrote:
> > > 
> > > 
> > > Please review this small change.
> > > JDK-8164482 [REDO] G1 does not implement millis_since_last_gc
> > > which
> > > is needed by RMI GC
> > > webrev:  http://cr.openjdk.java.net/~jprovino/8164482/webrev.00
> >   looks good.
>   after having another look, I do not think the change is
> good: millis_since_last_gc() tracks the time since any last gc
> (probably collection pause), so the last gc time needs to be updated
> at the end of every gc, not only full gcs.
> 
> The member and the getter name should also reflect this.

  also, what kind of testing did this change go through?

Please always mention this during the first review request email.

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list