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

[Thomas Stuefe]

On Mon, 28 Aug 2023 08:37:51 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.
>
> Afshin Zafari has updated the pull request incrementally with one additional commit since the last revision:
> 
>   move the copy_to function from header to cpp file.

still good.

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

Marked as reviewed by stuefe (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15306#pullrequestreview-1597815821