RFR: 8157023: Integrate NMT with JFR [v4]
Stefan Johansson
sjohanss at openjdk.org
Wed Dec 7 15:05:14 UTC 2022
On Wed, 7 Dec 2022 13:45:43 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:
>> Stefan Johansson has updated the pull request incrementally with three additional commits since the last revision:
>>
>> - Rename the files
>> - Renaming and updating comments
>> - tstuefe review - no rename
>
> src/hotspot/share/services/memJfrReporter.hpp line 45:
>
>> 43: // Helper class to avoid refreshing the NMTUsage to often and allow
>> 44: // the two JFR events to use the same data.
>> 45: class MemJFRCurrentUsage : public AllStatic {
>
> Move to .cpp file to because only used internally. Does not even have to be a class, can only be static fields and accessors. But no big deal, at your will.
Moved it but kept the class structure to better show how things hang together :)
-------------
PR: https://git.openjdk.org/jdk/pull/11449
More information about the hotspot-dev
mailing list