[src/trunk]: src/sys/kern We need to process SA_STOP signals immediately, and...

[christos <christos%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/7b7ce0982147
branches:  trunk
changeset: 769203:7b7ce0982147
user:      christos <christos%NetBSD.org@localhost>
date:      Sat Sep 03 10:28:33 2011 +0000

description:
We need to process SA_STOP signals immediately, and not deliver them to
the process. Instead of re-structuring the code to do that, call issignal()
like before in that case. (tail -F /file^Zfg should not get interrupted).

diffstat:

 sys/kern/kern_sleepq.c |  8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diffs (29 lines):

diff -r c034fa8f0cef -r 7b7ce0982147 sys/kern/kern_sleepq.c
--- a/sys/kern/kern_sleepq.c    Sat Sep 03 09:02:20 2011 +0000
+++ b/sys/kern/kern_sleepq.c    Sat Sep 03 10:28:33 2011 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: kern_sleepq.c,v 1.42 2011/08/31 16:09:55 christos Exp $        */
+/*     $NetBSD: kern_sleepq.c,v 1.43 2011/09/03 10:28:33 christos Exp $        */
 
 /*-
  * Copyright (c) 2006, 2007, 2008, 2009 The NetBSD Foundation, Inc.
@@ -35,7 +35,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_sleepq.c,v 1.42 2011/08/31 16:09:55 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_sleepq.c,v 1.43 2011/09/03 10:28:33 christos Exp $");
 
 #include <sys/param.h>
 #include <sys/kernel.h>
@@ -284,7 +284,9 @@
                         * not recurse again.
                         */
                        mutex_enter(p->p_lock);
-                       if ((sig = sigispending(l, 0)) != 0)
+                       if (((sig = sigispending(l, 0)) != 0 &&
+                           (sigprop[sig] & SA_STOP) == 0) ||
+                           (sig = issignal(l)) != 0)
                                error = sleepq_sigtoerror(l, sig);
                        mutex_exit(p->p_lock);
                }