[jira] [Commented] (MESOS-2104) Correct naming of cgroup memory statistics

["Ian Downes (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MESOS-2104?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14217125#comment-14217125 ] 

Ian Downes commented on MESOS-2104:
-----------------------------------

Proposed fields (not included field numbers, yet):

{noformat}
// Current fields:
optional uint64 mem_rss_bytes = 5;          // memory.usage_in_bytes
optional uint64 mem_limit_bytes = 6;        // Resources::mem()
optional uint64 mem_file_bytes = 10;        // memory.stat: total_cache
optional uint64 mem_anon_bytes = 11;        // memory.stat: total_rss
optional uint64 mem_mapped_file_bytes = 12; // memory.stat: total_mapped_file
 
// ------------------------------------------------------------------------- //
 
// Suggested fields:
optional uint64 mem_limit_bytes;             // memory.limit_in_bytes
optional uint64 mem_soft_limit_bytes;        // memory.soft_limit_in_bytes
 
optional uint64 mem_total_bytes;             // memory.usage_in_bytes
optional uint64 mem_total_memsw_bytes;       // memory.memsw.usage_in_bytes
 
// Report the total usage, i.e., including any child cgroups. If child
// cgroups are used then the application inside the container should
// report the child cgroup statistics.
optional uint64 mem_rss_bytes;              // memory.stat: total_rss
optional uint64 mem_cache_bytes;            // memory.stat: total_cache
optional uint64 mem_mapped_file_bytes;      // memory.stat: total_mapped_file_bytes
optional uint64 mem_swap_bytes;             // memory.stat: total_swap
{noformat}

> Correct naming of cgroup memory statistics
> ------------------------------------------
>
>                 Key: MESOS-2104
>                 URL: https://issues.apache.org/jira/browse/MESOS-2104
>             Project: Mesos
>          Issue Type: Improvement
>          Components: isolation
>    Affects Versions: 0.20.0
>            Reporter: Ian Downes
>            Assignee: Ian Downes
>              Labels: twitter
>             Fix For: 0.22.0
>
>
> mem_rss_bytes is *not* RSS but is the total memory usage (memory.usage_in_bytes) of the cgroup, including file cache etc. Actual RSS is reported as mem_anon_bytes. These, and others, should be consistently named.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)