On Wed, 15 Feb 2023 10:50:11 GMT, Thomas Stuefe 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
On Wed, 15 Feb 2023 10:50:11 GMT, Thomas Stuefe 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
> 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.
>
> Test
> 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.
>
> Test
On Wed, 15 Feb 2023 07:49:24 GMT, David Holmes wrote:
> Given the reason this bug likely arose is the fact the parameters to
> `calculate_jfr_stats` say they are "bytes" then fixing that is not a mere
> cleanup IMO. Can we at least fix that in this PR?
Hi David, I reformed the ill-named functi
On Tue, 14 Feb 2023 13:40:40 GMT, Thomas Stuefe 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
On Wed, 15 Feb 2023 05:49:14 GMT, David Holmes wrote:
> That said, given:
>
> ```
> // This only exists for JFR and jcmd VM.classloader_stats. We may want to
> // change this. Capacity as a stat is of questionable use since it may
> // contain committed and uncommitted areas. For now we do thi
On Tue, 14 Feb 2023 13:40:40 GMT, Thomas Stuefe 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
On Tue, 14 Feb 2023 13:40:40 GMT, Thomas Stuefe 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
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
10 matches
Mail list logo
