On Wed, 13 Mar 2024 16:40:33 GMT, Richard Reingruber <rr...@openjdk.org> wrote:

> @MBaesken found 2 more locations in jvmti that need switching to `WXWrite`
> 
> JvmtiExport::get_jvmti_interface GetCarrierThread
> 
> Both use `ThreadInVMfromNative`.

Should we address those 2 more findings in this PR ? Or open a separate JBS 
issue ?

btw those were the jtreg tests triggering the 2 additional findings / asserts 

runtime/Thread/AsyncExceptionOnMonitorEnter.java
runtime/Thread/AsyncExceptionTest.java
serviceability/jvmti/RedefineClasses/RedefineSharedClassJFR.java
runtime/handshake/HandshakeDirectTest.java
runtime/handshake/SuspendBlocked.java
runtime/jni/terminatedThread/TestTerminatedThread.java
runtime/lockStack/TestStackWalk.java
serviceability/jvmti/vthread/GetThreadState/GetThreadStateTest.java#default
serviceability/jvmti/vthread/GetThreadState/GetThreadStateTest.java#no-vmcontinuations
serviceability/jvmti/vthread/GetThreadStateMountedTest/GetThreadStateMountedTest.java
serviceability/jvmti/vthread/RawMonitorTest/RawMonitorTest.java
serviceability/jvmti/vthread/SuspendWithInterruptLock/SuspendWithInterruptLock.java#default
serviceability/jvmti/vthread/SuspendWithInterruptLock/SuspendWithInterruptLock.java#xint
serviceability/jvmti/vthread/ThreadStateTest/ThreadStateTest.java
serviceability/jvmti/vthread/WaitNotifySuspendedVThreadTest/WaitNotifySuspendedVThreadTest.java

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

PR Comment: https://git.openjdk.org/jdk/pull/18238#issuecomment-1996996689

Reply via email to