On Tue, 20 Jan 2026 00:12:20 GMT, David Holmes <[email protected]> wrote:
>> Addressed. > > You didn't expand the comment. IIUC if the state is TS_ENTER then we were > notified and moved from TS_WAIT to TS_ENTER before being unparked. But, again > IIUC, that will only happen if we don't have to post the event else the > notification will unpark us directly and move us to TS_RUN. Which means that > checking `!= TS_ENTER` is redundant because if we should post the event then > we can't be in state TS_ENTER. Sorry for missing that. Yes, you are right, it is redundant, as `JvmtiExport::should_post_monitor_waited()` is mutually exclusive with being in the `TS_ENTER` state now. I added this to the comment. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27040#discussion_r2707570006
