From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kamezawa Hiroyuki Subject: Re: [PATCH 2/2] memcg: replace cgroup_lock with memcg specific memcg_lock fix Date: Wed, 13 Feb 2013 19:43:30 +0900 Message-ID: <511B6E52.1090800@jp.fujitsu.com> References: <1360569889-843-1-git-send-email-glommer@parallels.com> <1360569889-843-3-git-send-email-glommer@parallels.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1360569889-843-3-git-send-email-glommer-bzQdu9zFT3WakBO8gow8eQ@public.gmane.org> Sender: cgroups-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Glauber Costa Cc: Andrew Morton , Tejun Heo , linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Michal Hocko , Johannes Weiner (2013/02/11 17:04), Glauber Costa wrote: > Signed-off-by: Glauber Costa > Reported-by: Dan Carpenter I'm sorry I missed this... Acked-by: KAMEZAWA Hiroyuki > --- > mm/memcontrol.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 28252c9..03ebf68 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -5884,7 +5884,7 @@ static int mem_cgroup_oom_control_write(struct cgroup *cgrp, > mutex_lock(&memcg_create_mutex); > /* oom-kill-disable is a flag for subhierarchy. */ > if ((parent->use_hierarchy) || memcg_has_children(memcg)) { > - cgroup_unlock(); > + mutex_unlock(&memcg_create_mutex); > return -EINVAL; > } > memcg->oom_kill_disable = val; >