RFR: 8362602: Add test.timeout.factor to CompileFactory to avoid test timeouts [v2]
David Briemann
dbriemann at openjdk.org
Mon Jul 21 09:04:41 UTC 2025
On Mon, 21 Jul 2025 08:36:08 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:
>> David Briemann has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update copyright header
>
> test/hotspot/jtreg/compiler/lib/compile_framework/Compile.java line 186:
>
>> 184: try {
>> 185: Process process = builder.start();
>> 186: long timeout = COMPILE_TIMEOUT * (long)Math.pow(2, timeoutFactor-1);
>
> Is there a reason for scaling the timeout exponentially instead of linearly, as jtreg does it and most users would expect?
I thought that I saw this documented somewhere but I cannot find it anymore and in other places it is used linearly, like you say.
I adapted it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26388#discussion_r2218578639
More information about the hotspot-compiler-dev
mailing list