diff options
author | Oleg Nesterov <oleg@tv-sign.ru> | 2008-04-30 03:53:02 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-30 11:29:37 -0400 |
commit | 193191035ad6268db9f561e81e3474b8be89a5ba (patch) | |
tree | ac4af28bf65dd7fc6c40d7260806725d9f3330ba /kernel/signal.c | |
parent | 2e2ba22ea4fd4bb85f0fa37c521066db6775cbef (diff) |
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/signal.c')
-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, | |||
551 | !capable(CAP_KILL)) { | 551 | !capable(CAP_KILL)) { |
552 | switch (sig) { | 552 | switch (sig) { |
553 | case SIGCONT: | 553 | case SIGCONT: |
554 | read_lock(&tasklist_lock); | ||
555 | sid = task_session(t); | 554 | sid = task_session(t); |
556 | read_unlock(&tasklist_lock); | ||
557 | /* | 555 | /* |
558 | * We don't return the error if sid == NULL. The | 556 | * We don't return the error if sid == NULL. The |
559 | * task was unhashed, the caller must notice this. | 557 | * task was unhashed, the caller must notice this. |