RFR: 8157023: Integrate NMT with JFR [v3]

Stefan Johansson sjohanss at openjdk.org
Tue Dec 6 15:02:32 UTC 2022


On Tue, 6 Dec 2022 14:56:44 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> I assume you don't think `NMTSnapshot` is good much better =) I can't come up with something much better than your suggestions, `NMTUsage` is not perfect either (sound like the usage cause by NMT). If we don't go with the NMT-prefix I think something like `MemUsage`or `NativeMemoryUsage` would make sense. Looking at the other reporting classes connected to NMT they are named `Mem`-something. So I think `MemUsage`, what do you think?
>
> Good. Up to you.
> 
> I wish we would allow C++ namespaces, having all in something like "namespace nmt". Then it does not hurt so much if names are a bit more generic.

I'll go with `MemUsage`, normally I would have spent some more time trying to come up with something better, but Thursday is getting closer :)

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

PR: https://git.openjdk.org/jdk/pull/11449


More information about the hotspot-dev mailing list