RFR: 8378194: Protect process_pending_interp_only() work with JvmtiThreadState_lock [v5]
Serguei Spitsyn
sspitsyn at openjdk.org
Wed Feb 25 23:20:49 UTC 2026
On Wed, 25 Feb 2026 19:50:55 GMT, Alex Menkov <amenkov at openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>>
>> review: convert volatile bool to Atomic<bool>
>
> src/hotspot/share/prims/jvmtiThreadState.hpp line 207:
>
>> 205: // Return true if any thread has entered interp_only_mode at any point during the JVMs execution.
>> 206: static bool seen_interp_only_mode() {
>> 207: return _seen_interp_only_mode.load_relaxed();
>
> Not sure `load_relaxed` is enough here.
> I think we need `load_acquire`/`release_store`
Okay, thanks. Replaced with `load_acquire`/`release_store`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29800#discussion_r2855984545
More information about the serviceability-dev
mailing list