RFR(XS) 8212074: Add method to peek the remaining tasks in task queues
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Oct 15 13:05:20 UTC 2018
Hi,
On Mon, 2018-10-15 at 09:03 -0400, Zhengyu Gu wrote:
> Hi Thomas,
>
> Thanks for reviewing.
>
> On 10/15/2018 05:06 AM, Thomas Schatzl wrote:
> > Hi Zhengyu,
> >
> > On Fri, 2018-10-12 at 09:28 -0400, Zhengyu Gu wrote:
> > > Hi All,
> > >
> > > Shenandoah implemented an variant of termination protocol based
> > > on
> > > [1], which requires to peek into task queues for remaining tasks,
> > > so
> > > that it can wake just enough workers to steal tasks.
> > >
> > > The patch adds methods for this purpose, and allow subclasses of
> > > ParallelTaskTerminator to access it members.
> > >
> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8212074
> > > Webrev: http://cr.openjdk.java.net/~zgu/8212074/webrev.00/
> >
> > could you please move line 451 of taskqueue.hpp in the obviously
> > following patch that adds the ShenandoahTaskTerminator to the
> > Shenandoah specific code? It does not seem to fit here.
>
> Agree.
>
> Updated webrev: http://cr.openjdk.java.net/~zgu/8212074/webrev.01/
Looks good.
Thomas
More information about the hotspot-gc-dev
mailing list