RFR: 8203359: Container level resources events [v8]
Jaroslav Bachorik
jbachorik at openjdk.java.net
Wed Apr 14 12:06:14 UTC 2021
On Wed, 14 Apr 2021 11:17:14 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:
>> Jaroslav Bachorik has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision:
>>
>> - Roll back conditional registration of container events
>> - Remove container events flag
>> - Remove trailing spaces
>> - Doh
>> - Report container type and register events conditionally
>> - Remove unused test files
>> - Initial test support for JFR container events
>> - Update the JFR control files
>> - Split off the CPU throttling metrics
>> - Formatting spaces
>> - ... and 1 more: https://git.openjdk.java.net/jdk/compare/b72abe91...67a61bd7
>
> src/jdk.jfr/share/classes/jdk/jfr/events/ContainerMemoryUsageEvent.java line 46:
>
>> 44: public final class ContainerMemoryUsageEvent extends AbstractJDKEvent {
>> 45: @Label("Memory Pressure")
>> 46: @Description("(attempts per second * 1000), if enabled, that the operating system tries to satisfy a memory request for any " +
>
> This unit seems a bit strange. Do we really need to multiply by 1000?
This is taken as reported by cgroups - I didn't want to change the semantics so it does not confuse people familiar with cgroups.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3126
More information about the hotspot-jfr-dev
mailing list