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

Guoxiong Li gli at openjdk.org
Wed May 10 05:19:26 UTC 2023


On Tue, 9 May 2023 22:17:50 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove unnecessary loop. Add bug id. Adjust the action time out.
>
> 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`.

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

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


More information about the hotspot-runtime-dev mailing list