diff options
author | Ben Blum <bblum@andrew.cmu.edu> | 2011-11-02 13:38:05 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-11-02 16:06:59 -0700 |
commit | 33ef6b6984403a688189317ef46bb3caab3b70e0 (patch) | |
tree | 43dff6dd0150b34ce69b1d681c92659c9b20eb5c /mm/page_cgroup.c | |
parent | 434a964daa14b9db083ce20404a4a2add54d037a (diff) | |
download | lwn-33ef6b6984403a688189317ef46bb3caab3b70e0.tar.gz lwn-33ef6b6984403a688189317ef46bb3caab3b70e0.zip |
cgroups: more safe tasklist locking in cgroup_attach_proc
Fix unstable tasklist locking in cgroup_attach_proc.
According to this thread - https://lkml.org/lkml/2011/7/27/243 - RCU is
not sufficient to guarantee the tasklist is stable w.r.t. de_thread and
exit. Taking tasklist_lock for reading, instead of rcu_read_lock, ensures
proper exclusion.
Signed-off-by: Ben Blum <bblum@andrew.cmu.edu>
Acked-by: Paul Menage <paul@paulmenage.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page_cgroup.c')
0 files changed, 0 insertions, 0 deletions