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

Guoxiong Li gli at openjdk.org
Thu May 11 18:24:46 UTC 2023


On Thu, 11 May 2023 04:59:02 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 timeout option. Adjust comment. Level up the log to info.
>
> I am okay with these changes.
> 
> I do worry that your slow dev board may expose a lot of internal timeout issues in tests - but tests should be using values modified by the timeoutFactor to compensate for slow systems where possible. Not sure how that would apply in the current case though.

@dholmes-ora @dcubed-ojdk Thanks for the review. Integrating.

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

PR Comment: https://git.openjdk.org/jdk/pull/13878#issuecomment-1544472222


More information about the hotspot-runtime-dev mailing list