[PATCH i-g-t 1/2] lib/igt_fb: Add IGT_FORMAT_MOD_PREFERRED helper modifier

[Jeevan B <jeevan.b@intel.com>]

Add IGT_FORMAT_MOD_PREFERRED as an internal FB modifier sentinel so tests
can request preferred tiling without hardcoding driver rules.

Resolve it in igt_fb_resolve_modifier() (never pass it to kernel) by
checking support for the requested format in this order:
X_TILED -> 4_TILED -> LINEAR.

v2:
- Switch from gen/driver policy to capability-based selection
- Drop Y_TILED preference

Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 lib/igt_fb.c | 38 +++++++++++++++++++++++++++++++++++++-
 lib/igt_fb.h | 18 ++++++++++++++++++
 2 files changed, 55 insertions(+), 1 deletion(-)

diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index fa9220953..1b0e3f358 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -790,6 +790,40 @@ static int fb_num_planes(const struct igt_fb *fb)
 	return num_planes;
 }
 
+static uint64_t igt_fb_resolve_modifier(int fd, uint32_t format,
+					uint64_t modifier)
+{
+	static const uint64_t preferred_modifiers[] = {
+		I915_FORMAT_MOD_X_TILED,
+		I915_FORMAT_MOD_4_TILED,
+		DRM_FORMAT_MOD_LINEAR,
+	};
+	igt_display_t display;
+	uint64_t resolved = DRM_FORMAT_MOD_LINEAR;
+	int i;
+
+	if (modifier != IGT_FORMAT_MOD_PREFERRED)
+		return modifier;
+
+	igt_display_require(&display, fd);
+
+	for (i = 0; i < ARRAY_SIZE(preferred_modifiers); i++) {
+		if (preferred_modifiers[i] == DRM_FORMAT_MOD_LINEAR ||
+		    igt_display_has_format_mod(&display, format,
+					       preferred_modifiers[i])) {
+			resolved = preferred_modifiers[i];
+			break;
+		}
+	}
+
+	igt_display_fini(&display);
+
+	igt_debug("IGT_FORMAT_MOD_PREFERRED resolved to %s\n",
+		  igt_fb_modifier_name(resolved));
+
+	return resolved;
+}
+
 void igt_init_fb(struct igt_fb *fb, int fd, int width, int height,
 		 uint32_t drm_format, uint64_t modifier,
 		 enum igt_color_encoding color_encoding,
@@ -803,7 +837,7 @@ void igt_init_fb(struct igt_fb *fb, int fd, int width, int height,
 
 	fb->width = width;
 	fb->height = height;
-	fb->modifier = modifier;
+	fb->modifier = igt_fb_resolve_modifier(fd, drm_format, modifier);
 	fb->drm_format = drm_format;
 	fb->fd = fd;
 	fb->num_planes = fb_num_planes(fb);
@@ -5266,6 +5300,8 @@ void igt_format_array_fill(uint32_t **formats_array, unsigned int *count,
 const char *igt_fb_modifier_name(uint64_t modifier)
 {
 	switch (modifier) {
+	case IGT_FORMAT_MOD_PREFERRED:
+		return "preferred";
 	case DRM_FORMAT_MOD_LINEAR:
 		return "linear";
 	case I915_FORMAT_MOD_X_TILED:
diff --git a/lib/igt_fb.h b/lib/igt_fb.h
index 8e5907dab..cd0923270 100644
--- a/lib/igt_fb.h
+++ b/lib/igt_fb.h
@@ -50,6 +50,24 @@ typedef struct _igt_crc igt_crc_t;
  */
 #define IGT_FORMAT_FLOAT fourcc_code('I', 'G', 'F', 'x')
 
+/**
+ * IGT_FORMAT_MOD_PREFERRED:
+ *
+ * Modifier used by tests to request the best tiling format for the
+ * current device. The actual modifier is selected inside igt_init_fb()
+ * based on device capabilities, so tests don't need to handle driver or
+ * generation-specific details.
+ *
+ * Default selection rules (capability-based):
+ *  - If X_TILED is supported for the requested format, use X_TILED
+ *  - Else if 4_TILED is supported for the requested format, use 4_TILED
+ *  - Else use LINEAR
+ *
+ * Note: This is a special placeholder value and must never be sent to
+ * the kernel. Modifier selection is logged in IGT debug output.
+ */
+#define IGT_FORMAT_MOD_PREFERRED  UINT64_C(0xfffffffffffffffe)
+
 #define IGT_FORMAT_FMT "%c%c%c%c(0x%08x)"
 #define IGT_FORMAT_ARGS(f) ((f) >> 0) & 0xff, ((f) >> 8) & 0xff, \
 		((f) >> 16) & 0xff, ((f) >> 24) & 0xff, (f)
-- 
2.43.0