lkml.org 
[lkml]   [2011]   [Dec]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 4/4] cgroup: remove extra calls to find_existing_css_set
On Thu, Dec 22, 2011 at 10:44:39AM +0100, Frederic Weisbecker wrote:
> > > if (ent.cgrp == cgrp)
> > > continue;
> > > + ent.cg = find_css_set(tsk->cgroups, cgrp);
> >
> > unfortunately This won't work, because we are holding tasklist_lock.
>
> I believe we can remove tasklist_lock now (in a seperate patch).
>
> It was there in order to protect while_each_thread() against exec but
> now we have threadgroup_lock().
>
> I think we only need to use rcu_read_lock() to protect against concurrent
> removal in exit.

Yeah, that should work and I really like this patch.

kernel/cgroup.c | 152 ++++++++++++-------------------------------------------
1 files changed, 32 insertions(+), 120 deletions(-)

Let's get it working. :)

Thanks.

--
tejun


\
 
 \ /
  Last update: 2011-12-22 16:43    [W:0.085 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site