RFR(XS) 8212074: Add method to peek the remaining tasks in task queues

Zhengyu Gu zgu at redhat.com
Mon Oct 15 13:24:31 UTC 2018


Thanks for reviewing, Roman.

-Zhengyu

On 10/15/2018 09:17 AM, Roman Kennke wrote:
> Hi Zhengyu.
> 
>>> 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 to me.
> 
> Thanks,
> Roman
> 
> 



More information about the hotspot-gc-dev mailing list