[patch attached as well due to space/tab breakage.
resent as list member.]

SIGTTOU was causing a hang in terminals with tostop set (which seems to be
the default for my local emacs configuration).

Signed-off-by: David Rochberg ([email protected])
BUG=chromiumos:7716
TEST=stty tostop, then run with and without this change.  Without the
change, tests will hang in a SIGTTOU/syscall-restarted loop.

Change-Id: Ib5317e238b68089d71e8e57418717e7553fd64d9

Review URL: http://codereview.chromium.org/6368022
---
 server/autoserv |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/server/autoserv b/server/autoserv
index b3e900b..1cec992 100755
--- a/server/autoserv
+++ b/server/autoserv
@@ -53,6 +53,9 @@ def run_autoserv(pid_file_manager, results, parser):
     # Set signal handler
     signal.signal(signal.SIGTERM, handle_sigterm)

+    # Ignore SIGTTOU's generated by output from forked children.
+    signal.signal(signal.SIGTTOU, signal.SIG_IGN)
+
     # Server side tests that call shell scripts often depend on $USER being
set
     # but depending on how you launch your autotest scheduler it may not be
set.
     os.environ['USER'] = getpass.getuser()
-- 
1.7.3.1
From 8a60d1e1da306702d0003e452494ab9bd1637d73 Mon Sep 17 00:00:00 2001
From: David Rochberg <[email protected]>
Date: Tue, 1 Feb 2011 14:22:07 -0500
Subject: [PATCH] Ignore SIGTTOU.

SIGTTOU was causing a hang in terminals with tostop set (which seems to be the default for my local emacs configuration).

Signed-off-by: David Rochberg ([email protected])
BUG=chromiumos:7716
TEST=stty tostop, then run with and without this change.  Without the change, tests will hang in a SIGTTOU/syscall-restarted loop.

Change-Id: Ib5317e238b68089d71e8e57418717e7553fd64d9

Review URL: http://codereview.chromium.org/6368022
---
 server/autoserv |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/server/autoserv b/server/autoserv
index b3e900b..1cec992 100755
--- a/server/autoserv
+++ b/server/autoserv
@@ -53,6 +53,9 @@ def run_autoserv(pid_file_manager, results, parser):
     # Set signal handler
     signal.signal(signal.SIGTERM, handle_sigterm)
 
+    # Ignore SIGTTOU's generated by output from forked children.
+    signal.signal(signal.SIGTTOU, signal.SIG_IGN)
+
     # Server side tests that call shell scripts often depend on $USER being set
     # but depending on how you launch your autotest scheduler it may not be set.
     os.environ['USER'] = getpass.getuser()
-- 
1.7.3.1

_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to