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