[Intel-gfx] [PATCH i-g-t 4/5] lib/igt_core: remove igt_disable/enable_exit_handlers

Daniel Vetter daniel.vetter at ffwll.ch
Thu Feb 18 12:39:28 UTC 2016


No longer needed, and also not really a safe idea.

Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
---
 lib/igt_core.c | 45 ---------------------------------------------
 lib/igt_core.h |  2 --
 2 files changed, 47 deletions(-)

diff --git a/lib/igt_core.c b/lib/igt_core.c
index 8e0bd2e87ecd..26ef0cc8edbf 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -1590,51 +1590,6 @@ err:
 	igt_assert_f(0, "failed to install the signal handler\n");
 }
 
-/**
- * igt_disable_exit_handler:
- *
- * Temporarily disable all exit handlers. Useful for library code doing tricky
- * things.
- */
-void igt_disable_exit_handler(void)
-{
-	sigset_t set;
-	int i;
-
-	if (exit_handler_disabled)
-		return;
-
-	sigemptyset(&set);
-	for (i = 0; i < ARRAY_SIZE(handled_signals); i++)
-		sigaddset(&set, handled_signals[i].number);
-
-	if (sigprocmask(SIG_BLOCK, &set, &saved_sig_mask)) {
-		perror("sigprocmask");
-		return;
-	}
-
-	exit_handler_disabled = true;
-}
-
-/**
- * igt_enable_exit_handler:
- *
- * Re-enable all exit handlers temporarily disabled with
- * igt_disable_exit_handler().
- */
-void igt_enable_exit_handler(void)
-{
-	if (!exit_handler_disabled)
-		return;
-
-	if (sigprocmask(SIG_SETMASK, &saved_sig_mask, NULL)) {
-		perror("sigprocmask");
-		return;
-	}
-
-	exit_handler_disabled = false;
-}
-
 /* simulation enviroment support */
 
 /**
diff --git a/lib/igt_core.h b/lib/igt_core.h
index 8f297e06a068..b3ecb0545916 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -666,8 +666,6 @@ typedef void (*igt_exit_handler_t)(int sig);
 
 /* reliable atexit helpers, also work when killed by a signal (if possible) */
 void igt_install_exit_handler(igt_exit_handler_t fn);
-void igt_enable_exit_handler(void);
-void igt_disable_exit_handler(void);
 
 /* helpers to automatically reduce test runtime in simulation */
 bool igt_run_in_simulation(void);
-- 
2.7.0



More information about the Intel-gfx mailing list