Messages in this thread | | | From | Naoya Horiguchi <> | Subject | Re: [PATCH] smaps: fill missing fields for vma(VM_HUGETLB) | Date | Thu, 6 Aug 2015 07:44:44 +0000 |
| |
On Wed, Aug 05, 2015 at 07:18:44PM -0700, David Rientjes wrote: ... > Hmm, wouldn't this be confusing since VmRSS in /proc/pid/status doesn't > match the rss shown in smaps, since hugetlb mappings aren't accounted in > get_mm_rss()? > > Not sure this is a good idea, I think consistency amongst rss values would > be more important.
Right, so one option is making get_mm_rss() count hugetlb, but that could make oom/memcg less efficient or broken as you stated in a previous email. So another one is to add "VmHugetlbRSS:" field in /proc/pid/status?
Thanks, Naoya Horiguchi | |