RFR: JDK-8317772: NMT: Make peak values available in release builds
Johan Sjölen
jsjolen at openjdk.org
Thu Oct 12 09:16:15 UTC 2023
On Tue, 10 Oct 2023 12:18:15 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> We added peak counts to NMT with [JDK-8297958](https://bugs.openjdk.org/browse/JDK-8297958). That feature is *extremely* helpful in analyzing native memory footprints since it gives you a glance into the past. For instance, it makes libc memory retention easier to explain.
>
> Initially, I restricted this feature to debug JVMs out of vague concerns about performance when running with NMT. However, after measuring, it turns out that the increase in cost is very small. This makes sense since we only update the peak values when we actually reach a peak.
>
> Since the payoff is huge, it makes sense to have this feature always enabled. We don't need a new switch for that. As a side effect, NMT code becomes a bit clearer since we can lose a number of assert ifdefs.
Hi,
I'm happy with this going in, thanks.
-------------
Marked as reviewed by jsjolen (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16119#pullrequestreview-1673685684
More information about the hotspot-runtime-dev
mailing list