From: Oleg Nesterov <oleg@tv-sign.ru>
Date: Wed, 30 Apr 2008 07:53:02 +0000 (-0700)
Subject: signals: check_kill_permission: remove tasklist_lock
X-Git-Tag: v2.6.26-rc1~277
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=193191035ad6268db9f561e81e3474b8be89a5ba;p=linux-2.6

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

diff --git a/kernel/signal.c b/kernel/signal.c
index 24be82c0aa..02ef3548ae 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.