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