diff options
-rw-r--r-- | include/linux/sched.h | 2 | ||||
-rw-r--r-- | kernel/fork.c | 12 | ||||
-rw-r--r-- | kernel/signal.c | 19 |
3 files changed, 14 insertions, 19 deletions
diff --git a/include/linux/sched.h b/include/linux/sched.h index 7dd430b697aa..921148277da9 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h | |||
@@ -1150,8 +1150,8 @@ extern void exit_thread(void); | |||
1150 | 1150 | ||
1151 | extern void exit_files(struct task_struct *); | 1151 | extern void exit_files(struct task_struct *); |
1152 | extern void __cleanup_signal(struct signal_struct *); | 1152 | extern void __cleanup_signal(struct signal_struct *); |
1153 | extern void cleanup_sighand(struct task_struct *); | ||
1153 | extern void __exit_signal(struct task_struct *); | 1154 | extern void __exit_signal(struct task_struct *); |
1154 | extern void __exit_sighand(struct task_struct *); | ||
1155 | extern void exit_itimers(struct signal_struct *); | 1155 | extern void exit_itimers(struct signal_struct *); |
1156 | 1156 | ||
1157 | extern NORET_TYPE void do_group_exit(int); | 1157 | extern NORET_TYPE void do_group_exit(int); |
diff --git a/kernel/fork.c b/kernel/fork.c index 0aff28cdbadd..12cdd9fc9d02 100644 --- a/kernel/fork.c +++ b/kernel/fork.c | |||
@@ -803,6 +803,16 @@ static inline int copy_sighand(unsigned long clone_flags, struct task_struct * t | |||
803 | return 0; | 803 | return 0; |
804 | } | 804 | } |
805 | 805 | ||
806 | void cleanup_sighand(struct task_struct *tsk) | ||
807 | { | ||
808 | struct sighand_struct * sighand = tsk->sighand; | ||
809 | |||
810 | /* Ok, we're done with the signal handlers */ | ||
811 | tsk->sighand = NULL; | ||
812 | if (atomic_dec_and_test(&sighand->count)) | ||
813 | kmem_cache_free(sighand_cachep, sighand); | ||
814 | } | ||
815 | |||
806 | static inline int copy_signal(unsigned long clone_flags, struct task_struct * tsk) | 816 | static inline int copy_signal(unsigned long clone_flags, struct task_struct * tsk) |
807 | { | 817 | { |
808 | struct signal_struct *sig; | 818 | struct signal_struct *sig; |
@@ -1224,7 +1234,7 @@ bad_fork_cleanup_mm: | |||
1224 | bad_fork_cleanup_signal: | 1234 | bad_fork_cleanup_signal: |
1225 | cleanup_signal(p); | 1235 | cleanup_signal(p); |
1226 | bad_fork_cleanup_sighand: | 1236 | bad_fork_cleanup_sighand: |
1227 | __exit_sighand(p); | 1237 | cleanup_sighand(p); |
1228 | bad_fork_cleanup_fs: | 1238 | bad_fork_cleanup_fs: |
1229 | exit_fs(p); /* blocking */ | 1239 | exit_fs(p); /* blocking */ |
1230 | bad_fork_cleanup_files: | 1240 | bad_fork_cleanup_files: |
diff --git a/kernel/signal.c b/kernel/signal.c index ca1fa854e469..b29c868bd5ee 100644 --- a/kernel/signal.c +++ b/kernel/signal.c | |||
@@ -310,9 +310,7 @@ static void flush_sigqueue(struct sigpending *queue) | |||
310 | /* | 310 | /* |
311 | * Flush all pending signals for a task. | 311 | * Flush all pending signals for a task. |
312 | */ | 312 | */ |
313 | 313 | void flush_signals(struct task_struct *t) | |
314 | void | ||
315 | flush_signals(struct task_struct *t) | ||
316 | { | 314 | { |
317 | unsigned long flags; | 315 | unsigned long flags; |
318 | 316 | ||
@@ -326,19 +324,6 @@ flush_signals(struct task_struct *t) | |||
326 | /* | 324 | /* |
327 | * This function expects the tasklist_lock write-locked. | 325 | * This function expects the tasklist_lock write-locked. |
328 | */ | 326 | */ |
329 | void __exit_sighand(struct task_struct *tsk) | ||
330 | { | ||
331 | struct sighand_struct * sighand = tsk->sighand; | ||
332 | |||
333 | /* Ok, we're done with the signal handlers */ | ||
334 | tsk->sighand = NULL; | ||
335 | if (atomic_dec_and_test(&sighand->count)) | ||
336 | kmem_cache_free(sighand_cachep, sighand); | ||
337 | } | ||
338 | |||
339 | /* | ||
340 | * This function expects the tasklist_lock write-locked. | ||
341 | */ | ||
342 | void __exit_signal(struct task_struct *tsk) | 327 | void __exit_signal(struct task_struct *tsk) |
343 | { | 328 | { |
344 | struct signal_struct *sig = tsk->signal; | 329 | struct signal_struct *sig = tsk->signal; |
@@ -386,7 +371,7 @@ void __exit_signal(struct task_struct *tsk) | |||
386 | } | 371 | } |
387 | 372 | ||
388 | tsk->signal = NULL; | 373 | tsk->signal = NULL; |
389 | __exit_sighand(tsk); | 374 | cleanup_sighand(tsk); |
390 | spin_unlock(&sighand->siglock); | 375 | spin_unlock(&sighand->siglock); |
391 | rcu_read_unlock(); | 376 | rcu_read_unlock(); |
392 | 377 | ||