diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-03-28 19:11:25 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-28 21:36:44 -0500 |
commit | 47e65328a7b1cdfc4e3102e50d60faf94ebba7d3 (patch) | |
tree | 78e00a5321cca87767806a91e623e71b5c6637c7 /kernel/exit.c | |
parent | 88531f725bd52e37a7be726860e4ff3f09031d89 (diff) |
[PATCH] pids: kill PIDTYPE_TGID
This patch kills PIDTYPE_TGID pid_type thus saving one hash table in
kernel/pid.c and speeding up subthreads create/destroy a bit. It is also a
preparation for the further tref/pids rework.
This patch adds 'struct list_head thread_group' to 'struct task_struct'
instead.
We don't detach group leader from PIDTYPE_PID namespace until another
thread inherits it's ->pid == ->tgid, so we are safe wrt premature
free_pidmap(->tgid) call.
Currently there are no users of find_task_by_pid_type(PIDTYPE_TGID).
Should the need arise, we can use find_task_by_pid()->group_leader.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Acked-By: Eric 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 | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/kernel/exit.c b/kernel/exit.c index aea23e713cf4..22399caf7574 100644 --- a/kernel/exit.c +++ b/kernel/exit.c | |||
@@ -51,7 +51,6 @@ static void __unhash_process(struct task_struct *p) | |||
51 | { | 51 | { |
52 | nr_threads--; | 52 | nr_threads--; |
53 | detach_pid(p, PIDTYPE_PID); | 53 | detach_pid(p, PIDTYPE_PID); |
54 | detach_pid(p, PIDTYPE_TGID); | ||
55 | if (thread_group_leader(p)) { | 54 | if (thread_group_leader(p)) { |
56 | detach_pid(p, PIDTYPE_PGID); | 55 | detach_pid(p, PIDTYPE_PGID); |
57 | detach_pid(p, PIDTYPE_SID); | 56 | detach_pid(p, PIDTYPE_SID); |
@@ -59,7 +58,7 @@ static void __unhash_process(struct task_struct *p) | |||
59 | list_del_init(&p->tasks); | 58 | list_del_init(&p->tasks); |
60 | __get_cpu_var(process_counts)--; | 59 | __get_cpu_var(process_counts)--; |
61 | } | 60 | } |
62 | 61 | list_del_rcu(&p->thread_group); | |
63 | remove_parent(p); | 62 | remove_parent(p); |
64 | } | 63 | } |
65 | 64 | ||
@@ -964,13 +963,6 @@ asmlinkage long sys_exit(int error_code) | |||
964 | do_exit((error_code&0xff)<<8); | 963 | do_exit((error_code&0xff)<<8); |
965 | } | 964 | } |
966 | 965 | ||
967 | task_t fastcall *next_thread(const task_t *p) | ||
968 | { | ||
969 | return pid_task(p->pids[PIDTYPE_TGID].pid_list.next, PIDTYPE_TGID); | ||
970 | } | ||
971 | |||
972 | EXPORT_SYMBOL(next_thread); | ||
973 | |||
974 | /* | 966 | /* |
975 | * Take down every thread in the group. This is called by fatal signals | 967 | * Take down every thread in the group. This is called by fatal signals |
976 | * as well as by sys_exit_group (below). | 968 | * as well as by sys_exit_group (below). |