RFR: 8361103: java_lang_Thread::async_get_stack_trace does not properly protect JavaThread [v4]

[David Holmes]

On Thu, 10 Jul 2025 21:36:53 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

>> The fix updates `java_lang_Thread::async_get_stack_trace()` (used by `java.lang.Thread.getStackTrace()` to get stack trace for platform and mounted virtual threads) to correctly use `ThreadListHandle` for thread protection.
>> 
>> Testing: tier1..5
>
> Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   carrier in TLH

> Added extra argument to cv_internal_thread_to_JavaThread to return carrrier's JavaThread.

@alexmenkov  I think we need a separate JBS issue for this and have runtime fix it. I want to be sure we are fixing it in the right/best way. I don't think we need the extra argument, but need to examine the existing usages. Chances are that any code trying to deal with virtual thread's via the API is actually doing it wrong and will need fixing anyway. I will file a bug.

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

PR Comment: https://git.openjdk.org/jdk/pull/26119#issuecomment-3059286832