RFR: 8290211: jdk/internal/vm/Continuation/Fuzz.java failed with "AssertionError: Failed to compile int Fuzz.com_int(int, int) in 5000ms"

[Jie Fu]

On Thu, 11 Aug 2022 20:02:06 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

> A trivial fix so that Continuation/Fuzz.java honors the timeoutFactor JTREG setting
> when waiting for a compilation to finish.
> 
> This fix is being tested in my jdk-20+10 stress testing run.
> 
> The usual Mach5 timeoutFactor is 4.0 with slower configurations using a timeoutFactor
> of 10.0. In my stress testing, I use release-bits: 4.0, fastdebug-bits: 6.0 and slowdebug-bits: 12.0.

> So you're trying to make sure we have a minimum COMPILATION_TIMEOUT value of 5 seconds,
but I'm not sure why you want that.

With the current patch, `COMPILATION_TIMEOUT` may become less than 5_000, which may lead to more failures on various platforms.
That's why I suggest let `COMPILATION_TIMEOUT >= 5_000`, which won't make things worse.

Thanks.

> Definitely not. Traditional default value for no specified timeoutFactor value is 1.0.

Okay, you're right.
This is because the timeoutFactor may be used by other tests too.

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

PR: https://git.openjdk.org/jdk/pull/9844