[igt-dev] [PATCH i-g-t v2 1/2] tests/kms_plane: Don't test all format+modifier combinations

[Ville Syrjala <ville.syrjala@linux.intel.com>]

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

When running the non-extended tests let's use igt_reduce_format()
and test each format "class" only once for each modifier except
linear. This follows the earlier pattern of doing all the YCbCr
limited vs. full range tests only for linear and skipping for
other modifiers.

This should maintian sufficient coverage to catch most watermark
related bugs and whatnot.

On icl:
$ time kms_plane --r pixel-format-pipe-A-planes
- real	0m9.390s
+ real	0m7.244s

Full run of all pixel-format* tests on the same machine
goes from ~58 seconds down to ~38 seconds.

v2: Test the format the plane reported, not the reduced format

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 tests/kms_plane.c | 73 +++++++++++++++++++++++++++++++++--------------
 1 file changed, 51 insertions(+), 22 deletions(-)

diff --git a/tests/kms_plane.c b/tests/kms_plane.c
index c87983a4e539..f20f5bee918b 100644
--- a/tests/kms_plane.c
+++ b/tests/kms_plane.c
@@ -25,6 +25,7 @@
  */
 
 #include "igt.h"
+#include "igt_vec.h"
 #include <errno.h>
 #include <stdbool.h>
 #include <stdio.h>
@@ -816,16 +817,21 @@ enum crc_set { PACKED_CRC_SET,
 	       PLANAR_CRC_SET,
 	       MAX_CRC_SET };
 
+struct format_mod {
+	uint64_t modifier;
+	uint32_t format;
+};
+
 static bool test_format_plane(data_t *data, enum pipe pipe,
 			      igt_output_t *output, igt_plane_t *plane, igt_fb_t *primary_fb)
 {
 	struct igt_fb fb = {};
 	struct igt_fb *clear_fb = plane->type == DRM_PLANE_TYPE_PRIMARY ? primary_fb : NULL;
 	drmModeModeInfo *mode;
-	uint32_t format, ref_format;
-	uint64_t modifier, ref_modifier;
 	uint64_t width, height;
 	igt_crc_t ref_crc[MAX_CRC_SET][ARRAY_SIZE(colors_extended)];
+	struct igt_vec tested_formats;
+	struct format_mod ref = {};
 	igt_crc_t* crcset;
 	bool result = true;
 
@@ -835,12 +841,14 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
 	if (data->crop != 0 && plane->type == DRM_PLANE_TYPE_CURSOR)
 		return true;
 
+	igt_vec_init(&tested_formats, sizeof(struct format_mod));
+
 	mode = igt_output_get_mode(output);
 	if (plane->type != DRM_PLANE_TYPE_CURSOR) {
 		width = mode->hdisplay;
 		height = mode->vdisplay;
-		ref_format = format = DRM_FORMAT_XRGB8888;
-		ref_modifier = modifier = DRM_FORMAT_MOD_NONE;
+		ref.format = DRM_FORMAT_XRGB8888;
+		ref.modifier = DRM_FORMAT_MOD_NONE;
 	} else {
 		if (!plane->drm_plane) {
 			igt_debug("Only legacy cursor ioctl supported, skipping cursor plane\n");
@@ -848,8 +856,8 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
 		}
 		do_or_die(drmGetCap(data->drm_fd, DRM_CAP_CURSOR_WIDTH, &width));
 		do_or_die(drmGetCap(data->drm_fd, DRM_CAP_CURSOR_HEIGHT, &height));
-		ref_format = format = DRM_FORMAT_ARGB8888;
-		ref_modifier = modifier = DRM_FORMAT_MOD_NONE;
+		ref.format = DRM_FORMAT_ARGB8888;
+		ref.modifier = DRM_FORMAT_MOD_NONE;
 	}
 
 	igt_debug("Testing connector %s on %s plane %s.%u\n",
@@ -859,15 +867,15 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
 	igt_pipe_crc_start(data->pipe_crc);
 
 	igt_info("Testing format " IGT_FORMAT_FMT " / modifier 0x%" PRIx64 " on %s.%u\n",
-		 IGT_FORMAT_ARGS(format), modifier,
+		 IGT_FORMAT_ARGS(ref.format), ref.modifier,
 		 kmstest_pipe_name(pipe), plane->index);
 
 	if (data->display.is_atomic) {
 		struct igt_fb test_fb;
 		int ret;
 
-		igt_create_fb(data->drm_fd, 64, 64, format,
-			      LOCAL_DRM_FORMAT_MOD_NONE, &test_fb);
+		igt_create_fb(data->drm_fd, 64, 64, ref.format,
+			      DRM_FORMAT_MOD_LINEAR, &test_fb);
 
 		igt_plane_set_fb(plane, &test_fb);
 
@@ -885,12 +893,12 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
 		igt_remove_fb(data->drm_fd, &test_fb);
 	}
 
-	capture_format_crcs_packed(data, pipe, plane, format, modifier,
+	capture_format_crcs_packed(data, pipe, plane, ref.format, ref.modifier,
 				   width, height, IGT_COLOR_YCBCR_BT709,
 				   IGT_COLOR_YCBCR_LIMITED_RANGE,
 				   ref_crc[PACKED_CRC_SET], &fb);
 
-	capture_format_crcs_planar(data, pipe, plane, format, modifier,
+	capture_format_crcs_planar(data, pipe, plane, ref.format, ref.modifier,
 				   width, height, IGT_COLOR_YCBCR_BT709,
 				   IGT_COLOR_YCBCR_LIMITED_RANGE,
 				   ref_crc[PLANAR_CRC_SET], &fb);
@@ -903,36 +911,55 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
 	igt_require(num_unique_crcs(ref_crc[PLANAR_CRC_SET], data->num_colors) > 1);
 
 	for (int i = 0; i < plane->format_mod_count; i++) {
-		format = plane->formats[i];
-		modifier = plane->modifiers[i];
+		struct format_mod f = {
+			.format = plane->formats[i],
+			.modifier = plane->modifiers[i],
+		};
 
-		if (format == ref_format &&
-		    modifier == ref_modifier)
+		if (f.format == ref.format &&
+		    f.modifier == ref.modifier)
 			continue;
 
-		if (format == DRM_FORMAT_C8) {
+		/* test each format "class" only once in non-extended tests */
+		if (!data->extended && f.modifier != DRM_FORMAT_MOD_LINEAR) {
+			struct format_mod rf = {
+				.format = igt_reduce_format(f.format),
+				.modifier = f.modifier,
+			};
+
+			if (igt_vec_index(&tested_formats, &rf) >= 0) {
+				igt_info("Skipping format " IGT_FORMAT_FMT " / modifier 0x%" PRIx64 " on %s.%u\n",
+					 IGT_FORMAT_ARGS(f.format), f.modifier,
+					 kmstest_pipe_name(pipe), plane->index);
+				continue;
+			}
+
+			igt_vec_push(&tested_formats, &rf);
+		}
+
+		if (f.format == DRM_FORMAT_C8) {
 			if (!set_c8_legacy_lut(data, pipe, LUT_MASK))
 				continue;
 		} else {
-			if (!igt_fb_supported_format(format))
+			if (!igt_fb_supported_format(f.format))
 				continue;
 		}
 
-		crcset = ref_crc[(igt_format_is_yuv_semiplanar(format)
+		crcset = ref_crc[(igt_format_is_yuv_semiplanar(f.format)
 				 ? PLANAR_CRC_SET : PACKED_CRC_SET)];
 
-		if (igt_format_is_yuv(format))
+		if (igt_format_is_yuv(f.format))
 			result &= test_format_plane_yuv(data, pipe, plane,
-							format, modifier,
+							f.format, f.modifier,
 							width, height,
 							crcset, &fb);
 		else
 			result &= test_format_plane_rgb(data, pipe, plane,
-							format, modifier,
+							f.format, f.modifier,
 							width, height,
 							crcset, &fb);
 
-		if (format == DRM_FORMAT_C8)
+		if (f.format == DRM_FORMAT_C8)
 			set_legacy_lut(data, pipe, LUT_MASK);
 	}
 
@@ -941,6 +968,8 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
 	igt_plane_set_fb(plane, clear_fb);
 	igt_remove_fb(data->drm_fd, &fb);
 
+	igt_vec_fini(&tested_formats);
+
 	return result;
 }
 
-- 
2.26.3

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