On Tue, 25 Mar 2025 16:38:55 GMT, Viktor Klang <vkl...@openjdk.org> wrote:
>> 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 47 additional commits >> since the last revision: >> >> - Merge branch 'openjdk:master' into JDK-8319447 >> - Match indent of naster changes >> - Use TC_MASK in accord with https://bugs.openjdk.org/browse/JDK-8330017 >> (Unnecessarily for now.) >> - Reword javadoc >> - Use SharedSecrets for ThreadLocalRandomProbe; other tweaks >> - Disambiguate caller-runs vs Interruptible >> - Merge branch 'openjdk:master' into JDK-8319447 >> - Associate probes with carriers if Virtual (no doc updates yet) >> - Reduce volatile reads >> - Address review comments; reactivation tweak >> - ... and 37 more: https://git.openjdk.org/jdk/compare/82df2bda...b552c225 > > test/jdk/java/util/concurrent/tck/ForkJoinPool20Test.java line 414: > >> 412: final CountDownLatch done = new CountDownLatch(cycles); >> 413: final Runnable task = new CheckedRunnable() { >> 414: public void realRun() { done.countDown(); }}; > > This does not throw if the task executes *too many times*—maybe it should? Well , it does fail if outer loop terminates ((int delay = 1; delay <= LONG_DELAY_MS; delay *= 3)), which seems good enough here (since we can't otherwise calibrate). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23702#discussion_r2012755804
