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