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