diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-03-28 19:11:06 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-28 21:36:41 -0500 |
commit | c97d98931ac52ef110b62d9b75c6a6f2bfbc1898 (patch) | |
tree | f811f8b2862692c2eb7ee92e62e8f0afcfd37a2d /kernel/exit.c | |
parent | 9b678ece42893b53aae5ed7cb8d7cb261cacb72c (diff) |
[PATCH] kill SET_LINKS/REMOVE_LINKS
Both SET_LINKS() and SET_LINKS/REMOVE_LINKS() have exactly one caller, and
these callers already check thread_group_leader().
This patch kills theese macros, they mix two different things: setting
process's parent and registering it in init_task.tasks list. Callers are
updated to do these actions by hand.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Cc: "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 'kernel/exit.c')
-rw-r--r-- | kernel/exit.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index 5b5e8b67680e..f436a6bd3fb7 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
@@ -54,11 +54,13 @@ static void __unhash_process(struct task_struct *p) | |||
54 | if (thread_group_leader(p)) { | 54 | if (thread_group_leader(p)) { |
55 | detach_pid(p, PIDTYPE_PGID); | 55 | detach_pid(p, PIDTYPE_PGID); |
56 | detach_pid(p, PIDTYPE_SID); | 56 | detach_pid(p, PIDTYPE_SID); |
57 | |||
58 | list_del_init(&p->tasks); | ||
57 | if (p->pid) | 59 | if (p->pid) |
58 | __get_cpu_var(process_counts)--; | 60 | __get_cpu_var(process_counts)--; |
59 | } | 61 | } |
60 | 62 | ||
61 | REMOVE_LINKS(p); | 63 | remove_parent(p); |
62 | } | 64 | } |
63 | 65 | ||
64 | void release_task(struct task_struct * p) | 66 | void release_task(struct task_struct * p) |