On Fri, 8 Sep 2023 03:03:56 GMT, Leonid Mesnik <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains two additional
>> commits since the last revision:
>>
>> - Merge
>> - 8312174: missing JVMTI events from vthreads parked during JVMTI attach
>
> test/hotspot/jtreg/serviceability/jvmti/vthread/VThreadEventTest/VThreadEventTest.java
> line 86:
>
>> 84: log("WARNING: test expects at least 8 processors.");
>> 85: }
>> 86: Counter ready1 = new Counter(THREAD_CNT);
>
> I think that CountDownLatch should works fine here and no need to use custom
> Counter.
Thanks. Replaced it with the CountDownLatch now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15467#discussion_r1320422749
