RFR: 8364343: Virtual Thread transition management needs to be independent of JVM TI [v3]
David Holmes
dholmes at openjdk.org
Wed Nov 26 07:32:52 UTC 2025
On Tue, 25 Nov 2025 19:45:08 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:
>> src/hotspot/share/classfile/javaClasses.cpp line 1764:
>>
>>> 1762: jint* addr = java_thread->field_addr<jint>(_VTMS_transition_disable_count_offset);
>>> 1763: int val = AtomicAccess::load(addr);
>>> 1764: AtomicAccess::store(addr, val - 1);
>>
>> Suggestion:
>>
>> AtomicAccess::dec(addr);
>
> I’d prefer to leave it as a plain store to avoid the unnecessary extra fence.
But it isn't then an atomic update.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28361#discussion_r2563699480
More information about the graal-dev
mailing list