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