diff options
author | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-10-19 13:01:03 +0200 |
---|---|---|
committer | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-11-07 15:44:08 +0100 |
commit | a3ede1b535ac8137f6949c468edd7054453d5dae (patch) | |
tree | 4414f7817f9c24f93ee5e0a7cd3f560333e9a675 /tests/kms_frontbuffer_tracking.c | |
parent | cc41f4c921e56c62c85ec5349c47022ae9b5f008 (diff) |
tests/kms_frontbuffer_tracking: Really do not skip on 4k displays, v2.
The previous fixes to kms_frontbuffer_tracking fixed the sharedfb tests,
but the stridechange test still creates its own fb. Use the existing 'big'
fb to force a stride change, without ending up with an excessive stride.
Changes since v1:
- Use the big fb used by the shrfb tests. It already exists and has
the right dimensions, so why not?
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Diffstat (limited to 'tests/kms_frontbuffer_tracking.c')
-rw-r--r-- | tests/kms_frontbuffer_tracking.c | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c index bf4a68ff..276ef83c 100644 --- a/tests/kms_frontbuffer_tracking.c +++ b/tests/kms_frontbuffer_tracking.c @@ -2915,7 +2915,7 @@ static void badstride_subtest(const struct test_mode *t) */ static void stridechange_subtest(const struct test_mode *t) { - struct igt_fb new_fb, *old_fb; + struct igt_fb *new_fb, *old_fb; struct modeset_params *params = pick_params(t); int rc; @@ -2923,15 +2923,14 @@ static void stridechange_subtest(const struct test_mode *t) old_fb = params->primary.fb; - create_fb(t->format, params->primary.fb->width + 512, params->primary.fb->height, - opt.tiling, t->plane, &new_fb); - fill_fb(&new_fb, COLOR_PRIM_BG); - - igt_assert(old_fb->strides[0] != new_fb.strides[0]); - /* We can't assert that FBC will be enabled since there may not be * enough space for the CFB, but we can check the CRC. */ - params->primary.fb = &new_fb; + new_fb = &fbs[t->format].big; + igt_assert(old_fb->strides[0] != new_fb->strides[0]); + + params->primary.fb = new_fb; + fill_fb_region(¶ms->primary, COLOR_PRIM_BG); + set_mode_for_params(params); do_assertions(DONT_ASSERT_FEATURE_STATUS); @@ -2941,7 +2940,7 @@ static void stridechange_subtest(const struct test_mode *t) do_assertions(0); /* This operation is the same as above, but with the planes API. */ - params->primary.fb = &new_fb; + params->primary.fb = new_fb; set_prim_plane_for_params(params); do_assertions(DONT_ASSERT_FEATURE_STATUS); @@ -2953,11 +2952,9 @@ static void stridechange_subtest(const struct test_mode *t) * Try to set a new stride. with the page flip api. This is allowed * with the atomic page flip helper, but not with the legacy page flip. */ - rc = drmModePageFlip(drm.fd, drm.display.pipes[params->pipe].crtc_id, new_fb.fb_id, 0, NULL); + rc = drmModePageFlip(drm.fd, drm.display.pipes[params->pipe].crtc_id, new_fb->fb_id, 0, NULL); igt_assert(rc == -EINVAL || rc == 0); do_assertions(0); - - igt_remove_fb(drm.fd, &new_fb); } /** |