RFR: 8346120: VirtualThreadPinned event recorded for Object.wait may have wrong duration or may record second event [v3]
Alan Bateman
alanb at openjdk.org
Sat Dec 14 16:27:19 UTC 2024
> A jdk.VirtualPinnedEvent JFR event is recorded by Object::wait when a virtual thread waits in Object.wait while pinned. The posting of the event in ObjectMonitor::wait is done after waiting but it can block again in enter/ReenterI before it re-acquires. If that happens then the event duration won't be accurate, and for the off-list case, the enter may be contended and a jdk.VirtualPinnedEvent event will be recorded for contended monitor enter. The changes to fix this are small: move the post of the event for Object.wait to end of the wait function, and only post an event for contended monitor enter when not waiting.
>
> I've added a test to check the pinned event that is recored for several contended monitor enter and Object.wait cases. It checks the event duration and that only one event is recorded for each case.
Alan Bateman has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
- Merge branch 'master' into JDK-8346120
- Fix test comments
- testObjectWait2 test already owns lock
- Min duration check needs to have some tolerance
- Rename test, check more event fields
- Filter out VirtualThread.getAndClearInterrupt
- Initial commit
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/22718/files
- new: https://git.openjdk.org/jdk/pull/22718/files/47643726..51ef032d
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=22718&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=22718&range=01-02
Stats: 2244 lines in 113 files changed: 1604 ins; 345 del; 295 mod
Patch: https://git.openjdk.org/jdk/pull/22718.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/22718/head:pull/22718
PR: https://git.openjdk.org/jdk/pull/22718
More information about the core-libs-dev
mailing list