diff options
author | Imre Deak <imre.deak@intel.com> | 2020-02-06 21:05:41 +0200 |
---|---|---|
committer | Imre Deak <imre.deak@intel.com> | 2020-02-13 10:20:45 +0200 |
commit | c4e70ed4703e628198e6e82684a3fb9e53f4dca1 (patch) | |
tree | 77b113966b998d4f2d0254c260ec7b19ba195e9e /tests/kms_frontbuffer_tracking.c | |
parent | 76c068120a8db3dbe9a1e0e2bd535a2c4faa8359 (diff) |
tests/kms_frontbuffer_tracking: Skip GTT subtests on platforms w/o aperture
Subtests that draw through a GTT mapping are not relevent on a platform w/o
GTT aperture, so skip them.
basic_subtest() so far expected that each method drew a new rectangle to
the FB (see the pattern::frames_stack flag), so fix up things allowing
the skip now.
v2:
- Remove the check for aperture where it's not relevant, or where it
would skip the test incorrectly (badformat_subtest()).
v3:
- Don't skip the whole subtest where multiple draw methods are used.
(Ashutosh).
Cc: Ashutosh Dixit <ashutosh.dixit@intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com> (v2)
Acked-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Diffstat (limited to 'tests/kms_frontbuffer_tracking.c')
-rw-r--r-- | tests/kms_frontbuffer_tracking.c | 26 |
1 files changed, 25 insertions, 1 deletions
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c index 724f5d16..c4b4af43 100644 --- a/tests/kms_frontbuffer_tracking.c +++ b/tests/kms_frontbuffer_tracking.c @@ -1934,6 +1934,9 @@ static void draw_subtest(const struct test_mode *t) struct modeset_params *params = pick_params(t); struct fb_region *target; + igt_skip_on(t->method == IGT_DRAW_MMAP_GTT && + !gem_has_mappable_ggtt(drm.fd)); + switch (t->screen) { case SCREEN_PRIM: if (t->method != IGT_DRAW_MMAP_GTT && t->plane == PLANE_PRI) @@ -2032,6 +2035,12 @@ static void multidraw_subtest(const struct test_mode *t) igt_debug("Methods %s and %s\n", igt_draw_get_method_name(m1), igt_draw_get_method_name(m2)); + + if ((m1 == IGT_DRAW_MMAP_GTT || + m2 == IGT_DRAW_MMAP_GTT) && + !gem_has_mappable_ggtt(drm.fd)) + continue; + for (r = 0; r < pattern->n_rects; r++) { used_method = (r % 2 == 0) ? m1 : m2; @@ -2276,6 +2285,9 @@ static void flip_subtest(const struct test_mode *t) struct draw_pattern_info *pattern = &pattern1; enum color bg_color; + igt_skip_on(t->method == IGT_DRAW_MMAP_GTT && + !gem_has_mappable_ggtt(drm.fd)); + switch (t->screen) { case SCREEN_PRIM: assertions |= ASSERT_LAST_ACTION_CHANGED; @@ -2335,6 +2347,9 @@ static void fliptrack_subtest(const struct test_mode *t, enum flip_type type) struct modeset_params *params = pick_params(t); struct draw_pattern_info *pattern = &pattern1; + igt_skip_on(t->method == IGT_DRAW_MMAP_GTT && + !gem_has_mappable_ggtt(drm.fd)); + prepare_subtest(t, pattern); create_fb(t->format, params->primary.fb->width, params->primary.fb->height, @@ -2744,6 +2759,9 @@ static void farfromfence_subtest(const struct test_mode *t) int max_height, assertions = 0; int gen = intel_gen(intel_get_drm_devid(drm.fd)); + igt_skip_on(t->method == IGT_DRAW_MMAP_GTT && + !gem_has_mappable_ggtt(drm.fd)); + switch (gen) { case 2: max_height = 2048; @@ -3024,7 +3042,11 @@ static void basic_subtest(const struct test_mode *t) opt.tiling, t->plane, &fb2); fb1 = params->primary.fb; - for (r = 0, method = 0; method < IGT_DRAW_METHOD_COUNT; method++, r++) { + for (r = 0, method = 0; method < IGT_DRAW_METHOD_COUNT; method++) { + if (method == IGT_DRAW_MMAP_GTT && + !gem_has_mappable_ggtt(drm.fd)) + continue; + if (r == pattern->n_rects) { params->primary.fb = (params->primary.fb == fb1) ? &fb2 : fb1; @@ -3040,6 +3062,8 @@ static void basic_subtest(const struct test_mode *t) draw_rect(pattern, ¶ms->primary, method, r); update_wanted_crc(t, &pattern->crcs[t->format][r]); do_assertions(assertions); + + r++; } igt_remove_fb(drm.fd, &fb2); |