[RFC i-g-t v2 0/5] intel-gpu-tools: Add support for the Chamelium

[RFC i-g-t v2 0/5] intel-gpu-tools: Add support for the Chamelium

[Lyude]

This is the next version of the patchset for adding Chamelium support into
intel-gpu-tools. The major changes (described in more details in the actual
patches):
- Connector mappings are specified in a configuration file, no more guessing
- Some fixes for tests that were throwing false negatives
- Cleanups
- Split into slightly smaller patches

As well, it was suggested that we use libsoup for the XMLRPC interactions with
the Chamelium since we already include glib in igt's build deps, however I
decided against it because (at least this seems to be the case on Fedora)
libsoup is actually packaged seperately from glib, so using it would still
require adding another dependency anyway. If I'm wrong here, feel free to let me
know.

As a final note: a few of the DisplayPort tests can prove to be unreliable on
some hardware, and the dp-display test will only work once by default. This is
due to a couple of bugs with chameleond's handling of the chamelium's DP
receiver. These problems can mostly be alleviated by using the patches I've got
here

	https://github.com/Lyude/chameleond/tree/wip/chameleon-fixes

All of which I'll be sending to Google at some point (whenever I get their
strange depot-tools setup). As well, all of the display tests won't work on
non-intel hardware due to a couple of errorneous intel-hardware specific
assumptions made in igt_kms.c, which I should have fixed soon.

Lyude (5):
  igt_aux: Add igt_skip_without_suspend_support()
  igt_aux: Add igt_set_autoresume_delay()
  igt_aux: Add some list helpers from wayland
  igt_kms: Add helpers for watching for sysfs hotplug events
  Add support for hotplug testing with the Chamelium

 configure.ac           |  13 +
 lib/Makefile.am        |  10 +-
 lib/igt.h              |   3 +
 lib/igt_aux.c          |  94 +++++++
 lib/igt_aux.h          |  41 +++
 lib/igt_chamelium.c    | 689 +++++++++++++++++++++++++++++++++++++++++++++++++
 lib/igt_chamelium.h    |  69 +++++
 lib/igt_kms.c          | 109 ++++++++
 lib/igt_kms.h          |   6 +
 scripts/run-tests.sh   |   4 +-
 tests/Makefile.am      |   5 +-
 tests/Makefile.sources |   1 +
 tests/chamelium.c      | 407 +++++++++++++++++++++++++++++
 13 files changed, 1447 insertions(+), 4 deletions(-)
 create mode 100644 lib/igt_chamelium.c
 create mode 100644 lib/igt_chamelium.h
 create mode 100644 tests/chamelium.c

-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[RFC i-g-t v2 1/5] igt_aux: Add igt_skip_without_suspend_support()

[Lyude]

Since all of the chamelium calls are blocking, we need to be able to
make suspend/resume tests with it multi-threaded. As such, it's not the
best idea to rely on igt_system_suspend_autoresume() for skipping tests
on systems without suspend/resume support since we could accidentally
leave the thread controlling the chamelium running after the test gets
skipped.

Signed-off-by: Lyude <lyude@redhat.com>
---
 lib/igt_aux.c | 21 +++++++++++++++++++++
 lib/igt_aux.h |  2 ++
 2 files changed, 23 insertions(+)

diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 421f6d4..9754148 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -743,6 +743,27 @@ static uint32_t get_supported_suspend_states(int power_dir)
 }
 
 /**
+ * igt_skip_without_suspend_state:
+ * @state: an #igt_suspend_state to check for
+ *
+ * Check whether or not the system supports the given @state, and skip the
+ * current test if it doesn't. Useful for tests we want to skip before
+ * attempting to call #igt_system_suspend_autoresume.
+ */
+void igt_skip_without_suspend_support(enum igt_suspend_state state,
+				      enum igt_suspend_test test)
+{
+	int power_dir;
+
+	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
+	igt_require(get_supported_suspend_states(power_dir) & (1 << state));
+	igt_require(test == SUSPEND_TEST_NONE ||
+		    faccessat(power_dir, "pm_test", R_OK | W_OK, 0) == 0);
+
+	close(power_dir);
+}
+
+/**
  * igt_system_suspend_autoresume:
  * @state: an #igt_suspend_state, the target suspend state
  * @test: an #igt_suspend_test, test point at which to complete the suspend
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index d30196b..973a9cd 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -170,6 +170,8 @@ enum igt_suspend_test {
 	SUSPEND_TEST_NUM,
 };
 
+void igt_skip_without_suspend_support(enum igt_suspend_state state,
+				      enum igt_suspend_test test);
 void igt_system_suspend_autoresume(enum igt_suspend_state state,
 				   enum igt_suspend_test test);
 
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[RFC i-g-t v2 2/5] igt_aux: Add igt_set_autoresume_delay()

[Lyude]

The default autoresume delay is about 5 seconds. It's possible on a
system that's not very fast this might not be a long enough time, since
an asynchronous hotplug event we scheduled on the chamelium that was
intended to happen during suspend could happen before we actually manage
to suspend. So, add a function that allows us to increase the autoresume
time to ensure this never happens during suspend/resume tests with the
chamelium.

Signed-off-by: Lyude <lyude@redhat.com>

Changes since v1:
- Use igt_require, not assert
---
 lib/igt_aux.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++
 lib/igt_aux.h |  1 +
 2 files changed, 47 insertions(+)

diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 9754148..67432c6 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -812,6 +812,52 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
 	close(power_dir);
 }
 
+static int original_autoresume_delay;
+
+static void igt_restore_autoresume_delay(int sig)
+{
+	int delay_fd;
+	char delay_str[10];
+
+	igt_require((delay_fd = open("/sys/module/suspend/parameters/pm_test_delay",
+				    O_WRONLY)) >= 0);
+
+	snprintf(delay_str, sizeof(delay_str), "%d", original_autoresume_delay);
+	igt_require(write(delay_fd, delay_str, strlen(delay_str)));
+
+	close(delay_fd);
+}
+
+/**
+ * igt_set_autoresume_delay:
+ * @delay_secs: The delay in seconds before resuming the system
+ *
+ * Sets how long we wait to resume the system after suspending it, using the
+ * suspend.pm_test_delay variable. On exit, the original delay value is
+ * restored.
+ */
+void igt_set_autoresume_delay(int delay_secs)
+{
+	int delay_fd;
+	char delay_str[10];
+
+	igt_skip_on_simulation();
+
+	igt_require((delay_fd = open("/sys/module/suspend/parameters/pm_test_delay",
+				    O_RDWR)) >= 0);
+
+	if (!original_autoresume_delay) {
+		igt_require(read(delay_fd, delay_str, sizeof(delay_str)));
+		original_autoresume_delay = atoi(delay_str);
+		igt_install_exit_handler(igt_restore_autoresume_delay);
+	}
+
+	snprintf(delay_str, sizeof(delay_str), "%d", delay_secs);
+	igt_require(write(delay_fd, delay_str, strlen(delay_str)));
+
+	close(delay_fd);
+}
+
 /**
  * igt_drop_root:
  *
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index 973a9cd..7cee901 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -174,6 +174,7 @@ void igt_skip_without_suspend_support(enum igt_suspend_state state,
 				      enum igt_suspend_test test);
 void igt_system_suspend_autoresume(enum igt_suspend_state state,
 				   enum igt_suspend_test test);
+void igt_set_autoresume_delay(int delay_secs);
 
 /* dropping priviledges */
 void igt_drop_root(void);
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[RFC i-g-t v2 3/5] igt_aux: Add some list helpers from wayland

[Lyude]

Since we're going to be using lists for keeping track of EDIDs we've
allocated on the chamelium, add some generic list helpers from the
wayland project.

Signed-off-by: Lyude <lyude@redhat.com>

Changes since v1:
- Rename list helpers to be more like those from the Linux kernel
---
 lib/igt_aux.c | 27 +++++++++++++++++++++++++++
 lib/igt_aux.h | 38 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 65 insertions(+)

diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 67432c6..6e44218 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -1343,3 +1343,30 @@ double igt_stop_siglatency(struct igt_mean *result)
 
 	return mean;
 }
+
+void igt_list_init(struct igt_list *igt_list)
+{
+	igt_list->prev = igt_list;
+	igt_list->next = igt_list;
+}
+
+void igt_list_add(struct igt_list *igt_list, struct igt_list *elm)
+{
+	elm->prev = igt_list;
+	elm->next = igt_list->next;
+	igt_list->next = elm;
+	elm->next->prev = elm;
+}
+
+void igt_list_del(struct igt_list *elm)
+{
+	elm->prev->next = elm->next;
+	elm->next->prev = elm->prev;
+	elm->next = NULL;
+	elm->prev = NULL;
+}
+
+bool igt_list_empty(const struct igt_list *igt_list)
+{
+	return igt_list->next == igt_list;
+}
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index 7cee901..c021e3a 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -267,4 +267,42 @@ double igt_stop_siglatency(struct igt_mean *result);
 void igt_set_module_param(const char *name, const char *val);
 void igt_set_module_param_int(const char *name, int val);
 
+/*
+ * This list data structure is a verbatim copy from wayland-util.h from the
+ * Wayland project; except that wl_ prefix has been removed.
+ */
+
+struct igt_list {
+	struct igt_list *prev;
+	struct igt_list *next;
+};
+
+void igt_list_init(struct igt_list *list);
+void igt_list_add(struct igt_list *list, struct igt_list *elm);
+void igt_list_del(struct igt_list *elm);
+bool igt_list_empty(const struct igt_list *list);
+
+#ifdef __GNUC__
+#define container_of(ptr, sample, member)				\
+	(__typeof__(sample))((char *)(ptr)	-			\
+		 ((char *)&(sample)->member - (char *)(sample)))
+#else
+#define container_of(ptr, sample, member)				\
+	(void *)((char *)(ptr)	-				        \
+		 ((char *)&(sample)->member - (char *)(sample)))
+#endif
+
+#define igt_list_for_each(pos, head, member)				\
+	for (pos = 0, pos = container_of((head)->next, pos, member);	\
+	     &pos->member != (head);					\
+	     pos = container_of(pos->member.next, pos, member))
+
+#define igt_list_for_each_safe(pos, tmp, head, member)			\
+	for (pos = 0, tmp = 0, 						\
+	     pos = container_of((head)->next, pos, member),		\
+	     tmp = container_of((pos)->member.next, tmp, member);	\
+	     &pos->member != (head);					\
+	     pos = tmp,							\
+	     tmp = container_of(pos->member.next, tmp, member))
+
 #endif /* IGT_AUX_H */
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[RFC i-g-t v2 4/5] igt_kms: Add helpers for watching for sysfs hotplug events

[Lyude]

This adds some basic helpers for connecting to udev and watching for
sysfs hotplug events.

Signed-off-by: Lyude <lyude@redhat.com>
---
 lib/igt_kms.c | 109 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 lib/igt_kms.h |   6 ++++
 2 files changed, 115 insertions(+)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 989704e..433a721 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -38,6 +38,10 @@
 #elif HAVE_SYS_KD_H
 #include <sys/kd.h>
 #endif
+#ifdef HAVE_UDEV
+#include <libudev.h>
+#include <poll.h>
+#endif
 #include <errno.h>
 #include <time.h>
 
@@ -2760,6 +2764,111 @@ void igt_reset_connectors(void)
 			      "detect");
 }
 
+#ifdef HAVE_UDEV
+static struct udev_monitor *hotplug_mon;
+
+/**
+ * igt_watch_hotplug:
+ *
+ * Begin monitoring udev for hotplug events.
+ */
+void igt_watch_hotplug(void)
+{
+	struct udev *udev;
+	int ret, flags, fd;
+
+	if (hotplug_mon)
+		igt_cleanup_hotplug();
+
+	udev = udev_new();
+	igt_assert(udev != NULL);
+
+	hotplug_mon = udev_monitor_new_from_netlink(udev, "udev");
+	igt_assert(hotplug_mon != NULL);
+
+	ret = udev_monitor_filter_add_match_subsystem_devtype(hotplug_mon,
+							      "drm",
+							      "drm_minor");
+	igt_assert_eq(ret, 0);
+	ret = udev_monitor_filter_update(hotplug_mon);
+	igt_assert_eq(ret, 0);
+	ret = udev_monitor_enable_receiving(hotplug_mon);
+	igt_assert_eq(ret, 0);
+
+	/* Set the fd for udev as non blocking */
+	fd = udev_monitor_get_fd(hotplug_mon);
+	flags = fcntl(fd, F_GETFL, 0);
+	igt_assert(flags);
+
+	flags |= O_NONBLOCK;
+	igt_assert_neq(fcntl(fd, F_SETFL, flags), -1);
+}
+
+/**
+ * igt_hotplug_detected:
+ * @timeout_secs: How long to wait for a hotplug event to occur.
+ *
+ * Assert that a hotplug event was received since we last checked the monitor.
+ *
+ * Returns: true if a sysfs hotplug event was received, false if we timed out
+ */
+bool igt_hotplug_detected(int timeout_secs)
+{
+	struct udev_device *dev;
+	const char *hotplug_val;
+	struct pollfd fd = {
+		.fd = udev_monitor_get_fd(hotplug_mon),
+		.events = POLLIN
+	};
+	bool hotplug_received = false;
+
+	/* Go through all of the events pending on the udev monitor. Once we
+	 * receive a hotplug, we continue going through the rest of the events
+	 * so that redundant hotplug events don't change the results of future
+	 * checks
+	 */
+	while (!hotplug_received && poll(&fd, 1, timeout_secs * 1000)) {
+		dev = udev_monitor_receive_device(hotplug_mon);
+
+		hotplug_val = udev_device_get_property_value(dev, "HOTPLUG");
+		if (hotplug_val && atoi(hotplug_val) == 1)
+			hotplug_received = true;
+
+		udev_device_unref(dev);
+	}
+
+	return hotplug_received;
+}
+
+/**
+ * igt_flush_hotplugs:
+ * @mon: A udev monitor created by #igt_watch_hotplug
+ *
+ * Get rid of any pending hotplug events waiting on the udev monitor
+ */
+void igt_flush_hotplugs(void)
+{
+	struct udev_device *dev;
+
+	while ((dev = udev_monitor_receive_device(hotplug_mon)))
+		udev_device_unref(dev);
+}
+
+/**
+ * igt_cleanup_hotplug:
+ *
+ * Cleanup the resources allocated by #igt_watch_hotplug
+ */
+void igt_cleanup_hotplug(void)
+{
+	struct udev *udev = udev_monitor_get_udev(hotplug_mon);
+
+	udev_monitor_unref(hotplug_mon);
+	hotplug_mon = NULL;
+	udev_unref(udev);
+}
+#endif
+
 /**
  * kmstest_get_vbl_flag:
  * @pipe_id: Pipe to convert to flag representation.
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index 6422adc..95395cd 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -479,5 +479,11 @@ 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
+void igt_watch_hotplug(void);
+bool igt_hotplug_detected(int timeout_secs);
+void igt_flush_hotplugs(void);
+void igt_cleanup_hotplug(void);
+#endif
 
 #endif /* __IGT_KMS_H__ */
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[RFC i-g-t v2 5/5] Add support for hotplug testing with the Chamelium

[Lyude]

For the purpose of testing things such as hotplugging and bad monitors,
the ChromeOS team ended up designing a neat little device known as the
Chamelium. More information on this can be found here:

	https://www.chromium.org/chromium-os/testing/chamelium

This adds support for a couple of things to intel-gpu-tools:
 - igt library functions for connecting to udev and monitoring it for
   hotplug events, loosely based off of the unfinished hotplugging
   implementation in testdisplay
 - Library functions for controlling the chamelium in tests using
   xmlrpc. A couple of RPC calls were ommitted here, mainly because they
   didn't seem very useful for our needs or because they're just plain
   broken
 - A set of basic tests using the chamelium.

Because there's no sure fire way to figure out the mappings between
chamelium connectors and the DRM connectors on the machine, we rely on
the user specifying these mappings manually in the Chamelium
configuration file.

Running these tests requires (of course) a working Chamelium, along with
the RPC URL for the chamelium being specified in the environment
variable CHAMELIUM_HOST. If no URL is specified, the tests will just
skip on their own. As well, tests for connectors which are not actually
present on the system or the chamelium will skip on their own as well.

Signed-off-by: Lyude <lyude@redhat.com>

Changes since v1:
- Don't try to guess connector mappings, have the user specify them
  manually using a configuration file
- Open DRM fd using DRIVER_ANY, not DRIVER_INTEL
- Lower the hotplug timeout a little bit, since 30 seconds was leftover
  from debugging these tests anyway
- Don't try to keep track of the original state of the chamelium ports,
  and just leave them plugged in after each run. This makes more sense
  to me, since I'd imagine in automated testing setups using chameliums
  that all of the extra monitors will probably be provided by the
  Chamelium to begin with, so keeping them plugged in would make sure
  tests running afterwards that require >1 monitor don't get skipped.
- Add wait_for_connector() to the chamelium tests. After some more
  testing, I found that depending on the system some tests would throw
  false negatives due to us not waiting long enough for the system to
  detect that we connected something to it. This mainly happened with
  VGA connectors, since their lack of HPD makes them take significantly
  longer for the hardware to notice. wait_for_connector() fixes this by
  continually reprobing the status of the desired connector (without
  relying on a hpd event happening, since that might never come) until
  we get what we want, or we time out and fail.
- Use kmstest_get_property() for retrieving EDIDs instead of doing it by
  hand
- Don't hardcode PIPE_A for bringing up the display, use kmstest to find
  an appropriate CRTC to use.
---
 configure.ac           |  13 +
 lib/Makefile.am        |  10 +-
 lib/igt.h              |   3 +
 lib/igt_chamelium.c    | 689 +++++++++++++++++++++++++++++++++++++++++++++++++
 lib/igt_chamelium.h    |  69 +++++
 scripts/run-tests.sh   |   4 +-
 tests/Makefile.am      |   5 +-
 tests/Makefile.sources |   1 +
 tests/chamelium.c      | 407 +++++++++++++++++++++++++++++
 9 files changed, 1197 insertions(+), 4 deletions(-)
 create mode 100644 lib/igt_chamelium.c
 create mode 100644 lib/igt_chamelium.h
 create mode 100644 tests/chamelium.c

diff --git a/configure.ac b/configure.ac
index e181c83..53d7074 100644
--- a/configure.ac
+++ b/configure.ac
@@ -262,6 +262,18 @@ if test "x$with_libunwind" = xyes; then
 			  AC_MSG_ERROR([libunwind not found. Use --without-libunwind to disable libunwind support.]))
 fi
 
+# enable support for using the chamelium
+AC_ARG_ENABLE(chamelium,
+	      AS_HELP_STRING([--without-chamelium],
+			     [Build tests without chamelium support]),
+	      [], [with_chamelium=yes])
+
+AM_CONDITIONAL(HAVE_CHAMELIUM, [test "x$with_chamelium" = xyes])
+if test "x$with_chamelium" = xyes; then
+	AC_DEFINE(HAVE_CHAMELIUM, 1, [chamelium suport])
+	PKG_CHECK_MODULES(XMLRPC, xmlrpc_client)
+fi
+
 # enable debug symbols
 AC_ARG_ENABLE(debug,
 	      AS_HELP_STRING([--disable-debug],
@@ -359,6 +371,7 @@ echo "       Assembler          : ${enable_assembler}"
 echo "       Debugger           : ${enable_debugger}"
 echo "       Overlay            : X: ${enable_overlay_xlib}, Xv: ${enable_overlay_xvlib}"
 echo "       x86-specific tools : ${build_x86}"
+echo "       Chamelium support  : ${with_chamelium}"
 echo ""
 echo " • API-Documentation      : ${enable_gtk_doc}"
 echo " • Fail on warnings       : ${enable_werror}"
diff --git a/lib/Makefile.am b/lib/Makefile.am
index 4c0893d..aeac43a 100644
--- a/lib/Makefile.am
+++ b/lib/Makefile.am
@@ -22,8 +22,14 @@ if !HAVE_LIBDRM_INTEL
         stubs/drm/intel_bufmgr.h
 endif
 
+if HAVE_CHAMELIUM
+    libintel_tools_la_SOURCES +=	\
+	igt_chamelium.c			\
+	igt_chamelium.h
+endif
+
 AM_CPPFLAGS = -I$(top_srcdir)
-AM_CFLAGS = $(CWARNFLAGS) $(DRM_CFLAGS) $(PCIACCESS_CFLAGS) $(LIBUNWIND_CFLAGS) $(DEBUG_CFLAGS) \
+AM_CFLAGS = $(CWARNFLAGS) $(DRM_CFLAGS) $(PCIACCESS_CFLAGS) $(LIBUNWIND_CFLAGS) $(DEBUG_CFLAGS) $(XMLRPC_CFLAGS) $(UDEV_CFLAGS) \
 	    -DIGT_SRCDIR=\""$(abs_top_srcdir)/tests"\" \
 	    -DIGT_DATADIR=\""$(pkgdatadir)"\" \
 	    -DIGT_LOG_DOMAIN=\""$(subst _,-,$*)"\" \
@@ -38,5 +44,7 @@ libintel_tools_la_LIBADD = \
 	$(LIBUDEV_LIBS) \
 	$(LIBUNWIND_LIBS) \
 	$(TIMER_LIBS) \
+	$(XMLRPC_LIBS) \
+	$(UDEV_LIBS) \
 	-lm
 
diff --git a/lib/igt.h b/lib/igt.h
index d751f24..ec4632f 100644
--- a/lib/igt.h
+++ b/lib/igt.h
@@ -30,6 +30,9 @@
 #include "igt_aux.h"
 #include "igt_core.h"
 #include "igt_core.h"
+#ifdef HAVE_CHAMELIUM
+#include "igt_chamelium.h"
+#endif
 #include "igt_debugfs.h"
 #include "igt_draw.h"
 #include "igt_fb.h"
diff --git a/lib/igt_chamelium.c b/lib/igt_chamelium.c
new file mode 100644
index 0000000..6697054
--- /dev/null
+++ b/lib/igt_chamelium.c
@@ -0,0 +1,689 @@
+/*
+ * Copyright © 2016 Red Hat Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ * Authors:
+ *  Lyude Paul <lyude@redhat.com>
+ */
+
+#include "config.h"
+
+#include <string.h>
+#include <errno.h>
+#include <xmlrpc-c/base.h>
+#include <xmlrpc-c/client.h>
+#include <glib.h>
+
+#include "igt.h"
+
+#define check_rpc() \
+	igt_assert_f(!env.fault_occurred, "Chamelium RPC call failed: %s\n", \
+		     env.fault_string);
+
+/**
+ * chamelium_ports:
+ *
+ * Contains information on all of the ports that are physically connected from
+ * the chamelium to the system. This information is initialized when
+ * #chamelium_init is called.
+ */
+struct chamelium_port *chamelium_ports;
+
+/**
+ * chamelium_port_count:
+ *
+ * How many ports are physically connected from the chamelium to the system.
+ */
+int chamelium_port_count;
+
+static const char *chamelium_url;
+static xmlrpc_env env;
+
+struct chamelium_edid {
+	int id;
+	struct igt_list link;
+};
+struct chamelium_edid *allocated_edids;
+
+/**
+ * chamelium_plug:
+ * @id: The ID of the port on the chamelium to plug in
+ *
+ * Simulate a display connector being plugged into the system using the
+ * chamelium.
+ */
+void chamelium_plug(int id)
+{
+	xmlrpc_value *res;
+
+	igt_debug("Plugging port %d\n", id);
+	res = xmlrpc_client_call(&env, chamelium_url, "Plug", "(i)", id);
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+}
+
+/**
+ * chamelium_unplug:
+ * @id: The ID of the port on the chamelium to unplug
+ *
+ * Simulate a display connector being unplugged from the system using the
+ * chamelium.
+ */
+void chamelium_unplug(int id)
+{
+	xmlrpc_value *res;
+
+	igt_debug("Unplugging port %d\n", id);
+	res = xmlrpc_client_call(&env, chamelium_url, "Unplug", "(i)", id);
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+}
+
+/**
+ * chamelium_is_plugged:
+ * @id: The ID of the port on the chamelium to check the status of
+ *
+ * Check whether or not the given port has been plugged into the system using
+ * #chamelium_plug.
+ *
+ * Returns: True if the connector is set to plugged in, false otherwise.
+ */
+bool chamelium_is_plugged(int id)
+{
+	xmlrpc_value *res;
+	xmlrpc_bool is_plugged;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "IsPlugged", "(i)", id);
+	check_rpc();
+
+	xmlrpc_read_bool(&env, res, &is_plugged);
+	xmlrpc_DECREF(res);
+
+	return is_plugged;
+}
+
+/**
+ * chamelium_port_wait_video_input_stable:
+ * @id: The ID of the port on the chamelium to check the status of
+ * @timeout_secs: How long to wait for a video signal to appear before timing
+ * out
+ *
+ * Waits for a video signal to appear on the given port. This is useful for
+ * checking whether or not we've setup a monitor correctly.
+ *
+ * Returns: True if a video signal was detected, false if we timed out
+ */
+bool chamelium_port_wait_video_input_stable(int id, int timeout_secs)
+{
+	xmlrpc_value *res;
+	xmlrpc_bool is_on;
+
+	igt_debug("Waiting for video input to stabalize on port %d\n", id);
+
+	res = xmlrpc_client_call(&env, chamelium_url, "WaitVideoInputStable",
+				 "(ii)", id, timeout_secs);
+	check_rpc();
+
+	xmlrpc_read_bool(&env, res, &is_on);
+	xmlrpc_DECREF(res);
+
+	return is_on;
+}
+
+/**
+ * chamelium_fire_hpd_pulses:
+ * @id: The ID of the port to fire hotplug pulses on
+ * @width_msec: How long each pulse should last
+ * @count: The number of pulses to send
+ *
+ * A convienence function for sending multiple hotplug pulses to the system.
+ * The pulses start at low (e.g. connector is disconnected), and then alternate
+ * from high (e.g. connector is plugged in) to low. This is the equivalent of
+ * repeatedly calling #chamelium_plug and #chamelium_unplug, waiting
+ * @width_msec between each call.
+ *
+ * If @count is even, the last pulse sent will be high, and if it's odd then it
+ * will be low. Resetting the HPD line back to it's previous state, if desired,
+ * is the responsibility of the caller.
+ */
+void chamelium_fire_hpd_pulses(int port, int width_msec, int count)
+{
+	xmlrpc_value *pulse_widths = xmlrpc_array_new(&env),
+		     *width = xmlrpc_int_new(&env, width_msec), *res;
+	int i;
+
+	igt_debug("Firing %d HPD pulses with width of %d msec on port %d\n",
+		  count, width_msec, port);
+
+	for (i = 0; i < count; i++)
+		xmlrpc_array_append_item(&env, pulse_widths, width);
+
+	res = xmlrpc_client_call(&env, chamelium_url, "FireMixedHpdPulses",
+				 "(iA)", port, pulse_widths);
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+	xmlrpc_DECREF(width);
+	xmlrpc_DECREF(pulse_widths);
+}
+
+/**
+ * chamelium_fire_mixed_hpd_pulses:
+ * @id: The ID of the port to fire hotplug pulses on
+ * @...: The length of each pulse in milliseconds, terminated with a %0
+ *
+ * Does the same thing as #chamelium_fire_hpd_pulses, but allows the caller to
+ * specify the length of each individual pulse.
+ */
+void chamelium_fire_mixed_hpd_pulses(int id, ...)
+{
+	va_list args;
+	xmlrpc_value *pulse_widths = xmlrpc_array_new(&env), *width, *res;
+	int arg;
+
+	igt_debug("Firing mixed HPD pulses on port %d\n", id);
+
+	va_start(args, id);
+	for (arg = va_arg(args, int); arg; arg = va_arg(args, int)) {
+		width = xmlrpc_int_new(&env, arg);
+		xmlrpc_array_append_item(&env, pulse_widths, width);
+		xmlrpc_DECREF(width);
+	}
+	va_end(args);
+
+	res = xmlrpc_client_call(&env, chamelium_url, "FireMixedHpdPulses",
+				 "(iA)", id, pulse_widths);
+	check_rpc();
+	xmlrpc_DECREF(res);
+
+	xmlrpc_DECREF(pulse_widths);
+}
+
+static void async_rpc_handler(const char *server_url, const char *method_name,
+			      xmlrpc_value *param_array, void *user_data,
+			      xmlrpc_env *fault, xmlrpc_value *result)
+{
+	/* We don't care about the responses */
+}
+
+/**
+ * chamelium_async_hpd_pulse_start:
+ * @id: The ID of the port to fire a hotplug pulse on
+ * @high: Whether to fire a high pulse (e.g. simulate a connect), or a low
+ * pulse (e.g. simulate a disconnect)
+ * @delay_secs: How long to wait before sending the HPD pulse.
+ *
+ * Instructs the chamelium to send an hpd pulse after @delay_secs seconds have
+ * passed, without waiting for the chamelium to finish. This is useful for
+ * testing things such as hpd after a suspend/resume cycle, since we can't tell
+ * the chamelium to send a hotplug at the same time that our system is
+ * suspended.
+ *
+ * It is required that the user eventually call
+ * #chamelium_async_hpd_pulse_finish, to clean up the leftover XML-RPC
+ * responses from the chamelium.
+ */
+void chamelium_async_hpd_pulse_start(int id, bool high, int delay_secs)
+{
+	xmlrpc_value *pulse_widths = xmlrpc_array_new(&env), *width;
+
+	/* TODO: Actually implement something in the chameleon server to allow
+	 * for delayed actions such as hotplugs. This would work a bit better
+	 * and allow us to test suspend/resume on ports without hpd like VGA
+	 */
+
+	igt_debug("Sending HPD pulse (%s) on port %d with %d second delay\n",
+		  high ? "high->low" : "low->high", id, delay_secs);
+
+	/* If we're starting at high, make the first pulse width 0 so we keep
+	 * the port connected */
+	if (high) {
+		width = xmlrpc_int_new(&env, 0);
+		xmlrpc_array_append_item(&env, pulse_widths, width);
+		xmlrpc_DECREF(width);
+	}
+
+	width = xmlrpc_int_new(&env, delay_secs * 1000);
+	xmlrpc_array_append_item(&env, pulse_widths, width);
+	xmlrpc_DECREF(width);
+
+	xmlrpc_client_call_asynch(chamelium_url, "FireMixedHpdPulses",
+				  async_rpc_handler, NULL, "(iA)",
+				  id, pulse_widths);
+	xmlrpc_DECREF(pulse_widths);
+}
+
+/**
+ * chamelium_async_hpd_pulse_finish:
+ *
+ * Waits for any asynchronous RPC started by #chamelium_async_hpd_pulse_start
+ * to complete, and then cleans up any leftover responses from the chamelium.
+ * If all of the RPC calls have already completed, this function returns
+ * immediately.
+ */
+void chamelium_async_hpd_pulse_finish(void)
+{
+	xmlrpc_client_event_loop_finish_asynch();
+}
+
+/**
+ * chamelium_new_edid:
+ * @edid: The edid blob to upload to the chamelium
+ *
+ * Uploads and registers a new EDID with the chamelium. The EDID will be
+ * destroyed automatically when #chamelium_deinit is called.
+ *
+ * Returns: The ID of the EDID uploaded to the chamelium.
+ */
+int chamelium_new_edid(const unsigned char *edid)
+{
+	xmlrpc_value *res;
+	struct chamelium_edid *allocated_edid;
+	int edid_id;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "CreateEdid",
+				 "(6)", edid, EDID_LENGTH);
+	check_rpc();
+
+	xmlrpc_read_int(&env, res, &edid_id);
+	xmlrpc_DECREF(res);
+
+	allocated_edid = malloc(sizeof(struct chamelium_edid));
+	igt_assert(allocated_edid);
+
+	allocated_edid->id = edid_id;
+	if (allocated_edids) {
+		igt_list_add(&allocated_edids->link, &allocated_edid->link);
+	} else {
+		igt_list_init(&allocated_edid->link);
+		allocated_edids = allocated_edid;
+	}
+
+	return edid_id;
+}
+
+static void chamelium_destroy_edid(int edid_id)
+{
+	xmlrpc_value *res;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "DestroyEdid",
+				 "(i)", edid_id);
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+}
+
+/**
+ * chamelium_port_set_edid:
+ * @id: The ID of the port to set the EDID on
+ * @edid_id: The ID of an EDID on the chamelium created with
+ * #chamelium_new_edid, or 0 to disable the EDID on the port
+ *
+ * Sets a port on the chamelium to use the specified EDID. This does not fire a
+ * hotplug pulse on it's own, and merely changes what EDID the chamelium port
+ * will report to us the next time we probe it. Users will need to reprobe the
+ * connectors themselves if they want to see the EDID reported by the port
+ * change.
+ */
+void chamelium_port_set_edid(int id, int edid_id)
+{
+	xmlrpc_value *res;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "ApplyEdid",
+				 "(ii)", id, edid_id);
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+}
+
+/**
+ * chamelium_port_set_ddc_state:
+ * @id: The ID of the port whose DDC bus we want to modify
+ * @enabled: Whether or not to enable the DDC bus
+ *
+ * This disables the DDC bus (e.g. the i2c line on the connector that gives us
+ * an EDID) of the specified port on the chamelium. This is useful for testing
+ * behavior on legacy connectors such as VGA, where the presence of a DDC bus
+ * is not always guaranteed.
+ */
+void chamelium_port_set_ddc_state(int port, bool enabled)
+{
+	xmlrpc_value *res;
+
+	igt_debug("%sabling DDC bus on port %d\n",
+		  enabled ? "En" : "Dis", port);
+
+	res = xmlrpc_client_call(&env, chamelium_url, "SetDdcState",
+				 "(ib)", port, enabled);
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+}
+
+/**
+ * chamelium_port_get_ddc_state:
+ * @id: The ID of the port whose DDC bus we want to check the status of
+ *
+ * Check whether or not the DDC bus on the specified chamelium port is enabled
+ * or not.
+ *
+ * Returns: True if the DDC bus is enabled, false otherwise.
+ */
+bool chamelium_port_get_ddc_state(int id)
+{
+	xmlrpc_value *res;
+	xmlrpc_bool enabled;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "IsDdcEnabled",
+				 "(i)", id);
+	check_rpc();
+
+	xmlrpc_read_bool(&env, res, &enabled);
+
+	xmlrpc_DECREF(res);
+	return enabled;
+}
+
+/**
+ * chamelium_port_get_resolution:
+ * @id: The ID of the port whose display resolution we want to check
+ * @x: Where to store the horizontal resolution of the port
+ * @y: Where to store the verical resolution of the port
+ *
+ * Check the current reported display resolution of the specified port on the
+ * chamelium. This information is provided by the chamelium itself, not DRM.
+ * Useful for verifying that we really are scanning out at the resolution we
+ * think we are.
+ */
+void chamelium_port_get_resolution(int id, int *x, int *y)
+{
+	xmlrpc_value *res, *res_x, *res_y;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "DetectResolution",
+				 "(i)", id);
+	check_rpc();
+
+	xmlrpc_array_read_item(&env, res, 0, &res_x);
+	xmlrpc_array_read_item(&env, res, 1, &res_y);
+	xmlrpc_read_int(&env, res_x, x);
+	xmlrpc_read_int(&env, res_y, y);
+
+	xmlrpc_DECREF(res_x);
+	xmlrpc_DECREF(res_y);
+	xmlrpc_DECREF(res);
+}
+
+/**
+ * chamelium_get_crc_for_area:
+ * @id: The ID of the port from which we want to retrieve the CRC
+ * @x: The X coordinate on the emulated display to start calculating the CRC
+ * from
+ * @y: The Y coordinate on the emulated display to start calculating the CRC
+ * from
+ * @w: The width of the area to fetch the CRC from
+ * @h: The height of the area to fetch the CRC from
+ *
+ * Reads back the pixel CRC for an area on the specified chamelium port. This
+ * is the same as using the CRC readback from a GPU, the main difference being
+ * the data is provided by the chamelium and also allows us to specify a region
+ * of the screen to use as opposed to the entire thing.
+ *
+ * Returns: The CRC read back from the chamelium
+ */
+unsigned int chamelium_get_crc_for_area(int id, int x, int y, int w, int h)
+{
+	xmlrpc_value *res;
+	unsigned int crc;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "ComputePixelChecksum",
+				 "(iiiii)", id, x, y, w, h);
+	check_rpc();
+
+	xmlrpc_read_int(&env, res, (int*)(&crc));
+
+	xmlrpc_DECREF(res);
+	return crc;
+}
+
+static unsigned int chamelium_get_port_type(int port)
+{
+	xmlrpc_value *res;
+	const char *port_type_str;
+	unsigned int port_type;
+
+	res = xmlrpc_client_call(&env, chamelium_url, "GetConnectorType",
+				 "(i)", port);
+	check_rpc();
+
+	xmlrpc_read_string(&env, res, &port_type_str);
+	igt_debug("Port %d is of type '%s'\n", port, port_type_str);
+
+	if (strcmp(port_type_str, "DP") == 0)
+		port_type = DRM_MODE_CONNECTOR_DisplayPort;
+	else if (strcmp(port_type_str, "HDMI") == 0)
+		port_type = DRM_MODE_CONNECTOR_HDMIA;
+	else if (strcmp(port_type_str, "VGA") == 0)
+		port_type = DRM_MODE_CONNECTOR_VGA;
+	else
+		port_type = DRM_MODE_CONNECTOR_Unknown;
+
+	free((void*)port_type_str);
+	xmlrpc_DECREF(res);
+
+	return port_type;
+}
+
+static void chamelium_read_port_mappings(int drm_fd, GKeyFile *key_file)
+{
+	drmModeRes *res;
+	drmModeConnector *connector;
+	struct chamelium_port *port;
+	GError *error = NULL;
+	char **group_list;
+	char *group;
+	size_t group_list_len;
+	int port_i, i, j;
+
+	group_list = g_key_file_get_groups(key_file, &group_list_len);
+	chamelium_port_count = group_list_len - 1;
+	chamelium_ports = calloc(sizeof(struct chamelium_port),
+				 chamelium_port_count);
+	memset(chamelium_ports, 0,
+	       sizeof(struct chamelium_port) * chamelium_port_count);
+	port_i = 0;
+	res = drmModeGetResources(drm_fd);
+
+	for (i = 0; group_list[i] != NULL; i++) {
+		group = group_list[i];
+
+		if (strcmp(group, "Chamelium") == 0)
+			continue;
+
+		port = &chamelium_ports[port_i++];
+		port->connector_name = strdup(group);
+		port->id = g_key_file_get_integer(key_file, group,
+						  "ChameliumPortID",
+						  &error);
+		igt_require_f(port->id,
+			      "Failed to read chamelium port ID for %s: %s\n",
+			      group, error->message);
+
+		port->type = chamelium_get_port_type(port->id);
+		igt_require_f(port->type != DRM_MODE_CONNECTOR_Unknown,
+			      "Unable to retrieve the physical port type from the Chamelium for '%s'\n",
+			      group);
+
+		for (j = 0;
+		     j < res->count_connectors && !port->connector_id;
+		     j++) {
+			char connector_name[50];
+
+			connector = drmModeGetConnectorCurrent(
+			    drm_fd, res->connectors[j]);
+
+			/* We have to generate the connector name on our own */
+			snprintf(connector_name, 50, "%s-%u",
+				 kmstest_connector_type_str(connector->connector_type),
+				 connector->connector_type_id);
+
+			if (strcmp(connector_name, group) == 0)
+				port->connector_id = connector->connector_id;
+
+			drmModeFreeConnector(connector);
+		}
+		igt_assert_f(port->connector_id,
+			     "No connector found with name '%s'\n", group);
+
+		igt_debug("Port '%s' with physical type '%s' mapped to Chamelium port %d\n",
+			  group, kmstest_connector_type_str(port->type),
+			  port->id);
+	}
+
+	drmModeFreeResources(res);
+	g_strfreev(group_list);
+}
+
+static void chamelium_read_config(int drm_fd)
+{
+	GKeyFile *key_file = g_key_file_new();
+	GError *error = NULL;
+	char *key_file_loc;
+	int rc;
+
+	key_file_loc = getenv("CHAMELIUM_CONFIG_PATH");
+	if (!key_file_loc) {
+		igt_require(key_file_loc = alloca(100));
+		snprintf(key_file_loc, 100, "%s/.igt_chamelium_rc",
+			 g_get_home_dir());
+	}
+
+	rc = g_key_file_load_from_file(key_file, key_file_loc,
+				       G_KEY_FILE_NONE, &error);
+	igt_require_f(rc, "Failed to read chamelium configuration file: %s\n",
+		      error->message);
+
+	chamelium_url = g_key_file_get_string(key_file, "Chamelium", "URL",
+					      &error);
+	igt_require_f(chamelium_url,
+		      "Couldn't read chamelium URL from config file: %s\n",
+		      error->message);
+
+	chamelium_read_port_mappings(drm_fd, key_file);
+
+	g_key_file_free(key_file);
+}
+
+/**
+ * chamelium_reset:
+ *
+ * Resets the chamelium's IO board. As well, this also has the effect of
+ * causing all of the chamelium ports to get set to unplugged
+ */
+void chamelium_reset(void)
+{
+	xmlrpc_value *res;
+
+	igt_debug("Resetting the chamelium\n");
+
+	res = xmlrpc_client_call(&env, chamelium_url, "Reset", "()");
+	check_rpc();
+
+	xmlrpc_DECREF(res);
+}
+
+static void chamelium_exit_handler(int sig)
+{
+	chamelium_deinit();
+}
+
+/**
+ * chamelium_init:
+ *
+ * Sets up a connection with a chamelium, using the url provided in the
+ * CHAMELIUM_HOST enviornment variable. This must be called first before trying
+ * to use the chamelium. When the connection is no longer needed, the user
+ * should call #chamelium_deinit to free the resources used by the connection.
+ *
+ * If we fail to establish a connection with the chamelium, we fail the current
+ * test.
+ */
+void chamelium_init(int drm_fd)
+{
+	xmlrpc_env_init(&env);
+
+	xmlrpc_client_init2(&env, XMLRPC_CLIENT_NO_FLAGS, PACKAGE,
+			    PACKAGE_VERSION, NULL, 0);
+	igt_fail_on_f(env.fault_occurred,
+		      "Failed to init xmlrpc: %s\n",
+		      env.fault_string);
+
+	chamelium_read_config(drm_fd);
+	chamelium_reset();
+
+	igt_install_exit_handler(chamelium_exit_handler);
+}
+
+/**
+ * chamelium_deinit:
+ *
+ * Frees the resources used by a connection to the chamelium that was set up
+ * with #chamelium_init. As well, this function restores the state of the
+ * chamelium like it was before calling #chamelium_init. This function is also
+ * called as an exit handler, so users only need to call manually if they don't
+ * want the chamelium interfering with other tests in the same file.
+ */
+void chamelium_deinit(void)
+{
+	int i;
+	struct chamelium_edid *pos, *tmp;
+
+	if (!chamelium_url)
+		return;
+
+	/* We want to make sure we leave all of the ports plugged in, since
+	 * testing setups requiring multiple monitors are probably using the
+	 * chamelium to provide said monitors
+	 */
+	chamelium_reset();
+	for (i = 0; i < chamelium_port_count; i++)
+		chamelium_plug(chamelium_ports[i].id);
+
+	/* Destroy any EDIDs we created to make sure we don't leak them */
+	igt_list_for_each_safe(pos, tmp, &allocated_edids->link, link) {
+		chamelium_destroy_edid(pos->id);
+		free(pos);
+	}
+
+	xmlrpc_client_cleanup();
+	xmlrpc_env_clean(&env);
+
+	for (i = 0; i < chamelium_port_count; i++)
+		free(chamelium_ports[i].connector_name);
+
+	free(chamelium_ports);
+	allocated_edids = NULL;
+	chamelium_url = NULL;
+	chamelium_ports = NULL;
+	chamelium_port_count = 0;
+}
+
diff --git a/lib/igt_chamelium.h b/lib/igt_chamelium.h
new file mode 100644
index 0000000..8c1d4a2
--- /dev/null
+++ b/lib/igt_chamelium.h
@@ -0,0 +1,69 @@
+/*
+ * Copyright © 2016 Red Hat Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ * Authors: Lyude Paul <lyude@redhat.com>
+ */
+
+#ifndef IGT_CHAMELIUM_H
+#define IGT_CHAMELIUM_H
+
+#include "config.h"
+#include "igt.h"
+#include <stdbool.h>
+
+/**
+ * chamelium_port:
+ * @type: The DRM connector type of the chamelium port (not the host's)
+ * @id: The ID of the chamelium port
+ * @connector_id: The ID of the DRM connector connected to this port
+ * @connector_name: The name of the DRM connector
+ */
+struct chamelium_port {
+	unsigned int type;
+	int id;
+	int connector_id;
+	char *connector_name;
+};
+
+extern int chamelium_port_count;
+extern struct chamelium_port *chamelium_ports;
+
+void chamelium_init(int drm_fd);
+void chamelium_deinit(void);
+void chamelium_reset(void);
+
+void chamelium_plug(int id);
+void chamelium_unplug(int id);
+bool chamelium_is_plugged(int id);
+bool chamelium_port_wait_video_input_stable(int id, int timeout_secs);
+void chamelium_fire_mixed_hpd_pulses(int id, ...);
+void chamelium_fire_hpd_pulses(int id, int width, int count);
+void chamelium_async_hpd_pulse_start(int id, bool high, int delay_secs);
+void chamelium_async_hpd_pulse_finish(void);
+int chamelium_new_edid(const unsigned char *edid);
+void chamelium_port_set_edid(int id, int edid_id);
+bool chamelium_port_get_ddc_state(int id);
+void chamelium_port_set_ddc_state(int id, bool enabled);
+void chamelium_port_get_resolution(int id, int *x, int *y);
+unsigned int chamelium_get_crc_for_area(int id, int x, int y, int w, int h);
+
+#endif /* IGT_CHAMELIUM_H */
diff --git a/scripts/run-tests.sh b/scripts/run-tests.sh
index 97ba9e5..6539bf9 100755
--- a/scripts/run-tests.sh
+++ b/scripts/run-tests.sh
@@ -122,10 +122,10 @@ if [ ! -x "$PIGLIT" ]; then
 fi
 
 if [ "x$RESUME" != "x" ]; then
-	sudo IGT_TEST_ROOT="$IGT_TEST_ROOT" "$PIGLIT" resume "$RESULTS" $NORETRY
+	sudo IGT_TEST_ROOT="$IGT_TEST_ROOT" CHAMELIUM_HOST="$CHAMELIUM_HOST" "$PIGLIT" resume "$RESULTS" $NORETRY
 else
 	mkdir -p "$RESULTS"
-	sudo IGT_TEST_ROOT="$IGT_TEST_ROOT" "$PIGLIT" run igt -o "$RESULTS" -s $VERBOSE $EXCLUDE $FILTER
+	sudo IGT_TEST_ROOT="$IGT_TEST_ROOT" CHAMELIUM_HOST="$CHAMELIUM_HOST" "$PIGLIT" run igt -o "$RESULTS" -s $VERBOSE $EXCLUDE $FILTER
 fi
 
 if [ "$SUMMARY" == "html" ]; then
diff --git a/tests/Makefile.am b/tests/Makefile.am
index a408126..06a8e6b 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -63,7 +63,7 @@ AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) -Wno-unused-result $(DEBUG_CFLAGS)\
 	$(LIBUNWIND_CFLAGS) $(WERROR_CFLAGS) \
 	$(NULL)
 
-LDADD = ../lib/libintel_tools.la $(GLIB_LIBS)
+LDADD = ../lib/libintel_tools.la $(GLIB_LIBS) $(XMLRPC_LIBS)
 
 AM_CFLAGS += $(CAIRO_CFLAGS) $(LIBUDEV_CFLAGS) $(GLIB_CFLAGS)
 AM_LDFLAGS = -Wl,--as-needed
@@ -119,5 +119,8 @@ vc4_wait_bo_CFLAGS = $(AM_CFLAGS) $(DRM_VC4_CFLAGS)
 vc4_wait_bo_LDADD = $(LDADD) $(DRM_VC4_LIBS)
 vc4_wait_seqno_CFLAGS = $(AM_CFLAGS) $(DRM_VC4_CFLAGS)
 vc4_wait_seqno_LDADD = $(LDADD) $(DRM_VC4_LIBS)
+
+chamelium_CFLAGS = $(AM_CFLAGS) $(XMLRPC_CFLAGS) $(UDEV_CFLAGS)
+chamelium_LDADD = $(LDADD) $(XMLRPC_LIBS) $(UDEV_LIBS)
 endif
 
diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index 65e0792..7d09b54 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -134,6 +134,7 @@ TESTS_progs_M = \
 	template \
 	vgem_basic \
 	vgem_slow \
+	chamelium \
 	$(NULL)
 
 TESTS_progs_XM = \
diff --git a/tests/chamelium.c b/tests/chamelium.c
new file mode 100644
index 0000000..e0f645a
--- /dev/null
+++ b/tests/chamelium.c
@@ -0,0 +1,407 @@
+/*
+ * Copyright © 2016 Red Hat Inc.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ * Authors:
+ *    Lyude Paul <lyude@redhat.com>
+ */
+
+#include "config.h"
+#include "igt.h"
+
+#include <fcntl.h>
+#include <string.h>
+
+typedef struct {
+	int drm_fd;
+} data_t;
+
+#define HOTPLUG_TIMEOUT 20 /* seconds */
+
+static void
+require_connector_present(data_t *data, unsigned int type)
+{
+	int i;
+	bool found = false;
+
+	for (i = 0; i < chamelium_port_count && !found; i++) {
+		if (chamelium_ports[i].type == type)
+			found = true;
+	}
+
+	igt_require_f(found, "No port of type %s was found\n",
+		      kmstest_connector_type_str(type));
+}
+
+static drmModeConnection
+reprobe_connector(data_t *data, struct chamelium_port *port)
+{
+	drmModeConnector *connector;
+	drmModeConnection status;
+
+	igt_debug("Reprobing %s...\n", port->connector_name);
+	connector = drmModeGetConnector(data->drm_fd, port->connector_id);
+	igt_assert(connector);
+	status = connector->connection;
+
+	drmModeFreeConnector(connector);
+	return status;
+}
+
+static void
+wait_for_connector(data_t *data, struct chamelium_port *port,
+		   drmModeConnection status)
+{
+	bool finished = false;
+
+	igt_debug("Waiting for %s to %sconnect...\n", port->connector_name,
+		  status == DRM_MODE_DISCONNECTED ? "dis" : "");
+
+	/*
+	 * Rely on simple reprobing so we don't fail tests that don't require
+	 * that hpd events work in the event that hpd doesn't work on the system
+	 */
+	igt_until_timeout(HOTPLUG_TIMEOUT) {
+		if (reprobe_connector(data, port) == status) {
+			finished = true;
+			return;
+		}
+
+		sleep(1);
+	}
+
+	igt_assert(finished);
+}
+
+static void
+reset_state(data_t *data, struct chamelium_port *port)
+{
+	igt_reset_connectors();
+	chamelium_reset();
+	wait_for_connector(data, port, DRM_MODE_DISCONNECTED);
+}
+
+static void
+test_basic_hotplug(data_t *data, struct chamelium_port *port)
+{
+	int i;
+
+	reset_state(data, port);
+
+	igt_watch_hotplug();
+
+	for (i = 0; i < 15; i++) {
+		igt_flush_hotplugs();
+
+		/* Check if we get a sysfs hotplug event */
+		chamelium_plug(port->id);
+		igt_assert(igt_hotplug_detected(HOTPLUG_TIMEOUT));
+		igt_assert(reprobe_connector(data, port) == DRM_MODE_CONNECTED);
+
+		igt_flush_hotplugs();
+
+		/* Now check if we get a hotplug from disconnection */
+		chamelium_unplug(port->id);
+		igt_assert(igt_hotplug_detected(HOTPLUG_TIMEOUT));
+		igt_assert(reprobe_connector(data, port) ==
+			   DRM_MODE_DISCONNECTED);
+
+		/* Sleep so we don't accidentally cause an hpd storm */
+		sleep(1);
+	}
+}
+
+static void
+test_edid_read(data_t *data, struct chamelium_port *port,
+	       int edid_id, const unsigned char *edid)
+{
+	drmModePropertyBlobPtr edid_blob = NULL;
+	uint64_t edid_blob_id;
+
+	reset_state(data, port);
+
+	chamelium_port_set_edid(port->id, edid_id);
+	chamelium_plug(port->id);
+	wait_for_connector(data, port, DRM_MODE_CONNECTED);
+
+	igt_assert(kmstest_get_property(data->drm_fd, port->connector_id,
+					DRM_MODE_OBJECT_CONNECTOR, "EDID", NULL,
+					&edid_blob_id, NULL));
+	igt_assert(edid_blob = drmModeGetPropertyBlob(data->drm_fd,
+						      edid_blob_id));
+
+	/* Compare the EDID from the connector to what we expect */
+	igt_assert(memcmp(edid, edid_blob->data, EDID_LENGTH) == 0);
+
+	drmModeFreePropertyBlob(edid_blob);
+}
+
+static void
+test_suspend_resume_hpd(data_t *data, struct chamelium_port *port,
+			enum igt_suspend_state state,
+			enum igt_suspend_test test)
+{
+	int delay = 7;
+
+	igt_skip_without_suspend_support(state, test);
+	reset_state(data, port);
+	igt_watch_hotplug();
+
+	igt_set_autoresume_delay(15);
+
+	/* Make sure we notice new connectors after resuming */
+	chamelium_async_hpd_pulse_start(port->id, false, delay);
+	igt_system_suspend_autoresume(state, test);
+	chamelium_async_hpd_pulse_finish();
+
+	igt_assert(igt_hotplug_detected(HOTPLUG_TIMEOUT));
+	igt_assert(reprobe_connector(data, port) == DRM_MODE_CONNECTED);
+
+	igt_flush_hotplugs();
+
+	/* Now make sure we notice disconnected connectors after resuming */
+	chamelium_async_hpd_pulse_start(port->id, true, delay);
+	igt_system_suspend_autoresume(state, test);
+	chamelium_async_hpd_pulse_finish();
+
+	igt_assert(igt_hotplug_detected(HOTPLUG_TIMEOUT));
+	igt_assert(reprobe_connector(data, port) == DRM_MODE_DISCONNECTED);
+}
+
+static void
+test_suspend_resume_edid_change(data_t *data, struct chamelium_port *port,
+				enum igt_suspend_state state,
+				enum igt_suspend_test test,
+				int edid_id,
+				int alt_edid_id)
+{
+	igt_skip_without_suspend_support(state, test);
+	reset_state(data, port);
+	igt_watch_hotplug();
+
+	/* First plug in the port */
+	chamelium_port_set_edid(port->id, edid_id);
+	chamelium_plug(port->id);
+	wait_for_connector(data, port, DRM_MODE_CONNECTED);
+
+	igt_flush_hotplugs();
+
+	/*
+	 * Change the edid before we suspend. On resume, the machine should
+	 * notice the EDID change and fire a hotplug event.
+	 */
+	chamelium_port_set_edid(port->id, alt_edid_id);
+
+	igt_system_suspend_autoresume(state, test);
+	igt_assert(igt_hotplug_detected(HOTPLUG_TIMEOUT));
+}
+
+static void
+test_display(data_t *data, struct chamelium_port *port)
+{
+	igt_display_t display;
+	igt_output_t *output;
+	igt_plane_t *primary;
+	struct igt_fb fb;
+	drmModeRes *res;
+	drmModeModeInfo *mode;
+	drmModeConnector *connector;
+	uint32_t crtc_id;
+	int fb_id;
+
+	reset_state(data, port);
+
+	chamelium_plug(port->id);
+	wait_for_connector(data, port, DRM_MODE_CONNECTED);
+	igt_assert(res = drmModeGetResources(data->drm_fd));
+	kmstest_unset_all_crtcs(data->drm_fd, res);
+
+	igt_display_init(&display, data->drm_fd);
+
+	/* Find the output struct for this connector */
+	for_each_connected_output(&display, output) {
+		if (output->config.connector->connector_id ==
+		    port->connector_id)
+			break;
+	}
+
+	connector = drmModeGetConnectorCurrent(data->drm_fd,
+					       port->connector_id);
+
+	/* Find a spare CRTC to use for the display */
+	crtc_id = kmstest_find_crtc_for_connector(data->drm_fd, res, connector,
+						  0);
+
+	/* Setup the display */
+	igt_output_set_pipe(output, kmstest_get_pipe_from_crtc_id(data->drm_fd,
+								  crtc_id));
+	mode = igt_output_get_mode(output);
+	primary = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
+	igt_assert(primary);
+
+	fb_id = igt_create_pattern_fb(data->drm_fd,
+				      mode->hdisplay,
+				      mode->vdisplay,
+				      DRM_FORMAT_XRGB8888,
+				      LOCAL_DRM_FORMAT_MOD_NONE,
+				      &fb);
+	igt_assert(fb_id > 0);
+	igt_plane_set_fb(primary, &fb);
+
+	igt_display_commit(&display);
+
+	igt_assert(chamelium_port_wait_video_input_stable(port->id,
+							  HOTPLUG_TIMEOUT));
+
+	drmModeFreeResources(res);
+	drmModeFreeConnector(connector);
+	igt_display_fini(&display);
+}
+
+static void
+test_hpd_without_ddc(data_t *data, struct chamelium_port *port)
+{
+	reset_state(data, port);
+	igt_watch_hotplug();
+
+	/* Disable the DDC on the connector and make sure we still get a
+	 * hotplug
+	 */
+	chamelium_port_set_ddc_state(port->id, false);
+	chamelium_plug(port->id);
+
+	igt_assert(igt_hotplug_detected(HOTPLUG_TIMEOUT));
+	igt_assert(reprobe_connector(data, port) == DRM_MODE_CONNECTED);
+}
+
+#define for_each_port(p, port)                  \
+	for (p = 0, port = &chamelium_ports[p]; \
+	     p < chamelium_port_count;          \
+	     p++, port = &chamelium_ports[p])   \
+
+#define connector_subtest(name__, type__) \
+	igt_subtest(name__)               \
+		for_each_port(p, port)    \
+			if (port->type == DRM_MODE_CONNECTOR_ ## type__)
+
+#define define_common_connector_tests(type_str__, type__)                     \
+	connector_subtest(type_str__ "-hpd", type__)                          \
+		test_basic_hotplug(&data, port);                              \
+                                                                              \
+	connector_subtest(type_str__ "-edid-read", type__) {                  \
+		test_edid_read(&data, port, edid_id,                          \
+			       igt_kms_get_base_edid());                      \
+		test_edid_read(&data, port, alt_edid_id,                      \
+			       igt_kms_get_alt_edid());                       \
+	}                                                                     \
+                                                                              \
+	connector_subtest(type_str__ "-hpd-after-suspend", type__)            \
+		test_suspend_resume_hpd(&data, port,                          \
+					SUSPEND_STATE_MEM,                    \
+					SUSPEND_TEST_NONE);                   \
+                                                                              \
+	connector_subtest(type_str__ "-hpd-after-hibernate", type__)          \
+		test_suspend_resume_hpd(&data, port,                          \
+					SUSPEND_STATE_DISK,                   \
+					SUSPEND_TEST_DEVICES);                \
+                                                                              \
+	connector_subtest(type_str__ "-edid-change-during-suspend", type__)   \
+		test_suspend_resume_edid_change(&data, port,                  \
+						SUSPEND_STATE_MEM,            \
+						SUSPEND_TEST_NONE,            \
+						edid_id, alt_edid_id);        \
+                                                                              \
+	connector_subtest(type_str__ "-edid-change-during-hibernate", type__) \
+		test_suspend_resume_edid_change(&data, port,                  \
+						SUSPEND_STATE_DISK,           \
+						SUSPEND_TEST_DEVICES,         \
+						edid_id, alt_edid_id);        \
+                                                                              \
+	connector_subtest(type_str__ "-display", type__)                      \
+		test_display(&data, port);
+
+static data_t data;
+
+igt_main
+{
+	struct chamelium_port *port;
+	int edid_id, alt_edid_id, p;
+
+	igt_fixture {
+		igt_skip_on_simulation();
+
+		data.drm_fd = drm_open_driver_master(DRIVER_ANY);
+
+		chamelium_init(data.drm_fd);
+
+		edid_id = chamelium_new_edid(igt_kms_get_base_edid());
+		alt_edid_id = chamelium_new_edid(igt_kms_get_alt_edid());
+
+		/* So fbcon doesn't try to reprobe things itself */
+		kmstest_set_vt_graphics_mode();
+	}
+
+	igt_subtest_group {
+		igt_fixture {
+			require_connector_present(
+			    &data, DRM_MODE_CONNECTOR_DisplayPort);
+		}
+
+		define_common_connector_tests("dp", DisplayPort);
+	}
+
+	igt_subtest_group {
+		igt_fixture {
+			require_connector_present(
+			    &data, DRM_MODE_CONNECTOR_HDMIA);
+		}
+
+		define_common_connector_tests("hdmi", HDMIA);
+	}
+
+	igt_subtest_group {
+		igt_fixture {
+			require_connector_present(
+			    &data, DRM_MODE_CONNECTOR_VGA);
+		}
+
+		connector_subtest("vga-hpd", VGA)
+			test_basic_hotplug(&data, port);
+
+		connector_subtest("vga-edid-read", VGA) {
+			test_edid_read(&data, port, edid_id,
+				       igt_kms_get_base_edid());
+			test_edid_read(&data, port, alt_edid_id,
+				       igt_kms_get_alt_edid());
+		}
+
+		/* FIXME: Right now there isn't a way to do any sort of delayed
+		 * psuedo-hotplug with VGA, so testing detection after a
+		 * suspend/resume cycle isn't possible yet
+		 */
+
+		connector_subtest("vga-hpd-without-ddc", VGA)
+			test_hpd_without_ddc(&data, port);
+
+		connector_subtest("vga-display", VGA)
+			test_display(&data, port);
+	}
+}
-- 
2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx