linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 5/4] tracehook_notify_death: use task_detached() helper
@ 2009-02-11 21:12 Oleg Nesterov
  2009-02-20  1:38 ` Roland McGrath
  0 siblings, 1 reply; 2+ messages in thread
From: Oleg Nesterov @ 2009-02-11 21:12 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Eric W. Biederman, Metzger, Markus T, Roland McGrath, linux-kernel

Now that task_detached() is exported, change tracehook_notify_death()
to use this helper, nobody else checks ->exit_signal == -1 by hand.

Signed-off-by: Oleg Nesterov <oleg@redhat.com>

--- 6.29-rc3/include/linux/tracehook.h~5_NOTIFY_USE_HELPER	2008-10-10 00:13:53.000000000 +0200
+++ 6.29-rc3/include/linux/tracehook.h	2009-02-11 21:54:57.000000000 +0100
@@ -507,7 +507,7 @@ static inline int tracehook_notify_jctl(
 static inline int tracehook_notify_death(struct task_struct *task,
 					 void **death_cookie, int group_dead)
 {
-	if (task->exit_signal == -1)
+	if (task_detached(task))
 		return task->ptrace ? SIGCHLD : DEATH_REAP;
 
 	/*


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 5/4] tracehook_notify_death: use task_detached() helper
  2009-02-11 21:12 [PATCH 5/4] tracehook_notify_death: use task_detached() helper Oleg Nesterov
@ 2009-02-20  1:38 ` Roland McGrath
  0 siblings, 0 replies; 2+ messages in thread
From: Roland McGrath @ 2009-02-20  1:38 UTC (permalink / raw)
  To: Oleg Nesterov
  Cc: Andrew Morton, Eric W. Biederman, Metzger, Markus T, linux-kernel

Acked-By: Roland McGrath <roland@redhat.com>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-02-20  1:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-02-11 21:12 [PATCH 5/4] tracehook_notify_death: use task_detached() helper Oleg Nesterov
2009-02-20  1:38 ` Roland McGrath

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).