RFR: 8224664: Parallel GC: Use WorkGang (6: PSRefProcTaskProxy)

Leo Korinth leo.korinth at oracle.com
Mon Aug 5 16:13:02 UTC 2019


Hi!

On 28/07/2019 19:54, Thomas Schatzl wrote:
> Hi,
> 
> On 27.05.19 10:34, Leo Korinth wrote:
>> Hi,
>>
>> Here is the sixth patch in a proposed patch series of eight that
>> removes gcTaskManager and uses the WorkGang abstraction instead.
>>
>> Here, the new PSRefProcTask is composed of PSRefProcTaskProxy and the
>> old StealTask (that was partially moved before).
>>
>> Now both psTasks.* and pcTasks.* are removed.
>>
>> Enhancement:
>>    https://bugs.openjdk.java.net/browse/JDK-8224664
>>
>> Webrev:
>>
>> http://cr.openjdk.java.net/~lkorinth/workgang/0/_8224664-Parallel-GC-Use-WorkGang-6-PSRefProcTaskProxy/ 
>>
>>    http://cr.openjdk.java.net/~lkorinth/workgang/0/all/
>>
>> Testing (on the whole patch series):
>>    mach5 remote-build-and-test --build-profiles 
>> linux-x64,linux-x64-debug,macosx-x64,solaris-sparcv9,windows-x64 
>> --test open/test/hotspot/jtreg/:hotspot_gc -a -XX:+UseParallelGC
>>    gc test suite
> 
> 
> http://cr.openjdk.java.net/~lkorinth/workgang/2/_8224664-Parallel-GC-Use-WorkGang-6-PSRefProcTaskProxy 
> 
> 
> Potentially psCardTable.cpp could get a copyright update, it's up to 
> you. No need for an extra webrev for this change.

Copyright gets updated in patch #7 though it should have been made 
already in #6 as you noted, I will not move the copyright update as it 
will be more for you to review. I guess you prefer it that way.

Thanks for reviewing,
Leo

> 
> Thanks,
>    Thomas
> 
> 
> 



More information about the hotspot-gc-dev mailing list