On Wed, 8 Nov 2023 16:02:08 GMT, Serguei Spitsyn <[email protected]> wrote:
>> src/hotspot/share/prims/jvmtiEnvBase.cpp line 2416:
>>
>>> 2414: if (!JvmtiEnvBase::is_vthread_alive(_target_h())) {
>>> 2415: return; // JVMTI_ERROR_THREAD_NOT_ALIVE (default)
>>> 2416: }
>>
>> Don't we have this check already in JvmtiHandshake::execute()? Same with the
>> other converted functions.
>
> Good suggestion, thanks.
> I'm a little bit paranoid about terminated vthreads. :)
> Will try to get rid of it and retest all tiers.
I've removed the extra checks for `JvmtiEnvBase::is_vthread_alive()`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16460#discussion_r1396115170
