RFR: 8378194: Protect process_pending_interp_only() work with JvmtiThreadState_lock [v3]
Serguei Spitsyn
sspitsyn at openjdk.org
Sat Feb 21 10:26:28 UTC 2026
On Sat, 21 Feb 2026 09:06:25 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>>
>> review: missed volatile keyword in the var definition
>
> src/hotspot/share/prims/jvmtiThreadState.hpp line 215:
>
>> 213: // It is cleared by EnterInterpOnlyModeClosure handshake.
>> 214: bool is_pending_interp_only_mode() {
>> 215: return AtomicAccess::load(&_pending_interp_only_mode);
>
> Does it matter to made
> `Atomic<bool> _pending_interp_only_mode `
> ?
>
> Seems the AtomicAccess is converted to `Atomic<T>` anywhere in the hotspot.
Yes, thanks. Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29800#discussion_r2836056482
More information about the serviceability-dev
mailing list