[igt-dev] [PATCH v2] lib/igt_fb: Add support for the gen12 media compressed modifier

Imre Deak imre.deak at intel.com
Tue Dec 17 00:00:33 UTC 2019


Media compressed framebuffers don't have a CCS CC plane. Add helpers to
select the different types of CCS planes and make sure we setup the
planes correctly for MC framebuffers too.

Note that the order of MC framebuffer planes this change assumes is

plane 0: Y plane
plane 1: UV plane
plane 2: CCS plane for plane 0
plane 3: CCS plane for plane 1

unlike the order defined in the latest decompression kernel patchset.
The above order is the logical one that allows us to keep the existing
way of handling the Y/UV planes.

v2:
- Fix is_ccs_plane() adding the missing is_ccs_modifier() check.

Cc: Mika Kahola <mika.kahola at intel.com>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
Signed-off-by: Imre Deak <imre.deak at intel.com>
---
 lib/igt_fb.c | 92 ++++++++++++++++++++++++++++++++++++----------------
 1 file changed, 64 insertions(+), 28 deletions(-)

diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index 3b141b93..1dccdc66 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -487,18 +487,50 @@ static bool is_ccs_modifier(uint64_t modifier)
 		modifier == I915_FORMAT_MOD_Yf_TILED_CCS;
 }
 
+static bool is_ccs_plane(const struct igt_fb *fb, int plane)
+{
+	if (is_gen12_mc_ccs_modifier(fb->modifier))
+		return plane >= fb->num_planes / 2;
+
+	return is_ccs_modifier(fb->modifier) && plane > 0;
+}
+
+static bool is_gen12_ccs_plane(const struct igt_fb *fb, int plane)
+{
+	return is_gen12_ccs_modifier(fb->modifier) && is_ccs_plane(fb, plane);
+}
+
+static bool is_gen12_ccs_cc_plane(const struct igt_fb *fb, int plane)
+{
+	return fb->modifier == I915_FORMAT_MOD_Y_TILED_GEN12_RC_CCS_CC &&
+	       plane == 2;
+}
+
+static int ccs_to_main_plane(const struct igt_fb *fb, int plane)
+{
+	if (is_gen12_ccs_cc_plane(fb, plane))
+		return 0;
+
+	return plane - fb->num_planes / 2;
+}
+
 static unsigned fb_plane_width(const struct igt_fb *fb, int plane)
 {
 	const struct format_desc_struct *format = lookup_drm_format(fb->drm_format);
 
-	if (is_ccs_modifier(fb->modifier) && plane == 1) {
-		if (is_gen12_ccs_modifier(fb->modifier))
-			return DIV_ROUND_UP(fb->width,
-					    512 / (fb->plane_bpp[0] / 8)) * 64;
-		else
-			return DIV_ROUND_UP(fb->width, 1024) * 128;
-	} else if (is_gen12_ccs_modifier(fb->modifier) && plane == 2) {
+	if (is_gen12_ccs_cc_plane(fb, plane)) {
 		return 64;
+	} if (is_gen12_ccs_plane(fb, plane)) {
+		int main_plane = ccs_to_main_plane(fb, plane);
+		int width = fb->width;
+
+		if (main_plane)
+			width = DIV_ROUND_UP(width, format->hsub);
+
+		return DIV_ROUND_UP(width,
+				    512 / (fb->plane_bpp[main_plane] / 8)) * 64;
+	} else if (is_ccs_plane(fb, plane)) {
+		 return DIV_ROUND_UP(fb->width, 1024) * 128;
 	}
 
 	if (plane == 0)
@@ -511,7 +543,7 @@ static unsigned fb_plane_bpp(const struct igt_fb *fb, int plane)
 {
 	const struct format_desc_struct *format = lookup_drm_format(fb->drm_format);
 
-	if (is_ccs_modifier(fb->modifier) && (plane == 1 || plane == 2))
+	if (is_ccs_plane(fb, plane))
 		return 8;
 	else
 		return format->plane_bpp[plane];
@@ -521,13 +553,18 @@ static unsigned fb_plane_height(const struct igt_fb *fb, int plane)
 {
 	const struct format_desc_struct *format = lookup_drm_format(fb->drm_format);
 
-	if (is_ccs_modifier(fb->modifier) && plane == 1) {
-		if (is_gen12_ccs_modifier(fb->modifier))
-			return DIV_ROUND_UP(fb->height, 128) * 4;
-		else
-			return DIV_ROUND_UP(fb->height, 512) * 32;
-	} else if (is_gen12_ccs_modifier(fb->modifier) && plane == 2)
+	if (is_gen12_ccs_cc_plane(fb, plane)) {
 		return 1;
+	} else if (is_gen12_ccs_plane(fb, plane)) {
+		int height = fb->height;
+
+		if (ccs_to_main_plane(fb, plane))
+			height = DIV_ROUND_UP(height, format->vsub);
+
+		return DIV_ROUND_UP(height, 128) * 4;
+	} else if (is_ccs_plane(fb, plane)) {
+		return DIV_ROUND_UP(fb->height, 512) * 32;
+	}
 
 	if (plane == 0)
 		return fb->height;
@@ -537,16 +574,15 @@ static unsigned fb_plane_height(const struct igt_fb *fb, int plane)
 
 static int fb_num_planes(const struct igt_fb *fb)
 {
-	const struct format_desc_struct *format = lookup_drm_format(fb->drm_format);
+	int num_planes = lookup_drm_format(fb->drm_format)->num_planes;
 
-	if (is_ccs_modifier(fb->modifier)) {
-		if (fb->modifier == I915_FORMAT_MOD_Y_TILED_GEN12_RC_CCS_CC)
-			return 3;
-		else
-			return 2;
-	} else {
-		return format->num_planes;
-	}
+	if (is_ccs_modifier(fb->modifier))
+		num_planes *= 2;
+
+	if (fb->modifier == I915_FORMAT_MOD_Y_TILED_GEN12_RC_CCS_CC)
+		num_planes++;
+
+	return num_planes;
 }
 
 void igt_init_fb(struct igt_fb *fb, int fd, int width, int height,
@@ -604,12 +640,12 @@ static uint32_t calc_plane_stride(struct igt_fb *fb, int plane)
 		 * so the easiest way is to align the luma stride to 256.
 		 */
 		return ALIGN(min_stride, 256);
-	} else if (is_gen12_ccs_modifier(fb->modifier) && plane == 1) {
-		/* A main surface using a CCS AUX surface must be 4x4 tiles aligned. */
-		return ALIGN(min_stride, 64);
-	} else if (is_gen12_ccs_modifier(fb->modifier) && plane == 2) {
+	} else if (is_gen12_ccs_cc_plane(fb, plane)) {
 		/* clear color always fixed to 64 bytes */
 		return 64;
+	} else if (is_gen12_ccs_plane(fb, plane)) {
+		/* A main surface using a CCS AUX surface must be 4x4 tiles aligned. */
+		return ALIGN(min_stride, 64);
 	} else {
 		unsigned int tile_width, tile_height;
 
@@ -644,7 +680,7 @@ static uint64_t calc_plane_size(struct igt_fb *fb, int plane)
 		size = roundup_power_of_two(size);
 
 		return size;
-	} else if (is_gen12_ccs_modifier(fb->modifier) && (plane == 1 || plane == 2)) {
+	} else if (is_gen12_ccs_plane(fb, plane)) {
 		/* The AUX CCS surface must be page aligned */
 		return (uint64_t)fb->strides[plane] *
 			ALIGN(fb->plane_height[plane], 64);
-- 
2.17.1



More information about the igt-dev mailing list