RFR: 8264605: vmTestbase/nsk/jvmti/SuspendThread/suspendthrd003/TestDescription.java failed with "agent_tools.cpp, 471: (foundThread = (jthread) jni_env->NewGlobalRef(foundThread)) != NULL"

[David Holmes]

On Tue, 23 Nov 2021 16:01:27 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> The agentThread variable might have been set by the new agentThread iteration and then deleted in the wrapper from the previous iteration.
>> 
>> The deletion of agentThread variable should be synced with the termination of the agent thread.
>
> The new thread is started in nsk_jvmti_runAgentThread which is executed under monitors also as well as nsk_jvmti_resetAgentData. 
> Could you please point exact location which is not protected? Seems all thread_state changes and agentThread changes are protected.

Sorry @lmesnik I missed the higher-level use of the monitor when starting the thread.

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

PR: https://git.openjdk.java.net/jdk/pull/6515