diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2016-11-10 08:03:30 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2016-11-10 21:16:55 +0000 |
| commit | b37aad82b2d662770ad55ab594b74048509b0692 (patch) | |
| tree | 04452d1c4c2384cf0e2583805392cb42e28e87ea | |
| parent | 050c00d53f391aa41931229ffe230e4872a5cb4f (diff) | |
igt/kms_frontbuffer_tracking: Drop unsynchronized pageflip test
A raw pageflip is nonblocking and asynchronous, but
kms_frontbuffer_tracking persumed that it was synchronous and completed
before the funtion returns. It doesn't, so the CRC could be sampled
before the flip completed.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
| -rw-r--r-- | tests/kms_frontbuffer_tracking.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c index bc42701a..343b1242 100644 --- a/tests/kms_frontbuffer_tracking.c +++ b/tests/kms_frontbuffer_tracking.c @@ -122,7 +122,6 @@ struct test_mode { * and they exercise slightly different codepaths inside the Kernel. */ enum flip_type { FLIP_PAGEFLIP, - FLIP_PAGEFLIP_EVENT, FLIP_MODESET, FLIP_PLANES, FLIP_COUNT, @@ -2322,11 +2321,6 @@ static void page_flip_for_params(struct modeset_params *params, switch (type) { case FLIP_PAGEFLIP: rc = drmModePageFlip(drm.fd, params->crtc_id, - params->fb.fb->fb_id, 0, NULL); - igt_assert_eq(rc, 0); - break; - case FLIP_PAGEFLIP_EVENT: - rc = drmModePageFlip(drm.fd, params->crtc_id, params->fb.fb->fb_id, DRM_MODE_PAGE_FLIP_EVENT, NULL); igt_assert_eq(rc, 0); @@ -3366,8 +3360,6 @@ static const char *flip_str(enum flip_type flip) switch (flip) { case FLIP_PAGEFLIP: return "pg"; - case FLIP_PAGEFLIP_EVENT: - return "ev"; case FLIP_MODESET: return "ms"; case FLIP_PLANES: |