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


On Sat, 29 Mar 2025 07:53:37 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> 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

@iklam could you take a look at this small change?

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

PR Comment: https://git.openjdk.org/jdk/pull/24311#issuecomment-2764642142


More information about the hotspot-runtime-dev mailing list