RFR: 8296324: JVMTI GetStackTrace truncates vthread stack trace for agents loaded into running VM [v3]

[Serguei Spitsyn]

On Fri, 18 Nov 2022 19:34:19 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   ajust condition when init_static_notify_jvmti_events() is called
>
> src/hotspot/share/prims/jvmtiExport.cpp line 390:
> 
>> 388:       java_lang_VirtualThread::init_static_notify_jvmti_events();
>> 389:     }
>> 390:   }
> 
> Yes, this looks good now. Removing the `if (!java_lang_VirtualThread ::notify_jvmti_events())` check means the `init_static_notify_jvmti_events()` can still be called on subsequent calls to this method. So if `init_static_notify_jvmti_events()` was not called the first time (due to not being in the LIVE phase), then it can still be called on subsequent calls to this method if not in the LIVE phase.

Okay. Thank you for the review, Chris,

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

PR: https://git.openjdk.org/jdk/pull/11204