diff options
| author | Eric W. Biederman <ebiederm@xmission.com> | 2006-04-18 22:20:16 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-04-19 09:13:49 -0700 |
| commit | 5e85d4abe3f43bb5362f384bab0e20ef082ce0b5 (patch) | |
| tree | cd3a29086e5274fd08bc8d22d15568deab144755 /fs/exec.c | |
| parent | 181ae4005d0a4010802be534d929b38c42b9ac06 (diff) | |
| download | kernel-common-5e85d4abe3f43bb5362f384bab0e20ef082ce0b5.tar.gz kernel-common-5e85d4abe3f43bb5362f384bab0e20ef082ce0b5.tar.bz2 kernel-common-5e85d4abe3f43bb5362f384bab0e20ef082ce0b5.zip | |
[PATCH] task: Make task list manipulations RCU safe
While we can currently walk through thread groups, process groups, and
sessions with just the rcu_read_lock, this opens the door to walking the
entire task list.
We already have all of the other RCU guarantees so there is no cost in
doing this, this should be enough so that proc can stop taking the
tasklist lock during readdir.
prev_task was killed because it has no users, and using it will miss new
tasks when doing an rcu traversal.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/exec.c')
| -rw-r--r-- | fs/exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/exec.c b/fs/exec.c index 4121bb559739..3a79d97ac234 100644 --- a/fs/exec.c +++ b/fs/exec.c @@ -712,7 +712,7 @@ static int de_thread(struct task_struct *tsk) attach_pid(current, PIDTYPE_PID, current->pid); attach_pid(current, PIDTYPE_PGID, current->signal->pgrp); attach_pid(current, PIDTYPE_SID, current->signal->session); - list_add_tail(¤t->tasks, &init_task.tasks); + list_add_tail_rcu(¤t->tasks, &init_task.tasks); current->group_leader = current; leader->group_leader = current; |