On Tue, 24 Feb 2026 02:08:25 GMT, Serguei Spitsyn <[email protected]> wrote:

>> The function `JvmtiThreadState::process_pending_interp_only()` should 
>> protect the check with `state->is_pending_interp_only_mode()` and call to 
>> `JvmtiEventController::enter_interp_only_mode(state)` with the 
>> `JvmtiThreadState_lock`. Some level of optimization with a check of 
>> `seen_interp_only_mode()` is used as the code path of the 
>> `process_pending_interp_only()` is hot. Then the `seen_interp_only_mode()` 
>> has to be set somewhat earlier when the `is_pending_interp_only_mode()` is 
>> set.
>> This issue was discovered when in the work on the PR update of 
>> https://bugs.openjdk.org/browse/JDK-8373367 .
>> 
>> Testing:
>>  - TBD: mach5 tiers 1-5
>
> Serguei Spitsyn has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   review: convert volatile bool to Atomic<bool>

Marked as reviewed by lmesnik (Reviewer).

-------------

PR Review: https://git.openjdk.org/jdk/pull/29800#pullrequestreview-3855976015

Reply via email to