[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4FC72CA4.6080708@parallels.com>
Date: Thu, 31 May 2012 12:32:36 +0400
From: Glauber Costa <glommer@...allels.com>
To: Gao feng <gaofeng@...fujitsu.com>
CC: KOSAKI Motohiro <kosaki.motohiro@...il.com>,
David Rientjes <rientjes@...gle.com>,
Kamezawa Hiroyuki <kamezawa.hiroyu@...fujitsu.com>,
<hannes@...xchg.org>, <mhocko@...e.cz>, <bsingharora@...il.com>,
<akpm@...ux-foundation.org>, <linux-kernel@...r.kernel.org>,
<cgroups@...r.kernel.org>, <linux-mm@...ck.org>,
<containers@...ts.linux-foundation.org>
Subject: Re: [PATCH] meminfo: show /proc/meminfo base on container's memcg
On 05/31/2012 11:58 AM, Gao feng wrote:
>> > It's one of a option. But, I seriously doubt fuse can make simpler than kamezawa-san's
>> > idea. But yeah, I might NACK kamezawa-san's one if he will post ugly patch.
>> >
> It seams I should do some homework to make the implement beautifully.
>
> I think kamezawa-san's idea is more simpler.
> thanks for your advice.
>
One think to keep in mind: A file in memcg does not need to follow the
same format of /proc/meminfo so we can bind mount. We should be able to
reconstruct that in userspace based on information available from the
kernel. You can even collect that from multiple locations, and *then*
you bind mount.
It helps to keep the churn out of the kernel, and in case of meminfo,
you might need no extra kernel patches at all. And in the case of other
files like /proc/stat, the relevant information comes from more than one
cgroup anyway, so there is not too much way around it.
--
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