On Mon, 11 Nov 2024 11:06:17 GMT, Kevin Walls <kev...@openjdk.org> wrote:

> Test timeout change to cope with -Xcomp runs.
> Long-standing problemlisted test.
> 
> With -Xcomp locally I could see a failure on the second attempt.
> After this change, ran more runs locally and 40 runs per platform in CI are 
> OK.

Looks reasonable to me. A possible alternative would be to use 
`jdk.test.lib.Utils.adjustTimeout(long)` to take into account the 
`timeoutFactor`. IIRC tests run with an icreased `timeoutFactor` when run with 
-Xcomp

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

PR Review: https://git.openjdk.org/jdk/pull/22008#pullrequestreview-2427486147

Reply via email to