From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx180.postini.com [74.125.245.180]) by kanga.kvack.org (Postfix) with SMTP id 9FC9D6B0078 for ; Wed, 27 Jun 2012 04:42:42 -0400 (EDT) Message-ID: <4FEAC6DA.1010806@parallels.com> Date: Wed, 27 Jun 2012 12:39:54 +0400 From: Glauber Costa MIME-Version: 1.0 Subject: Re: [PATCH 00/11] kmem controller for memcg: stripped down version References: <1340633728-12785-1-git-send-email-glommer@parallels.com> <20120625162745.eabe4f03.akpm@linux-foundation.org> <4FE9621D.2050002@parallels.com> <20120626145539.eeeab909.akpm@linux-foundation.org> In-Reply-To: Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: David Rientjes Cc: Andrew Morton , cgroups@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Frederic Weisbecker , Pekka Enberg , Michal Hocko , Johannes Weiner , Christoph Lameter , devel@openvz.org, kamezawa.hiroyu@jp.fujitsu.com, Tejun Heo On 06/27/2012 05:08 AM, David Rientjes wrote: > On Tue, 26 Jun 2012, Andrew Morton wrote: > >> mm, maybe. Kernel developers tend to look at code from the point of >> view "does it work as designed", "is it clean", "is it efficient", "do >> I understand it", etc. We often forget to step back and really >> consider whether or not it should be merged at all. >> > > It's appropriate for true memory isolation so that applications cannot > cause an excess of slab to be consumed. This allows other applications to > have higher reservations without the risk of incurring a global oom > condition as the result of the usage of other memcgs. Just a note for Andrew, we we're in the same page: The slab cache limitation is not included in *this* particular series. The goal was always to have other kernel resources limited as well, and the general argument from David holds: we want a set of applications to run truly independently from others, without creating memory pressure on the global system. The way history develop in this series, I started from the slab cache, and a page-level tracking appeared on that series. I then figured it would be better to start tracking something that is totally page-based, such as the stack - that already accounts for 70 % of the infrastructure, and then merge the slab code later. In this sense, it was just a strategy inversion. But both are, and were, in the goals. > I'm not sure whether it would ever be appropriate to limit the amount of > slab for an individual slab cache, however, instead of limiting the sum of > all slab for a set of processes. With cache merging in slub this would > seem to be difficult to do correctly. Yes, I do agree. -- 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