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 -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
