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