From 8408e00e623b52f18ad0795122593741197fa2f4 Mon Sep 17 00:00:00 2001 From: Thomas Wood Date: Thu, 24 Jul 2014 17:45:48 +0100 Subject: testdisplay: only set terminal attributes when in foreground process group The Piglit test runner for intel-gpu-tools creates a new process group for the test processes, so attempting to set terminal attributes causes the process to receive SIGTTOU and be stopped. Since the test is not run interactively in this case, the issue can be avoided by not setting terminal attributes if the process is not in the foreground process group. Signed-off-by: Thomas Wood --- tests/testdisplay.c | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'tests/testdisplay.c') diff --git a/tests/testdisplay.c b/tests/testdisplay.c index 6d8fe3ad..f60e66d5 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -725,6 +725,11 @@ static void set_termio_mode(void) { struct termios tio; + /* don't attempt to set terminal attributes if not in the foreground + * process group */ + if (getpgrp() != tcgetpgrp(STDOUT_FILENO)) + return; + tio_fd = dup(STDIN_FILENO); tcgetattr(tio_fd, &saved_tio); igt_install_exit_handler(restore_termio_mode); -- cgit v1.2.3