RFR: 8249262: Initialize InstanceKlass::_package_entry during CDS dump time [v2]

Ioi Lam iklam at openjdk.java.net
Wed Jan 27 23:10:44 UTC 2021


On Tue, 26 Jan 2021 23:23:59 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> With JDK-8244778, the `PackageEntry` of all archived classes (for the 3 built-in class loaders) are all created during CDS dump time. We can remember this in `InstanceKlass::_package_entry` during dump time to avoid the hashtable lookup at runtime in `InstanceKlass::set_package()` and `SystemDictionaryShared::get_package_entry_from_class_name()`.
>> 
>> Passed tiers 1 - 4 testing.
>> 
>> Just a very small perf improvement with HelloWorld:
>> instr delta =        -9514    -0.0145%
>> time  delta =       -0.272 ms -0.6916%
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   1. updates per review comments from Ioi.
>   2. copyright update.

Looks good to me.

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

Marked as reviewed by iklam (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2206


More information about the hotspot-runtime-dev mailing list