Re: RFR: 8323792: ThreadSnapshot::initialize can cause assert in Thread::check_for_dangling_thread_pointer (possibility of dangling Thread pointer) [v9]

[David Holmes]

On Wed, 18 Mar 2026 07:07:05 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Zhengyu Gu has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update ThreadInfoTest.java from @kevinjwalls
>
> test/hotspot/jtreg/serviceability/threads/ThreadInfoTest.java line 141:
> 
>> 139:             while (true) {
>> 140:                 if (replacing) {
>> 141:                     int replaced = replaceThreads(ids, infos);
> 
> The return value is unused and serves no purpose.

And surely this whole loop can simplify to:

while (true) {
    if (count < 10) {
        replaceThreads(ids, infos);
    }
    goSleep(1);
}

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30105#discussion_r2951377977