RFR: 8353225: Add a way to iterate Klass inside the loaded CDS archive

Thomas Stuefe stuefe at openjdk.org
Sun Mar 30 16:49:02 UTC 2025


Please consider.

This patch adds a way to iterate all Klass structures in the loaded CDS archive (`SystemDictionaryShared::iterate_klasses_in_shared_archive(ConstKlassClosure* klassClosure, bool is_static);`). This uses the same mechanism that `PrintSharedArchiveAndExit` uses. I  need this mechanism for some future GC work.

Then, it re-implements `PrintSharedArchiveAndExit` to reuse the new iteration API instead. This also serves as a way to test, since there are pre-existing tests for this flag.

It also adds a new, rather basic, gtest.

Finally, the new gtest picked up a small pre-existing bug in the implementation of `PrintSharedArchiveAndExit`: it prints all array classes of all InstanceKlass inside the archive, for all dimensions, without testing if that array class is part of the archive. The array class may have been created dynamically, however. The patch fixes that.

----

Testing:
- I ensured that the output of `PrintSharedArchiveAndExit`, before and after this patch, are identical apart from the small error fix concerning array classes
- I ran all runtime/cds tests on Linux x64
- GHAs

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

Commit messages:
 - iterate_klasses

Changes: https://git.openjdk.org/jdk/pull/24311/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24311&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8353225
  Stats: 195 lines in 10 files changed: 146 ins; 16 del; 33 mod
  Patch: https://git.openjdk.org/jdk/pull/24311.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24311/head:pull/24311

PR: https://git.openjdk.org/jdk/pull/24311


More information about the hotspot-runtime-dev mailing list