RFR: 8364314: java_lang_Thread::get_thread_status fails assert(base != nullptr) failed: Invalid base [v2]

[David Holmes]

On Thu, 31 Jul 2025 14:12:19 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> David Holmes 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 three additional commits since the last revision:
>> 
>>  - Remove from ProblemList
>>  - Merge branch 'master' into 8364314-threadSMR
>>  - 8364314: java_lang_Thread::get_thread_status fails assert(base != nullptr) failed: Invalid base
>
> src/hotspot/share/services/threadService.cpp line 1281:
> 
>> 1279:         // mounted vthread, use carrier thread state
>> 1280:         oop carrier_thread = java_lang_VirtualThread::carrier_thread(_thread_h());
>> 1281:         assert(carrier_thread != nullptr, "should only get here for a mounted vthread");
> 
> This assert will trigger if unmounted when the handshake op runs. This is one of the transition cases. In the loom this is now changed to return so that the caller retries.

Yes but I am not trying to address the case of the disabler not functioning.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26544#discussion_r2250145163