RFR: 8373367: interp-only mechanism fails to work for carrier threads in a corner case [v2]

David Holmes dholmes at openjdk.org
Fri Jan 30 00:00:53 UTC 2026


On Thu, 29 Jan 2026 23:56:32 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review: some tweaks for carrier thread JvmtiThreadState initialization
>
> src/hotspot/share/prims/jvmtiThreadState.cpp line 61:
> 
>> 59: 
>> 60:   if (JvmtiEnvBase::is_thread_carrying_vthread(thread, thread_oop)) {
>> 61:     // Carrier and virtual threads can temporary share the same JavaThread.
> 
> Suggestion:
> 
>     // Carrier and virtual threads can temporarily share the same JavaThread.

Also I think this comment block should come before the if statement as it seems to be explaining both the if and else cases.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29436#discussion_r2744016478


More information about the hotspot-dev mailing list