On Wed, 21 Jan 2026 04:48:00 GMT, Patricio Chilano Mateo
<[email protected]> wrote:
>> src/hotspot/share/runtime/objectMonitor.cpp line 1933:
>>
>>> 1931: // Post monitor waited event. Note that this is past-tense, we
>>> are done waiting.
>>> 1932: // A thread which should post monitor_waited event is never in
>>> TS_ENTER state.
>>> 1933: if (JvmtiExport::should_post_monitor_waited()) {
>>
>> Is it worth asserting that `node.TState != ObjectWaiter::TS_ENTER` inside
>> the if-statement?
>
> I think we will have to add it as a branch condition since the event could
> have been enabled after notification.
If the event can be enabled (and disabled?) dynamically then that changes a
lot! We can't use the event state as a proxy for TS_RUN state (nor vice-versa).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27040#discussion_r2711012799
