On Sun, 11 Jan 2026 17:11:38 GMT, Doug Lea <[email protected]> wrote:

>> Changes signal filtering to avoid possible starvation
>
> 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 33 additional commits since 
> the last revision:
> 
>  - Merge branch 'openjdk:master' into JDK-8373118
>  - reunify push; improve contention vs activation vs park balance
>  - Undo unrelated change
>  - Re-introduce acquiring array reads; re-arrange to rely on volatile base 
> index
>  - Change signalWork fencing; in-progress activation changes
>  - Merge branch 'openjdk:master' into JDK-8373118
>  - Split external push
>  - Undo/redo ordering changes
>  - Strengthen some orderings
>  - Merge branch 'openjdk:master' into JDK-8373118
>  - ... and 23 more: https://git.openjdk.org/jdk/compare/849376f7...f42d2475

src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 1260:

> 1258:                 U.putReferenceVolatile(a, slotOffset(m & s), task);
> 1259:                 if (unlock != 1)              // release external lock
> 1260:                     U.putInt(this, PHASE, unlock);

Doesn't this need to at least be a `putIntRelease`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28797#discussion_r2681978895

Reply via email to