On Sun, 26 May 2024 14:24:27 GMT, SendaoYan <s...@openjdk.org> wrote:

> > That would mean it's not tested. I suspect the 
> > java_lang_VirtualThread::is_instance checks will need to be changed to test 
> > with is_a(vmClasses::BaseVirtualThread_klass()) to allow for the 
> > alternative implementation.
> 
> Do you mean change like this:

No, I meant in JvmtiEnvBase::get_object_monitor_usage that you'll need to go 
through all places where it tests if the thread is a virtual thread, they will 
need to be changed to support the alternative implementation.

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

PR Comment: https://git.openjdk.org/jdk/pull/19405#issuecomment-2132281742

Reply via email to