RFR: 8275035: Clean up worker thread infrastructure [v2]

Per Liden pliden at openjdk.java.net
Thu Oct 14 14:08:56 UTC 2021


On Thu, 14 Oct 2021 09:03:18 GMT, Per Liden <pliden at openjdk.org> wrote:

>> I propose that we clean up our GangWorker/WorkGang and related classes, to remove abstractions we no longer need (after CMS was removed, MutexDispatcher was removed, Parallel is now using WorkGang, etc) and adjusting names as follows:
>> 
>> * Rename AbstractGangTask to WorkerTask
>> * Rename WorkGang to WorkerThreads
>> * Fold GangWorker into WorkerThread
>> * Fold WorkManager into WorkerThreads
>> * Move SubTaskDone and friends to a new workerUtils.hpp/cpp
>> 
>> I've split things up into several commits to make it easier to review.
>> 
>> Testing: Passes Tier 1-3 on all Oracle platforms.
>
> Per Liden has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 21 commits:
> 
>  - Merge master
>  - Clean up naming in ReferenceProcessor
>  - Clean up naming in Shenandoah
>  - Clean up naming in HeapDumper/HeapInspector
>  - Clean up naming in G1
>  - Clean up naming in pretouch
>  - Clean up naming in WeakProcessor
>  - Clean up naming in ZGC
>  - Remove unused log tag
>  - Rename workgroup.hpp/cpp to workerThread.hpp/cpp
>  - ... and 11 more: https://git.openjdk.java.net/jdk/compare/8b1b6f9f...9a89f785

Thanks all for reviewing!

-------------

PR: https://git.openjdk.java.net/jdk/pull/5886


More information about the serviceability-dev mailing list