debug cgroup: use task_css_set instead of rcu_dereference

This macro `task_css_set` verifies that the caller is
inside proper critical section if the kernel set CONFIG_PROVE_RCU=y.

Signed-off-by: Wang Long <wanglong19@meituan.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
This commit is contained in:
Wang Long 2017-11-19 16:08:37 -05:00 committed by Tejun Heo
parent 1599a185f0
commit ddf7005f32
1 changed files with 2 additions and 2 deletions

View File

@ -50,7 +50,7 @@ static int current_css_set_read(struct seq_file *seq, void *v)
spin_lock_irq(&css_set_lock); spin_lock_irq(&css_set_lock);
rcu_read_lock(); rcu_read_lock();
cset = rcu_dereference(current->cgroups); cset = task_css_set(current);
refcnt = refcount_read(&cset->refcount); refcnt = refcount_read(&cset->refcount);
seq_printf(seq, "css_set %pK %d", cset, refcnt); seq_printf(seq, "css_set %pK %d", cset, refcnt);
if (refcnt > cset->nr_tasks) if (refcnt > cset->nr_tasks)
@ -96,7 +96,7 @@ static int current_css_set_cg_links_read(struct seq_file *seq, void *v)
spin_lock_irq(&css_set_lock); spin_lock_irq(&css_set_lock);
rcu_read_lock(); rcu_read_lock();
cset = rcu_dereference(current->cgroups); cset = task_css_set(current);
list_for_each_entry(link, &cset->cgrp_links, cgrp_link) { list_for_each_entry(link, &cset->cgrp_links, cgrp_link) {
struct cgroup *c = link->cgrp; struct cgroup *c = link->cgrp;