On Mon, 1 Mar 2021 19:30:00 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> 
wrote:

>> This simple fix reduces the amount of concurrency on the foreign memory 
>> TestHandshake test. As this test spins a new accessor thread for each 
>> available processors, on machines which feature an high number of available 
>> processors (because of multi-threading), and which are slower in forking new 
>> threads (e.g. Windows), the test can sometimes timeout.
>> 
>> The sensible step, for now, is to introduce an hard limit on the number of 
>> concurrent accessor threads being created. I've looked at the test logs 
>> quite in depth, and there's nothing suggesting that something else is amiss 
>> - the number of concurrent threads just seem to be too high in some 
>> instances.
>
> Maurizio Cimadamore has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Address review comments

Marked as reviewed by psandoz (Reviewer).

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

PR: https://git.openjdk.java.net/jdk/pull/2748

Reply via email to