On Wed, 8 Jul 2026 21:58:38 GMT, Patricio Chilano Mateo <[email protected]> wrote:
>> Agreed > > This `do_vthread` method is only called for the case where the target is an > instance of `java_lang_VirtualThread` > [[1]](https://github.com/openjdk/jdk/blob/1911bd7782e075f01eca7578c07d3f805c41c21d/src/hotspot/share/prims/jvmtiEnvBase.cpp#L2016). > Method `JvmtiEnvBase::is_vthread_suspended` only uses the oop to check for > suspension. That is a bit too subtle and we should not have to inspect `is_vthread_suspended` to reason about the safety of passing null in this case. Moving the null check seems prudent. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31759#discussion_r3548343399
