diff options
author | Paul Jackson <pj@sgi.com> | 2006-03-31 05:30:51 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-31 15:18:55 -0500 |
commit | 2741a559a01e1ba9bf87285569dc1a104d134ecf (patch) | |
tree | 8c61321d5fa1a0c3517219eaa3089e223d5cd943 /kernel/cpuset.c | |
parent | 4a01c8d5be628ac20cfd432c21808d76be5813e6 (diff) |
[PATCH] cpuset: unsafe mm reference fix
Fix unsafe reference to a tasks mm struct, by moving the reference inside of a
convenient nearby properly guarded code block.
Signed-off-by: Paul Jackson <pj@sgi.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel/cpuset.c')
-rw-r--r-- | kernel/cpuset.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/cpuset.c b/kernel/cpuset.c index 2523a4b6a8c6..bf42381a4195 100644 --- a/kernel/cpuset.c +++ b/kernel/cpuset.c | |||
@@ -1183,11 +1183,11 @@ static int attach_task(struct cpuset *cs, char *pidbuf, char **ppathbuf) | |||
1183 | mm = get_task_mm(tsk); | 1183 | mm = get_task_mm(tsk); |
1184 | if (mm) { | 1184 | if (mm) { |
1185 | mpol_rebind_mm(mm, &to); | 1185 | mpol_rebind_mm(mm, &to); |
1186 | if (is_memory_migrate(cs)) | ||
1187 | do_migrate_pages(mm, &from, &to, MPOL_MF_MOVE_ALL); | ||
1186 | mmput(mm); | 1188 | mmput(mm); |
1187 | } | 1189 | } |
1188 | 1190 | ||
1189 | if (is_memory_migrate(cs)) | ||
1190 | do_migrate_pages(tsk->mm, &from, &to, MPOL_MF_MOVE_ALL); | ||
1191 | put_task_struct(tsk); | 1191 | put_task_struct(tsk); |
1192 | synchronize_rcu(); | 1192 | synchronize_rcu(); |
1193 | if (atomic_dec_and_test(&oldcs->count)) | 1193 | if (atomic_dec_and_test(&oldcs->count)) |