RFR: 8307653: Adjust delay time and gc log argument in TestAbortOnVMOperationTimeout [v2]

[Daniel D. Daugherty]

On Wed, 10 May 2023 05:16:20 GMT, Guoxiong Li <gli at openjdk.org> wrote:

>> test/hotspot/jtreg/runtime/Safepoint/TestAbortOnVMOperationTimeout.java line 35:
>> 
>>> 33:  * @modules java.base/jdk.internal.misc
>>> 34:  *          java.management
>>> 35:  * @run driver/timeout=600 TestAbortOnVMOperationTimeout
>> 
>> Is this change really needed? Do you see timeouts without it?
>
> Yes, local env shows `time out`. The jtreg's default timeout is `480s`, so I increase it to `600s`.

Not quite. The default JTREG timeout is 120 seconds and the default timeoutFactor
value that we use is 4 for a total timeout value of 480 seconds. If your test runs
indicate the need for a total timeout value of 600 seconds, then you should use:

 * @run driver/timeout=150

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13878#discussion_r1190046321