[jdk17u-dev] RFR: 8227060: Optimize safepoint cleanup subtask order

Fri May 19 15:37:02 UTC 2023

On Fri, 19 May 2023 13:36:08 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> Unclean backport to improve the safepoint cleanup times. Provides the grounds for the ultimate improvement in safepoint cleanup ([JDK-8306738](https://bugs.openjdk.org/browse/JDK-8306738)). The backport is not clean, because there are copyright years conflicts, the rename of `AbstractGangTask` to `WorkerTask` is missing, `WorkGang` -> `WorkerThreads` rename is missing, `JavaThread::cast` cleanup is missing.
> 
> Additional testing:
>  - [x] Ad-hoc benchmarks
>  - [x] Linux x86_64 fastdebug `tier1 tier2 tier3`
> 
> The safepoint cleanup times are slightly better on small tests, and there seem to be no regressions.
> ![plot](https://github.com/openjdk/jdk17u-dev/assets/1858943/7e012482-71f8-459f-aca0-4c043b791e92)

Lgtm.

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

Marked as reviewed by phh (Reviewer).

PR Review: https://git.openjdk.org/jdk17u-dev/pull/1377#pullrequestreview-1434651728