> This bug was introduced by my recent changes for > [JDK-8265028](https://bugs.openjdk.java.net/browse/JDK-8265028), which > attempted to speed up ThreadNode lookups by not looking in the runningThreads > list if the TLS lookup failed. At the time it was thought that the thread > could not possibly be on the list, but it turns out sometimes it can. > > For now I'm just doing a quick fix to replace the assert being triggered with > a lookup instead, which is pretty much how it worked before JDK-8265028. > However, I eventually want to get back to not having to do the lookup, but > first I need to better understand why this is happening in the first place, > and the tests are failing too often to wait for that, thus the quick fix.
Chris Plummer has updated the pull request incrementally with one additional commit since the last revision: Minor formatting fix. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3634/files - new: https://git.openjdk.java.net/jdk/pull/3634/files/784962da..b65a64af Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3634&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3634&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/3634.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3634/head:pull/3634 PR: https://git.openjdk.java.net/jdk/pull/3634
