On Sun, 26 May 2024 16:43:50 GMT, Alan Bateman <[email protected]> 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.

Thanks, I will check it later.

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

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

Reply via email to