RFR: 8329351: Add runtime/Monitor/TestRecursiveLocking.java for recursive Java monitor stress testing [v4]

David Holmes dholmes at openjdk.org
Fri Nov 22 12:16:17 UTC 2024


On Thu, 21 Nov 2024 11:59:18 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> From [JDK-8329351](https://bugs.openjdk.org/browse/JDK-8329351): _"`.../StressWrapper_TestRecursiveLocking_36M.java` is a wrapper that executes `.../TestRecursiveLocking.java` for 2 minutes for each of the 18 configs for a total of 36 minutes of execution time..."_
>> 
>> I think there is some value in looking at a test file's name, and thereby know how long it will take to run.
>> 
>> @dcubed-ojdk created and named the file, maybe he has an opinion.
>
> There's precedent for this naming in our internal tests (runthese and kitchensink).

Sure but if we add/remove a test case we have to rename the test file. It is not obvious what numbers combine to produce 36M.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22238#discussion_r1853823517


More information about the hotspot-dev mailing list