On Fri, 1 May 2026 02:54:38 GMT, Leonid Mesnik <[email protected]> wrote:
>> David Holmes has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix typo
>
> src/hotspot/share/runtime/javaThread.hpp line 980:
>
>> 978: void exit_jni_deferred_suspension() {
>> 979: assert(Thread::current() == this, "this must be current thread");
>> 980: _jni_deferred_suspension_count--;
>
> Why
> `AtomicAccess::inc(&_jni_deferred_suspension_count);`
> is not used here? (Same for dec())
> I am not sure I see how this access is protected.
Because only the current thread modifies its own count.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30936#discussion_r3172427667
