On Thu, 26 Feb 2026 00:02:21 GMT, Leonid Mesnik <[email protected]> wrote:
>> Okay, thanks. Replaced with `load_acquire`/`release_store`. > > Well, I think that load_relaxed should be enough here. > > The `_seen_interp_only_mode` and `_pending_interp_only_mode` are updated with > the JvmtiThreadState_lock and changes should be visible together after lock > is released. > > So even `_seen_interp_only_mode` is relaxed load/store and might be reordered > with `_pending_interp_only_mode`, it is not possible that > `_pending_interp_only_mode` is true in the > `JvmtiThreadState::enter_interp_only_mode()` > while > `_seen_interp_only_mode` > is still false. > > It might be possible that it becomes 'true' while the recompute enabled is > still in process, but it is ok. Thank you, Leonid. I have already pushed the update. Keeping `load_acquire/release_store` should be okay for now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/29800#discussion_r2856731478
