diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2018-11-22 20:08:00 +0200 |
---|---|---|
committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2019-03-11 22:03:45 +0200 |
commit | 50f9d1f76dc60e34788a86ba4aa1c4213bc7857a (patch) | |
tree | f001ab4e81fc8e7c9b321e5004e12f65936da495 /tests/kms_flip.c | |
parent | e347d24a3a741391d1d488b8efb4b2de32eb1632 (diff) |
tests/kms_flip: Validate the vbl sequence numbers
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>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'tests/kms_flip.c')
-rwxr-xr-x | tests/kms_flip.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/tests/kms_flip.c b/tests/kms_flip.c index b9729dfd..05ea3df0 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); |