On Mon, 23 Aug 2021 11:33:35 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:

> See the RFE for discussion.
> 
> Current PR improves the test time like this:
> 
> 
> $  make run-test TEST=java/lang/invoke/LFCaching/
> 
> # Before
> real  3m51.608s
> user  5m21.612s
> sys   0m5.391s
> 
> # After
> real  1m13.606s
> user  2m26.827s
> sys   0m4.761s

the reason we tie time-budget in this test (and other similar stress tests) to 
timeout is to give a test chance to do actual testing in slow configurations 
(which will set higher timeout factor), for example, runs w/ -Xcomp on debug 
builds. if we use hardcoded value, the test might spend (almost) all its 
allocated time to just init and wouldn't perform any testing. ]

so instead of using the hardcode limit, I'd prefer to adjust the 
multiplication, `0.25` will give you the same 60s w/ current default timeout 
factor.

-- Igor

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

PR: https://git.openjdk.java.net/jdk/pull/5214

Reply via email to