From mboxrd@z Thu Jan 1 00:00:00 1970 From: KOSAKI Motohiro Subject: Re: [PATCH] meminfo: show /proc/meminfo base on container's memcg Date: Thu, 31 May 2012 02:37:25 -0400 Message-ID: <4FC711A5.4090003@gmail.com> References: <1338260214-21919-1-git-send-email-gaofeng@cn.fujitsu.com> <4FC6B68C.2070703@jp.fujitsu.com> <4FC6BC3E.5010807@jp.fujitsu.com> <4FC6C111.2060108@jp.fujitsu.com> <4FC6D881.4090706@jp.fujitsu.com> <4FC70355.70805@jp.fujitsu.com> <4FC70E5E.1010003@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: David Rientjes Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, mhocko-AlSwsSmVLrQ@public.gmane.org, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, hannes-druUgvl0LCNAfugRpC6u6w@public.gmane.org, cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, KOSAKI Motohiro , akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org List-Id: containers.vger.kernel.org (5/31/12 2:28 AM), David Rientjes wrote: > On Thu, 31 May 2012, KOSAKI Motohiro wrote: > >>> An application should always know the cgroup that its attached to and be >>> able to read its state using the command that I gave earlier. >> >> No. you don't need why userland folks want namespaces. Even though you don't >> need namespaces. It doesn't good reason to refuse another use case. >> > > 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. But anyway, I dislike current implementaion. So, I NAK this patch too.