RFR: JDK-8261297: NMT: Final report should use scale 1 [v3]

Thomas Stuefe stuefe at openjdk.java.net
Wed Feb 10 15:45:50 UTC 2021


> If PrintNMTStatistics is specified, we write a NMT report to stdout before exiting the VM. This is useful for leak analysis.
> 
> However, the report uses the standard "K" scale, which is shouldn't, since this omits small leaks < 1K. Hence it should use scale=1. This would provide us with a precise exit report. Note that for this to work JDK-8261238 has to be fixed too, which is a separate issue.
> 
> This patch:
> - changes the scale of the final report to 1
> - does some minor cleanup work in the MemReporter class hierarchy (const-ifying scale and the output stream, and establishing 0 as "default scale" as to avoid having the default scale hard coded in four places.

Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:

  Missed another one

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2450/files
  - new: https://git.openjdk.java.net/jdk/pull/2450/files/d2479461..c343316c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2450&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2450&range=01-02

  Stats: 4 lines in 1 file changed: 0 ins; 2 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2450.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2450/head:pull/2450

PR: https://git.openjdk.java.net/jdk/pull/2450


More information about the hotspot-runtime-dev mailing list