[PATCH i-g-t 20/21] tests/intel/xe_eudebug: Add render node testing

[Mika Kuoppala <mika.kuoppala@linux.intel.com>]

We need basic-connect to also check if render node is
available for connection.

Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
 tests/intel/xe_eudebug.c | 41 ++++++++++++++++++++++++++++++++++------
 1 file changed, 35 insertions(+), 6 deletions(-)

diff --git a/tests/intel/xe_eudebug.c b/tests/intel/xe_eudebug.c
index 4e7ba0348d..01410f3197 100644
--- a/tests/intel/xe_eudebug.c
+++ b/tests/intel/xe_eudebug.c
@@ -460,6 +460,9 @@ static int __debug_connect(int fd, int *debugfd, struct drm_xe_eudebug_connect *
 	return ret;
 }
 
+
+#define TEST_CONNECT_RENDER BIT(0)
+
 /**
  * SUBTEST: basic-connect
  * Functionality: attach
@@ -467,23 +470,34 @@ static int __debug_connect(int fd, int *debugfd, struct drm_xe_eudebug_connect *
  *	Exercise XE_EUDEBUG_CONNECT ioctl with passing
  *	valid and invalid params.
  */
-static void test_connect(int fd)
+static void test_connect(int fd, unsigned int flags)
 {
 	struct drm_xe_eudebug_connect param = {};
-	int debugfd, ret;
+	int debugfd, ret, render_fd, target_fd;
 	pid_t *pid;
 
 	pid = mmap(NULL, sizeof(pid_t), PROT_WRITE,
 		   MAP_SHARED | MAP_ANON, -1, 0);
 
+	if (flags & TEST_CONNECT_RENDER)
+		render_fd = drm_open_driver_render(DRIVER_XE);
+
 	/* get fresh unrelated pid */
-	igt_fork(child, 1)
+	igt_fork(child, 1) {
+		if (flags & TEST_CONNECT_RENDER)
+			close(fd);
 		*pid = getpid();
+		igt_info("client in pid %d\n", *pid);
+		sleep(5);
+	}
 
-	igt_waitchildren();
-	param.fd = xe_eudebug_fd_for_pid(*pid);
+	/* XXX build a sync mechanism */
+	sleep(1);
+	target_fd = xe_eudebug_fd_for_pid(*pid);
+	igt_assert(target_fd >= 0);
 	munmap(pid, sizeof(pid_t));
 
+	param.fd = target_fd + random() + 1;
 	ret = __debug_connect(fd, &debugfd, &param);
 	igt_assert(debugfd == -1);
 	igt_assert_eq(ret, -EBADFD);
@@ -493,7 +507,15 @@ static void test_connect(int fd)
 	igt_assert(debugfd == -1);
 	igt_assert_eq(ret, -EINVAL);
 
+	param.fd = target_fd;
+	ret = __debug_connect(fd, &debugfd, &param);
+	igt_assert_neq(debugfd, -1);
+	igt_assert_eq(ret, 0);
+
+	close(debugfd);
+
 	param.fd = xe_eudebug_fd_for_pid(getpid());
+	igt_assert((int)param.fd >= 0);
 	param.version = -1;
 	ret = __debug_connect(fd, &debugfd, &param);
 	igt_assert(debugfd == -1);
@@ -517,6 +539,10 @@ static void test_connect(int fd)
 	igt_assert_eq(ret, 0);
 
 	close(debugfd);
+	if (flags & TEST_CONNECT_RENDER)
+		close(render_fd);
+
+	igt_waitchildren();
 }
 
 static void switch_user(__uid_t uid, __gid_t gid)
@@ -2789,7 +2815,10 @@ int igt_main()
 		test_sysfs_toggle(fd);
 
 	igt_subtest("basic-connect")
-		test_connect(fd);
+		test_connect(fd, 0);
+
+	igt_subtest("basic-connect-render")
+		test_connect(fd, TEST_CONNECT_RENDER);
 
 	igt_subtest("connect-user")
 		test_connect_user(fd);
-- 
2.43.0