Integrated: 8269268: JDWP: Properly fix thread lookup assert in findThread()

Chris Plummer cjplummer at openjdk.java.net
Tue Jun 29 18:49:08 UTC 2021


On Thu, 24 Jun 2021 05:17:02 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

> Re-enable the assert that was disabled (with some overhead) by [JDK-8265683](https://bugs.openjdk.java.net/browse/JDK-8265683). Explanation is in the CR and also in comments included with the changes.
> 
> I tested by running `vmTestbase/nsk/jdb/suspend/suspend001/suspend001.java` and `vmTestbase/nsk/jdb/wherei/wherei001/wherei001.java` 100's of times, and did not see any failures. I also verified the original issue was still reproducible by temporarily not setting `gdata->handlingVMDeath = JNI_TRUE`, which did trigger the assert as expected.

This pull request has now been integrated.

Changeset: 7ca753bf
Author:    Chris Plummer <cjplummer at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/7ca753bf0cb58ff0f82a7e9baee177a6da9f56e7
Stats:     38 lines in 3 files changed: 31 ins; 0 del; 7 mod

8269268: JDWP: Properly fix thread lookup assert in findThread()

Reviewed-by: kevinw, amenkov, sspitsyn

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

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


More information about the serviceability-dev mailing list