RFR: 8315923: pretouch_memory by atomic-add-0 fragments huge pages unexpectedly [v22]
Liming Liu
duke at openjdk.org
Fri Jan 12 08:20:03 UTC 2024
> As described at [JDK-8315923](https://bugs.openjdk.org/browse/JDK-8315923), this patch uses madvise with MADV_POPULATE_WRITE to pretouch memory when supported (since kernel 5.14).
>
> Ran the newly added jtreg test on 64c Neoverse-N1 machines with kernel 4.18, 5.13 and 6.1, and observed that transparent huge pages formed right after pretouch on kernel 6.1. Recorded the time spent on the test in *seconds* with `VERBOSE=time` as the table below, and got that the patch takes improvements when the system call is supported, while does not hurt if not supported:
>
> <table>
> <tr>
> <th>Kernel</th>
> <th colspan="2"><tt>-XX:-TransparentHugePages</tt></th>
> <th colspan="2"><tt>-XX:+TransparentHugePages</tt></th>
> </tr>
> <tr><td></td><td>Unpatched</td><td>Patched</td><td>Unpatched</td><td>Patched</td></tr>
> <tr><td>4.18</td><td>11.30</td><td>11.30</td><td>0.25</td><td>0.25</td></tr>
> <tr><td>5.13</td><td>0.22</td><td>0.22</td><td>3.42</td><td>3.42</td></tr>
> <tr><td>6.1</td><td>0.27</td><td>0.33</td><td>3.54</td><td>0.33</td></tr>
> </table>
Liming Liu 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 29 additional commits since the last revision:
- Use serial GC for the test
- Use pthread instead
- Remove the deletion
- Fix type errors
- Try to add a thread to use memory
- Replace to char* when type casting
- Fix the typo
- Use char* instead
- Fix the function arguments
- Try to add a testcase to cover concurrent pretouch
- ... and 19 more: https://git.openjdk.org/jdk/compare/01f5a923...6447a70e
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/15781/files
- new: https://git.openjdk.org/jdk/pull/15781/files/f974a393..6447a70e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=15781&range=21
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=15781&range=20-21
Stats: 29318 lines in 1043 files changed: 19415 ins; 5250 del; 4653 mod
Patch: https://git.openjdk.org/jdk/pull/15781.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/15781/head:pull/15781
PR: https://git.openjdk.org/jdk/pull/15781
More information about the hotspot-runtime-dev
mailing list