[jdk17u-dev] RFR: 8313638: Add test for dump of resolved references
Goetz Lindenmaier
goetz at openjdk.org
Tue Oct 1 08:31:03 UTC 2024
I had to resolve the Whitebox coding. The method added by the backported change already exists in 17.
Originally, WB_GetResolvedReferences was added by "https://bugs.openjdk.org/browse/JDK-8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive" in 10 which did not directly use it, there were closed tests depending on it. See "https://bugs.openjdk.org/browse/JDK-8188791: Move AppCDS from closed repo to open repo". It was removed in "https://bugs.openjdk.org/browse/JDK-8298048: Combine CDS archive heap into a single block" in 21 and added again by this change.
I resolved the code to use the version of this new change to reduce differences to 21.
The old Whitebox code is used in
runtime/cds/appcds/cacheObject/CheckCachedResolvedReferencesApp.java
runtime/cds/appcds/cacheObject/GCStressApp.java
All runtime/cds/appcds/cacheObject/ are still passing.
-------------
Commit messages:
- Backport 83dca6296e3fc7b9912ef7b82e443ce1415a7bcc
Changes: https://git.openjdk.org/jdk17u-dev/pull/2924/files
Webrev: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=2924&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8313638
Stats: 216 lines in 5 files changed: 202 ins; 14 del; 0 mod
Patch: https://git.openjdk.org/jdk17u-dev/pull/2924.diff
Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/2924/head:pull/2924
PR: https://git.openjdk.org/jdk17u-dev/pull/2924
More information about the jdk-updates-dev
mailing list