[igt-dev] [PATCH i-g-t] tests/testdisplay: Speed up testdisplay for ci

Ville Syrjala ville.syrjala at linux.intel.com
Mon Jan 21 14:49:25 UTC 2019


From: Ville Syrjälä <ville.syrjala at linux.intel.com>

No point in sleeping 5 seconds between each modeset when running in ci.
So let's just make the default sleep interval 0 seconds and leave it up
to the user to bump it via -s when running interactively.

On a KBL + LSPCON + a 4k HDMI display with 36 modes listed:
time ./tests/testdisplay
- real	3m12,026s
- user	0m1,920s
- sys	0m1,241s
+ real	0m14,681s
+ user	0m3,135s
+ sys	0m1,340s

Cc: Martin Peres <martin.peres at linux.intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
---
 tests/testdisplay.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/testdisplay.c b/tests/testdisplay.c
index 9972fb3cea63..b36572642bee 100644
--- a/tests/testdisplay.c
+++ b/tests/testdisplay.c
@@ -81,7 +81,7 @@ int drm_fd, modes;
 int test_all_modes = 0, test_preferred_mode = 0, force_mode = 0, test_plane,
     test_stereo_modes;
 uint64_t tiling = LOCAL_DRM_FORMAT_MOD_NONE;
-int sleep_between_modes = 5;
+int sleep_between_modes = 0;
 int do_dpms = 0; /* This aliases to DPMS_ON */
 uint32_t depth = 24, stride, bpp;
 int qr_code = 0;
@@ -525,7 +525,7 @@ static void __attribute__((noreturn)) usage(char *name, char opt)
 	igt_info("usage: %s [-hiasdpmtf]\n", name);
 	igt_info("\t-i\tdump info\n");
 	igt_info("\t-a\ttest all modes\n");
-	igt_info("\t-s\t<duration>\tsleep between each mode test\n");
+	igt_info("\t-s\t<duration>\tsleep between each mode test (default: 0)\n");
 	igt_info("\t-d\t<depth>\tbit depth of scanout buffer\n");
 	igt_info("\t-p\t<planew,h>,<crtcx,y>,<crtcw,h> test overlay plane\n");
 	igt_info("\t-m\ttest the preferred mode\n");
-- 
2.19.2



More information about the igt-dev mailing list