RFR: 8272838: Move CriticalJNI tests out of tier1 [v2]
Aleksey Shipilev
shade at openjdk.java.net
Fri Aug 27 10:39:53 UTC 2021
> See the bug report for more discussion. These tests are tied in Epsilon and Shenandoah test groups, so I rehashed those a bit. Note there are no `tier2` or `tier3` test groups in Hotspot yet, which means these tests would only run as part of Epsilon and Shenandoah test suites. I would rehash tier2/3 in [JDK-8272914](https://bugs.openjdk.java.net/browse/JDK-8272914), while hooking up more runtime and GC tests there.
>
> Motivational run time improvements:
>
>
> $ make run-test TEST=hotspot:tier1
>
> # Before
> real 8m9.678s
> user 289m59.304s
> sys 27m46.347s
>
> # After
> real 7m53.359s # 15 seconds saved
> user 278m49.104s # 11 minutes saved
> sys 28m2.516s
>
>
> Additional testing:
> - [x] Checked `hotspot:tier1` run test lists before/after the patch, the difference is only these two lists
> - [x] Some test performance measurements.
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 three additional commits since the last revision:
- Make sure CriticalJNI tests are in tier2 in both Epsilon and Shenandoah blocks
- Merge branch 'master' into JDK-8272838-perf-test-criticaljni
- 8272838: Remove CriticalJNI tests from tier1
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/5219/files
- new: https://git.openjdk.java.net/jdk/pull/5219/files/a9556d7a..8c628bf4
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5219&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5219&range=00-01
Stats: 7100 lines in 339 files changed: 4941 ins; 1108 del; 1051 mod
Patch: https://git.openjdk.java.net/jdk/pull/5219.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/5219/head:pull/5219
PR: https://git.openjdk.java.net/jdk/pull/5219
More information about the hotspot-dev
mailing list