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