From: Chris Wilson <chris@chris-wilson.co.uk>
Date: Fri, 21 Jun 2013 15:52:53 +0000 (+0100)
Subject: kms_render: Skip unsupported display configurations
X-Git-Tag: intel-gpu-tools-1.4~359
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d2048819b9db8781c65399f28696f375448a03e6;p=profile%2Fextras%2Fintel-gpu-tools.git

kms_render: Skip unsupported display configurations

Certain modes may not be supported by certain combinations of pipes.
This is impossible to determine upfront, and we await an atomic
modesetting query operation. In the meantime, if we fail to set a mode,
just skip that test.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=66000
---

diff --git a/tests/kms_render.c b/tests/kms_render.c
index 707ce27..3867a0b 100644
--- a/tests/kms_render.c
+++ b/tests/kms_render.c
@@ -124,9 +124,10 @@ static int test_format(const char *test_name,
 	if (!kmstest_create_fb2(drm_fd, width, height, format, false, &fb[1]))
 		goto err2;
 
-	do_or_die(drmModeSetCrtc(drm_fd, cconf->crtc->crtc_id, fb[0].fb_id,
+	if (drmModeSetCrtc(drm_fd, cconf->crtc->crtc_id, fb[0].fb_id,
 				 0, 0, &cconf->connector->connector_id, 1,
-				 mode));
+				 mode))
+		goto err2;
 	do_or_die(drmModePageFlip(drm_fd, cconf->crtc->crtc_id, fb[0].fb_id,
 				  0, NULL));
 	sleep(2);
@@ -152,8 +153,10 @@ static int test_format(const char *test_name,
 err2:
 	kmstest_remove_fb(drm_fd, &fb[0]);
 err1:
-	fprintf(stderr, "skip testing unsupported format %s\n",
-		kmstest_format_str(format));
+	printf("Test %s with %s on %s: SKIPPED\n",
+		test_name, mode_format_str, cconf_str);
+	free(mode_format_str);
+	free(cconf_str);
 
 	return -1;
 }