[src/trunk]: src/sys/kern When closing a tty, limit the amount of time spent ...

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

details:   https://anonhg.NetBSD.org/src/rev/d946f68160cd
branches:  trunk
changeset: 338864:d946f68160cd
user:      gson <gson%NetBSD.org@localhost>
date:      Fri Jun 12 17:28:53 2015 +0000

description:
When closing a tty, limit the amount of time spent waiting for the
output to drain to five seconds so that exiting processes with
buffered output for a serial port blocked by flow control do not
hang indefinitely.  Should fix PR kern/12534.  OK christos.

diffstat:

 sys/kern/tty.c |  29 +++++++++++++++++++++--------
 1 files changed, 21 insertions(+), 8 deletions(-)

diffs (80 lines):

diff -r 3e48aac76805 -r d946f68160cd sys/kern/tty.c
--- a/sys/kern/tty.c    Fri Jun 12 17:24:02 2015 +0000
+++ b/sys/kern/tty.c    Fri Jun 12 17:28:53 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: tty.c,v 1.262 2014/09/05 05:33:39 matt Exp $   */
+/*     $NetBSD: tty.c,v 1.263 2015/06/12 17:28:53 gson Exp $   */
 
 /*-
  * Copyright (c) 2008 The NetBSD Foundation, Inc.
@@ -63,7 +63,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: tty.c,v 1.262 2014/09/05 05:33:39 matt Exp $");
+__KERNEL_RCSID(0, "$NetBSD: tty.c,v 1.263 2015/06/12 17:28:53 gson Exp $");
 
 #include "opt_compat_netbsd.h"
 
@@ -102,6 +102,7 @@
 #endif /* COMPAT_60 */
 
 static int     ttnread(struct tty *);
+static int     ttywait_timo(struct tty *, int timo);
 static void    ttyblock(struct tty *);
 static void    ttyecho(int, struct tty *);
 static void    ttyrubo(struct tty *, int);
@@ -1536,10 +1537,10 @@
 }
 
 /*
- * Wait for output to drain.
+ * Wait for output to drain, or if this times out, flush it.
  */
-int
-ttywait(struct tty *tp)
+static int
+ttywait_timo(struct tty *tp, int timo)
 {
        int     error;
 
@@ -1549,9 +1550,11 @@
        while ((tp->t_outq.c_cc || ISSET(tp->t_state, TS_BUSY)) &&
            CONNECTED(tp) && tp->t_oproc) {
                (*tp->t_oproc)(tp);
-               error = ttysleep(tp, &tp->t_outcv, true, 0);
-               if (error)
+               error = ttysleep(tp, &tp->t_outcv, true, timo);
+               if (error) {
+                       ttyflush(tp, FWRITE);
                        break;
+               }
        }
        mutex_spin_exit(&tty_lock);
 
@@ -1559,6 +1562,15 @@
 }
 
 /*
+ * Wait for output to drain.
+ */
+int
+ttywait(struct tty *tp)
+{
+       return ttywait_timo(tp, 0);
+}
+
+/*
  * Flush if successfully wait.
  */
 int
@@ -1566,7 +1578,8 @@
 {
        int     error;
 
-       if ((error = ttywait(tp)) == 0) {
+       error = ttywait_timo(tp, 5 * hz);
+       if (error == 0 || error == EWOULDBLOCK) {
                mutex_spin_enter(&tty_lock);
                ttyflush(tp, FREAD);
                mutex_spin_exit(&tty_lock);