> 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 incrementally with one additional 
commit since the last revision:

  testObjectWait2 test already owns lock

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22718/files
  - new: https://git.openjdk.org/jdk/pull/22718/files/2dfee538..47643726

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22718&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22718&range=00-01

  Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 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

Reply via email to