On Thu, 13 Jul 2023 19:21:01 GMT, Doug Lea <d...@openjdk.org> wrote: >> This now uses Thread.isVirtual to distinguish spin vs immediate block cases, >> enabling re-introduction of spin control from the previous version, removing >> anomalies like this one. > > 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 seven additional commits since > the last revision: > > - 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 > - Overhaul LTQ and SQ to use common blocking and matching mechanics > - Merge branch 'openjdk:master' into JDK-8301341 > - Use Thread.isVirtual to distinguish spin vs immediate block cases
Need to digest this a bit. Will give more concrete feedback ASAP :) src/java.base/share/classes/java/util/concurrent/LinkedTransferQueue.java line 180: > 178: * increase CAS contention and overhead. Using the smallest > 179: * non-zero value of one is both simple and empirically a good > 180: * choice in most applicatkions. The slack value is hard-wired: a s/applicatkions/applications ------------- PR Review: https://git.openjdk.org/jdk/pull/14317#pullrequestreview-1510867492 PR Review Comment: https://git.openjdk.org/jdk/pull/14317#discussion_r1263111307