On Mon, 23 Jun 2025 17:04:37 GMT, Leonid Mesnik <lmes...@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
