On Fri, 26 May 2023 16:41:32 GMT, Paul Hohensee <p...@openjdk.org> wrote:
>> Please review this addition to com.sun.management.ThreadMXBean that returns
>> the total number of bytes allocated on the Java heap since JVM launch by
>> both terminated and live threads.
>>
>> Because this PR adds a new interface method, I've updated the JMM_VERSION to
>> 4, but would be happy to update it to 3_1 instead.
>
> Paul Hohensee has updated the pull request with a new target base due to a
> merge or a rebase. The pull request now contains 100 commits:
>
> - 8304074: Remove extra file
> - 8304074: Add getTotalAllocatedBytes high water mark
> - Merge branch 'master' into 8304074
> - 8304074: Refactor test exception handling
> - Merge master
> - Merge master
> - 8304074: Change UnsupportedOperationException in javadoc comment to {@code
> UnsupportedOperationException}
> - 8306507: [linux] Print number of memory mappings in error reports
>
> Reviewed-by: adinn, sgehwolf
> - 8300086: Replace NULL with nullptr in share/c1/
>
> Reviewed-by: thartmann, chagedorn
> - 8308465: Reduce memory accesses in AArch64 MD5 intrinsic
>
> Reviewed-by: aph, phh
> - ... and 90 more: https://git.openjdk.org/jdk/compare/77c5adb0...5f2f86bb
Changed the MonitoringSupport_lock comment to "Protects updates to the
serviceability memory pools and allocated memory high water mark". The lock was
introduced to serialize updates to the G1 memory pool stats, but given that
doesn't happen very often, and the name, imo it's reasonable to use it in a
more general fashion.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/13814#issuecomment-1567380618
