RFR: 8259068: Streamline class loader locking
Coleen Phillimore
coleenp at openjdk.java.net
Fri Jan 15 00:03:13 UTC 2021
On Thu, 14 Jan 2021 23:42:44 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> src/hotspot/share/classfile/systemDictionaryShared.cpp line 1042:
>>
>>> 1040: Handle lockObject = compute_loader_lock_object(THREAD, class_loader);
>>> 1041: check_loader_lock_contention(THREAD, lockObject);
>>> 1042: ObjectLocker ol(lockObject, THREAD);
>>
>> I think we can replace the above 3 lines with something like
>> assert(!is_parallelCapable(class_loader), "ObjectLocker not required");
>
> Yes, I agree. I updated the comment slightly also.
It's assert(is_parallelCapable(class_loader)).
-------------
PR: https://git.openjdk.java.net/jdk/pull/2071
More information about the hotspot-dev
mailing list