RFR (M): 6672778: G1 should trim task queues more aggressively during evacuation pauses

[Thomas Schatzl]

Hi all,

  I unfortunately found another issue with timing:

when calculating the "Other" time, we add the SATBFiltering phase to
the known worker time - however that time is already included in the
ext root scan time, so it is double-counted, and you occasionally get
slightly negative "Other" times.

This problem had been introduced in the refactoring too :(

This change fixes that.

Webrev:
http://cr.openjdk.java.net/~tschatzl/6672778/webrev.3_to_4/ (diff)
http://cr.openjdk.java.net/~tschatzl/6672778/webrev.4/ (full)

Thanks,
  Thomas

On Mon, 2018-04-16 at 14:58 +0200, Stefan Johansson wrote:
> Hi Thomas,
> 
> On 2018-04-16 13:08, Thomas Schatzl wrote:
> > Hi all,
> > 
> > On Fri, 2018-04-13 at 14:25 +0200, Stefan Johansson wrote:
> > > 
> > > On 2018-04-13 10:35, Thomas Schatzl wrote:
> > > > Hi Stefan,
> > > > 
> > > >     thanks for your review... :)
> > > > 
> > > > On Thu, 2018-04-12 at 17:15 +0200, Stefan Johansson wrote:
> > > > > Hi Thomas,
> > > > > 
> > > > > On 2018-04-11 13:46, Thomas Schatzl wrote:
> > > > > > Hi all,
> > > > > > 
> > > > > >      I updated and (hopefully) improved the change a bit
> > > > > > after
> > > > > > some
> > > > > > more thinking.
> > > > 
> > > > [...]
> > 
> > Also fixed a problem with the "-" operator of Tickspan introduced
> > in
> > all this refactoring. This caused negative times being reported
> > sometimes in the logs.
> > 
> > http://cr.openjdk.java.net/~tschatzl/6672778/webrev.2_to_3/ (diff)
> > http://cr.openjdk.java.net/~tschatzl/6672778/webrev.3/ (full)
> > 
> > The change looks really nice now imho, thanks Stefan!
> 
> I agree, ship it :)
> 
> Thanks,
> Stefan
> > 
> > Thanks,
> >    Thomas
> >