On Sun, 24 Aug 2025 22:51:05 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/tiered/Level2RecompilationTest.java line 36:
>> 
>>> 34:  * @build jdk.test.whitebox.WhiteBox
>>> 35:  * @run driver jdk.test.lib.helpers.ClassFileInstaller 
>>> jdk.test.whitebox.WhiteBox
>>> 36:  * @run main/othervm/timeout=960 -Xbootclasspath/a:. 
>>> -XX:+TieredCompilation
>> 
>> Why not `480` in this case?
>
> Leo also states in the description:
> 
>> In a few places, I have added a bit more timeout so that it will work with 
>> the 0.7 timeout factor.

In a few places, I have got timeouts after adjusting the timeout value. This is 
most likely because I have used a timeout factor of 0.7 to minimise 
"flickering" behaviour before finally changing to a timeout factor of 1. One 
consequence of this is that a few test cases will have double the original 
timeout (those test that would not pass a reduction to 0.7). So in all the 
instances when the timeout factor is exactly two times the size, the reason is 
that I have realised that I have already adjusted the timeout and I do not want 
to quad it again.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26749#discussion_r2297765975

Reply via email to