On Wed, 22 May 2024 15:45:43 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 36 additional commits 
>> since the last revision:
>> 
>>  - Merge branch 'openjdk:master' into JDK-8322732
>>  - More performance tradoffs
>>  - Address review comments
>>  - Merge branch 'openjdk:master' into JDK-8322732
>>  - Repack some fields; adjust control flow
>>  - Merge branch 'openjdk:master' into JDK-8322732
>>  - Next version
>>  - Merge branch 'openjdk:master' into JDK-8322732
>>  - Reduce unneeded signals
>>  - Merge branch 'openjdk:master' into JDK-8322732
>>  - ... and 26 more: https://git.openjdk.org/jdk/compare/709b497f...f1fc4f3e
>
> src/java.base/share/classes/java/util/concurrent/ForkJoinPool.java line 586:
> 
>> 584:      * term. We use Marsaglia XorShifts, seeded with the Weyl sequence
>> 585:      * from ThreadLocalRandom probes, which are cheap and
>> 586:      * suffice. Each queue's polling attempt o avoid becoming stuck
> 
> Suggestion:
> 
>      * suffice. Each queue's polling attempt to avoid becoming stuck

thanks; fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19131#discussion_r1610782380

Reply via email to