From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail203.messagelabs.com (mail203.messagelabs.com [216.82.254.243]) by kanga.kvack.org (Postfix) with SMTP id A644C6B006A for ; Tue, 5 Jan 2010 22:21:51 -0500 (EST) Received: from m2.gw.fujitsu.co.jp ([10.0.50.72]) by fgwmail5.fujitsu.co.jp (Fujitsu Gateway) with ESMTP id o063Lmsr018140 for (envelope-from kamezawa.hiroyu@jp.fujitsu.com); Wed, 6 Jan 2010 12:21:48 +0900 Received: from smail (m2 [127.0.0.1]) by outgoing.m2.gw.fujitsu.co.jp (Postfix) with ESMTP id 8DC7845DE51 for ; Wed, 6 Jan 2010 12:21:48 +0900 (JST) Received: from s2.gw.fujitsu.co.jp (s2.gw.fujitsu.co.jp [10.0.50.92]) by m2.gw.fujitsu.co.jp (Postfix) with ESMTP id 4E4A445DE5D for ; Wed, 6 Jan 2010 12:21:48 +0900 (JST) Received: from s2.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s2.gw.fujitsu.co.jp (Postfix) with ESMTP id 257C31DB803F for ; Wed, 6 Jan 2010 12:21:48 +0900 (JST) Received: from ml14.s.css.fujitsu.com (ml14.s.css.fujitsu.com [10.249.87.104]) by s2.gw.fujitsu.co.jp (Postfix) with ESMTP id B04BDE38001 for ; Wed, 6 Jan 2010 12:21:47 +0900 (JST) Date: Wed, 6 Jan 2010 12:18:36 +0900 From: KAMEZAWA Hiroyuki Subject: Re: [PATCH -mm] Shared Page accounting for memory cgroup (v2) Message-Id: <20100106121836.40f3b3c0.kamezawa.hiroyu@jp.fujitsu.com> In-Reply-To: <20100106030752.GI3059@balbir.in.ibm.com> References: <20100105185226.GG3059@balbir.in.ibm.com> <20100106090708.f3ec9fd8.kamezawa.hiroyu@jp.fujitsu.com> <20100106030752.GI3059@balbir.in.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org To: balbir@linux.vnet.ibm.com Cc: Andrew Morton , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" List-ID: On Wed, 6 Jan 2010 08:37:52 +0530 Balbir Singh wrote: > * KAMEZAWA Hiroyuki [2010-01-06 09:07:08]: > > > On Wed, 6 Jan 2010 00:22:26 +0530 > > Balbir Singh wrote: > > > > > Hi, All, > > > > > > No major changes from v1, except for the use of get_mm_rss(). > > > Kamezawa-San felt that this can be done in user space and I responded > > > to him with my concerns of doing it in user space. The thread > > > can be found at http://thread.gmane.org/gmane.linux.kernel.mm/42367. > > > > > > If there are no major objections, can I ask for a merge into -mm. > > > Andrew, the patches are against mmotm 10 December 2009, if there > > > are some merge conflicts, please let me know, I can rebase after > > > you release the next mmotm. > > > > > > > The problem is that this isn't "shared" uasge but "considered to be shared" > > usage. Okay ? > > > > Could you give me your definition of "shared". From the mem cgroup > perspective, total_rss (which is accumulated) subtracted from the > count of pages in the LRU which are RSS and FILE_MAPPED is shared, no? You consider only "mapped" pages are shared page. That's wrong. And let's think about your "total_rss - RSS+MAPPED" In this typical case, fork() ---- process(A) -> fork() --- process(B) -> process(C) total_rss = rss(A) + rss(B) + rss(C) = 3 * rss(A) Then, total_rss - RSS_MAPPED = 2 * rss(A). How we call this number ? Is this "shared usage" ? I think no. If you want to do this, scan LRU and count the number of really shared pages. It's much better than detecting "shared pages" via process and will have no big issue if implemented in proper way. > I understand that some of the pages that might be shared, show up > in our LRU and accounting. These are not treated as shared by > our cgroup, but by other cgroups. > > > Then I don't want to provide this misleading value as "official report" from > > the kernel. And this can be done in userland. > > > > I explained some of the issues of doing this from user space, would > you be OK if I called them "non-private" pages? > I think I explained there is no issue to do this in user-land. Thanks, -Kame -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org