[PATCH] Remove another for_each_process loop

[Matthew Wilcox <willy@debian.org>]

Convert send_sigurg() to the for_each_task_pid() mechanism.  Also in
the case where we were trying to send a signal to a non-existent PID,
don't bother searching for -PID in the PGID array; we won't find it.

diff -urpNX dontdiff linux-2.5.40/fs/fcntl.c linux-2.5.40-willy/fs/fcntl.c
--- linux-2.5.40/fs/fcntl.c	2002-10-01 03:06:17.000000000 -0400
+++ linux-2.5.40-willy/fs/fcntl.c	2002-10-03 15:48:19.000000000 -0400
@@ -491,15 +491,17 @@ void send_sigio(struct fown_struct *fown
 		goto out_unlock_fown;
 	
 	read_lock(&tasklist_lock);
-	if ( (pid > 0) && (p = find_task_by_pid(pid)) ) {
-		send_sigio_to_task(p, fown, fd, band);
-		goto out_unlock_task;
-	}
-	for_each_task_pid(-pid, PIDTYPE_PGID, p, l, pidptr)
-		send_sigio_to_task(p, fown,fd,band);
-out_unlock_task:
+	if (pid > 0) {
+		if (p = find_task_by_pid(pid)) {
+			send_sigio_to_task(p, fown, fd, band);
+		}
+	} else {
+		for_each_task_pid(-pid, PIDTYPE_PGID, p, l, pidptr) {
+			send_sigio_to_task(p, fown, fd, band);
+		}
+	}
 	read_unlock(&tasklist_lock);
-out_unlock_fown:
+ out_unlock_fown:
 	read_unlock(&fown->lock);
 }
 
@@ -523,21 +525,17 @@ int send_sigurg(struct fown_struct *fown
 	ret = 1;
 	
 	read_lock(&tasklist_lock);
-	if ((pid > 0) && (p = find_task_by_pid(pid))) {
-		send_sigurg_to_task(p, fown);
-		goto out_unlock_task;
-	}
-	for_each_process(p) {
-		int match = p->pid;
-		if (pid < 0)
-			match = -p->pgrp;
-		if (pid != match)
-			continue;
-		send_sigurg_to_task(p, fown);
+	if (pid > 0) {
+		if (p = find_task_by_pid(pid)) {
+			send_sigurg_to_task(p, fown);
+		}
+	} else {
+		for_each_task_pid(-pid, PIDTYPE_PGID, p, l, pidptr) {
+			send_sigurg_to_task(p, fown);
+		}
 	}
-out_unlock_task:
 	read_unlock(&tasklist_lock);
-out_unlock_fown:
+ out_unlock_fown:
 	read_unlock(&fown->lock);
 	return ret;
 }

-- 
Revolutions do not require corporate support.