public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Use same_thread_group() in signalfd.c
@ 2007-08-29 13:19 Pavel Emelyanov
  2007-08-29 21:18 ` sukadev
  0 siblings, 1 reply; 2+ messages in thread
From: Pavel Emelyanov @ 2007-08-29 13:19 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Oleg Nesterov, Sukadev Bhattiprolu, Linux Containers,
	Linux Kernel Mailing List

This is a lost hunk of previous patch that isolated the
explicit usage of task->tgid in some places. The signalfd
code uses the tsk->tgid comparison.

Signed-off-by: Pavel Emelyanov <xemul@openvz.org>

---

diff --git a/fs/signalfd.c b/fs/signalfd.c
index a8e293d..5bfd2c5 100644
--- a/fs/signalfd.c
+++ b/fs/signalfd.c
@@ -64,7 +64,7 @@ static int signalfd_lock(struct signalfd
 		return 0;
 	}
 
-	if (lk->tsk->tgid == current->tgid)
+	if (same_thread_group(lk->tsk, current->tgid))
 		lk->tsk = current;
 
 	return 1;

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

* Re: [PATCH] Use same_thread_group() in signalfd.c
  2007-08-29 13:19 [PATCH] Use same_thread_group() in signalfd.c Pavel Emelyanov
@ 2007-08-29 21:18 ` sukadev
  0 siblings, 0 replies; 2+ messages in thread
From: sukadev @ 2007-08-29 21:18 UTC (permalink / raw)
  To: Pavel Emelyanov
  Cc: Andrew Morton, Oleg Nesterov, Linux Containers,
	Linux Kernel Mailing List

Pavel Emelianov [xemul@openvz.org] wrote:
| This is a lost hunk of previous patch that isolated the
| explicit usage of task->tgid in some places. The signalfd
| code uses the tsk->tgid comparison.
| 
| Signed-off-by: Pavel Emelyanov <xemul@openvz.org>

Acked-by: Sukadev Bhattiprolu <sukadev@us.ibm.com>

| 
| ---
| 
| diff --git a/fs/signalfd.c b/fs/signalfd.c
| index a8e293d..5bfd2c5 100644
| --- a/fs/signalfd.c
| +++ b/fs/signalfd.c
| @@ -64,7 +64,7 @@ static int signalfd_lock(struct signalfd
| 		return 0;
| 	}
| 
| -	if (lk->tsk->tgid == current->tgid)
| +	if (same_thread_group(lk->tsk, current->tgid))
| 		lk->tsk = current;
| 
| 	return 1;

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

end of thread, other threads:[~2007-08-29 21:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-29 13:19 [PATCH] Use same_thread_group() in signalfd.c Pavel Emelyanov
2007-08-29 21:18 ` sukadev

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox