RFR: 8259067: bootclasspath append takes out object lock

Zhengyu Gu zgu at openjdk.java.net
Mon Jan 4 18:48:56 UTC 2021


On Mon, 4 Jan 2021 17:28:30 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> See CR for details.
> I made the classpath append list lock-free.  Calling experts in Atomic operations...
> Tested with tier1-6.
> Thanks,
> Coleen

src/hotspot/share/classfile/classLoader.cpp line 843:

> 841:           assert(first_append_entry() == NULL, "boot loader's append class path entry list not empty");
> 842:           Atomic::store(&_first_append_entry_list, new_entry);
> 843:           return;

Not sure if it is a problem. There is a chance that an entry is added, e.g. _last_append_entry = new_entry, but _first_append_entry_list == NULL. If contains_append_entry() queries on this new entry, it may return false.

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

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


More information about the serviceability-dev mailing list