RFR: 8284027: vmTestbase/nsk/jvmti/GetAllThreads/allthr001/ is failing [v3]

Serguei Spitsyn sspitsyn at openjdk.java.net
Wed May 4 23:56:20 UTC 2022


On Wed, 4 May 2022 03:40:03 GMT, Alex Menkov <amenkov at 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


More information about the serviceability-dev mailing list