RFR: 8288532: additional review changes for JDK-8286830
Daniel D. Daugherty
dcubed at openjdk.org
Fri Jun 17 19:01:54 UTC 2022
On Fri, 17 Jun 2022 17:33:58 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:
>> Update a couple of tests that were modified by JDK-8286830. Now the tests properly
>> honor any specified time limit by splitting the time between both sub-tests.
>> Also pick up a sanity check based on JDK-8288497 that I've been using in recent
>> stress testing.
>
> Hi Dan,
>
> Some comments below.
>
> Thanks,
> Patricio
@pchilano - Thanks for the review!
> test/hotspot/jtreg/runtime/Thread/SuspendAtExit.java line 81:
>
>> 79: }
>> 80: }
>> 81: timeMax /= 2; // Split time between the two sub-tests.
>
> Same thing about DEF_TIME_MAX.
I try not to have the default config for a stress test run longer that 30 seconds total.
15 seconds per sub-test is fine.
-------------
PR: https://git.openjdk.org/jdk19/pull/32
More information about the hotspot-runtime-dev
mailing list