RFR: 8359437: Make users and test suite not able to set LockingMode flag [v2]
Anton Artemov
duke at openjdk.org
Tue Jun 24 09:18:17 UTC 2025
On Mon, 23 Jun 2025 17:04:37 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:
>> Anton Artemov has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - 8359437: Addressed reviewers' comments
>> - 8359437: Addressed reviewers' comments
>
> test/hotspot/jtreg/gtest/LockStackGtests.java line 26:
>
>> 24:
>> 25: /* @test
>> 26: * @summary Run LockStack gtests with LockingMode=2
>
> All gtests are executed with default vm flags in GTestWrapper.java so this while test should be just removed.
Addressed in the latest commits.
> test/hotspot/jtreg/runtime/Monitor/ConcurrentDeflation.java line 37:
>
>> 35: * @bug 8318757
>> 36: * @summary Test concurrent monitor deflation by MonitorDeflationThread and thread dumping
>> 37: * @library /test/lib /
>
> The '/' shouldn't be required for whitebox. Could you please remove it.
Addressed in the latest commits.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25847#discussion_r2163386476
PR Review Comment: https://git.openjdk.org/jdk/pull/25847#discussion_r2163386162
More information about the hotspot-dev
mailing list