RFR: 8298992: runtime/NMT/SummarySanityCheck.java failed with "Total commi…tted (MMMMMM) did not match the summarized committed (NNNNNN)

[Gerard Ziemski]

On Wed, 16 Aug 2023 12:18:35 GMT, Afshin Zafari <azafari at openjdk.org> wrote:

> During exhaustive tests, it is observed that during taking snapshot of NMT metrics it is possible that new allocations happen concurrently, although a `ThreadCritical` is used during copying current metrics to the snapshot.
> A loop is surrounding the copying and checks whether the copied and original are the same.

Changes requested by gziemski (Committer).

src/hotspot/share/services/mallocTracker.hpp line 205:

> 203:       }
> 204:     } while(s->_all_mallocs.size() != total_size && ++loop_counter < loop_limit);
> 205:     assert(s->_all_mallocs.size() == total_size, "Total != sum of parts");

Do we agree then that the assert on line 205 is not needed?

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

PR Review: https://git.openjdk.org/jdk/pull/15306#pullrequestreview-1590192014
PR Review Comment: https://git.openjdk.org/jdk/pull/15306#discussion_r1302107483