On Tue, 14 Feb 2023 13:40:40 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:

> VM.classloader_stats shows metaspace consumption in words, but should show 
> bytes. 
> 
> Patch fixes that, and adjusts the associated jtreg test to catch regressions 
> like this.
> 
> I manually tested the test with the unpatched hotspot to see if the 
> regression shows up, it does.
> 
> Tested the patch on x64 (fastdebug, release) and x86.
> 
> Note: "calculate_jfr_stats" is misnamed, actually only ever used by this 
> dcmd, not by jfr. I will clear this up in a separate RFE.

This pull request has now been integrated.

Changeset: 6e2d3c6c
Author:    Thomas Stuefe <stu...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/6e2d3c6c45ded862d818e30dd03d023980bf0ec2
Stats:     63 lines in 4 files changed: 37 ins; 2 del; 24 mod

8302455: VM.classloader_stats memory size values are wrong

Reviewed-by: coleenp, dholmes

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

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

Reply via email to