[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150729005332.GB17938@Sligo.logfs.org>
Date: Tue, 28 Jul 2015 17:53:32 -0700
From: Jörn Engel <joern@...estorage.com>
To: David Rientjes <rientjes@...gle.com>
Cc: Mike Kravetz <mike.kravetz@...cle.com>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: hugetlb pages not accounted for in rss
On Tue, Jul 28, 2015 at 04:30:19PM -0700, David Rientjes wrote:
>
> It's not only the oom killer, I don't believe hugeltb pages are accounted
> to the "rss" in memcg. They use the hugetlb_cgroup for that. Starting to
> account for them in existing memcg deployments would cause them to hit
> their memory limits much earlier. The "rss_huge" field in memcg only
> represents transparent hugepages.
>
> I agree with your comment that having done this when hugetlbfs was
> introduced would have been optimal.
>
> It's always difficult to add a new class of memory to an existing metric
> ("new" here because it's currently unaccounted).
>
> If we can add yet another process metric to track hugetlbfs memory mapped,
> then the test could be converted to use that. I'm not sure if the
> jusitifcation would be strong enough, but you could try.
Well, we definitely need something. Having a 100GB process show 3GB of
rss is not very useful. How would we notice a memory leak if it only
affects hugepages, for example?
Jörn
--
The object-oriented version of 'Spaghetti code' is, of course, 'Lasagna code'.
(Too many layers).
-- Roberto Waltman.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists