From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx195.postini.com [74.125.245.195]) by kanga.kvack.org (Postfix) with SMTP id 07F436B00E5 for ; Mon, 12 Dec 2011 08:11:20 -0500 (EST) Date: Mon, 12 Dec 2011 14:11:18 +0100 From: Michal Hocko Subject: Re: [PATCH] mm: memcg: keep root group unchanged if fail to create new Message-ID: <20111212131118.GA15249@tiehlicka.suse.cz> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Hillf Danton Cc: Hugh Dickins , Balbir Singh , David Rientjes , Andrea Arcangeli , Andrew Morton , Johannes Weiner , KAMEZAWA Hiroyuki , linux-mm@kvack.org, LKML On Sun 11-12-11 15:39:43, Hugh Dickins wrote: > On Sun, 11 Dec 2011, Hillf Danton wrote: > > > If the request is not to create root group and we fail to meet it, > > we'd leave the root unchanged. > > I didn't understand that at first: please say "we should" rather > than "we'd", which I take to be an abbreviation for "we would". > > > > > Signed-off-by: Hillf Danton > > Yes indeed, well caught: > Acked-by: Hugh Dickins > > I wonder what was going through the author's mind when he wrote it > that way? I wonder if it's one of those bugs that creeps in when > you start from a perfectly functional patch, then make refinements > to suit feedback from reviewers. > > On which topic: wouldn't this patch be better just to move the > "root_mem_cgroup = memcg;" two lines lower down (and of course > remove free_out's "root_mem_cgroup = NULL;" as you already did)? Yes would look nicer. > I can't see mem_cgroup_soft_limit_tree_init() relying on > root_mem_cgroup at all. It doesn't but it still needs some love to handle error case properly AFAICS. We do not deallocate softlimit trees for nodes that succeeded. [...] Hilf could you update the patch please? -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org