On Wed, 4 May 2022 03:40:03 GMT, Alex Menkov <amen...@openjdk.org> wrote:

>> The test counts all "system" threads before the execution and expects that 
>> this number remains the same during test execution.
>> This makes the test fragile - JVM may start internal threads, some threads 
>> may end.
>> 
>> The fix updates the test:
>> - the test checks only test threads, and verify that the live threads are 
>> reported by GetAllThreads and terminated threads are not reported;
>> - dropped "system" thread counting stuff;
>> - added proper deallocation of GetThreadInfo results.
>
> Alex Menkov has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   decreased scope of the variables

Update looks good.
Thanks,
Serguei

-------------

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8512

Reply via email to