[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.1205302351510.25774@chino.kir.corp.google.com>
Date: Wed, 30 May 2012 23:56:13 -0700 (PDT)
From: David Rientjes <rientjes@...gle.com>
To: KOSAKI Motohiro <kosaki.motohiro@...il.com>
cc: Kamezawa Hiroyuki <kamezawa.hiroyu@...fujitsu.com>,
Gao feng <gaofeng@...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 Thu, 31 May 2012, KOSAKI Motohiro wrote:
> > This is tangent to the discussion, we need to revisit why an application
> > other than a daemon managing a set of memcgs would ever need to know the
> > information in /proc/meminfo. No use-case was ever presented in the
> > changelog and its not clear how this is at all relevant. So before
> > changing the kernel, please describe how this actually matters in a real-
> > world scenario.
>
> Huh? Don't you know a meanings of a namespace ISOLATION? isolation mean,
> isolated container shouldn't be able to access global information. If you
> want to lean container/namespace concept, tasting openvz or solaris container
> is a good start.
>
As I said, LXC and namespace isolation is a tangent to the discussion of
faking the /proc/meminfo for the memcg context of a thread.
> But anyway, I dislike current implementaion. So, I NAK this patch too.
>
I'm glad you reached that conclusion, but I think you did so for a much
different (although unspecified) reason.
Thanks.
--
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