RFR: 8238665: Add JFR event for direct memory statistics

Alan Bateman Alan.Bateman at oracle.com
Wed Mar 18 15:46:17 UTC 2020



On 13/03/2020 14:54, Denghui Dong wrote:
> Good suggestion, moved.
> Webrev: http://cr.openjdk.java.net/~ddong/8238665/webrev.03/
>
This looks much better.

What would you think about renaming the JFR event to 
"DirectBufferStatistics"? The concern I have with the proposed naming is 
that it will be really awkward to extend it to support mapped buffers.

The implementation changes look okay, hopefully Erik will skim over 
them. One small suggestion for for ManagementFactoryHelper is that you 
can stream().collect(Collectors.toList()) to create the value for 
bufferPools. You could even change it to volatile so that 
getBufferMXBeans isn't a synchronized method.

-Alan.


More information about the hotspot-jfr-dev mailing list