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