RFR: 8301341: LinkedTransferQueue does not respect timeout for poll() [v10]
Doug Lea
dl at openjdk.org
Fri Jul 21 16:42:25 UTC 2023
> This update addresses performance issues across both LinkedTransferQueue and SynchronousQueue by creating a common basis for implementation across them (mainly in LinkedTransferQueue). Pasting from internal doc summary of changes:
> * * Class DualNode replaces Qnode, with fields and methods
> * that apply to any match-based dual data structure, and now
> * usable in other j.u.c classes. in particular, SynchronousQueue.
> * * Blocking control (in class DualNode) accommodates
> * VirtualThreads and (perhaps virtualized) uniprocessors.
> * * All fields of this class (LinkedTransferQueue) are
> * default-initializable (to null), allowing further extension
> * (in particular, SynchronousQueue.Transferer)
> * * Head and tail fields are lazily initialized rather than set
> * to a dummy node, while also reducing retries under heavy
> * contention and misorderings, and relaxing some accesses,
> * requiring accommodation in many places (as well as
> * adjustments in WhiteBox tests).
Doug Lea 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 13 additional commits since the last revision:
- Merge branch 'openjdk:master' into JDK-8301341
- Address more review comments
- Address review comments
- nitpicks
- Merge branch 'openjdk:master' into JDK-8301341
- Accommodate white-box tests; use consistent constructions; minor improvements
- Merge branch 'openjdk:master' into JDK-8301341
- Simplify contention handling; fix test
- Fix inverted test assert; improve internal documentation; simplify code
- Merge branch 'openjdk:master' into JDK-8301341
- ... and 3 more: https://git.openjdk.org/jdk/compare/3e585ff1...f53cee67
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/14317/files
- new: https://git.openjdk.org/jdk/pull/14317/files/7c64a933..f53cee67
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=14317&range=09
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=14317&range=08-09
Stats: 5915 lines in 182 files changed: 4676 ins; 702 del; 537 mod
Patch: https://git.openjdk.org/jdk/pull/14317.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/14317/head:pull/14317
PR: https://git.openjdk.org/jdk/pull/14317
More information about the core-libs-dev
mailing list