RFR(S): Merge GC thread pools

Zhengyu Gu zgu at redhat.com
Fri Feb 10 23:17:39 UTC 2017


On 02/10/2017 05:53 PM, Roman Kennke wrote:

> Finally!
>
> Patch looks good to me.
>
> Small thing:
> here: 
> http://cr.openjdk.java.net/~zgu/shenandoah/single-worker/webrev.00/src/share/vm/gc/shenandoah/shenandoahConcurrentThread.cpp.udiff.html 
> <http://cr.openjdk.java.net/%7Ezgu/shenandoah/single-worker/webrev.00/src/share/vm/gc/shenandoah/shenandoahConcurrentThread.cpp.udiff.html>
>
> You might want to use run_task(&task, heap->max_workers()) or 
> something like that?
run_task() code is inside heap->reset_next_mark_bitmap() method, you can 
not real do so here.

Next thing I want to do, is moving all dynamic worker calculation code 
up, and we should always use
active workers in the reset.

Thanks,

-Zhengyu

>
> Roman
>
>
> Am Freitag, den 10.02.2017, 15:37 -0500 schrieb Zhengyu Gu:
>> Hi,
>>
>> The patch merges parallel GC and concurrent GC thread pools.
>>
>> http://cr.openjdk.java.net/~zgu/shenandoah/single-worker/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Ezgu/shenandoah/single-worker/webrev.00/>
>>
>> Test:
>>
>>    Fastdebug: specjbb2015, specjvm Debry, Compiler.sunflow
>>    Release:   specjbb2015
>>
>>
>> Thanks,
>>
>> -Zhengyu
>>
>>



More information about the shenandoah-dev mailing list