Re: [PATCH] smaps: fill missing fields for vma(VM_HUGETLB)

From: Naoya Horiguchi
Date: Thu Aug 06 2015 - 03:45:52 EST


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