2008-03-21 16:06:01

by Oleg Nesterov

[permalink] [raw]
Subject: [PATCH] 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 <[email protected]>

--- 25/kernel/signal.c~5_CKP_NO_TASKLIST 2008-03-19 21:15:30.000000000 +0300
+++ 25/kernel/signal.c 2008-03-21 18:10:30.000000000 +0300
@@ -551,9 +551,7 @@ static int check_kill_permission(int sig
!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.