On Mon, 18 Aug 2025 09:15:29 GMT, Leo Korinth <lkori...@openjdk.org> wrote:

>> It is unclear to me if the author meant this to be `2.5` more than normal or 
>> `10` more than JTREG default, or a `multiplier that seems to work`. It does 
>> not bother me _more_ if it is a `10` then a `2.5`, as it needs to have a 
>> value that is not the multiplicative identity value. I will not change this, 
>> the change I have made is already large enough and I want this to be 
>> integrated ASAP.
>
> It is also something that can be changed later, in a follow up fix.

Take test test/hotspot/jtreg/compiler/arraycopy/stress/TestStressArrayCopy.java 
as example.
If there is a bug in jvm with -Xcomp option which will cause this test run time 
outed. Before this PR, it will take `7200*10` seconds to run this test finish 
and report time outed failure. But after this PR, it will take `28800*10` 
seconds to run this test finish ang then report timed out failure. I think the 
`28800*10` senonds too long and it's unacceptable.

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

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

Reply via email to