diff options
| author | Oleg Nesterov <oleg@tv-sign.ru> | 2008-04-30 00:53:02 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-30 08:29:37 -0700 |
| commit | 193191035ad6268db9f561e81e3474b8be89a5ba (patch) | |
| tree | ac4af28bf65dd7fc6c40d7260806725d9f3330ba /kernel | |
| parent | 2e2ba22ea4fd4bb85f0fa37c521066db6775cbef (diff) | |
| download | lwn-193191035ad6268db9f561e81e3474b8be89a5ba.tar.gz lwn-193191035ad6268db9f561e81e3474b8be89a5ba.zip | |
signals: check_kill_permission: remove tasklist_lock
Now that task_session() can't return a false NULL, check_kill_permission()
doesn't need tasklist_lock.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Cc: Roland McGrath <roland@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/signal.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/signal.c b/kernel/signal.c index 24be82c0aae3..02ef3548aeb0 100644 --- a/kernel/signal.c +++ b/kernel/signal.c @@ -551,9 +551,7 @@ static int check_kill_permission(int sig, struct siginfo *info, !capable(CAP_KILL)) { switch (sig) { case SIGCONT: - read_lock(&tasklist_lock); sid = task_session(t); - read_unlock(&tasklist_lock); /* * We don't return the error if sid == NULL. The * task was unhashed, the caller must notice this. |