On Tue, 27 Oct 2026 10:12:44 GMT, Viktor Klang <[email protected]> wrote:

>> Mat Carter has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update 
>> test/jdk/java/util/concurrent/LinkedTransferQueue/TransferRemove.java
>>   
>>   Co-authored-by: Viktor Klang <[email protected]>
>
> test/jdk/java/util/concurrent/LinkedTransferQueue/TransferRemove.java line 38:
> 
>> 36:  * @requires os.arch == "aarch64"
>> 37:  * @library /test/lib
>> 38:  * @run main/othervm -XX:-UseLSE -XX:ActiveProcessorCount=1 
>> TransferRemove
> 
> @macarte How about configuring these two test configurations to run with a 
> lower timeout, so that if they do hang, they aren't stalling for longer than 
> necessary? 🤔

The default value is 120, most tests either leave it or increase it; a few use 
60 and some 30 (the value is multiplied by the timeoutScalar; I've learnt that 
this is 1 for standalone jtreg tests, with make test its either 4 or 10 [with 
-Xcomp])

I picked 60 :)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33551#discussion_r4394996680

Reply via email to