[igt-dev] [PATCH i-g-t 4/5] tests/kms_flip: Validate the vbl sequence numbers

[Ville Syrjala <ville.syrjala@linux.intel.com>]

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Make sure we haven't already passed the seq numbers we're
requesting when doing the ts calibration.

Suggested-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 tests/kms_flip.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index cde8b739ea83..997957542329 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1118,8 +1118,17 @@ static void calibrate_ts(struct test_output *o, int crtc_idx)
 	wait.request.type |= DRM_VBLANK_ABSOLUTE | DRM_VBLANK_EVENT;
 	wait.request.sequence = last_seq;
 	for (n = 0; n < CALIBRATE_TS_STEPS; n++) {
+		drmVBlank check = {};
+
 		++wait.request.sequence;
 		do_or_die(drmWaitVBlank(drm_fd, &wait));
+
+		/* Double check that haven't already missed the vblank */
+		check.request.type = kmstest_get_vbl_flag(crtc_idx);
+		check.request.type |= DRM_VBLANK_RELATIVE;
+		do_or_die(drmWaitVBlank(drm_fd, &check));
+
+		igt_assert(!igt_vblank_after(check.reply.sequence, wait.request.sequence));
 	}
 
 	igt_stats_init_with_size(&stats, CALIBRATE_TS_STEPS);
-- 
2.19.2

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