infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-01-28 03:58:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

8372809: Test vmTestbase/nsk/jdi/ThreadReference/isSuspended/issuspended001/TestDescription.java failed: JVMTI_ERROR_THREAD_NOT_ALIVE

Browse Source 
Reviewed-by: amenkov, sspitsyn
This commit is contained in:
Chris Plummer 2025-12-03 16:37:10 +00:00
parent af8977e406
commit c432150397
1 changed files with 17 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/jdk.jdwp.agent/share/native/libjdwp/threadControl.c
View File

@ -157,11 +157,20 @@ setThreadLocalStorage(jthread thread, ThreadNode *node)
error = JVMTI_FUNC_PTR(gdata->jvmti,SetThreadLocalStorage)
(gdata->jvmti, thread, (void*)node);
if ( error == JVMTI_ERROR_THREAD_NOT_ALIVE && node == NULL) {
/* Just return. This can happen when clearing the TLS. */
return;
} else if ( error != JVMTI_ERROR_NONE ) {
/* The jthread object must be valid, so this must be a fatal error */
if (error == JVMTI_ERROR_THREAD_NOT_ALIVE) {
if (node == NULL) {
// Just return. This can happen when clearing the TLS.
return;
}
if (isVThread(thread)) {
// Just return. This can happen with a vthread that is running and we
// had to create a ThreadNode for it. By the time we get here, it may
// have already terminated.
return;
}
}
if (error != JVMTI_ERROR_NONE) {
// The jthread object must be valid, so this must be a fatal error.
EXIT_ERROR(error, "cannot set thread local storage");
}
}
@ -251,9 +260,10 @@ findThread(ThreadList *list, jthread thread)
* Otherwise the thread should not be on the runningThreads.
*/
if ( !gdata->jvmtiCallBacksCleared ) {
/* The thread better not be on either list if the TLS lookup failed. */
// The thread better not be on the runningThreads list if the TLS lookup failed.
// It might be on the runningVThreads list because of how ThreadNodes for vthreads
// can be recreated just before terminating, so we don't check runningVThreads.
JDI_ASSERT(!nonTlsSearch(getEnv(), &runningThreads, thread));
JDI_ASSERT(!nonTlsSearch(getEnv(), &runningVThreads, thread));
} else {
/*
* Search the runningThreads and runningVThreads lists. The TLS lookup may have