RFR: 8336254: Virtual thread implementation + test updates [v2]

Alan Bateman alanb at openjdk.org
Fri Jul 19 16:59:54 UTC 2024


> Bringover some of the changes accumulated in the loom repo to the main line, most of these changes are test updates and have been baking in the loom repo for several months. The motive is partly to reduce the large set of changes that have accumulated in the loom repo, and partly to improve robustness and test coverage in the main line. The changes don't include any of the larger changes in the loom repo that are part of future JEPs.
> 
> Implementation:
> - Robustness improvements to not throw OOME when unparking a virtual thread.
> - Robustness improvements to reduce class loading when a virtual thread parks or parks when pinned (jdk.internal.misc.VirtualThreads is removed, jdk.internal.event.VirtualThreadPinnedEvent is eagerly loaded)
> - VirtualThread changes to reduce contention on timer queues when doing timed-park
> 
> Tests:
> - New tests for monitor enter/exit/wait/notify (this is a subset of the tests in the loom repo, we can't move many tests because they depend on on the changes to the object monitor implementation)
> - New test infrastructure to allow tests use a custom scheduler. This updates many tests to make use of this infrastructure, the "local" ThreadBuidlers is removed.
> - More test scenarios added to ThreadAPI and JVMTI GetThreadStateTest.java 
> - New test for ThreadMXBean.getLockedMonitor with synchronized native methods
> - Reimplement of JVMTI VThreadEvent test to improve reliability
> - Rename some tests to get consistent naming
> - Diagnostic output in several stress tests to help trace progress in the event of a timeout
> 
> Testing: tier1-6

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 eight additional commits since the last revision:

 - Merge
 - Fix typo in comment, missing copyright update, test nits
 - Merge
 - Drop JLA updates for this update
 - Merge
 - Merge
 - Update copyright headers
 - Initial commit

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20143/files
  - new: https://git.openjdk.org/jdk/pull/20143/files/1fad7dff..2d3bbf46

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

  Stats: 4362 lines in 184 files changed: 2873 ins; 856 del; 633 mod
  Patch: https://git.openjdk.org/jdk/pull/20143.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20143/head:pull/20143

PR: https://git.openjdk.org/jdk/pull/20143


More information about the serviceability-dev mailing list