RFR: 8291840: Avoid JavaCalls for setting up _java_system_loader and _java_platform_loader [v3]
Calvin Cheung
ccheung at openjdk.org
Thu Aug 11 15:49:25 UTC 2022
On Thu, 11 Aug 2022 12:13:30 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> I was thinking:
>>
>> void SystemDictionary::set_system_loader(ClassLoaderData *cld) {
>> assert(_java_system_loader.is_empty(), "already set!");
>> _java_system_loader = cld->class_loader_handle();
>> }
>
> Wait, I just realized, how does this not get into trouble with GC? If we copy this OopHandle, then GC will follow it from two locations. This doesn't seem right.
Hi Coleen,
Let me know what needs to be done to get this right. I can file another bug to fix it together with David's suggestion above.
-------------
PR: https://git.openjdk.org/jdk/pull/9734
More information about the hotspot-runtime-dev
mailing list