RFR: JDK-8305414: gtest/NMTGtests.java is failing various sub-tests

[Roman Kennke]

On Mon, 3 Apr 2023 12:31:17 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> [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.

Looks good to me. Thanks!

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

Marked as reviewed by rkennke (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13296#pullrequestreview-1369134131