From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx145.postini.com [74.125.245.145]) by kanga.kvack.org (Postfix) with SMTP id CD7CE6B006C for ; Tue, 4 Sep 2012 10:41:25 -0400 (EDT) Message-ID: <50461241.5010300@parallels.com> Date: Tue, 4 Sep 2012 18:37:53 +0400 From: Glauber Costa MIME-Version: 1.0 Subject: Re: [PATCH v2] memcg: first step towards hierarchical controller References: <1346687211-31848-1-git-send-email-glommer@parallels.com> <20120903170806.GA21682@dhcp22.suse.cz> <5045BD25.10301@parallels.com> <20120904130905.GA15683@dhcp22.suse.cz> <504601B8.2050907@parallels.com> <20120904143552.GB15683@dhcp22.suse.cz> In-Reply-To: <20120904143552.GB15683@dhcp22.suse.cz> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, linux-mm@kvack.org, Dave Jones , Ben Hutchings , Peter Zijlstra , Paul Turner , Lennart Poettering , Kay Sievers , Kamezawa Hiroyuki , Johannes Weiner , Tejun Heo On 09/04/2012 06:35 PM, Michal Hocko wrote: > On Tue 04-09-12 17:27:20, Glauber Costa wrote: >> On 09/04/2012 05:09 PM, Michal Hocko wrote: >>> Not really. Do it slowly means that somebody actually _notices_ that >>> something is about to change and they have a lot of time for that. This >>> will be really hard with the config option saying N by default. People >>> will ignore that until it's too late. >>> We are interested in those users who would keep the config default N and >>> they are (ab)using use_hierarchy=0 in a way which is hard/impossible to >>> fix. This is where distributions might help and they should IMHO but why >>> to put an additional code into upstream? Isn't it sufficient that those >>> who would like to help (and take the risk) would just take the patch? >> >> At least Fedora, seem to frown upon heavily at non-upstream patches. > > OK, so what about the following approach instead? We won't change the > default but rather shout at people when they actually create subtrees > with use_hierarchy==0. This shouldn't make pointless noise. I do not > remember whether we have considered this previously so sorry if this was > shot down as well. The warning is fine, but just shouting won't achieve nothing. I believe it would be really great to have a way to turn the default to 1 - and stop the shouting. Even if you are doing it in OpenSUSE as a patch, an upstream patch means at least that every distribution is using the same patch, and those who rebase will just flip the config. I'd personally believe merging both our patches together would achieve a good result. -- 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