RFR: 8329351: Add runtime/Monitor/TestRecursiveLocking.java for recursive Java monitor stress testing [v4]
Daniel D. Daugherty
dcubed at openjdk.org
Mon Nov 25 21:36:38 UTC 2024
On Fri, 22 Nov 2024 12:13:23 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> 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.
It should be easy enough to add a comment to the wrapper to explain why the suffix is 36M. Yes, there is quite a bit of precedent for including the expected duration in the name of the wrapper.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22238#discussion_r1857368172
More information about the hotspot-runtime-dev
mailing list