RFR: 8291840: Avoid JavaCalls for setting up _java_system_loader and _java_platform_loader [v3]

Calvin Cheung ccheung at openjdk.org
Tue Aug 9 18:08:40 UTC 2022


On Fri, 5 Aug 2022 04:05:52 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   add assert to the _java_platform_loader case
>
> src/hotspot/share/classfile/systemDictionary.cpp line 185:
> 
>> 183:   if (_java_system_loader.is_empty()) {
>> 184:     _java_system_loader = cld->class_loader_handle();
>> 185:   }
> 
> Wouldn't it be an error to call this when the loader has already been set?

Hi David,

Are you thinking about adding an `else` case to the function and call the `vm_exit_during_initialization()` ?

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

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


More information about the hotspot-runtime-dev mailing list