[igt-dev] [PATCH i-g-t] igt: Make LibUdev mandatory

Antonio Argenziano antonio.argenziano at intel.com
Fri Feb 23 23:14:40 UTC 2018


Since more essential components use libUdev, make its dependency
mandatory.

Signed-off-by: Antonio Argenziano <antonio.argenziano at intel.com>
Cc: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
Cc: Petri Latvala <petri.latvala at intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler at intel.com>
---
 configure.ac                   |  9 +--------
 lib/igt_aux.c                  | 14 +-------------
 lib/igt_kms.c                  |  6 +-----
 lib/igt_kms.h                  |  2 --
 meson.build                    |  7 +------
 tests/testdisplay_hotplug.c    | 12 ------------
 tools/Makefile.am              |  2 --
 tools/intel_l3_parity.c        |  7 -------
 tools/intel_l3_parity.h        |  5 -----
 tools/intel_l3_udev_listener.c |  2 --
 10 files changed, 4 insertions(+), 62 deletions(-)

diff --git a/configure.ac b/configure.ac
index e13a3b74..1a6f5cb2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -176,11 +176,7 @@ PKG_CHECK_MODULES(XRANDR, xrandr >= 1.3, AC_DEFINE(HAVE_XRANDR, 1, [Have libXran
 
 # for testdisplay
 PKG_CHECK_MODULES(CAIRO, [cairo >= 1.12.0])
-PKG_CHECK_MODULES(LIBUDEV, [libudev], [udev=yes], [udev=no])
-AM_CONDITIONAL(HAVE_UDEV, [test "x$udev" = xyes])
-if test x"$udev" = xyes; then
-	AC_DEFINE(HAVE_UDEV,1,[Enable udev-based monitor hotplug detection])
-fi
+PKG_CHECK_MODULES(LIBUDEV, [libudev])
 PKG_CHECK_MODULES(GLIB, [glib-2.0], [glib=yes], [glib=no])
 if test x"$glib" = xyes; then
 	AC_DEFINE(HAVE_GLIB,1,[Enable glib support])
@@ -210,9 +206,6 @@ if test "x$enable_chamelium" = xyes; then
 	fi
 	PKG_CHECK_MODULES(PIXMAN, pixman-1, [],
 			  [AC_MSG_ERROR([Failed to find pixman, required by chamelium.])])
-	if test x"$udev" != xyes; then
-		AC_MSG_ERROR([Failed to find udev, required by chamelium.])
-	fi
 	if test x"$glib" != xyes; then
 		AC_MSG_ERROR([Failed to find glib, required by chamelium.])
 	fi
diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index a23375a4..aabf6e50 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -50,6 +50,7 @@
 #include <assert.h>
 
 #include <proc/readproc.h>
+#include <libudev.h>
 
 #include "drmtest.h"
 #include "i915_drm.h"
@@ -425,8 +426,6 @@ void igt_stop_shrink_helper(void)
 	igt_stop_helper(&shrink_helper);
 }
 
-#ifdef HAVE_UDEV
-#include <libudev.h>
 
 static struct igt_helper_process hang_detector;
 static void __attribute__((noreturn))
@@ -506,17 +505,6 @@ void igt_stop_hang_detector(void)
 {
 	igt_stop_helper(&hang_detector);
 }
-#else
-void igt_fork_hang_detector(int fd)
-{
-	if (igt_only_list_subtests())
-		return;
-}
-
-void igt_stop_hang_detector(void)
-{
-}
-#endif
 
 /**
  * igt_check_boolean_env_var:
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index e14ecf3b..9b2f94c1 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -39,9 +39,8 @@
 #elif HAVE_SYS_KD_H
 #include <sys/kd.h>
 #endif
-#ifdef HAVE_UDEV
+
 #include <libudev.h>
-#endif
 #include <poll.h>
 #include <errno.h>
 #include <time.h>
@@ -3695,8 +3694,6 @@ void igt_reset_connectors(void)
 			      "detect");
 }
 
-#ifdef HAVE_UDEV
-
 /**
  * igt_watch_hotplug:
  *
@@ -3800,7 +3797,6 @@ void igt_cleanup_hotplug(struct udev_monitor *mon)
 	mon = NULL;
 	udev_unref(udev);
 }
-#endif
 
 /**
  * kmstest_get_vbl_flag:
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index b7521039..c3f82d34 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -658,12 +658,10 @@ uint32_t kmstest_get_vbl_flag(uint32_t pipe_id);
 const unsigned char* igt_kms_get_base_edid(void);
 const unsigned char* igt_kms_get_alt_edid(void);
 
-#ifdef HAVE_UDEV
 struct udev_monitor *igt_watch_hotplug(void);
 bool igt_hotplug_detected(struct udev_monitor *mon,
 			  int timeout_secs);
 void igt_flush_hotplugs(struct udev_monitor *mon);
 void igt_cleanup_hotplug(struct udev_monitor *mon);
-#endif
 
 #endif /* __IGT_KMS_H__ */
diff --git a/meson.build b/meson.build
index 509ae5de..b0297d74 100644
--- a/meson.build
+++ b/meson.build
@@ -48,12 +48,7 @@ if valgrind.found()
 endif
 
 cairo = dependency('cairo', version : '>1.12.0', required : true)
-
-libudev = dependency('libudev', required : false)
-if libudev.found()
-	config.set('HAVE_UDEV', 1)
-endif
-
+libudev = dependency('libudev', required : true)
 glib = dependency('glib-2.0', required : false)
 if glib.found()
 	config.set('HAVE_GLIB', 1)
diff --git a/tests/testdisplay_hotplug.c b/tests/testdisplay_hotplug.c
index ca1d849d..581d987e 100644
--- a/tests/testdisplay_hotplug.c
+++ b/tests/testdisplay_hotplug.c
@@ -32,7 +32,6 @@
 #include "config.h"
 
 
-#ifdef HAVE_UDEV
 #include <libudev.h>
 static struct udev_monitor *uevent_monitor;
 static struct udev *udev;
@@ -125,14 +124,3 @@ void testdisplay_cleanup_hotplug(void)
 	if (udev)
 		udev_unref(udev);
 }
-#else
-gboolean testdisplay_setup_hotplug(void)
-{
-	igt_warn("no hotplug support on this platform\n");
-	return TRUE;
-}
-
-void testdisplay_cleanup_hotplug(void)
-{
-}
-#endif
diff --git a/tools/Makefile.am b/tools/Makefile.am
index dcf282ea..8de2a231 100644
--- a/tools/Makefile.am
+++ b/tools/Makefile.am
@@ -7,11 +7,9 @@ bin_PROGRAMS += $(LIBDRM_INTEL_BIN)
 intel_error_decode_LDFLAGS = -lz
 endif
 
-if HAVE_UDEV
 bin_PROGRAMS += intel_dp_compliance
 intel_dp_compliance_CFLAGS = $(AM_CFLAGS)
 intel_dp_compliance_LDADD = $(top_builddir)/lib/libintel_tools.la
-endif
 
 SUBDIRS = null_state_gen registers
 
diff --git a/tools/intel_l3_parity.c b/tools/intel_l3_parity.c
index 9df4fc24..d1a9bc44 100644
--- a/tools/intel_l3_parity.c
+++ b/tools/intel_l3_parity.c
@@ -39,10 +39,8 @@
 #include "igt_sysfs.h"
 #include "drmtest.h"
 #include "config.h"
-#ifdef HAVE_UDEV
 #include <libudev.h>
 #include <syslog.h>
-#endif
 #include "intel_l3_parity.h"
 
 static unsigned int devid;
@@ -318,10 +316,6 @@ int main(int argc, char *argv[])
 
 	/* Daemon doesn't work like the other commands */
 	if (action == 'L') {
-#ifndef HAVE_UDEV
-		fprintf(stderr, "Daemon requires udev support. Please reconfigure.\n");
-		exit(EXIT_FAILURE);
-#else
 		struct l3_parity par;
 		struct l3_location loc;
 		if (daemonize) {
@@ -332,7 +326,6 @@ int main(int argc, char *argv[])
 		assert(l3_uevent_setup(&par) == 0);
 		assert(l3_listen(&par, daemonize == 1, &loc) == 0);
 		exit(EXIT_SUCCESS);
-#endif
 	}
 
 	if (action == 'l')
diff --git a/tools/intel_l3_parity.h b/tools/intel_l3_parity.h
index 759c4f4b..275c9b4b 100644
--- a/tools/intel_l3_parity.h
+++ b/tools/intel_l3_parity.h
@@ -18,14 +18,9 @@ struct l3_location {
 	uint8_t subbank;
 };
 
-#ifdef HAVE_UDEV
 int l3_uevent_setup(struct l3_parity *par);
 /* Listens (blocks) for an l3 parity event. Returns the location of the error. */
 int l3_listen(struct l3_parity *par, bool daemon, struct l3_location *loc);
 #define l3_uevent_teardown(par) {}
-#else
-#define l3_uevent_setup(par, daemon, loc) -1
-#define l3_listen(par) -1
-#endif
 
 #endif
diff --git a/tools/intel_l3_udev_listener.c b/tools/intel_l3_udev_listener.c
index 94d931e5..ff62f407 100644
--- a/tools/intel_l3_udev_listener.c
+++ b/tools/intel_l3_udev_listener.c
@@ -23,7 +23,6 @@
 
 #include "config.h"
 
-#ifdef HAVE_UDEV
 #include <libudev.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -123,4 +122,3 @@ again:
 
 	return 0;
 }
-#endif
-- 
2.14.2



More information about the igt-dev mailing list