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