RFR: 8157023: Integrate NMT with JFR [v3]
Thomas Stuefe
stuefe at openjdk.org
Tue Dec 6 14:59:07 UTC 2022
On Tue, 6 Dec 2022 13:58:36 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:
>> src/hotspot/share/services/memSnapshot.hpp line 42:
>>
>>> 40: };
>>> 41:
>>> 42: class MemSnapshot : public CHeapObj<mtNMT> {
>>
>> Can we name this not Snapshot but something different? We already have MallocMemorySnapshot and methods like "as_snapshot", which have nothing to do with this layer.
>>
>> proposals:
>>
>> `NMTData`
>> `NMTStatistics`
>> `NMTStatSet`
>>
>> ....
>> ....
>
> 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.
-------------
PR: https://git.openjdk.org/jdk/pull/11449
More information about the hotspot-dev
mailing list