[jdk17u-dev] RFR: 8280705: Parallel: Full gc mark stack draining should prefer to make work available to other threads [v3]

[Aleksey Shipilev]

On Tue, 15 Aug 2023 09:36:48 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Clean backport to match what G1 is also doing. The change was in mainline for more than a year without related issues.
>> 
>> Additional testing:
>>  - [x] Linux x86_64 fastdebug `tier1` with `-XX:+UseParallelGC`
>>  - [x] Linux x86_64 fastdebug `tier2` with `-XX:+UseParallelGC`
>>  - [x] Linux x86_64 fastdebug `tier3` with `-XX:+UseParallelGC`
>
> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - Merge branch 'master' into JDK-8280705-parallel-fullgc-work
>  - Merge branch 'master' into JDK-8280705-parallel-fullgc-work
>  - Merge branch 'master' into JDK-8280705-parallel-fullgc-work
>  - Backport bdda43e066b8da0ebf9a8ef2f843eabb230f0800

Same with `naive-bayes`:


$ build/linux-x86_64-server-release/images/jdk/bin/java -XX:+UseParallelGC -Xmx4g -Xms4g -XX:+AlwaysPreTouch -Xlog:gc -jar ../renaissance-mit-0.14.2.jar naive-bayes 2>&1 | grep "Pause Full"

# Before
[24.500s][info][gc] GC(340) Pause Full (System.gc()) 1373M->1367M(4089M) 90.501ms
[25.173s][info][gc] GC(353) Pause Full (System.gc()) 1373M->1367M(4090M) 92.504ms
[25.844s][info][gc] GC(366) Pause Full (System.gc()) 1373M->1367M(4089M) 93.355ms
[26.518s][info][gc] GC(379) Pause Full (System.gc()) 1373M->1367M(4090M) 91.851ms
[27.215s][info][gc] GC(392) Pause Full (System.gc()) 1373M->1367M(4090M) 91.128ms
[27.900s][info][gc] GC(405) Pause Full (System.gc()) 1373M->1367M(4090M) 90.536ms
[28.570s][info][gc] GC(418) Pause Full (System.gc()) 1373M->1367M(4090M) 92.705ms
[29.247s][info][gc] GC(431) Pause Full (System.gc()) 1373M->1367M(4090M) 91.838ms
[29.926s][info][gc] GC(444) Pause Full (System.gc()) 1373M->1367M(4090M) 92.964ms
[30.599s][info][gc] GC(457) Pause Full (System.gc()) 1373M->1367M(4089M) 91.042ms

# After
[24.654s][info][gc] GC(338) Pause Full (System.gc()) 1373M->1367M(4090M) 105.490ms
[25.349s][info][gc] GC(351) Pause Full (System.gc()) 1373M->1367M(4090M) 106.263ms
[26.046s][info][gc] GC(364) Pause Full (System.gc()) 1373M->1367M(4090M) 106.324ms
[26.748s][info][gc] GC(377) Pause Full (System.gc()) 1373M->1367M(4090M) 106.146ms
[27.448s][info][gc] GC(390) Pause Full (System.gc()) 1373M->1367M(4090M) 108.895ms
[28.143s][info][gc] GC(403) Pause Full (System.gc()) 1373M->1367M(4090M) 107.850ms
[28.861s][info][gc] GC(416) Pause Full (System.gc()) 1373M->1367M(4090M) 106.542ms
[29.550s][info][gc] GC(429) Pause Full (System.gc()) 1373M->1367M(4090M) 109.635ms
[30.233s][info][gc] GC(442) Pause Full (System.gc()) 1373M->1367M(4090M) 107.652ms
[30.922s][info][gc] GC(455) Pause Full (System.gc()) 1374M->1367M(4090M) 108.397ms

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1521#issuecomment-1679313219