RFR: 8357632: CDS test failures on static JDK
Jiangli Zhou
jiangli at openjdk.org
Thu May 29 00:30:21 UTC 2025
Please review the fix in CDSConfig::default_archive_path to form the default .jsa path using Arguments::get_java_home on static-jdk. It basically reverts to the previous code before JDK-8353504, but for static-jdk case only. The change does not affect how default .jsa path is handled on regular JDK.
The PR removes the affected CDS tests from hotspot/jtreg/ProblemList-StaticJdk.txt.
It adds '@requires !jdk.static' to runtime/cds/NonJVMVariantLocation.java to skip the test on static-jdk. NonJVMVariantLocation.java tests with VM variants specified in jvm.cfg, which is not processed on static-jdk.
Testing:
Tested the affected tests locally on static-jdk.
https://github.com/jianglizhou/jdk/actions/runs/15312986693 is in progress.
-------------
Commit messages:
- Remove affected tests from hotspot/jtreg/ProblemList-StaticJdk.txt.
- - Change CDSConfig::default_archive_path to use Arguments::get_java_home when forming the path to default .jsa on static JDK.
Changes: https://git.openjdk.org/jdk/pull/25516/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25516&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8357632
Stats: 31 lines in 3 files changed: 17 ins; 13 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/25516.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25516/head:pull/25516
PR: https://git.openjdk.org/jdk/pull/25516
More information about the hotspot-runtime-dev
mailing list