2007-08-29 13:23:45

by Pavel Emelyanov

Subject: [PATCH] Use same_thread_group() in signalfd.c

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 <[email protected]>

---

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;

2007-08-29 21:19:18

by Sukadev Bhattiprolu

Subject: Re: [PATCH] Use same_thread_group() in signalfd.c

Pavel Emelianov [[email protected]] 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 <[email protected]>

Acked-by: Sukadev Bhattiprolu <[email protected]>

|
| ---
|
| 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;