On Fri, 28 Feb 2025 01:40:34 GMT, Jiangli Zhou <jian...@openjdk.org> wrote:

> Please review the test fix that removes `libVThreadEventTest` explicit 
> dependency to `libjvm`, by removing the call to `JNI_GetCreatedJavaVMs` in 
> `Agent_OnAttach`. There is a `vm` argument passed via `Agent_OnAttach`. With 
> the change, `libVThreadEventTest` no longer needs to be linked with 
> `libjvm.so` (or `jvm.dll`). `VThreadEventTest.java` now runs on both dynamic 
> and static JDK.
> 
> Tested on static JDK locally
> GHA: 
> https://github.com/jianglizhou/jdk/actions/runs/13577993687/job/37960245161

Thanks for the reviews and historical info!

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

PR Comment: https://git.openjdk.org/jdk/pull/23835#issuecomment-2695103531

Reply via email to