public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] tests/kms_plane_lowres: Skip if no suitable mode is possible
@ 2019-08-20 14:19 Matt Roper
  2019-08-20 17:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (4 more replies)
  0 siblings, 5 replies; 9+ messages in thread
From: Matt Roper @ 2019-08-20 14:19 UTC (permalink / raw)
  To: igt-dev

When choosing a low resolution mode to switch to, we search the
connector's mode list for a mode that's small enough that a plane
positioned in the bottom left corner of the screen in the original mode
will be fully offscreen in the new, smaller mode.  However if no
suitable modes are found in the modelist, we blindly fall back to a 10x7
mode without actually checking that the canary plane will now be
offscreen.  If, for example, the original mode was 13x7, then switching
to 10x7 will leave the plane visible onscreen, causing the test to fail.

Before falling back to a 10x7 mode, check whether that's actually small
enough.  If it isn't just skip this subtest.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105581
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
---
 tests/kms_plane_lowres.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/kms_plane_lowres.c b/tests/kms_plane_lowres.c
index 68b85025..3f20ed32 100644
--- a/tests/kms_plane_lowres.c
+++ b/tests/kms_plane_lowres.c
@@ -59,8 +59,10 @@ get_lowres_mode(int drmfd, igt_output_t *output, drmModeModeInfo *mode_default)
 		}
 	}
 
-	if (!found)
+	if (!found) {
+		igt_skip_on(limit <= 768);
 		return *igt_std_1024_mode_get();
+	}
 
 	return mode;
 }
-- 
2.20.1

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2019-08-30 15:58 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-08-20 14:19 [igt-dev] [PATCH i-g-t] tests/kms_plane_lowres: Skip if no suitable mode is possible Matt Roper
2019-08-20 17:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2019-08-21  3:35 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-08-27  7:46 ` [igt-dev] [PATCH i-g-t] " Ser, Simon
2019-08-29 20:52   ` [igt-dev] [PATCH i-g-t v2] " Matt Roper
2019-08-30  7:29     ` Ser, Simon
2019-08-30 15:58       ` Matt Roper
2019-08-29 21:19 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_plane_lowres: Skip if no suitable mode is possible (rev2) Patchwork
2019-08-30 14:59 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox