RFR: 8324259: Classes used by CDS at runtime should be archived [v2]

Ioi Lam iklam at openjdk.org
Tue Oct 1 20:32:36 UTC 2024


On Mon, 30 Sep 2024 14:56:12 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> The following classes are not included in the archive at dumptime but are used at runtime when loading the archive:
>> 
>> [0.248s][info][class,load] java.nio.file.FileSystems source: jrt:/java.base loader: boot_loader
>> [0.248s][info][class,load] java.nio.file.FileSystems$DefaultFileSystemHolder source: jrt:/java.base loader: boot_loader
>> [0.249s][info][class,load] java.nio.file.FileSystems$DefaultFileSystemHolder$1 source: jrt:/java.base loader: boot_loader
>> [0.254s][info][class,load] java.net.URI$Parser source: jrt:/java.base loader: boot_loader 
>> 
>> This patch makes a dummy call at dumptime to archive these classes. Verified with tier 1-5 tests.
>
> Matias Saavedra Silva 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 four additional commits since the last revision:
> 
>  - Merge branch 'master' into archive_8324259
>  - Ioi comment and test fix
>  - Added regression test
>  - 8324259: Classes used by CDS at runtime should be archived

LGTM

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

Marked as reviewed by iklam (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21232#pullrequestreview-2341393640


More information about the hotspot-runtime-dev mailing list