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