igt-dev.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] tests/kms_force_connector_basic: Verify planes are restored after load detection
@ 2018-03-19 16:55 Maarten Lankhorst
  2018-03-19 17:02 ` [igt-dev] [PATCH i-g-t] tests/kms_force_connector_basic: Verify planes are restored after load detection, v2 Maarten Lankhorst
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Maarten Lankhorst @ 2018-03-19 16:55 UTC (permalink / raw)
  To: igt-dev

There's a bug in our load detection in which we don't correctly
restore planes to their previous states. Strictly verify this
is the case by setting a fb on all planes.

Cc: Ville Syrjala <ville.syrjala@linux.intel.com>
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 tests/kms_force_connector_basic.c | 63 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)

diff --git a/tests/kms_force_connector_basic.c b/tests/kms_force_connector_basic.c
index d11a775a089d..3ea6763ed311 100644
--- a/tests/kms_force_connector_basic.c
+++ b/tests/kms_force_connector_basic.c
@@ -122,6 +122,14 @@ int main(int argc, char **argv)
 	}
 
 	igt_subtest("force-load-detect") {
+		int i, j, w = 64, h = 64;
+		drmModePlaneRes *plane_resources;
+		struct igt_fb xrgb_fb, argb_fb;
+
+		igt_create_fb(drm_fd, w, h, DRM_FORMAT_XRGB8888, 0, &xrgb_fb);
+		igt_create_fb(drm_fd, w, h, DRM_FORMAT_ARGB8888, 0, &argb_fb);
+		igt_assert(drmSetClientCap(drm_fd, DRM_CLIENT_CAP_UNIVERSAL_PLANES, 1) == 0);
+
 		/*
 		 * disable all outputs to make sure we have a
 		 * free crtc available for load detect
@@ -131,6 +139,42 @@ int main(int argc, char **argv)
 
 		igt_set_module_param_int("load_detect_test", 1);
 
+		plane_resources = drmModeGetPlaneResources(drm_fd);
+		igt_assert(plane_resources);
+
+		for (i = 0; i < plane_resources->count_planes; i++) {
+			drmModePlane *drm_plane;
+			bool found = false;
+			uint32_t plane_id = plane_resources->planes[i];
+
+			drm_plane = drmModeGetPlane(drm_fd, plane_id);
+			igt_assert(drm_plane);
+
+			for (j = 0; j < drm_plane->count_formats; j++) {
+				uint32_t format = drm_plane->formats[j];
+				uint32_t crtc = ffs(drm_plane->possible_crtcs) - 1;
+				uint32_t crtc_id = res->crtcs[crtc];
+
+				if (format == DRM_FORMAT_XRGB8888)
+					do_or_die(drmModeSetPlane(drm_fd, plane_id, crtc_id,
+							xrgb_fb.fb_id,
+							0, 0, 0, w, h,
+							0, 0, IGT_FIXED(w, 0), IGT_FIXED(h, 0)));
+				else if (format == DRM_FORMAT_ARGB8888)
+					do_or_die(drmModeSetPlane(drm_fd, plane_id, crtc_id,
+							argb_fb.fb_id,
+							0, 0, 0, w, h,
+							0, 0, IGT_FIXED(w, 0), IGT_FIXED(h, 0)));
+				else
+					continue;
+
+				found = true;
+				break;
+			}
+			drmModeFreePlane(drm_plane);
+			igt_assert(found);
+		}
+
 		/* This can't use drmModeGetConnectorCurrent
 		 * because connector probing is the point of this test.
 		 */
@@ -141,6 +185,25 @@ int main(int argc, char **argv)
 		igt_assert(temp->connection != DRM_MODE_UNKNOWNCONNECTION);
 
 		drmModeFreeConnector(temp);
+
+		/* Look if planes are unmodified. */
+		for (i = 0; i < plane_resources->count_planes; i++) {
+			drmModePlane *drm_plane;
+			bool found = false;
+
+			drm_plane = drmModeGetPlane(drm_fd,
+						    plane_resources->planes[i]);
+			igt_assert(drm_plane);
+
+			igt_assert(drm_plane->crtc_id);
+			igt_assert(drm_plane->fb_id);
+
+			if (drm_plane->fb_id != xrgb_fb.fb_id)
+				igt_assert_eq(drm_plane->fb_id, argb_fb.fb_id);
+
+			drmModeFreePlane(drm_plane);
+			igt_assert(found);
+		}
 	}
 
 	igt_subtest("force-connector-state") {
-- 
2.16.2

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2018-08-10 10:16 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-03-19 16:55 [igt-dev] [PATCH i-g-t] tests/kms_force_connector_basic: Verify planes are restored after load detection Maarten Lankhorst
2018-03-19 17:02 ` [igt-dev] [PATCH i-g-t] tests/kms_force_connector_basic: Verify planes are restored after load detection, v2 Maarten Lankhorst
2018-03-21 14:22   ` Ville Syrjälä
2018-08-10 10:16     ` Maarten Lankhorst
2018-03-19 21:28 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_force_connector_basic: Verify planes are restored after load detection (rev2) Patchwork
2018-07-25 12:22 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2018-07-25 13:50 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).