> This fixes a bug in the debug agent when there is a request to suspend a > virtual thread that has already terminated. The issue was that unless the > debug agent was currently under a "suspend all", it would not properly put > the virtual thread on the `otherThreads` list, and instead added it to > `runningVThreads`. This meant at the end of `insertThread()` the following > code tried to do a JVMTI `SetThreadLocalStorage`, which can't be done on a > terminated thread. > > > if (list != &otherThreads) { > setThreadLocalStorage(node->thread, (void*)node); > }
Chris Plummer has updated the pull request incrementally with one additional commit since the last revision: Rename thread to targetThread. Verify targetThread state when the thread dies and when the breakpoint after the join() is reached. ------------- Changes: - all: https://git.openjdk.org/jdk19/pull/88/files - new: https://git.openjdk.org/jdk19/pull/88/files/17e4658e..1ff59060 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk19&pr=88&range=08 - incr: https://webrevs.openjdk.org/?repo=jdk19&pr=88&range=07-08 Stats: 21 lines in 1 file changed: 13 ins; 3 del; 5 mod Patch: https://git.openjdk.org/jdk19/pull/88.diff Fetch: git fetch https://git.openjdk.org/jdk19 pull/88/head:pull/88 PR: https://git.openjdk.org/jdk19/pull/88