RFR: 8373837: "assert(thread->is_VM_thread() || thread->is_Java_thread()) failed: Must be VMThread or JavaThread" on macosx-aarch64-26

[David Holmes]

On Thu, 18 Dec 2025 15:33:38 GMT, Fredrik Bredberg <fbredberg at openjdk.org> wrote:

>> If the SR signal is delivered late, there is a small window where the JavaThread destructor has run (and so the referenced assert will fail), but the Thread destructor has not yet reached the point where `is_terminated` returns true, and so we proceed to the assertion, that then fails. The simple fix is to move the assertion into the branch for an active suspension request, which guarantees we **can't** be executing destructors yet. See JBS for gory details.
>> 
>> EDIT: s/can/can't/
>> 
>> Testing: tiers 1-3 (sanity)
>> 
>> Thanks
>
> Looks good to me.

Thanks for the reviews @fbredber and @coleenp !

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

PR Comment: https://git.openjdk.org/jdk/pull/28883#issuecomment-3673108331