RFR: JDK-8305414: gtest/NMTGtests.java is failing various sub-tests [v5]
Thomas Stuefe
stuefe at openjdk.org
Wed Apr 5 05:27:12 UTC 2023
> [JDK-8304815](https://bugs.openjdk.org/browse/JDK-8304815) added the ability to use NMT for better hs-err reporting: we now ask NMT - if enabled - to make sense of otherwise unknown memory locations.
>
> With C-heap blocks, the error printer now prints out a message if a memory location points into a live or dead malloc block. For details, see JDK-8304815 and the comments in mallocTracker.cpp.
>
> The change came with its own gtest, which promptly started to fail intermittently. That failure was caused by a real issue the test found where we accidentally returned from MallocTracker::print_pointer_information() if we encountered unmapped memory (this makes the error intermittent). The return bypassed the block printing code, which made the test fail.
>
> Patch fixes that, and also fixes some debug output in the test.
>
> Tests: Ran the test manually on Linux aarch64; was able to reproduce the errors without patch, no errors with patch.
Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
- Merge branch 'openjdk:master' into JDK-8305414-gtest-NMTGtests.java-is-failing-various-sub-tests
- Merge branch 'master' into JDK-8305414-gtest-NMTGtests.java-is-failing-various-sub-tests
- Revert problemlisting
- Merge branch 'master' into JDK-8305414-gtest-NMTGtests.java-is-failing-various-sub-tests
- Merge branch 'master' into JDK-8305414-gtest-NMTGtests.java-is-failing-various-sub-tests
- JDK-8305414-gtest-NMTGtests.java-is-failing-various-sub-tests
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/13296/files
- new: https://git.openjdk.org/jdk/pull/13296/files/775014c0..613d38a4
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=13296&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=13296&range=03-04
Stats: 2753 lines in 62 files changed: 2008 ins; 427 del; 318 mod
Patch: https://git.openjdk.org/jdk/pull/13296.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/13296/head:pull/13296
PR: https://git.openjdk.org/jdk/pull/13296
More information about the hotspot-runtime-dev
mailing list