[PATCH v3 00/13] drm: Color pipeline teardown and follow-up fixes/improvements

[PATCH v3 00/13] drm: Color pipeline teardown and follow-up fixes/improvements

[Chaitanya Kumar Borah]

This series contains follow-up fixes and improvements for the DRM color
pipeline infrastructure that was introduced in v6.19.[1][2]

The central handling of clean up of colorop from the mode_config list
is missing. While vkms calls drm_colorop_pipeline_destroy() in vkms_destroy(),
amd driver calls it only during failure of the init path and i915/xe driver
does not call it at all. This means amd and intel leaks these objects on
driver removal.

This series adds the teardown of mode_config.colorop_list in drm_mode_config_cleanup().
Since, i915/xe sub-classes the drm_colorop within intel_colorop it was not enough
to just use drm_colorop_pipeline_destroy(). Therefore, this series

- Introduces driver-managed destruction for drm_colorop objects and
  updates core helpers to use driver-provided destroy callbacks.
- Ensures all colorop objects are correctly torn down during
  mode_config cleanup and driver removal.

In addition to that following changes are made in the series
- Fixes enum name lifetime leaks in color pipeline init in i915, amdgpu_dm, and vkms
- Corrects the ordering of the 3D LUT block in the i915 plane color pipeline
- Refactors i915 plane color pipeline initialization to reliably clean
  up partially constructed pipelines on failure.

Thanks for taking a look. Feedback is welcome.

[1] https://lore.kernel.org/dri-devel/cbe00ac4-a535-47d3-813a-e2eda7e9b991@amd.com/
[2] https://lore.kernel.org/intel-gfx/20251203085211.3663374-1-uma.shankar@intel.com/

v2:
 - Re-arrange patches (Alex)
 - Re-factor code to avoid repitition in pipeline creation (Suraj)

v3:
 - Add documentation only to function definition (Jani)
 - s/nvl/xe3plpd (Suraj)

Chaitanya Kumar Borah (13):
  drm/i915/color: Place 3D LUT after CSC in plane color pipeline
  drm/amd/display: Fix color pipeline enum name leak
  drm/vkms: Fix color pipeline enum name leak
  drm/i915/display: Fix color pipeline enum name leak
  drm/colorop: Add destroy helper for colorop objects
  drm: Allow driver-managed destruction of colorop objects
  drm/amd/display: Hook up colorop destroy helper for plane pipelines
  drm/vkms: Hook up colorop destroy helper for plane pipelines
  drm/i915/display: Hook up intel_colorop_destroy
  drm: Clean up colorop objects during mode_config cleanup
  drm/vkms: Remove drm_colorop_pipeline_destroy() from vkms_destroy()
  drm/colorop: Use destroy callback for color pipeline teardown
  drm/i915/color: Add failure handling in plane color pipeline init

 .../amd/display/amdgpu_dm/amdgpu_dm_colorop.c |  31 ++-
 .../amd/display/amdgpu_dm/amdgpu_dm_plane.c   |  13 +-
 drivers/gpu/drm/drm_colorop.c                 |  49 +++--
 drivers/gpu/drm/drm_mode_config.c             |   6 +
 .../drm/i915/display/intel_color_pipeline.c   | 179 +++++++++++++-----
 drivers/gpu/drm/i915/display/intel_colorop.c  |   6 +
 drivers/gpu/drm/i915/display/intel_colorop.h  |   1 +
 drivers/gpu/drm/vkms/vkms_colorop.c           |  31 +--
 drivers/gpu/drm/vkms/vkms_drv.c               |   1 -
 include/drm/drm_colorop.h                     |  32 +++-
 10 files changed, 259 insertions(+), 90 deletions(-)

-- 
2.25.1

[PATCH v3 01/13] drm/i915/color: Place 3D LUT after CSC in plane color pipeline

[Chaitanya Kumar Borah]

Move the 3D LUT block to its correct position in the plane
color pipeline:

  [Pre-CSC] -> [CSC] -> [3DLUT] -> [Post-CSC]

Fixes: 65db7a1f9cf7 ("drm/i915/color: Add 3D LUT to color pipeline")
Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
---
 .../drm/i915/display/intel_color_pipeline.c    | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
index 942d9b9c93ce..684641c8323b 100644
--- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
+++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
@@ -39,6 +39,15 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	/* TODO: handle failures and clean up */
 	prev_op = &colorop->base;
 
+	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
+	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
+					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
+	if (ret)
+		return ret;
+
+	drm_colorop_set_next_property(prev_op, &colorop->base);
+	prev_op = &colorop->base;
+
 	if (DISPLAY_VER(display) >= 35 &&
 	    intel_color_crtc_has_3dlut(display, pipe) &&
 	    plane->type == DRM_PLANE_TYPE_PRIMARY) {
@@ -55,15 +64,6 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 		prev_op = &colorop->base;
 	}
 
-	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
-	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
-					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
-	if (ret)
-		return ret;
-
-	drm_colorop_set_next_property(prev_op, &colorop->base);
-	prev_op = &colorop->base;
-
 	colorop = intel_colorop_create(INTEL_PLANE_CB_POST_CSC_LUT);
 	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane,
 						  PLANE_GAMMA_SIZE,
-- 
2.25.1

[PATCH v3 02/13] drm/amd/display: Fix color pipeline enum name leak

[Chaitanya Kumar Borah]

dm_plane_init_colorops() allocates enum names for color pipelines.
These are eventually passed to drm_property_create_enum() which create
its own copies of the string. Free the strings after initialization
is done.

Also, allocate color pipeline enum names only after successfully creating
color pipeline.

Fixes: 9ba25915efba ("drm/amd/display: Add support for sRGB EOTF in DEGAM block")
Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
---
 .../drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c   |  4 +++-
 .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c | 13 +++++++++----
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
index d585618b8064..a2de3bba8346 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
@@ -79,7 +79,6 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 
 	list->type = ops[i]->base.id;
-	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", ops[i]->base.id);
 
 	i++;
 
@@ -197,6 +196,9 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 
 	drm_colorop_set_next_property(ops[i-1], ops[i]);
+
+	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", ops[0]->base.id);
+
 	return 0;
 
 cleanup:
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
index 2e3ee78999d9..7c4496fb4b9d 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
@@ -1790,12 +1790,13 @@ dm_atomic_plane_get_property(struct drm_plane *plane,
 static int
 dm_plane_init_colorops(struct drm_plane *plane)
 {
-	struct drm_prop_enum_list pipelines[MAX_COLOR_PIPELINES];
+	struct drm_prop_enum_list pipelines[MAX_COLOR_PIPELINES] = {};
 	struct drm_device *dev = plane->dev;
 	struct amdgpu_device *adev = drm_to_adev(dev);
 	struct dc *dc = adev->dm.dc;
 	int len = 0;
-	int ret;
+	int ret = 0;
+	int i;
 
 	if (plane->type == DRM_PLANE_TYPE_CURSOR)
 		return 0;
@@ -1806,7 +1807,7 @@ dm_plane_init_colorops(struct drm_plane *plane)
 		if (ret) {
 			drm_err(plane->dev, "Failed to create color pipeline for plane %d: %d\n",
 				plane->base.id, ret);
-			return ret;
+			goto out;
 		}
 		len++;
 
@@ -1814,7 +1815,11 @@ dm_plane_init_colorops(struct drm_plane *plane)
 		drm_plane_create_color_pipeline_property(plane, pipelines, len);
 	}
 
-	return 0;
+out:
+	for (i = 0; i < len; i++)
+		kfree(pipelines[i].name);
+
+	return ret;
 }
 #endif
 
-- 
2.25.1

[PATCH v3 03/13] drm/vkms: Fix color pipeline enum name leak

[Chaitanya Kumar Borah]

vkms_initialize_colorops() allocates enum names for color pipelines,
which are copied by drm_property_create_enum(). The temporary strings
were not freed, resulting in a memory leak.

Allocate enum names only after successful pipeline construction and
free them on all exit paths

Fixes: c1e578bd08da ("drm/vkms: Add enumerated 1D curve colorop")
Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
Reviewed-by: Louis Chauvet <louis.chauvet@bootlin.com>
---
 drivers/gpu/drm/vkms/vkms_colorop.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/vkms/vkms_colorop.c b/drivers/gpu/drm/vkms/vkms_colorop.c
index 5c3ffc78aea0..d03a1f2e9c41 100644
--- a/drivers/gpu/drm/vkms/vkms_colorop.c
+++ b/drivers/gpu/drm/vkms/vkms_colorop.c
@@ -37,7 +37,6 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 
 	list->type = ops[i]->base.id;
-	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", ops[i]->base.id);
 
 	i++;
 
@@ -88,6 +87,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 
 	drm_colorop_set_next_property(ops[i - 1], ops[i]);
 
+	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", ops[0]->base.id);
+
 	return 0;
 
 cleanup:
@@ -103,18 +104,18 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 
 int vkms_initialize_colorops(struct drm_plane *plane)
 {
-	struct drm_prop_enum_list pipeline;
-	int ret;
+	struct drm_prop_enum_list pipeline = {};
+	int ret = 0;
 
 	/* Add color pipeline */
 	ret = vkms_initialize_color_pipeline(plane, &pipeline);
 	if (ret)
-		return ret;
+		goto out;
 
 	/* Create COLOR_PIPELINE property and attach */
 	ret = drm_plane_create_color_pipeline_property(plane, &pipeline, 1);
-	if (ret)
-		return ret;
 
-	return 0;
+	kfree(pipeline.name);
+out:
+	return ret;
 }
-- 
2.25.1

[PATCH v3 04/13] drm/i915/display: Fix color pipeline enum name leak

[Chaitanya Kumar Borah]

intel_color_pipeline_plane_init() allocates enum names for color
pipelines, which are copied by drm_property_create_enum(). The temporary
strings were not freed, resulting in a memory leak.

Allocate enum names only after successful pipeline construction and free
them on all exit paths.

Fixes: ef105316819d ("drm/i915/color: Create a transfer function color pipeline")
Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
---
 .../drm/i915/display/intel_color_pipeline.c    | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
index 684641c8323b..04af552b3648 100644
--- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
+++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
@@ -34,7 +34,6 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 		return ret;
 
 	list->type = colorop->base.base.id;
-	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", colorop->base.base.id);
 
 	/* TODO: handle failures and clean up */
 	prev_op = &colorop->base;
@@ -74,6 +73,8 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 
 	drm_colorop_set_next_property(prev_op, &colorop->base);
 
+	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", list->type);
+
 	return 0;
 }
 
@@ -81,9 +82,10 @@ int intel_color_pipeline_plane_init(struct drm_plane *plane, enum pipe pipe)
 {
 	struct drm_device *dev = plane->dev;
 	struct intel_display *display = to_intel_display(dev);
-	struct drm_prop_enum_list pipelines[MAX_COLOR_PIPELINES];
+	struct drm_prop_enum_list pipelines[MAX_COLOR_PIPELINES] = {};
 	int len = 0;
-	int ret;
+	int ret = 0;
+	int i;
 
 	/* Currently expose pipeline only for HDR planes */
 	if (!icl_is_hdr_plane(display, to_intel_plane(plane)->id))
@@ -92,8 +94,14 @@ int intel_color_pipeline_plane_init(struct drm_plane *plane, enum pipe pipe)
 	/* Add pipeline consisting of transfer functions */
 	ret = _intel_color_pipeline_plane_init(plane, &pipelines[len], pipe);
 	if (ret)
-		return ret;
+		goto out;
 	len++;
 
-	return drm_plane_create_color_pipeline_property(plane, pipelines, len);
+	ret = drm_plane_create_color_pipeline_property(plane, pipelines, len);
+
+	for (i = 0; i < len; i++)
+		kfree(pipelines[i].name);
+
+out:
+	return ret;
 }
-- 
2.25.1

[PATCH v3 05/13] drm/colorop: Add destroy helper for colorop objects

[Chaitanya Kumar Borah]

Add a helper that performs common cleanup and frees the
associated object. This can be used by drivers if they do not
require any driver-specific teardown.

v2:
- Add function documentation only before definition (Jani)

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
---
 drivers/gpu/drm/drm_colorop.c | 15 +++++++++++++++
 include/drm/drm_colorop.h     |  2 ++
 2 files changed, 17 insertions(+)

diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c
index 44eb823585d2..c226870fde9e 100644
--- a/drivers/gpu/drm/drm_colorop.c
+++ b/drivers/gpu/drm/drm_colorop.c
@@ -178,6 +178,21 @@ void drm_colorop_cleanup(struct drm_colorop *colorop)
 }
 EXPORT_SYMBOL(drm_colorop_cleanup);
 
+/**
+ * drm_colorop_destroy - destroy colorop
+ * @colorop: drm colorop
+ *
+ * Destroys @colorop by performing common DRM cleanup and freeing the
+ * colorop object. This can be used by drivers if they do not
+ * require any driver-specific teardown.
+ */
+void drm_colorop_destroy(struct drm_colorop *colorop)
+{
+	drm_colorop_cleanup(colorop);
+	kfree(colorop);
+}
+EXPORT_SYMBOL(drm_colorop_destroy);
+
 /**
  * drm_colorop_pipeline_destroy - Helper for color pipeline destruction
  *
diff --git a/include/drm/drm_colorop.h b/include/drm/drm_colorop.h
index a3a32f9f918c..3056f3f02597 100644
--- a/include/drm/drm_colorop.h
+++ b/include/drm/drm_colorop.h
@@ -420,6 +420,8 @@ void drm_colorop_atomic_destroy_state(struct drm_colorop *colorop,
  */
 void drm_colorop_reset(struct drm_colorop *colorop);
 
+void drm_colorop_destroy(struct drm_colorop *colorop);
+
 /**
  * drm_colorop_index - find the index of a registered colorop
  * @colorop: colorop to find index for
-- 
2.25.1

[PATCH v3 06/13] drm: Allow driver-managed destruction of colorop objects

[Chaitanya Kumar Borah]

Some drivers might want to embed struct drm_colorop inside
driver-specific objects, similar to planes or CRTCs. In such
cases, freeing only the drm_colorop is incorrect.

Add a drm_colorop_funcs callback to allow drivers to provide a destroy
hook that cleans up the full enclosing object. Make changes in helper
functions to accept helper functions as argument. Pass NULL for now
to retain current behavior.

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
---
 .../amd/display/amdgpu_dm/amdgpu_dm_colorop.c | 18 ++++++-----
 drivers/gpu/drm/drm_colorop.c                 | 31 +++++++++++++------
 .../drm/i915/display/intel_color_pipeline.c   |  8 ++---
 drivers/gpu/drm/vkms/vkms_colorop.c           | 10 +++---
 include/drm/drm_colorop.h                     | 30 +++++++++++++++---
 5 files changed, 66 insertions(+), 31 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
index a2de3bba8346..dfdb4fb4219f 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
@@ -72,7 +72,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
 					      amdgpu_dm_supported_degam_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
@@ -89,7 +89,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_mult_init(dev, ops[i], plane, DRM_COLOROP_FLAG_ALLOW_BYPASS);
+	ret = drm_plane_colorop_mult_init(dev, ops[i], plane, NULL, DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
 
@@ -104,7 +104,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, DRM_COLOROP_FLAG_ALLOW_BYPASS);
+	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
+					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
 
@@ -120,7 +121,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 			goto cleanup;
 		}
 
-		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
+		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
 						amdgpu_dm_supported_shaper_tfs,
 						DRM_COLOROP_FLAG_ALLOW_BYPASS);
 		if (ret)
@@ -137,7 +138,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 			goto cleanup;
 		}
 
-		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, MAX_COLOR_LUT_ENTRIES,
+		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL,
+							MAX_COLOR_LUT_ENTRIES,
 							DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 							DRM_COLOROP_FLAG_ALLOW_BYPASS);
 		if (ret)
@@ -154,7 +156,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 			goto cleanup;
 		}
 
-		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane, LUT3D_SIZE,
+		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane, NULL, LUT3D_SIZE,
 					DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
 					DRM_COLOROP_FLAG_ALLOW_BYPASS);
 		if (ret)
@@ -172,7 +174,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
 					      amdgpu_dm_supported_blnd_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
@@ -189,7 +191,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, MAX_COLOR_LUT_ENTRIES,
+	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL, MAX_COLOR_LUT_ENTRIES,
 						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c
index c226870fde9e..2bce29176ab3 100644
--- a/drivers/gpu/drm/drm_colorop.c
+++ b/drivers/gpu/drm/drm_colorop.c
@@ -93,7 +93,8 @@ static const struct drm_prop_enum_list drm_colorop_lut3d_interpolation_list[] =
 /* Init Helpers */
 
 static int drm_plane_colorop_init(struct drm_device *dev, struct drm_colorop *colorop,
-				  struct drm_plane *plane, enum drm_colorop_type type,
+				  struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				  enum drm_colorop_type type,
 				  uint32_t flags)
 {
 	struct drm_mode_config *config = &dev->mode_config;
@@ -109,6 +110,7 @@ static int drm_plane_colorop_init(struct drm_device *dev, struct drm_colorop *co
 	colorop->type = type;
 	colorop->plane = plane;
 	colorop->next = NULL;
+	colorop->funcs = funcs;
 
 	list_add_tail(&colorop->head, &config->colorop_list);
 	colorop->index = config->num_colorop++;
@@ -218,6 +220,7 @@ EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
  * @dev: DRM device
  * @colorop: The drm_colorop object to initialize
  * @plane: The associated drm_plane
+ * @funcs: control functions for the new colorop
  * @supported_tfs: A bitfield of supported drm_plane_colorop_curve_1d_init enum values,
  *                 created using BIT(curve_type) and combined with the OR '|'
  *                 operator.
@@ -225,7 +228,8 @@ EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
  * @return zero on success, -E value on failure
  */
 int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct drm_colorop *colorop,
-				    struct drm_plane *plane, u64 supported_tfs, uint32_t flags)
+				    struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				    u64 supported_tfs, uint32_t flags)
 {
 	struct drm_prop_enum_list enum_list[DRM_COLOROP_1D_CURVE_COUNT];
 	int i, len;
@@ -246,7 +250,7 @@ int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct drm_colorop *
 		return -EINVAL;
 	}
 
-	ret = drm_plane_colorop_init(dev, colorop, plane, DRM_COLOROP_1D_CURVE, flags);
+	ret = drm_plane_colorop_init(dev, colorop, plane, funcs, DRM_COLOROP_1D_CURVE, flags);
 	if (ret)
 		return ret;
 
@@ -303,20 +307,23 @@ static int drm_colorop_create_data_prop(struct drm_device *dev, struct drm_color
  * @dev: DRM device
  * @colorop: The drm_colorop object to initialize
  * @plane: The associated drm_plane
+ * @funcs: control functions for new colorop
  * @lut_size: LUT size supported by driver
  * @interpolation: 1D LUT interpolation type
  * @flags: bitmask of misc, see DRM_COLOROP_FLAG_* defines.
  * @return zero on success, -E value on failure
  */
 int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct drm_colorop *colorop,
-					struct drm_plane *plane, uint32_t lut_size,
+					struct drm_plane *plane,
+					const struct drm_colorop_funcs *funcs,
+					uint32_t lut_size,
 					enum drm_colorop_lut1d_interpolation_type interpolation,
 					uint32_t flags)
 {
 	struct drm_property *prop;
 	int ret;
 
-	ret = drm_plane_colorop_init(dev, colorop, plane, DRM_COLOROP_1D_LUT, flags);
+	ret = drm_plane_colorop_init(dev, colorop, plane, funcs, DRM_COLOROP_1D_LUT, flags);
 	if (ret)
 		return ret;
 
@@ -354,11 +361,12 @@ int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct drm_color
 EXPORT_SYMBOL(drm_plane_colorop_curve_1d_lut_init);
 
 int drm_plane_colorop_ctm_3x4_init(struct drm_device *dev, struct drm_colorop *colorop,
-				   struct drm_plane *plane, uint32_t flags)
+				   struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				   uint32_t flags)
 {
 	int ret;
 
-	ret = drm_plane_colorop_init(dev, colorop, plane, DRM_COLOROP_CTM_3X4, flags);
+	ret = drm_plane_colorop_init(dev, colorop, plane, funcs, DRM_COLOROP_CTM_3X4, flags);
 	if (ret)
 		return ret;
 
@@ -378,16 +386,18 @@ EXPORT_SYMBOL(drm_plane_colorop_ctm_3x4_init);
  * @dev: DRM device
  * @colorop: The drm_colorop object to initialize
  * @plane: The associated drm_plane
+ * @funcs: control functions for the new colorop
  * @flags: bitmask of misc, see DRM_COLOROP_FLAG_* defines.
  * @return zero on success, -E value on failure
  */
 int drm_plane_colorop_mult_init(struct drm_device *dev, struct drm_colorop *colorop,
-				struct drm_plane *plane, uint32_t flags)
+				struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				uint32_t flags)
 {
 	struct drm_property *prop;
 	int ret;
 
-	ret = drm_plane_colorop_init(dev, colorop, plane, DRM_COLOROP_MULTIPLIER, flags);
+	ret = drm_plane_colorop_init(dev, colorop, plane, funcs, DRM_COLOROP_MULTIPLIER, flags);
 	if (ret)
 		return ret;
 
@@ -406,6 +416,7 @@ EXPORT_SYMBOL(drm_plane_colorop_mult_init);
 
 int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop *colorop,
 				 struct drm_plane *plane,
+				 const struct drm_colorop_funcs *funcs,
 				 uint32_t lut_size,
 				 enum drm_colorop_lut3d_interpolation_type interpolation,
 				 uint32_t flags)
@@ -413,7 +424,7 @@ int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop *col
 	struct drm_property *prop;
 	int ret;
 
-	ret = drm_plane_colorop_init(dev, colorop, plane, DRM_COLOROP_3D_LUT, flags);
+	ret = drm_plane_colorop_init(dev, colorop, plane, funcs, DRM_COLOROP_3D_LUT, flags);
 	if (ret)
 		return ret;
 
diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
index 04af552b3648..d3d73d60727c 100644
--- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
+++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
@@ -25,7 +25,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 
 	colorop = intel_colorop_create(INTEL_PLANE_CB_PRE_CSC_LUT);
 
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane,
+	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, NULL,
 						  PLANE_DEGAMMA_SIZE,
 						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
@@ -39,7 +39,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	prev_op = &colorop->base;
 
 	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
-	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
+	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane, NULL,
 					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		return ret;
@@ -52,7 +52,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	    plane->type == DRM_PLANE_TYPE_PRIMARY) {
 		colorop = intel_colorop_create(INTEL_PLANE_CB_3DLUT);
 
-		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base, plane, 17,
+		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base, plane, NULL, 17,
 						   DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
 						   true);
 		if (ret)
@@ -64,7 +64,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	}
 
 	colorop = intel_colorop_create(INTEL_PLANE_CB_POST_CSC_LUT);
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane,
+	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, NULL,
 						  PLANE_GAMMA_SIZE,
 						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
diff --git a/drivers/gpu/drm/vkms/vkms_colorop.c b/drivers/gpu/drm/vkms/vkms_colorop.c
index d03a1f2e9c41..9e9dd0494628 100644
--- a/drivers/gpu/drm/vkms/vkms_colorop.c
+++ b/drivers/gpu/drm/vkms/vkms_colorop.c
@@ -31,7 +31,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, supported_tfs,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL, supported_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
@@ -48,7 +48,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, DRM_COLOROP_FLAG_ALLOW_BYPASS);
+	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
+					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
 
@@ -64,7 +65,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, DRM_COLOROP_FLAG_ALLOW_BYPASS);
+	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
+					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
 
@@ -80,7 +82,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, supported_tfs,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL, supported_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
diff --git a/include/drm/drm_colorop.h b/include/drm/drm_colorop.h
index 3056f3f02597..bd082854ca74 100644
--- a/include/drm/drm_colorop.h
+++ b/include/drm/drm_colorop.h
@@ -187,6 +187,19 @@ struct drm_colorop_state {
 	struct drm_atomic_state *state;
 };
 
+/**
+ * struct drm_colorop_funcs - driver colorop control functions
+ */
+struct drm_colorop_funcs {
+	/**
+	 * @destroy:
+	 *
+	 * Clean up colorop resources. This is called at driver unload time
+	 * through drm_mode_config_cleanup()
+	 */
+	void (*destroy)(struct drm_colorop *colorop);
+};
+
 /**
  * struct drm_colorop - DRM color operation control structure
  *
@@ -362,6 +375,8 @@ struct drm_colorop {
 	 */
 	struct drm_property *next_property;
 
+	/** @funcs: colorop control functions */
+	const struct drm_colorop_funcs *funcs;
 };
 
 #define obj_to_colorop(x) container_of(x, struct drm_colorop, base)
@@ -390,17 +405,22 @@ void drm_colorop_pipeline_destroy(struct drm_device *dev);
 void drm_colorop_cleanup(struct drm_colorop *colorop);
 
 int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct drm_colorop *colorop,
-				    struct drm_plane *plane, u64 supported_tfs, uint32_t flags);
+				    struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				    u64 supported_tfs, uint32_t flags);
 int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct drm_colorop *colorop,
-					struct drm_plane *plane, uint32_t lut_size,
+					struct drm_plane *plane,
+					const struct drm_colorop_funcs *funcs,
+					uint32_t lut_size,
 					enum drm_colorop_lut1d_interpolation_type interpolation,
 					uint32_t flags);
 int drm_plane_colorop_ctm_3x4_init(struct drm_device *dev, struct drm_colorop *colorop,
-				   struct drm_plane *plane, uint32_t flags);
+				   struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				   uint32_t flags);
 int drm_plane_colorop_mult_init(struct drm_device *dev, struct drm_colorop *colorop,
-				struct drm_plane *plane, uint32_t flags);
+				struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
+				uint32_t flags);
 int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop *colorop,
-				 struct drm_plane *plane,
+				 struct drm_plane *plane, const struct drm_colorop_funcs *funcs,
 				 uint32_t lut_size,
 				 enum drm_colorop_lut3d_interpolation_type interpolation,
 				 uint32_t flags);
-- 
2.25.1

RE: [PATCH v3 06/13] drm: Allow driver-managed destruction of colorop objects

[Kandpal, Suraj]

> Subject: [PATCH v3 06/13] drm: Allow driver-managed destruction of colorop
> objects
> 
> Some drivers might want to embed struct drm_colorop inside driver-specific
> objects, similar to planes or CRTCs. In such cases, freeing only the drm_colorop
> is incorrect.
> 
> Add a drm_colorop_funcs callback to allow drivers to provide a destroy hook
> that cleans up the full enclosing object. Make changes in helper functions to
> accept helper functions as argument. Pass NULL for now to retain current
> behavior.
> 

Just to point out checkpatch asks us to use u32 instead of uint32_t

Regards,
Suraj Kandpal

> Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
> Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
> Reviewed-by: Uma Shankar <uma.shankar@intel.com>
> Reviewed-by: Alex Hung <alex.hung@amd.com>
> ---
>  .../amd/display/amdgpu_dm/amdgpu_dm_colorop.c | 18 ++++++-----
>  drivers/gpu/drm/drm_colorop.c                 | 31 +++++++++++++------
>  .../drm/i915/display/intel_color_pipeline.c   |  8 ++---
>  drivers/gpu/drm/vkms/vkms_colorop.c           | 10 +++---
>  include/drm/drm_colorop.h                     | 30 +++++++++++++++---
>  5 files changed, 66 insertions(+), 31 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
> index a2de3bba8346..dfdb4fb4219f 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
> @@ -72,7 +72,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
> 
> amdgpu_dm_supported_degam_tfs,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
> @@ -89,7 +89,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_mult_init(dev, ops[i], plane,
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
> +	ret = drm_plane_colorop_mult_init(dev, ops[i], plane, NULL,
> +DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		goto cleanup;
> 
> @@ -104,7 +104,8 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
> +	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
> +
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		goto cleanup;
> 
> @@ -120,7 +121,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  			goto cleanup;
>  		}
> 
> -		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
> +		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
> NULL,
> 
> 	amdgpu_dm_supported_shaper_tfs,
> 
> 	DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  		if (ret)
> @@ -137,7 +138,8 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  			goto cleanup;
>  		}
> 
> -		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
> MAX_COLOR_LUT_ENTRIES,
> +		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
> NULL,
> +
> 	MAX_COLOR_LUT_ENTRIES,
> 
> 	DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
> 
> 	DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  		if (ret)
> @@ -154,7 +156,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  			goto cleanup;
>  		}
> 
> -		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane,
> LUT3D_SIZE,
> +		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane, NULL,
> +LUT3D_SIZE,
> 
> 	DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
> 
> 	DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  		if (ret)
> @@ -172,7 +174,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
>  					      amdgpu_dm_supported_blnd_tfs,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
> @@ -189,7 +191,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
> drm_plane *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
> MAX_COLOR_LUT_ENTRIES,
> +	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL,
> +MAX_COLOR_LUT_ENTRIES,
> 
> DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
> diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c
> index c226870fde9e..2bce29176ab3 100644
> --- a/drivers/gpu/drm/drm_colorop.c
> +++ b/drivers/gpu/drm/drm_colorop.c
> @@ -93,7 +93,8 @@ static const struct drm_prop_enum_list
> drm_colorop_lut3d_interpolation_list[] =
>  /* Init Helpers */
> 
>  static int drm_plane_colorop_init(struct drm_device *dev, struct drm_colorop
> *colorop,
> -				  struct drm_plane *plane, enum
> drm_colorop_type type,
> +				  struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				  enum drm_colorop_type type,
>  				  uint32_t flags)
>  {
>  	struct drm_mode_config *config = &dev->mode_config; @@ -109,6
> +110,7 @@ static int drm_plane_colorop_init(struct drm_device *dev, struct
> drm_colorop *co
>  	colorop->type = type;
>  	colorop->plane = plane;
>  	colorop->next = NULL;
> +	colorop->funcs = funcs;
> 
>  	list_add_tail(&colorop->head, &config->colorop_list);
>  	colorop->index = config->num_colorop++; @@ -218,6 +220,7 @@
> EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
>   * @dev: DRM device
>   * @colorop: The drm_colorop object to initialize
>   * @plane: The associated drm_plane
> + * @funcs: control functions for the new colorop
>   * @supported_tfs: A bitfield of supported drm_plane_colorop_curve_1d_init
> enum values,
>   *                 created using BIT(curve_type) and combined with the OR '|'
>   *                 operator.
> @@ -225,7 +228,8 @@ EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
>   * @return zero on success, -E value on failure
>   */
>  int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> -				    struct drm_plane *plane, u64 supported_tfs,
> uint32_t flags)
> +				    struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				    u64 supported_tfs, uint32_t flags)
>  {
>  	struct drm_prop_enum_list
> enum_list[DRM_COLOROP_1D_CURVE_COUNT];
>  	int i, len;
> @@ -246,7 +250,7 @@ int drm_plane_colorop_curve_1d_init(struct
> drm_device *dev, struct drm_colorop *
>  		return -EINVAL;
>  	}
> 
> -	ret = drm_plane_colorop_init(dev, colorop, plane,
> DRM_COLOROP_1D_CURVE, flags);
> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
> +DRM_COLOROP_1D_CURVE, flags);
>  	if (ret)
>  		return ret;
> 
> @@ -303,20 +307,23 @@ static int drm_colorop_create_data_prop(struct
> drm_device *dev, struct drm_color
>   * @dev: DRM device
>   * @colorop: The drm_colorop object to initialize
>   * @plane: The associated drm_plane
> + * @funcs: control functions for new colorop
>   * @lut_size: LUT size supported by driver
>   * @interpolation: 1D LUT interpolation type
>   * @flags: bitmask of misc, see DRM_COLOROP_FLAG_* defines.
>   * @return zero on success, -E value on failure
>   */
>  int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> -					struct drm_plane *plane, uint32_t
> lut_size,
> +					struct drm_plane *plane,
> +					const struct drm_colorop_funcs
> *funcs,
> +					uint32_t lut_size,
>  					enum
> drm_colorop_lut1d_interpolation_type interpolation,
>  					uint32_t flags)
>  {
>  	struct drm_property *prop;
>  	int ret;
> 
> -	ret = drm_plane_colorop_init(dev, colorop, plane,
> DRM_COLOROP_1D_LUT, flags);
> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
> +DRM_COLOROP_1D_LUT, flags);
>  	if (ret)
>  		return ret;
> 
> @@ -354,11 +361,12 @@ int drm_plane_colorop_curve_1d_lut_init(struct
> drm_device *dev, struct drm_color
> EXPORT_SYMBOL(drm_plane_colorop_curve_1d_lut_init);
> 
>  int drm_plane_colorop_ctm_3x4_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> -				   struct drm_plane *plane, uint32_t flags)
> +				   struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				   uint32_t flags)
>  {
>  	int ret;
> 
> -	ret = drm_plane_colorop_init(dev, colorop, plane,
> DRM_COLOROP_CTM_3X4, flags);
> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
> +DRM_COLOROP_CTM_3X4, flags);
>  	if (ret)
>  		return ret;
> 
> @@ -378,16 +386,18 @@
> EXPORT_SYMBOL(drm_plane_colorop_ctm_3x4_init);
>   * @dev: DRM device
>   * @colorop: The drm_colorop object to initialize
>   * @plane: The associated drm_plane
> + * @funcs: control functions for the new colorop
>   * @flags: bitmask of misc, see DRM_COLOROP_FLAG_* defines.
>   * @return zero on success, -E value on failure
>   */
>  int drm_plane_colorop_mult_init(struct drm_device *dev, struct drm_colorop
> *colorop,
> -				struct drm_plane *plane, uint32_t flags)
> +				struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				uint32_t flags)
>  {
>  	struct drm_property *prop;
>  	int ret;
> 
> -	ret = drm_plane_colorop_init(dev, colorop, plane,
> DRM_COLOROP_MULTIPLIER, flags);
> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
> +DRM_COLOROP_MULTIPLIER, flags);
>  	if (ret)
>  		return ret;
> 
> @@ -406,6 +416,7 @@ EXPORT_SYMBOL(drm_plane_colorop_mult_init);
> 
>  int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop
> *colorop,
>  				 struct drm_plane *plane,
> +				 const struct drm_colorop_funcs *funcs,
>  				 uint32_t lut_size,
>  				 enum drm_colorop_lut3d_interpolation_type
> interpolation,
>  				 uint32_t flags)
> @@ -413,7 +424,7 @@ int drm_plane_colorop_3dlut_init(struct drm_device
> *dev, struct drm_colorop *col
>  	struct drm_property *prop;
>  	int ret;
> 
> -	ret = drm_plane_colorop_init(dev, colorop, plane,
> DRM_COLOROP_3D_LUT, flags);
> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
> +DRM_COLOROP_3D_LUT, flags);
>  	if (ret)
>  		return ret;
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> index 04af552b3648..d3d73d60727c 100644
> --- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> +++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
> @@ -25,7 +25,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane
> *plane, struct drm_prop_en
> 
>  	colorop = intel_colorop_create(INTEL_PLANE_CB_PRE_CSC_LUT);
> 
> -	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
> plane,
> +	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
> plane,
> +NULL,
>  						  PLANE_DEGAMMA_SIZE,
> 
> DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS); @@ -39,7 +39,7 @@ int
> _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
>  	prev_op = &colorop->base;
> 
>  	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
> -	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
> +	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
> NULL,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		return ret;
> @@ -52,7 +52,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane
> *plane, struct drm_prop_en
>  	    plane->type == DRM_PLANE_TYPE_PRIMARY) {
>  		colorop = intel_colorop_create(INTEL_PLANE_CB_3DLUT);
> 
> -		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base,
> plane, 17,
> +		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base,
> plane, NULL,
> +17,
> 
> DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
>  						   true);
>  		if (ret)
> @@ -64,7 +64,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane
> *plane, struct drm_prop_en
>  	}
> 
>  	colorop = intel_colorop_create(INTEL_PLANE_CB_POST_CSC_LUT);
> -	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
> plane,
> +	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
> plane,
> +NULL,
>  						  PLANE_GAMMA_SIZE,
> 
> DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS); diff --git
> a/drivers/gpu/drm/vkms/vkms_colorop.c
> b/drivers/gpu/drm/vkms/vkms_colorop.c
> index d03a1f2e9c41..9e9dd0494628 100644
> --- a/drivers/gpu/drm/vkms/vkms_colorop.c
> +++ b/drivers/gpu/drm/vkms/vkms_colorop.c
> @@ -31,7 +31,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane
> *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
> supported_tfs,
> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
> +supported_tfs,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		goto cleanup;
> @@ -48,7 +48,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane
> *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
> +	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
> +
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		goto cleanup;
> 
> @@ -64,7 +65,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane
> *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
> +	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
> +
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		goto cleanup;
> 
> @@ -80,7 +82,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane
> *plane, struct drm_pr
>  		goto cleanup;
>  	}
> 
> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
> supported_tfs,
> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
> +supported_tfs,
> 
> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>  	if (ret)
>  		goto cleanup;
> diff --git a/include/drm/drm_colorop.h b/include/drm/drm_colorop.h index
> 3056f3f02597..bd082854ca74 100644
> --- a/include/drm/drm_colorop.h
> +++ b/include/drm/drm_colorop.h
> @@ -187,6 +187,19 @@ struct drm_colorop_state {
>  	struct drm_atomic_state *state;
>  };
> 
> +/**
> + * struct drm_colorop_funcs - driver colorop control functions  */
> +struct drm_colorop_funcs {
> +	/**
> +	 * @destroy:
> +	 *
> +	 * Clean up colorop resources. This is called at driver unload time
> +	 * through drm_mode_config_cleanup()
> +	 */
> +	void (*destroy)(struct drm_colorop *colorop); };
> +
>  /**
>   * struct drm_colorop - DRM color operation control structure
>   *
> @@ -362,6 +375,8 @@ struct drm_colorop {
>  	 */
>  	struct drm_property *next_property;
> 
> +	/** @funcs: colorop control functions */
> +	const struct drm_colorop_funcs *funcs;
>  };
> 
>  #define obj_to_colorop(x) container_of(x, struct drm_colorop, base) @@ -
> 390,17 +405,22 @@ void drm_colorop_pipeline_destroy(struct drm_device
> *dev);  void drm_colorop_cleanup(struct drm_colorop *colorop);
> 
>  int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> -				    struct drm_plane *plane, u64 supported_tfs,
> uint32_t flags);
> +				    struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				    u64 supported_tfs, uint32_t flags);
>  int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> -					struct drm_plane *plane, uint32_t
> lut_size,
> +					struct drm_plane *plane,
> +					const struct drm_colorop_funcs
> *funcs,
> +					uint32_t lut_size,
>  					enum
> drm_colorop_lut1d_interpolation_type interpolation,
>  					uint32_t flags);
>  int drm_plane_colorop_ctm_3x4_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> -				   struct drm_plane *plane, uint32_t flags);
> +				   struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				   uint32_t flags);
>  int drm_plane_colorop_mult_init(struct drm_device *dev, struct drm_colorop
> *colorop,
> -				struct drm_plane *plane, uint32_t flags);
> +				struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
> +				uint32_t flags);
>  int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop
> *colorop,
> -				 struct drm_plane *plane,
> +				 struct drm_plane *plane, const struct
> drm_colorop_funcs *funcs,
>  				 uint32_t lut_size,
>  				 enum drm_colorop_lut3d_interpolation_type
> interpolation,
>  				 uint32_t flags);
> --
> 2.25.1

Re: [PATCH v3 06/13] drm: Allow driver-managed destruction of colorop objects

[Borah, Chaitanya Kumar]

On 1/14/2026 11:07 AM, Kandpal, Suraj wrote:
>> Subject: [PATCH v3 06/13] drm: Allow driver-managed destruction of colorop
>> objects
>>
>> Some drivers might want to embed struct drm_colorop inside driver-specific
>> objects, similar to planes or CRTCs. In such cases, freeing only the drm_colorop
>> is incorrect.
>>
>> Add a drm_colorop_funcs callback to allow drivers to provide a destroy hook
>> that cleans up the full enclosing object. Make changes in helper functions to
>> accept helper functions as argument. Pass NULL for now to retain current
>> behavior.
>>
> 
> Just to point out checkpatch asks us to use u32 instead of uint32_t

That thought crossed my mind as well, but the file already consistently 
uses uint32_t. If cleanup is needed, we can follow up with a separate 
cleanup patch.

==
Chaitanya

> 
> Regards,
> Suraj Kandpal
> 
>> Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
>> Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
>> Reviewed-by: Uma Shankar <uma.shankar@intel.com>
>> Reviewed-by: Alex Hung <alex.hung@amd.com>
>> ---
>>   .../amd/display/amdgpu_dm/amdgpu_dm_colorop.c | 18 ++++++-----
>>   drivers/gpu/drm/drm_colorop.c                 | 31 +++++++++++++------
>>   .../drm/i915/display/intel_color_pipeline.c   |  8 ++---
>>   drivers/gpu/drm/vkms/vkms_colorop.c           | 10 +++---
>>   include/drm/drm_colorop.h                     | 30 +++++++++++++++---
>>   5 files changed, 66 insertions(+), 31 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
>> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
>> index a2de3bba8346..dfdb4fb4219f 100644
>> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
>> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
>> @@ -72,7 +72,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
>> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
>>
>> amdgpu_dm_supported_degam_tfs,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>> @@ -89,7 +89,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_mult_init(dev, ops[i], plane,
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>> +	ret = drm_plane_colorop_mult_init(dev, ops[i], plane, NULL,
>> +DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		goto cleanup;
>>
>> @@ -104,7 +104,8 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>> +	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
>> +
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		goto cleanup;
>>
>> @@ -120,7 +121,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   			goto cleanup;
>>   		}
>>
>> -		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
>> +		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
>> NULL,
>>
>> 	amdgpu_dm_supported_shaper_tfs,
>>
>> 	DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   		if (ret)
>> @@ -137,7 +138,8 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   			goto cleanup;
>>   		}
>>
>> -		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
>> MAX_COLOR_LUT_ENTRIES,
>> +		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
>> NULL,
>> +
>> 	MAX_COLOR_LUT_ENTRIES,
>>
>> 	DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
>>
>> 	DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   		if (ret)
>> @@ -154,7 +156,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   			goto cleanup;
>>   		}
>>
>> -		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane,
>> LUT3D_SIZE,
>> +		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane, NULL,
>> +LUT3D_SIZE,
>>
>> 	DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
>>
>> 	DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   		if (ret)
>> @@ -172,7 +174,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
>> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
>>   					      amdgpu_dm_supported_blnd_tfs,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>> @@ -189,7 +191,7 @@ int amdgpu_dm_initialize_default_pipeline(struct
>> drm_plane *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
>> MAX_COLOR_LUT_ENTRIES,
>> +	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL,
>> +MAX_COLOR_LUT_ENTRIES,
>>
>> DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>> diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c
>> index c226870fde9e..2bce29176ab3 100644
>> --- a/drivers/gpu/drm/drm_colorop.c
>> +++ b/drivers/gpu/drm/drm_colorop.c
>> @@ -93,7 +93,8 @@ static const struct drm_prop_enum_list
>> drm_colorop_lut3d_interpolation_list[] =
>>   /* Init Helpers */
>>
>>   static int drm_plane_colorop_init(struct drm_device *dev, struct drm_colorop
>> *colorop,
>> -				  struct drm_plane *plane, enum
>> drm_colorop_type type,
>> +				  struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				  enum drm_colorop_type type,
>>   				  uint32_t flags)
>>   {
>>   	struct drm_mode_config *config = &dev->mode_config; @@ -109,6
>> +110,7 @@ static int drm_plane_colorop_init(struct drm_device *dev, struct
>> drm_colorop *co
>>   	colorop->type = type;
>>   	colorop->plane = plane;
>>   	colorop->next = NULL;
>> +	colorop->funcs = funcs;
>>
>>   	list_add_tail(&colorop->head, &config->colorop_list);
>>   	colorop->index = config->num_colorop++; @@ -218,6 +220,7 @@
>> EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
>>    * @dev: DRM device
>>    * @colorop: The drm_colorop object to initialize
>>    * @plane: The associated drm_plane
>> + * @funcs: control functions for the new colorop
>>    * @supported_tfs: A bitfield of supported drm_plane_colorop_curve_1d_init
>> enum values,
>>    *                 created using BIT(curve_type) and combined with the OR '|'
>>    *                 operator.
>> @@ -225,7 +228,8 @@ EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
>>    * @return zero on success, -E value on failure
>>    */
>>   int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct
>> drm_colorop *colorop,
>> -				    struct drm_plane *plane, u64 supported_tfs,
>> uint32_t flags)
>> +				    struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				    u64 supported_tfs, uint32_t flags)
>>   {
>>   	struct drm_prop_enum_list
>> enum_list[DRM_COLOROP_1D_CURVE_COUNT];
>>   	int i, len;
>> @@ -246,7 +250,7 @@ int drm_plane_colorop_curve_1d_init(struct
>> drm_device *dev, struct drm_colorop *
>>   		return -EINVAL;
>>   	}
>>
>> -	ret = drm_plane_colorop_init(dev, colorop, plane,
>> DRM_COLOROP_1D_CURVE, flags);
>> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
>> +DRM_COLOROP_1D_CURVE, flags);
>>   	if (ret)
>>   		return ret;
>>
>> @@ -303,20 +307,23 @@ static int drm_colorop_create_data_prop(struct
>> drm_device *dev, struct drm_color
>>    * @dev: DRM device
>>    * @colorop: The drm_colorop object to initialize
>>    * @plane: The associated drm_plane
>> + * @funcs: control functions for new colorop
>>    * @lut_size: LUT size supported by driver
>>    * @interpolation: 1D LUT interpolation type
>>    * @flags: bitmask of misc, see DRM_COLOROP_FLAG_* defines.
>>    * @return zero on success, -E value on failure
>>    */
>>   int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct
>> drm_colorop *colorop,
>> -					struct drm_plane *plane, uint32_t
>> lut_size,
>> +					struct drm_plane *plane,
>> +					const struct drm_colorop_funcs
>> *funcs,
>> +					uint32_t lut_size,
>>   					enum
>> drm_colorop_lut1d_interpolation_type interpolation,
>>   					uint32_t flags)
>>   {
>>   	struct drm_property *prop;
>>   	int ret;
>>
>> -	ret = drm_plane_colorop_init(dev, colorop, plane,
>> DRM_COLOROP_1D_LUT, flags);
>> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
>> +DRM_COLOROP_1D_LUT, flags);
>>   	if (ret)
>>   		return ret;
>>
>> @@ -354,11 +361,12 @@ int drm_plane_colorop_curve_1d_lut_init(struct
>> drm_device *dev, struct drm_color
>> EXPORT_SYMBOL(drm_plane_colorop_curve_1d_lut_init);
>>
>>   int drm_plane_colorop_ctm_3x4_init(struct drm_device *dev, struct
>> drm_colorop *colorop,
>> -				   struct drm_plane *plane, uint32_t flags)
>> +				   struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				   uint32_t flags)
>>   {
>>   	int ret;
>>
>> -	ret = drm_plane_colorop_init(dev, colorop, plane,
>> DRM_COLOROP_CTM_3X4, flags);
>> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
>> +DRM_COLOROP_CTM_3X4, flags);
>>   	if (ret)
>>   		return ret;
>>
>> @@ -378,16 +386,18 @@
>> EXPORT_SYMBOL(drm_plane_colorop_ctm_3x4_init);
>>    * @dev: DRM device
>>    * @colorop: The drm_colorop object to initialize
>>    * @plane: The associated drm_plane
>> + * @funcs: control functions for the new colorop
>>    * @flags: bitmask of misc, see DRM_COLOROP_FLAG_* defines.
>>    * @return zero on success, -E value on failure
>>    */
>>   int drm_plane_colorop_mult_init(struct drm_device *dev, struct drm_colorop
>> *colorop,
>> -				struct drm_plane *plane, uint32_t flags)
>> +				struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				uint32_t flags)
>>   {
>>   	struct drm_property *prop;
>>   	int ret;
>>
>> -	ret = drm_plane_colorop_init(dev, colorop, plane,
>> DRM_COLOROP_MULTIPLIER, flags);
>> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
>> +DRM_COLOROP_MULTIPLIER, flags);
>>   	if (ret)
>>   		return ret;
>>
>> @@ -406,6 +416,7 @@ EXPORT_SYMBOL(drm_plane_colorop_mult_init);
>>
>>   int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop
>> *colorop,
>>   				 struct drm_plane *plane,
>> +				 const struct drm_colorop_funcs *funcs,
>>   				 uint32_t lut_size,
>>   				 enum drm_colorop_lut3d_interpolation_type
>> interpolation,
>>   				 uint32_t flags)
>> @@ -413,7 +424,7 @@ int drm_plane_colorop_3dlut_init(struct drm_device
>> *dev, struct drm_colorop *col
>>   	struct drm_property *prop;
>>   	int ret;
>>
>> -	ret = drm_plane_colorop_init(dev, colorop, plane,
>> DRM_COLOROP_3D_LUT, flags);
>> +	ret = drm_plane_colorop_init(dev, colorop, plane, funcs,
>> +DRM_COLOROP_3D_LUT, flags);
>>   	if (ret)
>>   		return ret;
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
>> b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
>> index 04af552b3648..d3d73d60727c 100644
>> --- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
>> +++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
>> @@ -25,7 +25,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane
>> *plane, struct drm_prop_en
>>
>>   	colorop = intel_colorop_create(INTEL_PLANE_CB_PRE_CSC_LUT);
>>
>> -	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
>> plane,
>> +	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
>> plane,
>> +NULL,
>>   						  PLANE_DEGAMMA_SIZE,
>>
>> DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS); @@ -39,7 +39,7 @@ int
>> _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
>>   	prev_op = &colorop->base;
>>
>>   	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
>> -	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
>> +	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
>> NULL,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		return ret;
>> @@ -52,7 +52,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane
>> *plane, struct drm_prop_en
>>   	    plane->type == DRM_PLANE_TYPE_PRIMARY) {
>>   		colorop = intel_colorop_create(INTEL_PLANE_CB_3DLUT);
>>
>> -		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base,
>> plane, 17,
>> +		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base,
>> plane, NULL,
>> +17,
>>
>> DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
>>   						   true);
>>   		if (ret)
>> @@ -64,7 +64,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane
>> *plane, struct drm_prop_en
>>   	}
>>
>>   	colorop = intel_colorop_create(INTEL_PLANE_CB_POST_CSC_LUT);
>> -	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
>> plane,
>> +	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base,
>> plane,
>> +NULL,
>>   						  PLANE_GAMMA_SIZE,
>>
>> DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS); diff --git
>> a/drivers/gpu/drm/vkms/vkms_colorop.c
>> b/drivers/gpu/drm/vkms/vkms_colorop.c
>> index d03a1f2e9c41..9e9dd0494628 100644
>> --- a/drivers/gpu/drm/vkms/vkms_colorop.c
>> +++ b/drivers/gpu/drm/vkms/vkms_colorop.c
>> @@ -31,7 +31,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane
>> *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
>> supported_tfs,
>> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
>> +supported_tfs,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		goto cleanup;
>> @@ -48,7 +48,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane
>> *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>> +	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
>> +
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		goto cleanup;
>>
>> @@ -64,7 +65,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane
>> *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>> +	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
>> +
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		goto cleanup;
>>
>> @@ -80,7 +82,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane
>> *plane, struct drm_pr
>>   		goto cleanup;
>>   	}
>>
>> -	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane,
>> supported_tfs,
>> +	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
>> +supported_tfs,
>>
>> DRM_COLOROP_FLAG_ALLOW_BYPASS);
>>   	if (ret)
>>   		goto cleanup;
>> diff --git a/include/drm/drm_colorop.h b/include/drm/drm_colorop.h index
>> 3056f3f02597..bd082854ca74 100644
>> --- a/include/drm/drm_colorop.h
>> +++ b/include/drm/drm_colorop.h
>> @@ -187,6 +187,19 @@ struct drm_colorop_state {
>>   	struct drm_atomic_state *state;
>>   };
>>
>> +/**
>> + * struct drm_colorop_funcs - driver colorop control functions  */
>> +struct drm_colorop_funcs {
>> +	/**
>> +	 * @destroy:
>> +	 *
>> +	 * Clean up colorop resources. This is called at driver unload time
>> +	 * through drm_mode_config_cleanup()
>> +	 */
>> +	void (*destroy)(struct drm_colorop *colorop); };
>> +
>>   /**
>>    * struct drm_colorop - DRM color operation control structure
>>    *
>> @@ -362,6 +375,8 @@ struct drm_colorop {
>>   	 */
>>   	struct drm_property *next_property;
>>
>> +	/** @funcs: colorop control functions */
>> +	const struct drm_colorop_funcs *funcs;
>>   };
>>
>>   #define obj_to_colorop(x) container_of(x, struct drm_colorop, base) @@ -
>> 390,17 +405,22 @@ void drm_colorop_pipeline_destroy(struct drm_device
>> *dev);  void drm_colorop_cleanup(struct drm_colorop *colorop);
>>
>>   int drm_plane_colorop_curve_1d_init(struct drm_device *dev, struct
>> drm_colorop *colorop,
>> -				    struct drm_plane *plane, u64 supported_tfs,
>> uint32_t flags);
>> +				    struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				    u64 supported_tfs, uint32_t flags);
>>   int drm_plane_colorop_curve_1d_lut_init(struct drm_device *dev, struct
>> drm_colorop *colorop,
>> -					struct drm_plane *plane, uint32_t
>> lut_size,
>> +					struct drm_plane *plane,
>> +					const struct drm_colorop_funcs
>> *funcs,
>> +					uint32_t lut_size,
>>   					enum
>> drm_colorop_lut1d_interpolation_type interpolation,
>>   					uint32_t flags);
>>   int drm_plane_colorop_ctm_3x4_init(struct drm_device *dev, struct
>> drm_colorop *colorop,
>> -				   struct drm_plane *plane, uint32_t flags);
>> +				   struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				   uint32_t flags);
>>   int drm_plane_colorop_mult_init(struct drm_device *dev, struct drm_colorop
>> *colorop,
>> -				struct drm_plane *plane, uint32_t flags);
>> +				struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>> +				uint32_t flags);
>>   int drm_plane_colorop_3dlut_init(struct drm_device *dev, struct drm_colorop
>> *colorop,
>> -				 struct drm_plane *plane,
>> +				 struct drm_plane *plane, const struct
>> drm_colorop_funcs *funcs,
>>   				 uint32_t lut_size,
>>   				 enum drm_colorop_lut3d_interpolation_type
>> interpolation,
>>   				 uint32_t flags);
>> --
>> 2.25.1
> 

[PATCH v3 07/13] drm/amd/display: Hook up colorop destroy helper for plane pipelines

[Chaitanya Kumar Borah]

Provide a drm_colorop_funcs instance for amdgpu_dm color pipeline
objects and hook up the common drm_colorop_destroy() helper as the
destroy callback.

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
---
 .../amd/display/amdgpu_dm/amdgpu_dm_colorop.c | 25 +++++++++++++------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
index dfdb4fb4219f..5130962193d9 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
@@ -55,6 +55,10 @@ const u64 amdgpu_dm_supported_blnd_tfs =
 
 #define LUT3D_SIZE		17
 
+static const struct drm_colorop_funcs dm_colorop_funcs = {
+	.destroy = drm_colorop_destroy,
+};
+
 int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_prop_enum_list *list)
 {
 	struct drm_colorop *ops[MAX_COLOR_PIPELINE_OPS];
@@ -72,7 +76,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, &dm_colorop_funcs,
 					      amdgpu_dm_supported_degam_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
@@ -89,7 +93,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_mult_init(dev, ops[i], plane, NULL, DRM_COLOROP_FLAG_ALLOW_BYPASS);
+	ret = drm_plane_colorop_mult_init(dev, ops[i], plane, &dm_colorop_funcs,
+					  DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
 
@@ -104,7 +109,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
+	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane,
+					     &dm_colorop_funcs,
 					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
@@ -121,7 +127,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 			goto cleanup;
 		}
 
-		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
+		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, &dm_colorop_funcs,
 						amdgpu_dm_supported_shaper_tfs,
 						DRM_COLOROP_FLAG_ALLOW_BYPASS);
 		if (ret)
@@ -138,7 +144,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 			goto cleanup;
 		}
 
-		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL,
+		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
+							&dm_colorop_funcs,
 							MAX_COLOR_LUT_ENTRIES,
 							DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 							DRM_COLOROP_FLAG_ALLOW_BYPASS);
@@ -156,7 +163,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 			goto cleanup;
 		}
 
-		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane, NULL, LUT3D_SIZE,
+		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane,
+					&dm_colorop_funcs, LUT3D_SIZE,
 					DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
 					DRM_COLOROP_FLAG_ALLOW_BYPASS);
 		if (ret)
@@ -174,7 +182,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, &dm_colorop_funcs,
 					      amdgpu_dm_supported_blnd_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
@@ -191,7 +199,8 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL, MAX_COLOR_LUT_ENTRIES,
+	ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, &dm_colorop_funcs,
+						  MAX_COLOR_LUT_ENTRIES,
 						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
-- 
2.25.1

[PATCH v3 08/13] drm/vkms: Hook up colorop destroy helper for plane pipelines

[Chaitanya Kumar Borah]

Provide a drm_colorop_funcs instance for vkms color pipeline
objects and hook up the common drm_colorop_destroy() helper as the
destroy callback.

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
Reviewed-by: Louis Chauvet <louis.chauvet@bootlin.com>
---
 drivers/gpu/drm/vkms/vkms_colorop.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/vkms/vkms_colorop.c b/drivers/gpu/drm/vkms/vkms_colorop.c
index 9e9dd0494628..ba826ad384b7 100644
--- a/drivers/gpu/drm/vkms/vkms_colorop.c
+++ b/drivers/gpu/drm/vkms/vkms_colorop.c
@@ -12,6 +12,10 @@ static const u64 supported_tfs =
 	BIT(DRM_COLOROP_1D_CURVE_SRGB_EOTF) |
 	BIT(DRM_COLOROP_1D_CURVE_SRGB_INV_EOTF);
 
+static const struct drm_colorop_funcs vkms_colorop_funcs = {
+	.destroy = drm_colorop_destroy,
+};
+
 #define MAX_COLOR_PIPELINE_OPS 4
 
 static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_prop_enum_list *list)
@@ -31,7 +35,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL, supported_tfs,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, &vkms_colorop_funcs,
+					      supported_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
@@ -48,7 +53,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
+	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, &vkms_colorop_funcs,
 					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
@@ -65,7 +70,7 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, NULL,
+	ret = drm_plane_colorop_ctm_3x4_init(dev, ops[i], plane, &vkms_colorop_funcs,
 					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
@@ -82,7 +87,8 @@ static int vkms_initialize_color_pipeline(struct drm_plane *plane, struct drm_pr
 		goto cleanup;
 	}
 
-	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL, supported_tfs,
+	ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, &vkms_colorop_funcs,
+					      supported_tfs,
 					      DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		goto cleanup;
-- 
2.25.1

[PATCH v3 09/13] drm/i915/display: Hook up intel_colorop_destroy

[Chaitanya Kumar Borah]

i915 embeds struct drm_colorop inside struct intel_colorop, so the
default drm_colorop_destroy() helper cannot be used. Add an
intel_colorop_destroy() helper that performs common DRM cleanup and
frees intel_colorop object.

This ensures correct teardown of plane color pipeline objects.

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
---
 drivers/gpu/drm/i915/display/intel_color_pipeline.c | 13 +++++++++----
 drivers/gpu/drm/i915/display/intel_colorop.c        |  6 ++++++
 drivers/gpu/drm/i915/display/intel_colorop.h        |  1 +
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
index d3d73d60727c..8fecc53540ba 100644
--- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
+++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
@@ -13,6 +13,10 @@
 #define PLANE_DEGAMMA_SIZE 128
 #define PLANE_GAMMA_SIZE 32
 
+static const struct drm_colorop_funcs intel_colorop_funcs = {
+	.destroy = intel_colorop_destroy,
+};
+
 static
 int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_enum_list *list,
 				     enum pipe pipe)
@@ -25,7 +29,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 
 	colorop = intel_colorop_create(INTEL_PLANE_CB_PRE_CSC_LUT);
 
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, NULL,
+	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, &intel_colorop_funcs,
 						  PLANE_DEGAMMA_SIZE,
 						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
@@ -39,7 +43,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	prev_op = &colorop->base;
 
 	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
-	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane, NULL,
+	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane, &intel_colorop_funcs,
 					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
 		return ret;
@@ -52,7 +56,8 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	    plane->type == DRM_PLANE_TYPE_PRIMARY) {
 		colorop = intel_colorop_create(INTEL_PLANE_CB_3DLUT);
 
-		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base, plane, NULL, 17,
+		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base, plane,
+						   &intel_colorop_funcs, 17,
 						   DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
 						   true);
 		if (ret)
@@ -64,7 +69,7 @@ int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_en
 	}
 
 	colorop = intel_colorop_create(INTEL_PLANE_CB_POST_CSC_LUT);
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, NULL,
+	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, &intel_colorop_funcs,
 						  PLANE_GAMMA_SIZE,
 						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
 						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
diff --git a/drivers/gpu/drm/i915/display/intel_colorop.c b/drivers/gpu/drm/i915/display/intel_colorop.c
index 1d84933f05aa..9e54f51cfad8 100644
--- a/drivers/gpu/drm/i915/display/intel_colorop.c
+++ b/drivers/gpu/drm/i915/display/intel_colorop.c
@@ -35,3 +35,9 @@ struct intel_colorop *intel_colorop_create(enum intel_color_block id)
 
 	return colorop;
 }
+
+void intel_colorop_destroy(struct drm_colorop *colorop)
+{
+	drm_colorop_cleanup(colorop);
+	kfree(to_intel_colorop(colorop));
+}
diff --git a/drivers/gpu/drm/i915/display/intel_colorop.h b/drivers/gpu/drm/i915/display/intel_colorop.h
index 9276eee6e75a..638baf67d98d 100644
--- a/drivers/gpu/drm/i915/display/intel_colorop.h
+++ b/drivers/gpu/drm/i915/display/intel_colorop.h
@@ -13,5 +13,6 @@ struct intel_colorop;
 struct intel_colorop *to_intel_colorop(struct drm_colorop *colorop);
 struct intel_colorop *intel_colorop_alloc(void);
 struct intel_colorop *intel_colorop_create(enum intel_color_block id);
+void intel_colorop_destroy(struct drm_colorop *colorop);
 
 #endif /* __INTEL_COLOROP_H__ */
-- 
2.25.1

[PATCH v3 10/13] drm: Clean up colorop objects during mode_config cleanup

[Chaitanya Kumar Borah]

Tear down all registered drm_colorop objects during
drm_mode_config_cleanup() by invoking their destroy callbacks.

This ensures proper cleanup of color pipeline objects during DRM device
removal.

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
---
 drivers/gpu/drm/drm_mode_config.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/gpu/drm/drm_mode_config.c b/drivers/gpu/drm/drm_mode_config.c
index d12db9b0bab8..84ae8a23a367 100644
--- a/drivers/gpu/drm/drm_mode_config.c
+++ b/drivers/gpu/drm/drm_mode_config.c
@@ -524,6 +524,7 @@ void drm_mode_config_cleanup(struct drm_device *dev)
 	struct drm_property *property, *pt;
 	struct drm_property_blob *blob, *bt;
 	struct drm_plane *plane, *plt;
+	struct drm_colorop *colorop, *copt;
 
 	list_for_each_entry_safe(encoder, enct, &dev->mode_config.encoder_list,
 				 head) {
@@ -553,6 +554,11 @@ void drm_mode_config_cleanup(struct drm_device *dev)
 		drm_property_destroy(dev, property);
 	}
 
+	list_for_each_entry_safe(colorop, copt, &dev->mode_config.colorop_list,
+				 head) {
+		colorop->funcs->destroy(colorop);
+	}
+
 	list_for_each_entry_safe(plane, plt, &dev->mode_config.plane_list,
 				 head) {
 		plane->funcs->destroy(plane);
-- 
2.25.1

[PATCH v3 11/13] drm/vkms: Remove drm_colorop_pipeline_destroy() from vkms_destroy()

[Chaitanya Kumar Borah]

Now that colorops are cleaned from drm_mode_config_cleanup(), remove
drm_colorop_pipeline_destroy() from  vkms_destroy().

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Louis Chauvet <louis.chauvet@bootlin.com>
---
 drivers/gpu/drm/vkms/vkms_drv.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/vkms/vkms_drv.c b/drivers/gpu/drm/vkms/vkms_drv.c
index 434c295f44ba..95020765c4c2 100644
--- a/drivers/gpu/drm/vkms/vkms_drv.c
+++ b/drivers/gpu/drm/vkms/vkms_drv.c
@@ -259,7 +259,6 @@ void vkms_destroy(struct vkms_config *config)
 
 	fdev = config->dev->faux_dev;
 
-	drm_colorop_pipeline_destroy(&config->dev->drm);
 	drm_dev_unregister(&config->dev->drm);
 	drm_atomic_helper_shutdown(&config->dev->drm);
 	devres_release_group(&fdev->dev, NULL);
-- 
2.25.1

[PATCH v3 12/13] drm/colorop: Use destroy callback for color pipeline teardown

[Chaitanya Kumar Borah]

Switch drm_colorop_pipeline_destroy() to use the driver-provided
destroy callback instead of directly calling drm_colorop_cleanup()
and freeing the object.

This allows drivers that embed struct drm_colorop in driver-specific
objects to perform correct teardown.

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Reviewed-by: Alex Hung <alex.hung@amd.com>
---
 drivers/gpu/drm/drm_colorop.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_colorop.c b/drivers/gpu/drm/drm_colorop.c
index 2bce29176ab3..aa19de769eb2 100644
--- a/drivers/gpu/drm/drm_colorop.c
+++ b/drivers/gpu/drm/drm_colorop.c
@@ -208,8 +208,7 @@ void drm_colorop_pipeline_destroy(struct drm_device *dev)
 	struct drm_colorop *colorop, *next;
 
 	list_for_each_entry_safe(colorop, next, &config->colorop_list, head) {
-		drm_colorop_cleanup(colorop);
-		kfree(colorop);
+		colorop->funcs->destroy(colorop);
 	}
 }
 EXPORT_SYMBOL(drm_colorop_pipeline_destroy);
-- 
2.25.1

[PATCH v3 13/13] drm/i915/color: Add failure handling in plane color pipeline init

[Chaitanya Kumar Borah]

The plane color pipeline initialization built up multiple colorop blocks
inline, but did not reliably clean up partially constructed pipelines
when an intermediate step failed. This could lead to leaked colorop
objects and fragile error handling as the pipeline grows.

Refactor the pipeline construction to use a common helper for adding
colorop blocks. This centralizes allocation, initialization, and
teardown logic, allowing the caller to reliably unwind all previously
created colorops on failure.

v2:
 - Refactor code to avoid repetition (Suraj)

v3:
 - s/nvl/xe3plpd (Suraj)

Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
---
 .../drm/i915/display/intel_color_pipeline.c   | 164 +++++++++++++-----
 1 file changed, 117 insertions(+), 47 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_color_pipeline.c b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
index 8fecc53540ba..6cf8080ee800 100644
--- a/drivers/gpu/drm/i915/display/intel_color_pipeline.c
+++ b/drivers/gpu/drm/i915/display/intel_color_pipeline.c
@@ -2,6 +2,8 @@
 /*
  * Copyright © 2025 Intel Corporation
  */
+#include <drm/drm_print.h>
+
 #include "intel_color.h"
 #include "intel_colorop.h"
 #include "intel_color_pipeline.h"
@@ -10,6 +12,7 @@
 #include "skl_universal_plane.h"
 
 #define MAX_COLOR_PIPELINES 1
+#define MAX_COLOROP 4
 #define PLANE_DEGAMMA_SIZE 128
 #define PLANE_GAMMA_SIZE 32
 
@@ -17,70 +20,137 @@ static const struct drm_colorop_funcs intel_colorop_funcs = {
 	.destroy = intel_colorop_destroy,
 };
 
+/*
+ * 3DLUT can be bound to all three HDR planes. However, even with the latest
+ * color pipeline UAPI, there is no good way to represent a HW block which
+ * can be shared/attached at different stages of the pipeline. So right now,
+ * we expose 3DLUT only attached with the primary plane.
+ *
+ * That way we don't confuse the userspace with opaque commit failures
+ * on trying to enable it on multiple planes which would otherwise make
+ * the pipeline totally unusable.
+ */
+static const enum intel_color_block xe3plpd_primary_plane_pipeline[] = {
+	INTEL_PLANE_CB_PRE_CSC_LUT,
+	INTEL_PLANE_CB_CSC,
+	INTEL_PLANE_CB_3DLUT,
+	INTEL_PLANE_CB_POST_CSC_LUT,
+};
+
+static const enum intel_color_block hdr_plane_pipeline[] = {
+	INTEL_PLANE_CB_PRE_CSC_LUT,
+	INTEL_PLANE_CB_CSC,
+	INTEL_PLANE_CB_POST_CSC_LUT,
+};
+
+static bool plane_has_3dlut(struct intel_display *display, enum pipe pipe,
+			    struct drm_plane *plane)
+{
+	return (DISPLAY_VER(display) >= 35 &&
+		intel_color_crtc_has_3dlut(display, pipe) &&
+		plane->type == DRM_PLANE_TYPE_PRIMARY);
+}
+
 static
-int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_enum_list *list,
-				     enum pipe pipe)
+struct intel_colorop *intel_color_pipeline_plane_add_colorop(struct drm_plane *plane,
+							     struct intel_colorop *prev,
+							     enum intel_color_block id)
 {
 	struct drm_device *dev = plane->dev;
-	struct intel_display *display = to_intel_display(dev);
-	struct drm_colorop *prev_op;
 	struct intel_colorop *colorop;
 	int ret;
 
-	colorop = intel_colorop_create(INTEL_PLANE_CB_PRE_CSC_LUT);
-
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, &intel_colorop_funcs,
-						  PLANE_DEGAMMA_SIZE,
-						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
-						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
-
-	if (ret)
-		return ret;
-
-	list->type = colorop->base.base.id;
-
-	/* TODO: handle failures and clean up */
-	prev_op = &colorop->base;
-
-	colorop = intel_colorop_create(INTEL_PLANE_CB_CSC);
-	ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane, &intel_colorop_funcs,
-					     DRM_COLOROP_FLAG_ALLOW_BYPASS);
-	if (ret)
-		return ret;
-
-	drm_colorop_set_next_property(prev_op, &colorop->base);
-	prev_op = &colorop->base;
-
-	if (DISPLAY_VER(display) >= 35 &&
-	    intel_color_crtc_has_3dlut(display, pipe) &&
-	    plane->type == DRM_PLANE_TYPE_PRIMARY) {
-		colorop = intel_colorop_create(INTEL_PLANE_CB_3DLUT);
-
+	colorop = intel_colorop_create(id);
+
+	if (IS_ERR(colorop))
+		return colorop;
+
+	switch (id) {
+	case INTEL_PLANE_CB_PRE_CSC_LUT:
+		ret = drm_plane_colorop_curve_1d_lut_init(dev,
+							  &colorop->base, plane,
+							  &intel_colorop_funcs,
+							  PLANE_DEGAMMA_SIZE,
+							  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
+							  DRM_COLOROP_FLAG_ALLOW_BYPASS);
+		break;
+	case INTEL_PLANE_CB_CSC:
+		ret = drm_plane_colorop_ctm_3x4_init(dev, &colorop->base, plane,
+						     &intel_colorop_funcs,
+						     DRM_COLOROP_FLAG_ALLOW_BYPASS);
+		break;
+	case INTEL_PLANE_CB_3DLUT:
 		ret = drm_plane_colorop_3dlut_init(dev, &colorop->base, plane,
 						   &intel_colorop_funcs, 17,
 						   DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,
 						   true);
-		if (ret)
-			return ret;
-
-		drm_colorop_set_next_property(prev_op, &colorop->base);
-
-		prev_op = &colorop->base;
+		break;
+	case INTEL_PLANE_CB_POST_CSC_LUT:
+		ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane,
+							  &intel_colorop_funcs,
+							  PLANE_GAMMA_SIZE,
+							  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
+							  DRM_COLOROP_FLAG_ALLOW_BYPASS);
+		break;
+	default:
+		drm_err(plane->dev, "Invalid colorop id [%d]", id);
+		ret = -EINVAL;
 	}
 
-	colorop = intel_colorop_create(INTEL_PLANE_CB_POST_CSC_LUT);
-	ret = drm_plane_colorop_curve_1d_lut_init(dev, &colorop->base, plane, &intel_colorop_funcs,
-						  PLANE_GAMMA_SIZE,
-						  DRM_COLOROP_LUT1D_INTERPOLATION_LINEAR,
-						  DRM_COLOROP_FLAG_ALLOW_BYPASS);
 	if (ret)
-		return ret;
+		goto cleanup;
 
-	drm_colorop_set_next_property(prev_op, &colorop->base);
+	if (prev)
+		drm_colorop_set_next_property(&prev->base, &colorop->base);
 
-	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", list->type);
+	return colorop;
+
+cleanup:
+	intel_colorop_destroy(&colorop->base);
+	return ERR_PTR(ret);
+}
+
+static
+int _intel_color_pipeline_plane_init(struct drm_plane *plane, struct drm_prop_enum_list *list,
+				     enum pipe pipe)
+{
+	struct drm_device *dev = plane->dev;
+	struct intel_display *display = to_intel_display(dev);
+	struct intel_colorop *colorop[MAX_COLOROP];
+	struct intel_colorop *prev = NULL;
+	const enum intel_color_block *pipeline;
+	int pipeline_len;
+	int ret = 0;
+	int i;
+
+	if (plane_has_3dlut(display, pipe, plane)) {
+		pipeline = xe3plpd_primary_plane_pipeline;
+		pipeline_len = ARRAY_SIZE(xe3plpd_primary_plane_pipeline);
+	} else {
+		pipeline = hdr_plane_pipeline;
+		pipeline_len = ARRAY_SIZE(hdr_plane_pipeline);
+	}
+
+	for (i = 0; i < pipeline_len; i++) {
+		colorop[i] = intel_color_pipeline_plane_add_colorop(plane, prev,
+								    pipeline[i]);
+		if (IS_ERR(colorop[i])) {
+			ret = PTR_ERR(colorop[i]);
+			goto cleanup;
+		}
+
+		prev = colorop[i];
+	}
+
+	list->type = colorop[0]->base.base.id;
+	list->name = kasprintf(GFP_KERNEL, "Color Pipeline %d", colorop[0]->base.base.id);
 
 	return 0;
+
+cleanup:
+	while (--i >= 0)
+		intel_colorop_destroy(&colorop[i]->base);
+	return ret;
 }
 
 int intel_color_pipeline_plane_init(struct drm_plane *plane, enum pipe pipe)
-- 
2.25.1

✗ CI.checkpatch: warning for drm: Color pipeline teardown and follow-up fixes/improvements (rev3)

[Patchwork]

== Series Details ==

Series: drm: Color pipeline teardown and follow-up fixes/improvements (rev3)
URL   : https://patchwork.freedesktop.org/series/159263/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
ee83616c430ce70bd254bd2774d143a5733c8666
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit c73cf14db2c0ba24b1f3c82e3ef9a4c7ab142669
Author: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Date:   Tue Jan 13 15:53:03 2026 +0530

    drm/i915/color: Add failure handling in plane color pipeline init
    
    The plane color pipeline initialization built up multiple colorop blocks
    inline, but did not reliably clean up partially constructed pipelines
    when an intermediate step failed. This could lead to leaked colorop
    objects and fragile error handling as the pipeline grows.
    
    Refactor the pipeline construction to use a common helper for adding
    colorop blocks. This centralizes allocation, initialization, and
    teardown logic, allowing the caller to reliably unwind all previously
    created colorops on failure.
    
    v2:
     - Refactor code to avoid repetition (Suraj)
    
    v3:
     - s/nvl/xe3plpd (Suraj)
    
    Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
    Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
+ /mt/dim checkpatch f7424dfbcb8f05a13bb1da2bed8e2212d2b7a082 drm-intel
f166843fe069 drm/i915/color: Place 3D LUT after CSC in plane color pipeline
082162cf6dac drm/amd/display: Fix color pipeline enum name leak
a1b78ac88836 drm/vkms: Fix color pipeline enum name leak
adae6513f14a drm/i915/display: Fix color pipeline enum name leak
57a50762145f drm/colorop: Add destroy helper for colorop objects
d30796634823 drm: Allow driver-managed destruction of colorop objects
-:58: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#58: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c:125:
+		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, NULL,
 						amdgpu_dm_supported_shaper_tfs,

-:67: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#67: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c:142:
+		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane, NULL,
+							MAX_COLOR_LUT_ENTRIES,

-:77: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#77: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c:160:
+		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane, NULL, LUT3D_SIZE,
 					DRM_COLOROP_LUT3D_INTERPOLATION_TETRAHEDRAL,

-:161: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
#161: FILE: drivers/gpu/drm/drm_colorop.c:319:
+					uint32_t lut_size,

-:352: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
#352: FILE: include/drm/drm_colorop.h:413:
+					uint32_t lut_size,

total: 0 errors, 0 warnings, 5 checks, 300 lines checked
f5fe63ab7aa5 drm/amd/display: Hook up colorop destroy helper for plane pipelines
-:65: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#65: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c:131:
+		ret = drm_plane_colorop_curve_1d_init(dev, ops[i], plane, &dm_colorop_funcs,
 						amdgpu_dm_supported_shaper_tfs,

-:74: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#74: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c:148:
+		ret = drm_plane_colorop_curve_1d_lut_init(dev, ops[i], plane,
+							&dm_colorop_funcs,

-:84: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#84: FILE: drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c:167:
+		ret = drm_plane_colorop_3dlut_init(dev, ops[i], plane,
+					&dm_colorop_funcs, LUT3D_SIZE,

total: 0 errors, 0 warnings, 3 checks, 79 lines checked
9dd9367e1fd7 drm/vkms: Hook up colorop destroy helper for plane pipelines
188831a5f26a drm/i915/display: Hook up intel_colorop_destroy
cacb529a82d3 drm: Clean up colorop objects during mode_config cleanup
f8d27de88de5 drm/vkms: Remove drm_colorop_pipeline_destroy() from vkms_destroy()
354a590c9861 drm/colorop: Use destroy callback for color pipeline teardown
c73cf14db2c0 drm/i915/color: Add failure handling in plane color pipeline init

✓ CI.KUnit: success for drm: Color pipeline teardown and follow-up fixes/improvements (rev3)

[Patchwork]

== Series Details ==

Series: drm: Color pipeline teardown and follow-up fixes/improvements (rev3)
URL   : https://patchwork.freedesktop.org/series/159263/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[11:17:55] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[11:18:00] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[11:18:39] Starting KUnit Kernel (1/1)...
[11:18:39] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[11:18:39] ================== guc_buf (11 subtests) ===================
[11:18:39] [PASSED] test_smallest
[11:18:39] [PASSED] test_largest
[11:18:39] [PASSED] test_granular
[11:18:39] [PASSED] test_unique
[11:18:39] [PASSED] test_overlap
[11:18:39] [PASSED] test_reusable
[11:18:39] [PASSED] test_too_big
[11:18:39] [PASSED] test_flush
[11:18:39] [PASSED] test_lookup
[11:18:39] [PASSED] test_data
[11:18:39] [PASSED] test_class
[11:18:39] ===================== [PASSED] guc_buf =====================
[11:18:39] =================== guc_dbm (7 subtests) ===================
[11:18:39] [PASSED] test_empty
[11:18:39] [PASSED] test_default
[11:18:39] ======================== test_size  ========================
[11:18:39] [PASSED] 4
[11:18:39] [PASSED] 8
[11:18:39] [PASSED] 32
[11:18:39] [PASSED] 256
[11:18:39] ==================== [PASSED] test_size ====================
[11:18:39] ======================= test_reuse  ========================
[11:18:39] [PASSED] 4
[11:18:39] [PASSED] 8
[11:18:39] [PASSED] 32
[11:18:39] [PASSED] 256
[11:18:39] =================== [PASSED] test_reuse ====================
[11:18:39] =================== test_range_overlap  ====================
[11:18:39] [PASSED] 4
[11:18:39] [PASSED] 8
[11:18:39] [PASSED] 32
[11:18:39] [PASSED] 256
[11:18:39] =============== [PASSED] test_range_overlap ================
[11:18:39] =================== test_range_compact  ====================
[11:18:39] [PASSED] 4
[11:18:39] [PASSED] 8
[11:18:39] [PASSED] 32
[11:18:39] [PASSED] 256
[11:18:39] =============== [PASSED] test_range_compact ================
[11:18:39] ==================== test_range_spare  =====================
[11:18:39] [PASSED] 4
[11:18:39] [PASSED] 8
[11:18:39] [PASSED] 32
[11:18:39] [PASSED] 256
[11:18:39] ================ [PASSED] test_range_spare =================
[11:18:39] ===================== [PASSED] guc_dbm =====================
[11:18:39] =================== guc_idm (6 subtests) ===================
[11:18:39] [PASSED] bad_init
[11:18:39] [PASSED] no_init
[11:18:39] [PASSED] init_fini
[11:18:39] [PASSED] check_used
[11:18:39] [PASSED] check_quota
[11:18:39] [PASSED] check_all
[11:18:39] ===================== [PASSED] guc_idm =====================
[11:18:39] ================== no_relay (3 subtests) ===================
[11:18:39] [PASSED] xe_drops_guc2pf_if_not_ready
[11:18:39] [PASSED] xe_drops_guc2vf_if_not_ready
[11:18:39] [PASSED] xe_rejects_send_if_not_ready
[11:18:39] ==================== [PASSED] no_relay =====================
[11:18:39] ================== pf_relay (14 subtests) ==================
[11:18:39] [PASSED] pf_rejects_guc2pf_too_short
[11:18:39] [PASSED] pf_rejects_guc2pf_too_long
[11:18:39] [PASSED] pf_rejects_guc2pf_no_payload
[11:18:39] [PASSED] pf_fails_no_payload
[11:18:39] [PASSED] pf_fails_bad_origin
[11:18:39] [PASSED] pf_fails_bad_type
[11:18:39] [PASSED] pf_txn_reports_error
[11:18:39] [PASSED] pf_txn_sends_pf2guc
[11:18:39] [PASSED] pf_sends_pf2guc
[11:18:39] [SKIPPED] pf_loopback_nop
[11:18:39] [SKIPPED] pf_loopback_echo
[11:18:39] [SKIPPED] pf_loopback_fail
[11:18:39] [SKIPPED] pf_loopback_busy
[11:18:39] [SKIPPED] pf_loopback_retry
[11:18:39] ==================== [PASSED] pf_relay =====================
[11:18:39] ================== vf_relay (3 subtests) ===================
[11:18:39] [PASSED] vf_rejects_guc2vf_too_short
[11:18:39] [PASSED] vf_rejects_guc2vf_too_long
[11:18:39] [PASSED] vf_rejects_guc2vf_no_payload
[11:18:39] ==================== [PASSED] vf_relay =====================
[11:18:39] ================ pf_gt_config (6 subtests) =================
[11:18:39] [PASSED] fair_contexts_1vf
[11:18:39] [PASSED] fair_doorbells_1vf
[11:18:39] [PASSED] fair_ggtt_1vf
[11:18:39] ====================== fair_contexts  ======================
[11:18:39] [PASSED] 1 VF
[11:18:39] [PASSED] 2 VFs
[11:18:39] [PASSED] 3 VFs
[11:18:39] [PASSED] 4 VFs
[11:18:39] [PASSED] 5 VFs
[11:18:39] [PASSED] 6 VFs
[11:18:39] [PASSED] 7 VFs
[11:18:39] [PASSED] 8 VFs
[11:18:39] [PASSED] 9 VFs
[11:18:39] [PASSED] 10 VFs
[11:18:39] [PASSED] 11 VFs
[11:18:39] [PASSED] 12 VFs
[11:18:39] [PASSED] 13 VFs
[11:18:39] [PASSED] 14 VFs
[11:18:39] [PASSED] 15 VFs
[11:18:39] [PASSED] 16 VFs
[11:18:39] [PASSED] 17 VFs
[11:18:39] [PASSED] 18 VFs
[11:18:39] [PASSED] 19 VFs
[11:18:39] [PASSED] 20 VFs
[11:18:39] [PASSED] 21 VFs
[11:18:39] [PASSED] 22 VFs
[11:18:39] [PASSED] 23 VFs
[11:18:39] [PASSED] 24 VFs
[11:18:39] [PASSED] 25 VFs
[11:18:39] [PASSED] 26 VFs
[11:18:39] [PASSED] 27 VFs
[11:18:39] [PASSED] 28 VFs
[11:18:39] [PASSED] 29 VFs
[11:18:39] [PASSED] 30 VFs
[11:18:39] [PASSED] 31 VFs
[11:18:39] [PASSED] 32 VFs
[11:18:39] [PASSED] 33 VFs
[11:18:39] [PASSED] 34 VFs
[11:18:39] [PASSED] 35 VFs
[11:18:39] [PASSED] 36 VFs
[11:18:39] [PASSED] 37 VFs
[11:18:39] [PASSED] 38 VFs
[11:18:39] [PASSED] 39 VFs
[11:18:39] [PASSED] 40 VFs
[11:18:39] [PASSED] 41 VFs
[11:18:39] [PASSED] 42 VFs
[11:18:39] [PASSED] 43 VFs
[11:18:39] [PASSED] 44 VFs
[11:18:39] [PASSED] 45 VFs
[11:18:39] [PASSED] 46 VFs
[11:18:39] [PASSED] 47 VFs
[11:18:39] [PASSED] 48 VFs
[11:18:39] [PASSED] 49 VFs
[11:18:39] [PASSED] 50 VFs
[11:18:39] [PASSED] 51 VFs
[11:18:39] [PASSED] 52 VFs
[11:18:39] [PASSED] 53 VFs
[11:18:39] [PASSED] 54 VFs
[11:18:39] [PASSED] 55 VFs
[11:18:39] [PASSED] 56 VFs
[11:18:39] [PASSED] 57 VFs
[11:18:39] [PASSED] 58 VFs
[11:18:39] [PASSED] 59 VFs
[11:18:39] [PASSED] 60 VFs
[11:18:39] [PASSED] 61 VFs
[11:18:39] [PASSED] 62 VFs
[11:18:39] [PASSED] 63 VFs
[11:18:39] ================== [PASSED] fair_contexts ==================
[11:18:39] ===================== fair_doorbells  ======================
[11:18:39] [PASSED] 1 VF
[11:18:39] [PASSED] 2 VFs
[11:18:39] [PASSED] 3 VFs
[11:18:39] [PASSED] 4 VFs
[11:18:39] [PASSED] 5 VFs
[11:18:39] [PASSED] 6 VFs
[11:18:39] [PASSED] 7 VFs
[11:18:39] [PASSED] 8 VFs
[11:18:39] [PASSED] 9 VFs
[11:18:39] [PASSED] 10 VFs
[11:18:39] [PASSED] 11 VFs
[11:18:39] [PASSED] 12 VFs
[11:18:39] [PASSED] 13 VFs
[11:18:39] [PASSED] 14 VFs
[11:18:39] [PASSED] 15 VFs
[11:18:39] [PASSED] 16 VFs
[11:18:39] [PASSED] 17 VFs
[11:18:39] [PASSED] 18 VFs
[11:18:39] [PASSED] 19 VFs
[11:18:39] [PASSED] 20 VFs
[11:18:39] [PASSED] 21 VFs
[11:18:39] [PASSED] 22 VFs
[11:18:39] [PASSED] 23 VFs
[11:18:39] [PASSED] 24 VFs
[11:18:39] [PASSED] 25 VFs
[11:18:39] [PASSED] 26 VFs
[11:18:39] [PASSED] 27 VFs
[11:18:39] [PASSED] 28 VFs
[11:18:39] [PASSED] 29 VFs
[11:18:39] [PASSED] 30 VFs
[11:18:39] [PASSED] 31 VFs
[11:18:39] [PASSED] 32 VFs
[11:18:39] [PASSED] 33 VFs
[11:18:39] [PASSED] 34 VFs
[11:18:39] [PASSED] 35 VFs
[11:18:39] [PASSED] 36 VFs
[11:18:39] [PASSED] 37 VFs
[11:18:39] [PASSED] 38 VFs
[11:18:39] [PASSED] 39 VFs
[11:18:39] [PASSED] 40 VFs
[11:18:39] [PASSED] 41 VFs
[11:18:39] [PASSED] 42 VFs
[11:18:39] [PASSED] 43 VFs
[11:18:39] [PASSED] 44 VFs
[11:18:39] [PASSED] 45 VFs
[11:18:39] [PASSED] 46 VFs
[11:18:39] [PASSED] 47 VFs
[11:18:39] [PASSED] 48 VFs
[11:18:39] [PASSED] 49 VFs
[11:18:39] [PASSED] 50 VFs
[11:18:39] [PASSED] 51 VFs
[11:18:39] [PASSED] 52 VFs
[11:18:39] [PASSED] 53 VFs
[11:18:39] [PASSED] 54 VFs
[11:18:39] [PASSED] 55 VFs
[11:18:39] [PASSED] 56 VFs
[11:18:39] [PASSED] 57 VFs
[11:18:39] [PASSED] 58 VFs
[11:18:39] [PASSED] 59 VFs
[11:18:39] [PASSED] 60 VFs
[11:18:39] [PASSED] 61 VFs
[11:18:39] [PASSED] 62 VFs
[11:18:39] [PASSED] 63 VFs
[11:18:39] ================= [PASSED] fair_doorbells ==================
[11:18:39] ======================== fair_ggtt  ========================
[11:18:39] [PASSED] 1 VF
[11:18:39] [PASSED] 2 VFs
[11:18:39] [PASSED] 3 VFs
[11:18:39] [PASSED] 4 VFs
[11:18:39] [PASSED] 5 VFs
[11:18:39] [PASSED] 6 VFs
[11:18:39] [PASSED] 7 VFs
[11:18:39] [PASSED] 8 VFs
[11:18:39] [PASSED] 9 VFs
[11:18:39] [PASSED] 10 VFs
[11:18:39] [PASSED] 11 VFs
[11:18:39] [PASSED] 12 VFs
[11:18:39] [PASSED] 13 VFs
[11:18:39] [PASSED] 14 VFs
[11:18:39] [PASSED] 15 VFs
[11:18:39] [PASSED] 16 VFs
[11:18:39] [PASSED] 17 VFs
[11:18:39] [PASSED] 18 VFs
[11:18:39] [PASSED] 19 VFs
[11:18:39] [PASSED] 20 VFs
[11:18:39] [PASSED] 21 VFs
[11:18:39] [PASSED] 22 VFs
[11:18:39] [PASSED] 23 VFs
[11:18:39] [PASSED] 24 VFs
[11:18:39] [PASSED] 25 VFs
[11:18:39] [PASSED] 26 VFs
[11:18:39] [PASSED] 27 VFs
[11:18:39] [PASSED] 28 VFs
[11:18:39] [PASSED] 29 VFs
[11:18:39] [PASSED] 30 VFs
[11:18:39] [PASSED] 31 VFs
[11:18:39] [PASSED] 32 VFs
[11:18:39] [PASSED] 33 VFs
[11:18:39] [PASSED] 34 VFs
[11:18:39] [PASSED] 35 VFs
[11:18:39] [PASSED] 36 VFs
[11:18:39] [PASSED] 37 VFs
[11:18:39] [PASSED] 38 VFs
[11:18:39] [PASSED] 39 VFs
[11:18:39] [PASSED] 40 VFs
[11:18:39] [PASSED] 41 VFs
[11:18:39] [PASSED] 42 VFs
[11:18:39] [PASSED] 43 VFs
[11:18:39] [PASSED] 44 VFs
[11:18:39] [PASSED] 45 VFs
[11:18:39] [PASSED] 46 VFs
[11:18:39] [PASSED] 47 VFs
[11:18:39] [PASSED] 48 VFs
[11:18:39] [PASSED] 49 VFs
[11:18:39] [PASSED] 50 VFs
[11:18:39] [PASSED] 51 VFs
[11:18:39] [PASSED] 52 VFs
[11:18:39] [PASSED] 53 VFs
[11:18:39] [PASSED] 54 VFs
[11:18:39] [PASSED] 55 VFs
[11:18:39] [PASSED] 56 VFs
[11:18:39] [PASSED] 57 VFs
[11:18:39] [PASSED] 58 VFs
[11:18:39] [PASSED] 59 VFs
[11:18:39] [PASSED] 60 VFs
[11:18:39] [PASSED] 61 VFs
[11:18:39] [PASSED] 62 VFs
[11:18:39] [PASSED] 63 VFs
[11:18:39] ==================== [PASSED] fair_ggtt ====================
[11:18:39] ================== [PASSED] pf_gt_config ===================
[11:18:39] ===================== lmtt (1 subtest) =====================
[11:18:39] ======================== test_ops  =========================
[11:18:39] [PASSED] 2-level
[11:18:39] [PASSED] multi-level
[11:18:39] ==================== [PASSED] test_ops =====================
[11:18:39] ====================== [PASSED] lmtt =======================
[11:18:39] ================= pf_service (11 subtests) =================
[11:18:39] [PASSED] pf_negotiate_any
[11:18:39] [PASSED] pf_negotiate_base_match
[11:18:39] [PASSED] pf_negotiate_base_newer
[11:18:39] [PASSED] pf_negotiate_base_next
[11:18:39] [SKIPPED] pf_negotiate_base_older
[11:18:39] [PASSED] pf_negotiate_base_prev
[11:18:39] [PASSED] pf_negotiate_latest_match
[11:18:39] [PASSED] pf_negotiate_latest_newer
[11:18:39] [PASSED] pf_negotiate_latest_next
[11:18:39] [SKIPPED] pf_negotiate_latest_older
[11:18:39] [SKIPPED] pf_negotiate_latest_prev
[11:18:39] =================== [PASSED] pf_service ====================
[11:18:39] ================= xe_guc_g2g (2 subtests) ==================
[11:18:39] ============== xe_live_guc_g2g_kunit_default  ==============
[11:18:39] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[11:18:39] ============== xe_live_guc_g2g_kunit_allmem  ===============
[11:18:39] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[11:18:39] =================== [SKIPPED] xe_guc_g2g ===================
[11:18:39] =================== xe_mocs (2 subtests) ===================
[11:18:39] ================ xe_live_mocs_kernel_kunit  ================
[11:18:39] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[11:18:39] ================ xe_live_mocs_reset_kunit  =================
[11:18:39] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[11:18:39] ==================== [SKIPPED] xe_mocs =====================
[11:18:39] ================= xe_migrate (2 subtests) ==================
[11:18:39] ================= xe_migrate_sanity_kunit  =================
[11:18:39] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[11:18:39] ================== xe_validate_ccs_kunit  ==================
[11:18:39] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[11:18:39] =================== [SKIPPED] xe_migrate ===================
[11:18:39] ================== xe_dma_buf (1 subtest) ==================
[11:18:39] ==================== xe_dma_buf_kunit  =====================
[11:18:39] ================ [SKIPPED] xe_dma_buf_kunit ================
[11:18:39] =================== [SKIPPED] xe_dma_buf ===================
[11:18:39] ================= xe_bo_shrink (1 subtest) =================
[11:18:39] =================== xe_bo_shrink_kunit  ====================
[11:18:39] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[11:18:39] ================== [SKIPPED] xe_bo_shrink ==================
[11:18:39] ==================== xe_bo (2 subtests) ====================
[11:18:39] ================== xe_ccs_migrate_kunit  ===================
[11:18:39] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[11:18:39] ==================== xe_bo_evict_kunit  ====================
[11:18:39] =============== [SKIPPED] xe_bo_evict_kunit ================
[11:18:39] ===================== [SKIPPED] xe_bo ======================
[11:18:39] ==================== args (13 subtests) ====================
[11:18:39] [PASSED] count_args_test
[11:18:39] [PASSED] call_args_example
[11:18:39] [PASSED] call_args_test
[11:18:39] [PASSED] drop_first_arg_example
[11:18:39] [PASSED] drop_first_arg_test
[11:18:39] [PASSED] first_arg_example
[11:18:39] [PASSED] first_arg_test
[11:18:39] [PASSED] last_arg_example
[11:18:39] [PASSED] last_arg_test
[11:18:39] [PASSED] pick_arg_example
[11:18:39] [PASSED] if_args_example
[11:18:39] [PASSED] if_args_test
[11:18:39] [PASSED] sep_comma_example
[11:18:39] ====================== [PASSED] args =======================
[11:18:39] =================== xe_pci (3 subtests) ====================
[11:18:39] ==================== check_graphics_ip  ====================
[11:18:39] [PASSED] 12.00 Xe_LP
[11:18:39] [PASSED] 12.10 Xe_LP+
[11:18:39] [PASSED] 12.55 Xe_HPG
[11:18:39] [PASSED] 12.60 Xe_HPC
[11:18:39] [PASSED] 12.70 Xe_LPG
[11:18:39] [PASSED] 12.71 Xe_LPG
[11:18:39] [PASSED] 12.74 Xe_LPG+
[11:18:39] [PASSED] 20.01 Xe2_HPG
[11:18:39] [PASSED] 20.02 Xe2_HPG
[11:18:39] [PASSED] 20.04 Xe2_LPG
[11:18:39] [PASSED] 30.00 Xe3_LPG
[11:18:39] [PASSED] 30.01 Xe3_LPG
[11:18:39] [PASSED] 30.03 Xe3_LPG
[11:18:39] [PASSED] 30.04 Xe3_LPG
[11:18:39] [PASSED] 30.05 Xe3_LPG
[11:18:39] [PASSED] 35.11 Xe3p_XPC
[11:18:39] ================ [PASSED] check_graphics_ip ================
[11:18:39] ===================== check_media_ip  ======================
[11:18:39] [PASSED] 12.00 Xe_M
[11:18:39] [PASSED] 12.55 Xe_HPM
[11:18:39] [PASSED] 13.00 Xe_LPM+
[11:18:39] [PASSED] 13.01 Xe2_HPM
[11:18:39] [PASSED] 20.00 Xe2_LPM
[11:18:39] [PASSED] 30.00 Xe3_LPM
[11:18:39] [PASSED] 30.02 Xe3_LPM
[11:18:39] [PASSED] 35.00 Xe3p_LPM
[11:18:39] [PASSED] 35.03 Xe3p_HPM
[11:18:39] ================= [PASSED] check_media_ip ==================
[11:18:39] =================== check_platform_desc  ===================
[11:18:39] [PASSED] 0x9A60 (TIGERLAKE)
[11:18:39] [PASSED] 0x9A68 (TIGERLAKE)
[11:18:39] [PASSED] 0x9A70 (TIGERLAKE)
[11:18:39] [PASSED] 0x9A40 (TIGERLAKE)
[11:18:39] [PASSED] 0x9A49 (TIGERLAKE)
[11:18:39] [PASSED] 0x9A59 (TIGERLAKE)
[11:18:39] [PASSED] 0x9A78 (TIGERLAKE)
[11:18:39] [PASSED] 0x9AC0 (TIGERLAKE)
[11:18:39] [PASSED] 0x9AC9 (TIGERLAKE)
[11:18:39] [PASSED] 0x9AD9 (TIGERLAKE)
[11:18:39] [PASSED] 0x9AF8 (TIGERLAKE)
[11:18:39] [PASSED] 0x4C80 (ROCKETLAKE)
[11:18:39] [PASSED] 0x4C8A (ROCKETLAKE)
[11:18:39] [PASSED] 0x4C8B (ROCKETLAKE)
[11:18:39] [PASSED] 0x4C8C (ROCKETLAKE)
[11:18:39] [PASSED] 0x4C90 (ROCKETLAKE)
[11:18:39] [PASSED] 0x4C9A (ROCKETLAKE)
[11:18:39] [PASSED] 0x4680 (ALDERLAKE_S)
[11:18:39] [PASSED] 0x4682 (ALDERLAKE_S)
[11:18:39] [PASSED] 0x4688 (ALDERLAKE_S)
[11:18:39] [PASSED] 0x468A (ALDERLAKE_S)
[11:18:39] [PASSED] 0x468B (ALDERLAKE_S)
[11:18:39] [PASSED] 0x4690 (ALDERLAKE_S)
[11:18:39] [PASSED] 0x4692 (ALDERLAKE_S)
[11:18:39] [PASSED] 0x4693 (ALDERLAKE_S)
[11:18:39] [PASSED] 0x46A0 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46A1 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46A2 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46A3 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46A6 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46A8 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46AA (ALDERLAKE_P)
[11:18:39] [PASSED] 0x462A (ALDERLAKE_P)
[11:18:39] [PASSED] 0x4626 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x4628 (ALDERLAKE_P)
stty: 'standard input': Inappropriate ioctl for device
[11:18:39] [PASSED] 0x46B0 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46B1 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46B2 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46B3 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46C0 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46C1 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46C2 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46C3 (ALDERLAKE_P)
[11:18:39] [PASSED] 0x46D0 (ALDERLAKE_N)
[11:18:39] [PASSED] 0x46D1 (ALDERLAKE_N)
[11:18:39] [PASSED] 0x46D2 (ALDERLAKE_N)
[11:18:39] [PASSED] 0x46D3 (ALDERLAKE_N)
[11:18:39] [PASSED] 0x46D4 (ALDERLAKE_N)
[11:18:39] [PASSED] 0xA721 (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7A1 (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7A9 (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7AC (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7AD (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA720 (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7A0 (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7A8 (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7AA (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA7AB (ALDERLAKE_P)
[11:18:39] [PASSED] 0xA780 (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA781 (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA782 (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA783 (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA788 (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA789 (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA78A (ALDERLAKE_S)
[11:18:39] [PASSED] 0xA78B (ALDERLAKE_S)
[11:18:39] [PASSED] 0x4905 (DG1)
[11:18:39] [PASSED] 0x4906 (DG1)
[11:18:39] [PASSED] 0x4907 (DG1)
[11:18:39] [PASSED] 0x4908 (DG1)
[11:18:39] [PASSED] 0x4909 (DG1)
[11:18:39] [PASSED] 0x56C0 (DG2)
[11:18:39] [PASSED] 0x56C2 (DG2)
[11:18:39] [PASSED] 0x56C1 (DG2)
[11:18:39] [PASSED] 0x7D51 (METEORLAKE)
[11:18:39] [PASSED] 0x7DD1 (METEORLAKE)
[11:18:39] [PASSED] 0x7D41 (METEORLAKE)
[11:18:39] [PASSED] 0x7D67 (METEORLAKE)
[11:18:39] [PASSED] 0xB640 (METEORLAKE)
[11:18:39] [PASSED] 0x56A0 (DG2)
[11:18:39] [PASSED] 0x56A1 (DG2)
[11:18:39] [PASSED] 0x56A2 (DG2)
[11:18:39] [PASSED] 0x56BE (DG2)
[11:18:39] [PASSED] 0x56BF (DG2)
[11:18:39] [PASSED] 0x5690 (DG2)
[11:18:39] [PASSED] 0x5691 (DG2)
[11:18:39] [PASSED] 0x5692 (DG2)
[11:18:39] [PASSED] 0x56A5 (DG2)
[11:18:39] [PASSED] 0x56A6 (DG2)
[11:18:39] [PASSED] 0x56B0 (DG2)
[11:18:39] [PASSED] 0x56B1 (DG2)
[11:18:39] [PASSED] 0x56BA (DG2)
[11:18:39] [PASSED] 0x56BB (DG2)
[11:18:39] [PASSED] 0x56BC (DG2)
[11:18:39] [PASSED] 0x56BD (DG2)
[11:18:39] [PASSED] 0x5693 (DG2)
[11:18:39] [PASSED] 0x5694 (DG2)
[11:18:39] [PASSED] 0x5695 (DG2)
[11:18:39] [PASSED] 0x56A3 (DG2)
[11:18:39] [PASSED] 0x56A4 (DG2)
[11:18:39] [PASSED] 0x56B2 (DG2)
[11:18:39] [PASSED] 0x56B3 (DG2)
[11:18:39] [PASSED] 0x5696 (DG2)
[11:18:39] [PASSED] 0x5697 (DG2)
[11:18:39] [PASSED] 0xB69 (PVC)
[11:18:39] [PASSED] 0xB6E (PVC)
[11:18:39] [PASSED] 0xBD4 (PVC)
[11:18:39] [PASSED] 0xBD5 (PVC)
[11:18:39] [PASSED] 0xBD6 (PVC)
[11:18:39] [PASSED] 0xBD7 (PVC)
[11:18:39] [PASSED] 0xBD8 (PVC)
[11:18:39] [PASSED] 0xBD9 (PVC)
[11:18:39] [PASSED] 0xBDA (PVC)
[11:18:39] [PASSED] 0xBDB (PVC)
[11:18:39] [PASSED] 0xBE0 (PVC)
[11:18:39] [PASSED] 0xBE1 (PVC)
[11:18:39] [PASSED] 0xBE5 (PVC)
[11:18:39] [PASSED] 0x7D40 (METEORLAKE)
[11:18:39] [PASSED] 0x7D45 (METEORLAKE)
[11:18:39] [PASSED] 0x7D55 (METEORLAKE)
[11:18:39] [PASSED] 0x7D60 (METEORLAKE)
[11:18:39] [PASSED] 0x7DD5 (METEORLAKE)
[11:18:39] [PASSED] 0x6420 (LUNARLAKE)
[11:18:39] [PASSED] 0x64A0 (LUNARLAKE)
[11:18:39] [PASSED] 0x64B0 (LUNARLAKE)
[11:18:39] [PASSED] 0xE202 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE209 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE20B (BATTLEMAGE)
[11:18:39] [PASSED] 0xE20C (BATTLEMAGE)
[11:18:39] [PASSED] 0xE20D (BATTLEMAGE)
[11:18:39] [PASSED] 0xE210 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE211 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE212 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE216 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE220 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE221 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE222 (BATTLEMAGE)
[11:18:39] [PASSED] 0xE223 (BATTLEMAGE)
[11:18:39] [PASSED] 0xB080 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB081 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB082 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB083 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB084 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB085 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB086 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB087 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB08F (PANTHERLAKE)
[11:18:39] [PASSED] 0xB090 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB0A0 (PANTHERLAKE)
[11:18:39] [PASSED] 0xB0B0 (PANTHERLAKE)
[11:18:39] [PASSED] 0xFD80 (PANTHERLAKE)
[11:18:39] [PASSED] 0xFD81 (PANTHERLAKE)
[11:18:39] [PASSED] 0xD740 (NOVALAKE_S)
[11:18:39] [PASSED] 0xD741 (NOVALAKE_S)
[11:18:39] [PASSED] 0xD742 (NOVALAKE_S)
[11:18:39] [PASSED] 0xD743 (NOVALAKE_S)
[11:18:39] [PASSED] 0xD744 (NOVALAKE_S)
[11:18:39] [PASSED] 0xD745 (NOVALAKE_S)
[11:18:39] [PASSED] 0x674C (CRESCENTISLAND)
[11:18:39] =============== [PASSED] check_platform_desc ===============
[11:18:39] ===================== [PASSED] xe_pci ======================
[11:18:39] =================== xe_rtp (2 subtests) ====================
[11:18:39] =============== xe_rtp_process_to_sr_tests  ================
[11:18:39] [PASSED] coalesce-same-reg
[11:18:39] [PASSED] no-match-no-add
[11:18:39] [PASSED] match-or
[11:18:39] [PASSED] match-or-xfail
[11:18:39] [PASSED] no-match-no-add-multiple-rules
[11:18:39] [PASSED] two-regs-two-entries
[11:18:39] [PASSED] clr-one-set-other
[11:18:39] [PASSED] set-field
[11:18:39] [PASSED] conflict-duplicate
[11:18:39] [PASSED] conflict-not-disjoint
[11:18:39] [PASSED] conflict-reg-type
[11:18:39] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[11:18:39] ================== xe_rtp_process_tests  ===================
[11:18:39] [PASSED] active1
[11:18:39] [PASSED] active2
[11:18:39] [PASSED] active-inactive
[11:18:39] [PASSED] inactive-active
[11:18:39] [PASSED] inactive-1st_or_active-inactive
[11:18:39] [PASSED] inactive-2nd_or_active-inactive
[11:18:39] [PASSED] inactive-last_or_active-inactive
[11:18:39] [PASSED] inactive-no_or_active-inactive
[11:18:39] ============== [PASSED] xe_rtp_process_tests ===============
[11:18:39] ===================== [PASSED] xe_rtp ======================
[11:18:39] ==================== xe_wa (1 subtest) =====================
[11:18:39] ======================== xe_wa_gt  =========================
[11:18:39] [PASSED] TIGERLAKE B0
[11:18:39] [PASSED] DG1 A0
[11:18:39] [PASSED] DG1 B0
[11:18:39] [PASSED] ALDERLAKE_S A0
[11:18:39] [PASSED] ALDERLAKE_S B0
[11:18:39] [PASSED] ALDERLAKE_S C0
[11:18:39] [PASSED] ALDERLAKE_S D0
[11:18:39] [PASSED] ALDERLAKE_P A0
[11:18:39] [PASSED] ALDERLAKE_P B0
[11:18:39] [PASSED] ALDERLAKE_P C0
[11:18:39] [PASSED] ALDERLAKE_S RPLS D0
[11:18:39] [PASSED] ALDERLAKE_P RPLU E0
[11:18:39] [PASSED] DG2 G10 C0
[11:18:39] [PASSED] DG2 G11 B1
[11:18:39] [PASSED] DG2 G12 A1
[11:18:39] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[11:18:39] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[11:18:39] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[11:18:39] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[11:18:39] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[11:18:39] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[11:18:39] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[11:18:39] ==================== [PASSED] xe_wa_gt =====================
[11:18:39] ====================== [PASSED] xe_wa ======================
[11:18:39] ============================================================
[11:18:39] Testing complete. Ran 512 tests: passed: 494, skipped: 18
[11:18:39] Elapsed time: 43.855s total, 4.295s configuring, 39.042s building, 0.483s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[11:18:39] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[11:18:41] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[11:19:11] Starting KUnit Kernel (1/1)...
[11:19:11] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[11:19:12] ============ drm_test_pick_cmdline (2 subtests) ============
[11:19:12] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[11:19:12] =============== drm_test_pick_cmdline_named  ===============
[11:19:12] [PASSED] NTSC
[11:19:12] [PASSED] NTSC-J
[11:19:12] [PASSED] PAL
[11:19:12] [PASSED] PAL-M
[11:19:12] =========== [PASSED] drm_test_pick_cmdline_named ===========
[11:19:12] ============== [PASSED] drm_test_pick_cmdline ==============
[11:19:12] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[11:19:12] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[11:19:12] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[11:19:12] =========== drm_validate_clone_mode (2 subtests) ===========
[11:19:12] ============== drm_test_check_in_clone_mode  ===============
[11:19:12] [PASSED] in_clone_mode
[11:19:12] [PASSED] not_in_clone_mode
[11:19:12] ========== [PASSED] drm_test_check_in_clone_mode ===========
[11:19:12] =============== drm_test_check_valid_clones  ===============
[11:19:12] [PASSED] not_in_clone_mode
[11:19:12] [PASSED] valid_clone
[11:19:12] [PASSED] invalid_clone
[11:19:12] =========== [PASSED] drm_test_check_valid_clones ===========
[11:19:12] ============= [PASSED] drm_validate_clone_mode =============
[11:19:12] ============= drm_validate_modeset (1 subtest) =============
[11:19:12] [PASSED] drm_test_check_connector_changed_modeset
[11:19:12] ============== [PASSED] drm_validate_modeset ===============
[11:19:12] ====== drm_test_bridge_get_current_state (2 subtests) ======
[11:19:12] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[11:19:12] [PASSED] drm_test_drm_bridge_get_current_state_legacy
[11:19:12] ======== [PASSED] drm_test_bridge_get_current_state ========
[11:19:12] ====== drm_test_bridge_helper_reset_crtc (3 subtests) ======
[11:19:12] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[11:19:12] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[11:19:12] [PASSED] drm_test_drm_bridge_helper_reset_crtc_legacy
[11:19:12] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[11:19:12] ============== drm_bridge_alloc (2 subtests) ===============
[11:19:12] [PASSED] drm_test_drm_bridge_alloc_basic
[11:19:12] [PASSED] drm_test_drm_bridge_alloc_get_put
[11:19:12] ================ [PASSED] drm_bridge_alloc =================
[11:19:12] ================== drm_buddy (8 subtests) ==================
[11:19:12] [PASSED] drm_test_buddy_alloc_limit
[11:19:12] [PASSED] drm_test_buddy_alloc_optimistic
[11:19:12] [PASSED] drm_test_buddy_alloc_pessimistic
[11:19:12] [PASSED] drm_test_buddy_alloc_pathological
[11:19:12] [PASSED] drm_test_buddy_alloc_contiguous
[11:19:12] [PASSED] drm_test_buddy_alloc_clear
[11:19:12] [PASSED] drm_test_buddy_alloc_range_bias
[11:19:12] [PASSED] drm_test_buddy_fragmentation_performance
[11:19:12] ==================== [PASSED] drm_buddy ====================
[11:19:12] ============= drm_cmdline_parser (40 subtests) =============
[11:19:12] [PASSED] drm_test_cmdline_force_d_only
[11:19:12] [PASSED] drm_test_cmdline_force_D_only_dvi
[11:19:12] [PASSED] drm_test_cmdline_force_D_only_hdmi
[11:19:12] [PASSED] drm_test_cmdline_force_D_only_not_digital
[11:19:12] [PASSED] drm_test_cmdline_force_e_only
[11:19:12] [PASSED] drm_test_cmdline_res
[11:19:12] [PASSED] drm_test_cmdline_res_vesa
[11:19:12] [PASSED] drm_test_cmdline_res_vesa_rblank
[11:19:12] [PASSED] drm_test_cmdline_res_rblank
[11:19:12] [PASSED] drm_test_cmdline_res_bpp
[11:19:12] [PASSED] drm_test_cmdline_res_refresh
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[11:19:12] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[11:19:12] [PASSED] drm_test_cmdline_res_margins_force_on
[11:19:12] [PASSED] drm_test_cmdline_res_vesa_margins
[11:19:12] [PASSED] drm_test_cmdline_name
[11:19:12] [PASSED] drm_test_cmdline_name_bpp
[11:19:12] [PASSED] drm_test_cmdline_name_option
[11:19:12] [PASSED] drm_test_cmdline_name_bpp_option
[11:19:12] [PASSED] drm_test_cmdline_rotate_0
[11:19:12] [PASSED] drm_test_cmdline_rotate_90
[11:19:12] [PASSED] drm_test_cmdline_rotate_180
[11:19:12] [PASSED] drm_test_cmdline_rotate_270
[11:19:12] [PASSED] drm_test_cmdline_hmirror
[11:19:12] [PASSED] drm_test_cmdline_vmirror
[11:19:12] [PASSED] drm_test_cmdline_margin_options
[11:19:12] [PASSED] drm_test_cmdline_multiple_options
[11:19:12] [PASSED] drm_test_cmdline_bpp_extra_and_option
[11:19:12] [PASSED] drm_test_cmdline_extra_and_option
[11:19:12] [PASSED] drm_test_cmdline_freestanding_options
[11:19:12] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[11:19:12] [PASSED] drm_test_cmdline_panel_orientation
[11:19:12] ================ drm_test_cmdline_invalid  =================
[11:19:12] [PASSED] margin_only
[11:19:12] [PASSED] interlace_only
[11:19:12] [PASSED] res_missing_x
[11:19:12] [PASSED] res_missing_y
[11:19:12] [PASSED] res_bad_y
[11:19:12] [PASSED] res_missing_y_bpp
[11:19:12] [PASSED] res_bad_bpp
[11:19:12] [PASSED] res_bad_refresh
[11:19:12] [PASSED] res_bpp_refresh_force_on_off
[11:19:12] [PASSED] res_invalid_mode
[11:19:12] [PASSED] res_bpp_wrong_place_mode
[11:19:12] [PASSED] name_bpp_refresh
[11:19:12] [PASSED] name_refresh
[11:19:12] [PASSED] name_refresh_wrong_mode
[11:19:12] [PASSED] name_refresh_invalid_mode
[11:19:12] [PASSED] rotate_multiple
[11:19:12] [PASSED] rotate_invalid_val
[11:19:12] [PASSED] rotate_truncated
[11:19:12] [PASSED] invalid_option
[11:19:12] [PASSED] invalid_tv_option
[11:19:12] [PASSED] truncated_tv_option
[11:19:12] ============ [PASSED] drm_test_cmdline_invalid =============
[11:19:12] =============== drm_test_cmdline_tv_options  ===============
[11:19:12] [PASSED] NTSC
[11:19:12] [PASSED] NTSC_443
[11:19:12] [PASSED] NTSC_J
[11:19:12] [PASSED] PAL
[11:19:12] [PASSED] PAL_M
[11:19:12] [PASSED] PAL_N
[11:19:12] [PASSED] SECAM
[11:19:12] [PASSED] MONO_525
[11:19:12] [PASSED] MONO_625
[11:19:12] =========== [PASSED] drm_test_cmdline_tv_options ===========
[11:19:12] =============== [PASSED] drm_cmdline_parser ================
[11:19:12] ========== drmm_connector_hdmi_init (20 subtests) ==========
[11:19:12] [PASSED] drm_test_connector_hdmi_init_valid
[11:19:12] [PASSED] drm_test_connector_hdmi_init_bpc_8
[11:19:12] [PASSED] drm_test_connector_hdmi_init_bpc_10
[11:19:12] [PASSED] drm_test_connector_hdmi_init_bpc_12
[11:19:12] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[11:19:12] [PASSED] drm_test_connector_hdmi_init_bpc_null
[11:19:12] [PASSED] drm_test_connector_hdmi_init_formats_empty
[11:19:12] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[11:19:12] === drm_test_connector_hdmi_init_formats_yuv420_allowed  ===
[11:19:12] [PASSED] supported_formats=0x9 yuv420_allowed=1
[11:19:12] [PASSED] supported_formats=0x9 yuv420_allowed=0
[11:19:12] [PASSED] supported_formats=0x3 yuv420_allowed=1
[11:19:12] [PASSED] supported_formats=0x3 yuv420_allowed=0
[11:19:12] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[11:19:12] [PASSED] drm_test_connector_hdmi_init_null_ddc
[11:19:12] [PASSED] drm_test_connector_hdmi_init_null_product
[11:19:12] [PASSED] drm_test_connector_hdmi_init_null_vendor
[11:19:12] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[11:19:12] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[11:19:12] [PASSED] drm_test_connector_hdmi_init_product_valid
[11:19:12] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[11:19:12] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[11:19:12] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[11:19:12] ========= drm_test_connector_hdmi_init_type_valid  =========
[11:19:12] [PASSED] HDMI-A
[11:19:12] [PASSED] HDMI-B
[11:19:12] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[11:19:12] ======== drm_test_connector_hdmi_init_type_invalid  ========
[11:19:12] [PASSED] Unknown
[11:19:12] [PASSED] VGA
[11:19:12] [PASSED] DVI-I
[11:19:12] [PASSED] DVI-D
[11:19:12] [PASSED] DVI-A
[11:19:12] [PASSED] Composite
[11:19:12] [PASSED] SVIDEO
[11:19:12] [PASSED] LVDS
[11:19:12] [PASSED] Component
[11:19:12] [PASSED] DIN
[11:19:12] [PASSED] DP
[11:19:12] [PASSED] TV
[11:19:12] [PASSED] eDP
[11:19:12] [PASSED] Virtual
[11:19:12] [PASSED] DSI
[11:19:12] [PASSED] DPI
[11:19:12] [PASSED] Writeback
[11:19:12] [PASSED] SPI
[11:19:12] [PASSED] USB
[11:19:12] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[11:19:12] ============ [PASSED] drmm_connector_hdmi_init =============
[11:19:12] ============= drmm_connector_init (3 subtests) =============
[11:19:12] [PASSED] drm_test_drmm_connector_init
[11:19:12] [PASSED] drm_test_drmm_connector_init_null_ddc
[11:19:12] ========= drm_test_drmm_connector_init_type_valid  =========
[11:19:12] [PASSED] Unknown
[11:19:12] [PASSED] VGA
[11:19:12] [PASSED] DVI-I
[11:19:12] [PASSED] DVI-D
[11:19:12] [PASSED] DVI-A
[11:19:12] [PASSED] Composite
[11:19:12] [PASSED] SVIDEO
[11:19:12] [PASSED] LVDS
[11:19:12] [PASSED] Component
[11:19:12] [PASSED] DIN
[11:19:12] [PASSED] DP
[11:19:12] [PASSED] HDMI-A
[11:19:12] [PASSED] HDMI-B
[11:19:12] [PASSED] TV
[11:19:12] [PASSED] eDP
[11:19:12] [PASSED] Virtual
[11:19:12] [PASSED] DSI
[11:19:12] [PASSED] DPI
[11:19:12] [PASSED] Writeback
[11:19:12] [PASSED] SPI
[11:19:12] [PASSED] USB
[11:19:12] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[11:19:12] =============== [PASSED] drmm_connector_init ===============
[11:19:12] ========= drm_connector_dynamic_init (6 subtests) ==========
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_init
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_init_properties
[11:19:12] ===== drm_test_drm_connector_dynamic_init_type_valid  ======
[11:19:12] [PASSED] Unknown
[11:19:12] [PASSED] VGA
[11:19:12] [PASSED] DVI-I
[11:19:12] [PASSED] DVI-D
[11:19:12] [PASSED] DVI-A
[11:19:12] [PASSED] Composite
[11:19:12] [PASSED] SVIDEO
[11:19:12] [PASSED] LVDS
[11:19:12] [PASSED] Component
[11:19:12] [PASSED] DIN
[11:19:12] [PASSED] DP
[11:19:12] [PASSED] HDMI-A
[11:19:12] [PASSED] HDMI-B
[11:19:12] [PASSED] TV
[11:19:12] [PASSED] eDP
[11:19:12] [PASSED] Virtual
[11:19:12] [PASSED] DSI
[11:19:12] [PASSED] DPI
[11:19:12] [PASSED] Writeback
[11:19:12] [PASSED] SPI
[11:19:12] [PASSED] USB
[11:19:12] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[11:19:12] ======== drm_test_drm_connector_dynamic_init_name  =========
[11:19:12] [PASSED] Unknown
[11:19:12] [PASSED] VGA
[11:19:12] [PASSED] DVI-I
[11:19:12] [PASSED] DVI-D
[11:19:12] [PASSED] DVI-A
[11:19:12] [PASSED] Composite
[11:19:12] [PASSED] SVIDEO
[11:19:12] [PASSED] LVDS
[11:19:12] [PASSED] Component
[11:19:12] [PASSED] DIN
[11:19:12] [PASSED] DP
[11:19:12] [PASSED] HDMI-A
[11:19:12] [PASSED] HDMI-B
[11:19:12] [PASSED] TV
[11:19:12] [PASSED] eDP
[11:19:12] [PASSED] Virtual
[11:19:12] [PASSED] DSI
[11:19:12] [PASSED] DPI
[11:19:12] [PASSED] Writeback
[11:19:12] [PASSED] SPI
[11:19:12] [PASSED] USB
[11:19:12] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[11:19:12] =========== [PASSED] drm_connector_dynamic_init ============
[11:19:12] ==== drm_connector_dynamic_register_early (4 subtests) =====
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[11:19:12] ====== [PASSED] drm_connector_dynamic_register_early =======
[11:19:12] ======= drm_connector_dynamic_register (7 subtests) ========
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[11:19:12] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[11:19:12] ========= [PASSED] drm_connector_dynamic_register ==========
[11:19:12] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[11:19:12] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[11:19:12] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[11:19:12] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[11:19:12] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[11:19:12] ========== drm_test_get_tv_mode_from_name_valid  ===========
[11:19:12] [PASSED] NTSC
[11:19:12] [PASSED] NTSC-443
[11:19:12] [PASSED] NTSC-J
[11:19:12] [PASSED] PAL
[11:19:12] [PASSED] PAL-M
[11:19:12] [PASSED] PAL-N
[11:19:12] [PASSED] SECAM
[11:19:12] [PASSED] Mono
[11:19:12] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[11:19:12] [PASSED] drm_test_get_tv_mode_from_name_truncated
[11:19:12] ============ [PASSED] drm_get_tv_mode_from_name ============
[11:19:12] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[11:19:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[11:19:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[11:19:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[11:19:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[11:19:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[11:19:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[11:19:12] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid  =
[11:19:12] [PASSED] VIC 96
[11:19:12] [PASSED] VIC 97
[11:19:12] [PASSED] VIC 101
[11:19:12] [PASSED] VIC 102
[11:19:12] [PASSED] VIC 106
[11:19:12] [PASSED] VIC 107
[11:19:12] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[11:19:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[11:19:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[11:19:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[11:19:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[11:19:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[11:19:12] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[11:19:12] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[11:19:12] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name  ====
[11:19:12] [PASSED] Automatic
[11:19:12] [PASSED] Full
[11:19:12] [PASSED] Limited 16:235
[11:19:12] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[11:19:12] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[11:19:12] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[11:19:12] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[11:19:12] === drm_test_drm_hdmi_connector_get_output_format_name  ====
[11:19:12] [PASSED] RGB
[11:19:12] [PASSED] YUV 4:2:0
[11:19:12] [PASSED] YUV 4:2:2
[11:19:12] [PASSED] YUV 4:4:4
[11:19:12] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[11:19:12] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[11:19:12] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[11:19:12] ============= drm_damage_helper (21 subtests) ==============
[11:19:12] [PASSED] drm_test_damage_iter_no_damage
[11:19:12] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[11:19:12] [PASSED] drm_test_damage_iter_no_damage_src_moved
[11:19:12] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[11:19:12] [PASSED] drm_test_damage_iter_no_damage_not_visible
[11:19:12] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[11:19:12] [PASSED] drm_test_damage_iter_no_damage_no_fb
[11:19:12] [PASSED] drm_test_damage_iter_simple_damage
[11:19:12] [PASSED] drm_test_damage_iter_single_damage
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_outside_src
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_src_moved
[11:19:12] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[11:19:12] [PASSED] drm_test_damage_iter_damage
[11:19:12] [PASSED] drm_test_damage_iter_damage_one_intersect
[11:19:12] [PASSED] drm_test_damage_iter_damage_one_outside
[11:19:12] [PASSED] drm_test_damage_iter_damage_src_moved
[11:19:12] [PASSED] drm_test_damage_iter_damage_not_visible
[11:19:12] ================ [PASSED] drm_damage_helper ================
[11:19:12] ============== drm_dp_mst_helper (3 subtests) ==============
[11:19:12] ============== drm_test_dp_mst_calc_pbn_mode  ==============
[11:19:12] [PASSED] Clock 154000 BPP 30 DSC disabled
[11:19:12] [PASSED] Clock 234000 BPP 30 DSC disabled
[11:19:12] [PASSED] Clock 297000 BPP 24 DSC disabled
[11:19:12] [PASSED] Clock 332880 BPP 24 DSC enabled
[11:19:12] [PASSED] Clock 324540 BPP 24 DSC enabled
[11:19:12] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[11:19:12] ============== drm_test_dp_mst_calc_pbn_div  ===============
[11:19:12] [PASSED] Link rate 2000000 lane count 4
[11:19:12] [PASSED] Link rate 2000000 lane count 2
[11:19:12] [PASSED] Link rate 2000000 lane count 1
[11:19:12] [PASSED] Link rate 1350000 lane count 4
[11:19:12] [PASSED] Link rate 1350000 lane count 2
[11:19:12] [PASSED] Link rate 1350000 lane count 1
[11:19:12] [PASSED] Link rate 1000000 lane count 4
[11:19:12] [PASSED] Link rate 1000000 lane count 2
[11:19:12] [PASSED] Link rate 1000000 lane count 1
[11:19:12] [PASSED] Link rate 810000 lane count 4
[11:19:12] [PASSED] Link rate 810000 lane count 2
[11:19:12] [PASSED] Link rate 810000 lane count 1
[11:19:12] [PASSED] Link rate 540000 lane count 4
[11:19:12] [PASSED] Link rate 540000 lane count 2
[11:19:12] [PASSED] Link rate 540000 lane count 1
[11:19:12] [PASSED] Link rate 270000 lane count 4
[11:19:12] [PASSED] Link rate 270000 lane count 2
[11:19:12] [PASSED] Link rate 270000 lane count 1
[11:19:12] [PASSED] Link rate 162000 lane count 4
[11:19:12] [PASSED] Link rate 162000 lane count 2
[11:19:12] [PASSED] Link rate 162000 lane count 1
[11:19:12] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[11:19:12] ========= drm_test_dp_mst_sideband_msg_req_decode  =========
[11:19:12] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[11:19:12] [PASSED] DP_POWER_UP_PHY with port number
[11:19:12] [PASSED] DP_POWER_DOWN_PHY with port number
[11:19:12] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[11:19:12] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[11:19:12] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[11:19:12] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[11:19:12] [PASSED] DP_QUERY_PAYLOAD with port number
[11:19:12] [PASSED] DP_QUERY_PAYLOAD with VCPI
[11:19:12] [PASSED] DP_REMOTE_DPCD_READ with port number
[11:19:12] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[11:19:12] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[11:19:12] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[11:19:12] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[11:19:12] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[11:19:12] [PASSED] DP_REMOTE_I2C_READ with port number
[11:19:12] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[11:19:12] [PASSED] DP_REMOTE_I2C_READ with transactions array
[11:19:12] [PASSED] DP_REMOTE_I2C_WRITE with port number
[11:19:12] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[11:19:12] [PASSED] DP_REMOTE_I2C_WRITE with data array
[11:19:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[11:19:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[11:19:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[11:19:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[11:19:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[11:19:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[11:19:12] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[11:19:12] ================ [PASSED] drm_dp_mst_helper ================
[11:19:12] ================== drm_exec (7 subtests) ===================
[11:19:12] [PASSED] sanitycheck
[11:19:12] [PASSED] test_lock
[11:19:12] [PASSED] test_lock_unlock
[11:19:12] [PASSED] test_duplicates
[11:19:12] [PASSED] test_prepare
[11:19:12] [PASSED] test_prepare_array
[11:19:12] [PASSED] test_multiple_loops
[11:19:12] ==================== [PASSED] drm_exec =====================
[11:19:12] =========== drm_format_helper_test (17 subtests) ===========
[11:19:12] ============== drm_test_fb_xrgb8888_to_gray8  ==============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[11:19:12] ============= drm_test_fb_xrgb8888_to_rgb332  ==============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[11:19:12] ============= drm_test_fb_xrgb8888_to_rgb565  ==============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[11:19:12] ============ drm_test_fb_xrgb8888_to_xrgb1555  =============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[11:19:12] ============ drm_test_fb_xrgb8888_to_argb1555  =============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[11:19:12] ============ drm_test_fb_xrgb8888_to_rgba5551  =============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[11:19:12] ============= drm_test_fb_xrgb8888_to_rgb888  ==============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[11:19:12] ============= drm_test_fb_xrgb8888_to_bgr888  ==============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[11:19:12] ============ drm_test_fb_xrgb8888_to_argb8888  =============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[11:19:12] =========== drm_test_fb_xrgb8888_to_xrgb2101010  ===========
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[11:19:12] =========== drm_test_fb_xrgb8888_to_argb2101010  ===========
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[11:19:12] ============== drm_test_fb_xrgb8888_to_mono  ===============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[11:19:12] ==================== drm_test_fb_swab  =====================
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ================ [PASSED] drm_test_fb_swab =================
[11:19:12] ============ drm_test_fb_xrgb8888_to_xbgr8888  =============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[11:19:12] ============ drm_test_fb_xrgb8888_to_abgr8888  =============
[11:19:12] [PASSED] single_pixel_source_buffer
[11:19:12] [PASSED] single_pixel_clip_rectangle
[11:19:12] [PASSED] well_known_colors
[11:19:12] [PASSED] destination_pitch
[11:19:12] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[11:19:12] ================= drm_test_fb_clip_offset  =================
[11:19:12] [PASSED] pass through
[11:19:12] [PASSED] horizontal offset
[11:19:12] [PASSED] vertical offset
[11:19:12] [PASSED] horizontal and vertical offset
[11:19:12] [PASSED] horizontal offset (custom pitch)
[11:19:12] [PASSED] vertical offset (custom pitch)
[11:19:12] [PASSED] horizontal and vertical offset (custom pitch)
[11:19:12] ============= [PASSED] drm_test_fb_clip_offset =============
[11:19:12] =================== drm_test_fb_memcpy  ====================
[11:19:12] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[11:19:12] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[11:19:12] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[11:19:12] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[11:19:12] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[11:19:12] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[11:19:12] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[11:19:12] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[11:19:12] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[11:19:12] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[11:19:12] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[11:19:12] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[11:19:12] =============== [PASSED] drm_test_fb_memcpy ================
[11:19:12] ============= [PASSED] drm_format_helper_test ==============
[11:19:12] ================= drm_format (18 subtests) =================
[11:19:12] [PASSED] drm_test_format_block_width_invalid
[11:19:12] [PASSED] drm_test_format_block_width_one_plane
[11:19:12] [PASSED] drm_test_format_block_width_two_plane
[11:19:12] [PASSED] drm_test_format_block_width_three_plane
[11:19:12] [PASSED] drm_test_format_block_width_tiled
[11:19:12] [PASSED] drm_test_format_block_height_invalid
[11:19:12] [PASSED] drm_test_format_block_height_one_plane
[11:19:12] [PASSED] drm_test_format_block_height_two_plane
[11:19:12] [PASSED] drm_test_format_block_height_three_plane
[11:19:12] [PASSED] drm_test_format_block_height_tiled
[11:19:12] [PASSED] drm_test_format_min_pitch_invalid
[11:19:12] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[11:19:12] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[11:19:12] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[11:19:12] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[11:19:12] [PASSED] drm_test_format_min_pitch_two_plane
[11:19:12] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[11:19:12] [PASSED] drm_test_format_min_pitch_tiled
[11:19:12] =================== [PASSED] drm_format ====================
[11:19:12] ============== drm_framebuffer (10 subtests) ===============
[11:19:12] ========== drm_test_framebuffer_check_src_coords  ==========
[11:19:12] [PASSED] Success: source fits into fb
[11:19:12] [PASSED] Fail: overflowing fb with x-axis coordinate
[11:19:12] [PASSED] Fail: overflowing fb with y-axis coordinate
[11:19:12] [PASSED] Fail: overflowing fb with source width
[11:19:12] [PASSED] Fail: overflowing fb with source height
[11:19:12] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[11:19:12] [PASSED] drm_test_framebuffer_cleanup
[11:19:12] =============== drm_test_framebuffer_create  ===============
[11:19:12] [PASSED] ABGR8888 normal sizes
[11:19:12] [PASSED] ABGR8888 max sizes
[11:19:12] [PASSED] ABGR8888 pitch greater than min required
[11:19:12] [PASSED] ABGR8888 pitch less than min required
[11:19:12] [PASSED] ABGR8888 Invalid width
[11:19:12] [PASSED] ABGR8888 Invalid buffer handle
[11:19:12] [PASSED] No pixel format
[11:19:12] [PASSED] ABGR8888 Width 0
[11:19:12] [PASSED] ABGR8888 Height 0
[11:19:12] [PASSED] ABGR8888 Out of bound height * pitch combination
[11:19:12] [PASSED] ABGR8888 Large buffer offset
[11:19:12] [PASSED] ABGR8888 Buffer offset for inexistent plane
[11:19:12] [PASSED] ABGR8888 Invalid flag
[11:19:12] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[11:19:12] [PASSED] ABGR8888 Valid buffer modifier
[11:19:12] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[11:19:12] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] NV12 Normal sizes
[11:19:12] [PASSED] NV12 Max sizes
[11:19:12] [PASSED] NV12 Invalid pitch
[11:19:12] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[11:19:12] [PASSED] NV12 different  modifier per-plane
[11:19:12] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[11:19:12] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] NV12 Modifier for inexistent plane
[11:19:12] [PASSED] NV12 Handle for inexistent plane
[11:19:12] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[11:19:12] [PASSED] YVU420 Normal sizes
[11:19:12] [PASSED] YVU420 Max sizes
[11:19:12] [PASSED] YVU420 Invalid pitch
[11:19:12] [PASSED] YVU420 Different pitches
[11:19:12] [PASSED] YVU420 Different buffer offsets/pitches
[11:19:12] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[11:19:12] [PASSED] YVU420 Valid modifier
[11:19:12] [PASSED] YVU420 Different modifiers per plane
[11:19:12] [PASSED] YVU420 Modifier for inexistent plane
[11:19:12] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[11:19:12] [PASSED] X0L2 Normal sizes
[11:19:12] [PASSED] X0L2 Max sizes
[11:19:12] [PASSED] X0L2 Invalid pitch
[11:19:12] [PASSED] X0L2 Pitch greater than minimum required
[11:19:12] [PASSED] X0L2 Handle for inexistent plane
[11:19:12] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[11:19:12] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[11:19:12] [PASSED] X0L2 Valid modifier
[11:19:12] [PASSED] X0L2 Modifier for inexistent plane
[11:19:12] =========== [PASSED] drm_test_framebuffer_create ===========
[11:19:12] [PASSED] drm_test_framebuffer_free
[11:19:12] [PASSED] drm_test_framebuffer_init
[11:19:12] [PASSED] drm_test_framebuffer_init_bad_format
[11:19:12] [PASSED] drm_test_framebuffer_init_dev_mismatch
[11:19:12] [PASSED] drm_test_framebuffer_lookup
[11:19:12] [PASSED] drm_test_framebuffer_lookup_inexistent
[11:19:12] [PASSED] drm_test_framebuffer_modifiers_not_supported
[11:19:12] ================= [PASSED] drm_framebuffer =================
[11:19:12] ================ drm_gem_shmem (8 subtests) ================
[11:19:12] [PASSED] drm_gem_shmem_test_obj_create
[11:19:12] [PASSED] drm_gem_shmem_test_obj_create_private
[11:19:12] [PASSED] drm_gem_shmem_test_pin_pages
[11:19:12] [PASSED] drm_gem_shmem_test_vmap
[11:19:12] [PASSED] drm_gem_shmem_test_get_sg_table
[11:19:12] [PASSED] drm_gem_shmem_test_get_pages_sgt
[11:19:12] [PASSED] drm_gem_shmem_test_madvise
[11:19:12] [PASSED] drm_gem_shmem_test_purge
[11:19:12] ================== [PASSED] drm_gem_shmem ==================
[11:19:12] === drm_atomic_helper_connector_hdmi_check (27 subtests) ===
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[11:19:12] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420  =======
[11:19:12] [PASSED] Automatic
[11:19:12] [PASSED] Full
[11:19:12] [PASSED] Limited 16:235
[11:19:12] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[11:19:12] [PASSED] drm_test_check_disable_connector
[11:19:12] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[11:19:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[11:19:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[11:19:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[11:19:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[11:19:12] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[11:19:12] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[11:19:12] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[11:19:12] [PASSED] drm_test_check_output_bpc_dvi
[11:19:12] [PASSED] drm_test_check_output_bpc_format_vic_1
[11:19:12] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[11:19:12] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[11:19:12] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[11:19:12] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[11:19:12] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[11:19:12] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[11:19:12] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[11:19:12] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[11:19:12] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[11:19:12] [PASSED] drm_test_check_broadcast_rgb_value
[11:19:12] [PASSED] drm_test_check_bpc_8_value
[11:19:12] [PASSED] drm_test_check_bpc_10_value
[11:19:12] [PASSED] drm_test_check_bpc_12_value
[11:19:12] [PASSED] drm_test_check_format_value
[11:19:12] [PASSED] drm_test_check_tmds_char_value
[11:19:12] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[11:19:12] = drm_atomic_helper_connector_hdmi_mode_valid (4 subtests) =
[11:19:12] [PASSED] drm_test_check_mode_valid
[11:19:12] [PASSED] drm_test_check_mode_valid_reject
[11:19:12] [PASSED] drm_test_check_mode_valid_reject_rate
[11:19:12] [PASSED] drm_test_check_mode_valid_reject_max_clock
[11:19:12] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[11:19:12] ================= drm_managed (2 subtests) =================
[11:19:12] [PASSED] drm_test_managed_release_action
[11:19:12] [PASSED] drm_test_managed_run_action
[11:19:12] =================== [PASSED] drm_managed ===================
[11:19:12] =================== drm_mm (6 subtests) ====================
[11:19:12] [PASSED] drm_test_mm_init
[11:19:12] [PASSED] drm_test_mm_debug
[11:19:12] [PASSED] drm_test_mm_align32
[11:19:12] [PASSED] drm_test_mm_align64
[11:19:12] [PASSED] drm_test_mm_lowest
[11:19:12] [PASSED] drm_test_mm_highest
[11:19:12] ===================== [PASSED] drm_mm ======================
[11:19:12] ============= drm_modes_analog_tv (5 subtests) =============
[11:19:12] [PASSED] drm_test_modes_analog_tv_mono_576i
[11:19:12] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[11:19:12] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[11:19:12] [PASSED] drm_test_modes_analog_tv_pal_576i
[11:19:12] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[11:19:12] =============== [PASSED] drm_modes_analog_tv ===============
[11:19:12] ============== drm_plane_helper (2 subtests) ===============
[11:19:12] =============== drm_test_check_plane_state  ================
[11:19:12] [PASSED] clipping_simple
[11:19:12] [PASSED] clipping_rotate_reflect
[11:19:12] [PASSED] positioning_simple
[11:19:12] [PASSED] upscaling
[11:19:12] [PASSED] downscaling
[11:19:12] [PASSED] rounding1
[11:19:12] [PASSED] rounding2
[11:19:12] [PASSED] rounding3
[11:19:12] [PASSED] rounding4
[11:19:12] =========== [PASSED] drm_test_check_plane_state ============
[11:19:12] =========== drm_test_check_invalid_plane_state  ============
[11:19:12] [PASSED] positioning_invalid
[11:19:12] [PASSED] upscaling_invalid
[11:19:12] [PASSED] downscaling_invalid
[11:19:12] ======= [PASSED] drm_test_check_invalid_plane_state ========
[11:19:12] ================ [PASSED] drm_plane_helper =================
[11:19:12] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[11:19:12] ====== drm_test_connector_helper_tv_get_modes_check  =======
[11:19:12] [PASSED] None
[11:19:12] [PASSED] PAL
[11:19:12] [PASSED] NTSC
[11:19:12] [PASSED] Both, NTSC Default
[11:19:12] [PASSED] Both, PAL Default
[11:19:12] [PASSED] Both, NTSC Default, with PAL on command-line
[11:19:12] [PASSED] Both, PAL Default, with NTSC on command-line
[11:19:12] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[11:19:12] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[11:19:12] ================== drm_rect (9 subtests) ===================
[11:19:12] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[11:19:12] [PASSED] drm_test_rect_clip_scaled_not_clipped
[11:19:12] [PASSED] drm_test_rect_clip_scaled_clipped
[11:19:12] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[11:19:12] ================= drm_test_rect_intersect  =================
[11:19:12] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[11:19:12] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[11:19:12] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[11:19:12] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[11:19:12] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[11:19:12] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[11:19:12] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[11:19:12] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[11:19:12] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[11:19:12] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[11:19:12] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[11:19:12] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[11:19:12] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[11:19:12] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[11:19:12] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[11:19:12] ============= [PASSED] drm_test_rect_intersect =============
[11:19:12] ================ drm_test_rect_calc_hscale  ================
[11:19:12] [PASSED] normal use
[11:19:12] [PASSED] out of max range
[11:19:12] [PASSED] out of min range
[11:19:12] [PASSED] zero dst
[11:19:12] [PASSED] negative src
[11:19:12] [PASSED] negative dst
[11:19:12] ============ [PASSED] drm_test_rect_calc_hscale ============
[11:19:12] ================ drm_test_rect_calc_vscale  ================
[11:19:12] [PASSED] normal use
stty: 'standard input': Inappropriate ioctl for device
[11:19:12] [PASSED] out of max range
[11:19:12] [PASSED] out of min range
[11:19:12] [PASSED] zero dst
[11:19:12] [PASSED] negative src
[11:19:12] [PASSED] negative dst
[11:19:12] ============ [PASSED] drm_test_rect_calc_vscale ============
[11:19:12] ================== drm_test_rect_rotate  ===================
[11:19:12] [PASSED] reflect-x
[11:19:12] [PASSED] reflect-y
[11:19:12] [PASSED] rotate-0
[11:19:12] [PASSED] rotate-90
[11:19:12] [PASSED] rotate-180
[11:19:12] [PASSED] rotate-270
[11:19:12] ============== [PASSED] drm_test_rect_rotate ===============
[11:19:12] ================ drm_test_rect_rotate_inv  =================
[11:19:12] [PASSED] reflect-x
[11:19:12] [PASSED] reflect-y
[11:19:12] [PASSED] rotate-0
[11:19:12] [PASSED] rotate-90
[11:19:12] [PASSED] rotate-180
[11:19:12] [PASSED] rotate-270
[11:19:12] ============ [PASSED] drm_test_rect_rotate_inv =============
[11:19:12] ==================== [PASSED] drm_rect =====================
[11:19:12] ============ drm_sysfb_modeset_test (1 subtest) ============
[11:19:12] ============ drm_test_sysfb_build_fourcc_list  =============
[11:19:12] [PASSED] no native formats
[11:19:12] [PASSED] XRGB8888 as native format
[11:19:12] [PASSED] remove duplicates
[11:19:12] [PASSED] convert alpha formats
[11:19:12] [PASSED] random formats
[11:19:12] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[11:19:12] ============= [PASSED] drm_sysfb_modeset_test ==============
[11:19:12] ================== drm_fixp (2 subtests) ===================
[11:19:12] [PASSED] drm_test_int2fixp
[11:19:12] [PASSED] drm_test_sm2fixp
[11:19:12] ==================== [PASSED] drm_fixp =====================
[11:19:12] ============================================================
[11:19:12] Testing complete. Ran 624 tests: passed: 624
[11:19:12] Elapsed time: 32.589s total, 1.608s configuring, 30.514s building, 0.452s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[11:19:12] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[11:19:14] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[11:19:23] Starting KUnit Kernel (1/1)...
[11:19:23] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[11:19:23] ================= ttm_device (5 subtests) ==================
[11:19:23] [PASSED] ttm_device_init_basic
[11:19:23] [PASSED] ttm_device_init_multiple
[11:19:23] [PASSED] ttm_device_fini_basic
[11:19:23] [PASSED] ttm_device_init_no_vma_man
[11:19:23] ================== ttm_device_init_pools  ==================
[11:19:23] [PASSED] No DMA allocations, no DMA32 required
[11:19:23] [PASSED] DMA allocations, DMA32 required
[11:19:23] [PASSED] No DMA allocations, DMA32 required
[11:19:23] [PASSED] DMA allocations, no DMA32 required
[11:19:23] ============== [PASSED] ttm_device_init_pools ==============
[11:19:23] =================== [PASSED] ttm_device ====================
[11:19:23] ================== ttm_pool (8 subtests) ===================
[11:19:23] ================== ttm_pool_alloc_basic  ===================
[11:19:23] [PASSED] One page
[11:19:23] [PASSED] More than one page
[11:19:23] [PASSED] Above the allocation limit
[11:19:23] [PASSED] One page, with coherent DMA mappings enabled
[11:19:23] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[11:19:23] ============== [PASSED] ttm_pool_alloc_basic ===============
[11:19:23] ============== ttm_pool_alloc_basic_dma_addr  ==============
[11:19:23] [PASSED] One page
[11:19:23] [PASSED] More than one page
[11:19:23] [PASSED] Above the allocation limit
[11:19:23] [PASSED] One page, with coherent DMA mappings enabled
[11:19:23] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[11:19:23] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[11:19:23] [PASSED] ttm_pool_alloc_order_caching_match
[11:19:23] [PASSED] ttm_pool_alloc_caching_mismatch
[11:19:23] [PASSED] ttm_pool_alloc_order_mismatch
[11:19:23] [PASSED] ttm_pool_free_dma_alloc
[11:19:23] [PASSED] ttm_pool_free_no_dma_alloc
[11:19:23] [PASSED] ttm_pool_fini_basic
[11:19:23] ==================== [PASSED] ttm_pool =====================
[11:19:23] ================ ttm_resource (8 subtests) =================
[11:19:23] ================= ttm_resource_init_basic  =================
[11:19:23] [PASSED] Init resource in TTM_PL_SYSTEM
[11:19:23] [PASSED] Init resource in TTM_PL_VRAM
[11:19:23] [PASSED] Init resource in a private placement
[11:19:23] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[11:19:23] ============= [PASSED] ttm_resource_init_basic =============
[11:19:23] [PASSED] ttm_resource_init_pinned
[11:19:23] [PASSED] ttm_resource_fini_basic
[11:19:23] [PASSED] ttm_resource_manager_init_basic
[11:19:23] [PASSED] ttm_resource_manager_usage_basic
[11:19:23] [PASSED] ttm_resource_manager_set_used_basic
[11:19:23] [PASSED] ttm_sys_man_alloc_basic
[11:19:23] [PASSED] ttm_sys_man_free_basic
[11:19:23] ================== [PASSED] ttm_resource ===================
[11:19:23] =================== ttm_tt (15 subtests) ===================
[11:19:23] ==================== ttm_tt_init_basic  ====================
[11:19:23] [PASSED] Page-aligned size
[11:19:23] [PASSED] Extra pages requested
[11:19:23] ================ [PASSED] ttm_tt_init_basic ================
[11:19:23] [PASSED] ttm_tt_init_misaligned
[11:19:23] [PASSED] ttm_tt_fini_basic
[11:19:23] [PASSED] ttm_tt_fini_sg
[11:19:23] [PASSED] ttm_tt_fini_shmem
[11:19:23] [PASSED] ttm_tt_create_basic
[11:19:23] [PASSED] ttm_tt_create_invalid_bo_type
[11:19:23] [PASSED] ttm_tt_create_ttm_exists
[11:19:23] [PASSED] ttm_tt_create_failed
[11:19:23] [PASSED] ttm_tt_destroy_basic
[11:19:23] [PASSED] ttm_tt_populate_null_ttm
[11:19:23] [PASSED] ttm_tt_populate_populated_ttm
[11:19:23] [PASSED] ttm_tt_unpopulate_basic
[11:19:23] [PASSED] ttm_tt_unpopulate_empty_ttm
[11:19:23] [PASSED] ttm_tt_swapin_basic
[11:19:23] ===================== [PASSED] ttm_tt ======================
[11:19:23] =================== ttm_bo (14 subtests) ===================
[11:19:23] =========== ttm_bo_reserve_optimistic_no_ticket  ===========
[11:19:23] [PASSED] Cannot be interrupted and sleeps
[11:19:23] [PASSED] Cannot be interrupted, locks straight away
[11:19:23] [PASSED] Can be interrupted, sleeps
[11:19:23] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[11:19:23] [PASSED] ttm_bo_reserve_locked_no_sleep
[11:19:23] [PASSED] ttm_bo_reserve_no_wait_ticket
[11:19:23] [PASSED] ttm_bo_reserve_double_resv
[11:19:23] [PASSED] ttm_bo_reserve_interrupted
[11:19:23] [PASSED] ttm_bo_reserve_deadlock
[11:19:23] [PASSED] ttm_bo_unreserve_basic
[11:19:23] [PASSED] ttm_bo_unreserve_pinned
[11:19:23] [PASSED] ttm_bo_unreserve_bulk
[11:19:23] [PASSED] ttm_bo_fini_basic
[11:19:23] [PASSED] ttm_bo_fini_shared_resv
[11:19:23] [PASSED] ttm_bo_pin_basic
[11:19:23] [PASSED] ttm_bo_pin_unpin_resource
[11:19:23] [PASSED] ttm_bo_multiple_pin_one_unpin
[11:19:23] ===================== [PASSED] ttm_bo ======================
[11:19:23] ============== ttm_bo_validate (21 subtests) ===============
[11:19:23] ============== ttm_bo_init_reserved_sys_man  ===============
[11:19:23] [PASSED] Buffer object for userspace
[11:19:23] [PASSED] Kernel buffer object
[11:19:23] [PASSED] Shared buffer object
[11:19:23] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[11:19:23] ============== ttm_bo_init_reserved_mock_man  ==============
[11:19:23] [PASSED] Buffer object for userspace
[11:19:23] [PASSED] Kernel buffer object
[11:19:23] [PASSED] Shared buffer object
[11:19:23] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[11:19:23] [PASSED] ttm_bo_init_reserved_resv
[11:19:23] ================== ttm_bo_validate_basic  ==================
[11:19:23] [PASSED] Buffer object for userspace
[11:19:23] [PASSED] Kernel buffer object
[11:19:23] [PASSED] Shared buffer object
[11:19:23] ============== [PASSED] ttm_bo_validate_basic ==============
[11:19:23] [PASSED] ttm_bo_validate_invalid_placement
[11:19:23] ============= ttm_bo_validate_same_placement  ==============
[11:19:23] [PASSED] System manager
[11:19:23] [PASSED] VRAM manager
[11:19:23] ========= [PASSED] ttm_bo_validate_same_placement ==========
[11:19:23] [PASSED] ttm_bo_validate_failed_alloc
[11:19:23] [PASSED] ttm_bo_validate_pinned
[11:19:23] [PASSED] ttm_bo_validate_busy_placement
[11:19:23] ================ ttm_bo_validate_multihop  =================
[11:19:23] [PASSED] Buffer object for userspace
[11:19:23] [PASSED] Kernel buffer object
[11:19:23] [PASSED] Shared buffer object
[11:19:23] ============ [PASSED] ttm_bo_validate_multihop =============
[11:19:23] ========== ttm_bo_validate_no_placement_signaled  ==========
[11:19:23] [PASSED] Buffer object in system domain, no page vector
[11:19:23] [PASSED] Buffer object in system domain with an existing page vector
[11:19:23] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[11:19:23] ======== ttm_bo_validate_no_placement_not_signaled  ========
[11:19:23] [PASSED] Buffer object for userspace
[11:19:23] [PASSED] Kernel buffer object
[11:19:23] [PASSED] Shared buffer object
[11:19:23] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[11:19:23] [PASSED] ttm_bo_validate_move_fence_signaled
[11:19:23] ========= ttm_bo_validate_move_fence_not_signaled  =========
[11:19:23] [PASSED] Waits for GPU
[11:19:23] [PASSED] Tries to lock straight away
[11:19:23] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[11:19:23] [PASSED] ttm_bo_validate_happy_evict
[11:19:23] [PASSED] ttm_bo_validate_all_pinned_evict
[11:19:23] [PASSED] ttm_bo_validate_allowed_only_evict
[11:19:23] [PASSED] ttm_bo_validate_deleted_evict
[11:19:23] [PASSED] ttm_bo_validate_busy_domain_evict
[11:19:23] [PASSED] ttm_bo_validate_evict_gutting
[11:19:23] [PASSED] ttm_bo_validate_recrusive_evict
stty: 'standard input': Inappropriate ioctl for device
[11:19:23] ================= [PASSED] ttm_bo_validate =================
[11:19:23] ============================================================
[11:19:23] Testing complete. Ran 101 tests: passed: 101
[11:19:23] Elapsed time: 11.234s total, 1.631s configuring, 9.387s building, 0.179s running

+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel

✗ CI.checksparse: warning for drm: Color pipeline teardown and follow-up fixes/improvements (rev3)

[Patchwork]

== Series Details ==

Series: drm: Color pipeline teardown and follow-up fixes/improvements (rev3)
URL   : https://patchwork.freedesktop.org/series/159263/
State : warning

== Summary ==

+ trap cleanup EXIT
+ KERNEL=/kernel
+ MT=/root/linux/maintainer-tools
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools /root/linux/maintainer-tools
Cloning into '/root/linux/maintainer-tools'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ make -C /root/linux/maintainer-tools
make: Entering directory '/root/linux/maintainer-tools'
cc -O2 -g -Wextra -o remap-log remap-log.c
make: Leaving directory '/root/linux/maintainer-tools'
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ /root/linux/maintainer-tools/dim sparse --fast f7424dfbcb8f05a13bb1da2bed8e2212d2b7a082
Sparse version: 0.6.4 (Ubuntu: 0.6.4-4ubuntu3)
Fast mode used, each commit won't be checked separately.
+drivers/gpu/drm/i915/display/dvo_ch7017.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/dvo_ch7xxx.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/dvo_ivch.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/dvo_ns2501.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/dvo_sil164.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/dvo_tfp410.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/g4x_dp.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/g4x_hdmi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/hsw_ips.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/i9xx_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/i9xx_wm.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/icl_dsi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_dsi.h):
+drivers/gpu/drm/i915/display/intel_acpi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_alpm.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_atomic.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_audio.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_backlight.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_bios.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_bw.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_casf.c:147:21: error: too long token expansion
+drivers/gpu/drm/i915/display/intel_casf.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_cdclk.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_color.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_colorop.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_combo_phy.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_connector.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_crtc.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_crt.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_crtc_state_dump.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_cursor.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_cx0_phy.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dbuf_bw.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_ddi_buf_trans.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_ddi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_debugfs.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_device.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_driver.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_irq.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_display_power.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_power_map.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_power_well.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_reset.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_rps.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dmc.c:131:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:134:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:137:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:140:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:143:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:146:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:149:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:153:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:154:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:157:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:160:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:163:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:166:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:170:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:174:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:178:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:182:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c:186:1: error: bad constant expression
+drivers/gpu/drm/i915/display/intel_dmc.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp_aux.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp_hdcp.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dpio_phy.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp_link_training.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dpll.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dpll_mgr.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp_mst.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dpt.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dpt_common.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dp_test.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_drrs.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dsb.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dsi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_dsi.h):
+drivers/gpu/drm/i915/display/intel_dsi_dcs_backlight.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dsi_vbt.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_dvo.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_encoder.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_fb_bo.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_fbc.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_fb.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_fb_pin.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_fdi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_fifo_underrun.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_flipq.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_frontbuffer.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_global_state.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_gmbus.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_hdcp.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_hdcp_gsc_message.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_hdmi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_hotplug.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_hotplug_irq.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_initial_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_link_bw.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_load_detect.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_lspcon.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_lt_phy.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_lvds.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_modeset_lock.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_modeset_setup.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_modeset_verify.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_opregion.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_overlay.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_panel.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_pch_display.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_pch_refclk.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_pfit.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_pipe_crc.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_pmdemand.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/intel_pps.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_psr.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_quirks.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_sdvo.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_snps_hdmi_pll.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_snps_phy.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_sprite.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_sprite_uapi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_tc.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_tv.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_vblank.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_vdsc.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_vga.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_vrr.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_wm.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/skl_prefill.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/skl_scaler.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h, drivers/gpu/drm/i915/display/intel_display_trace.h):
+drivers/gpu/drm/i915/display/skl_universal_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/skl_watermark.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/vlv_clock.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/vlv_dsi.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/vlv_dsi_pll.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/vlv_sideband.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/gem/i915_gem_pages.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/i915_initial_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/i915_panic.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression

+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel

✓ Xe.CI.BAT: success for drm: Color pipeline teardown and follow-up fixes/improvements (rev3)

[Patchwork]

[-- Attachment #1: Type: text/plain, Size: 1951 bytes --]

== Series Details ==

Series: drm: Color pipeline teardown and follow-up fixes/improvements (rev3)
URL   : https://patchwork.freedesktop.org/series/159263/
State : success

== Summary ==

CI Bug Log - changes from xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998_BAT -> xe-pw-159263v3_BAT
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Participating hosts (12 -> 12)
------------------------------

  No changes in participating hosts

Known issues
------------

  Here are the changes found in xe-pw-159263v3_BAT that come from known issues:

### IGT changes ###

#### Possible fixes ####

  * igt@xe_waitfence@abstime:
    - bat-dg2-oem2:       [TIMEOUT][1] ([Intel XE#6506]) -> [PASS][2]
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/bat-dg2-oem2/igt@xe_waitfence@abstime.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/bat-dg2-oem2/igt@xe_waitfence@abstime.html

  * igt@xe_waitfence@engine:
    - bat-dg2-oem2:       [FAIL][3] ([Intel XE#6519]) -> [PASS][4]
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/bat-dg2-oem2/igt@xe_waitfence@engine.html
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/bat-dg2-oem2/igt@xe_waitfence@engine.html

  
  [Intel XE#6506]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6506
  [Intel XE#6519]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6519


Build changes
-------------

  * Linux: xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998 -> xe-pw-159263v3

  IGT_8699: 0b67ab25f2eb58b296872c8c34474b79353727d5 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998: 0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998
  xe-pw-159263v3: 159263v3

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/index.html

[-- Attachment #2: Type: text/html, Size: 2550 bytes --]

✗ Xe.CI.Full: failure for drm: Color pipeline teardown and follow-up fixes/improvements (rev3)

[Patchwork]

[-- Attachment #1: Type: text/plain, Size: 10106 bytes --]

== Series Details ==

Series: drm: Color pipeline teardown and follow-up fixes/improvements (rev3)
URL   : https://patchwork.freedesktop.org/series/159263/
State : failure

== Summary ==

CI Bug Log - changes from xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998_FULL -> xe-pw-159263v3_FULL
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with xe-pw-159263v3_FULL absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in xe-pw-159263v3_FULL, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Participating hosts (2 -> 2)
------------------------------

  No changes in participating hosts

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in xe-pw-159263v3_FULL:

### IGT changes ###

#### Possible regressions ####

  * igt@kms_atomic_transition@plane-all-modeset-transition-fencing:
    - shard-bmg:          [PASS][1] -> [INCOMPLETE][2] +1 other test incomplete
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-bmg-4/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-2/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html

  
Known issues
------------

  Here are the changes found in xe-pw-159263v3_FULL that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_big_fb@y-tiled-16bpp-rotate-90:
    - shard-bmg:          NOTRUN -> [SKIP][3] ([Intel XE#1124])
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_big_fb@y-tiled-16bpp-rotate-90.html

  * igt@kms_bw@connected-linear-tiling-4-displays-2560x1440p:
    - shard-bmg:          NOTRUN -> [SKIP][4] ([Intel XE#2314] / [Intel XE#2894])
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_bw@connected-linear-tiling-4-displays-2560x1440p.html

  * igt@kms_chamelium_hpd@common-hpd-after-hibernate:
    - shard-bmg:          NOTRUN -> [SKIP][5] ([Intel XE#2252])
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_chamelium_hpd@common-hpd-after-hibernate.html

  * igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions:
    - shard-bmg:          NOTRUN -> [SKIP][6] ([Intel XE#2286])
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions.html

  * igt@kms_frontbuffer_tracking@fbcdrrs-1p-offscreen-pri-indfb-draw-mmap-wc:
    - shard-bmg:          NOTRUN -> [SKIP][7] ([Intel XE#2311])
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_frontbuffer_tracking@fbcdrrs-1p-offscreen-pri-indfb-draw-mmap-wc.html

  * igt@kms_hdr@bpc-switch@pipe-a-dp-2:
    - shard-bmg:          [PASS][8] -> [ABORT][9] ([Intel XE#6740]) +5 other tests abort
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-bmg-4/igt@kms_hdr@bpc-switch@pipe-a-dp-2.html
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-8/igt@kms_hdr@bpc-switch@pipe-a-dp-2.html

  * igt@kms_panel_fitting@atomic-fastset:
    - shard-bmg:          NOTRUN -> [SKIP][10] ([Intel XE#2486])
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_panel_fitting@atomic-fastset.html

  * igt@kms_pm_backlight@fade-with-suspend:
    - shard-bmg:          NOTRUN -> [SKIP][11] ([Intel XE#870])
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_pm_backlight@fade-with-suspend.html

  * igt@kms_psr2_sf@psr2-overlay-plane-update-continuous-sf:
    - shard-bmg:          NOTRUN -> [SKIP][12] ([Intel XE#1406] / [Intel XE#1489])
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_psr2_sf@psr2-overlay-plane-update-continuous-sf.html

  * igt@kms_vrr@flipline:
    - shard-bmg:          NOTRUN -> [SKIP][13] ([Intel XE#1499])
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_vrr@flipline.html

  * igt@testdisplay:
    - shard-bmg:          NOTRUN -> [ABORT][14] ([Intel XE#6740] / [Intel XE#6976])
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@testdisplay.html

  * igt@xe_eudebug@multiple-sessions:
    - shard-bmg:          NOTRUN -> [SKIP][15] ([Intel XE#4837])
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@xe_eudebug@multiple-sessions.html

  * igt@xe_eudebug_online@writes-caching-sram-bb-sram-target-vram:
    - shard-bmg:          NOTRUN -> [SKIP][16] ([Intel XE#4837] / [Intel XE#6665])
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@xe_eudebug_online@writes-caching-sram-bb-sram-target-vram.html

  * igt@xe_exec_multi_queue@max-queues-preempt-mode-basic:
    - shard-bmg:          NOTRUN -> [SKIP][17] ([Intel XE#6874])
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@xe_exec_multi_queue@max-queues-preempt-mode-basic.html

  * igt@xe_exec_system_allocator@pat-index-madvise-pat-idx-uc-single-vma:
    - shard-lnl:          [PASS][18] -> [FAIL][19] ([Intel XE#5625])
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-lnl-2/igt@xe_exec_system_allocator@pat-index-madvise-pat-idx-uc-single-vma.html
   [19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-lnl-1/igt@xe_exec_system_allocator@pat-index-madvise-pat-idx-uc-single-vma.html

  * igt@xe_exec_system_allocator@process-many-large-mmap-huge:
    - shard-bmg:          NOTRUN -> [SKIP][20] ([Intel XE#4943]) +1 other test skip
   [20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@xe_exec_system_allocator@process-many-large-mmap-huge.html

  
#### Possible fixes ####

  * igt@kms_flip@2x-flip-vs-dpms-on-nop-interruptible:
    - shard-bmg:          [FAIL][21] ([Intel XE#3098]) -> [PASS][22] +1 other test pass
   [21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-bmg-4/igt@kms_flip@2x-flip-vs-dpms-on-nop-interruptible.html
   [22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-7/igt@kms_flip@2x-flip-vs-dpms-on-nop-interruptible.html

  * igt@kms_flip@flip-vs-expired-vblank@a-edp1:
    - shard-lnl:          [FAIL][23] ([Intel XE#301]) -> [PASS][24] +2 other tests pass
   [23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-lnl-7/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
   [24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-lnl-1/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html

  * igt@xe_evict@evict-mixed-many-threads-small:
    - shard-bmg:          [INCOMPLETE][25] ([Intel XE#6321]) -> [PASS][26] +1 other test pass
   [25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-bmg-3/igt@xe_evict@evict-mixed-many-threads-small.html
   [26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-8/igt@xe_evict@evict-mixed-many-threads-small.html

  
#### Warnings ####

  * igt@kms_tiled_display@basic-test-pattern-with-chamelium:
    - shard-bmg:          [SKIP][27] ([Intel XE#2426]) -> [SKIP][28] ([Intel XE#2509])
   [27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998/shard-bmg-1/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
   [28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/shard-bmg-6/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html

  
  [Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
  [Intel XE#1406]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1406
  [Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
  [Intel XE#1499]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1499
  [Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
  [Intel XE#2286]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2286
  [Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
  [Intel XE#2314]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2314
  [Intel XE#2426]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2426
  [Intel XE#2486]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2486
  [Intel XE#2509]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2509
  [Intel XE#2894]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2894
  [Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
  [Intel XE#3098]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3098
  [Intel XE#4837]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4837
  [Intel XE#4943]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4943
  [Intel XE#5625]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5625
  [Intel XE#6321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6321
  [Intel XE#6665]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6665
  [Intel XE#6740]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6740
  [Intel XE#6874]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6874
  [Intel XE#6976]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6976
  [Intel XE#870]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/870


Build changes
-------------

  * Linux: xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998 -> xe-pw-159263v3

  IGT_8699: 0b67ab25f2eb58b296872c8c34474b79353727d5 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  xe-4375-0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998: 0e1b1ce1c8adbc0a8ad1ee763a8a9eeaee297998
  xe-pw-159263v3: 159263v3

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159263v3/index.html

[-- Attachment #2: Type: text/html, Size: 11266 bytes --]

Re: [PATCH v3 00/13] drm: Color pipeline teardown and follow-up fixes/improvements

[Jani Nikula]

On Tue, 13 Jan 2026, Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com> wrote:
> This series contains follow-up fixes and improvements for the DRM color
> pipeline infrastructure that was introduced in v6.19.[1][2]
>
> The central handling of clean up of colorop from the mode_config list
> is missing. While vkms calls drm_colorop_pipeline_destroy() in vkms_destroy(),
> amd driver calls it only during failure of the init path and i915/xe driver
> does not call it at all. This means amd and intel leaks these objects on
> driver removal.
>
> This series adds the teardown of mode_config.colorop_list in drm_mode_config_cleanup().
> Since, i915/xe sub-classes the drm_colorop within intel_colorop it was not enough
> to just use drm_colorop_pipeline_destroy(). Therefore, this series
>
> - Introduces driver-managed destruction for drm_colorop objects and
>   updates core helpers to use driver-provided destroy callbacks.
> - Ensures all colorop objects are correctly torn down during
>   mode_config cleanup and driver removal.
>
> In addition to that following changes are made in the series
> - Fixes enum name lifetime leaks in color pipeline init in i915, amdgpu_dm, and vkms
> - Corrects the ordering of the 3D LUT block in the i915 plane color pipeline
> - Refactors i915 plane color pipeline initialization to reliably clean
>   up partially constructed pipelines on failure.
>
> Thanks for taking a look. Feedback is welcome.

I did not do detailed review, but

Acked-by: Jani Nikula <jani.nikula@intel.com>

for merging via drm-misc.

Please coordinate with drm and drm-misc maintainers on which branch
these should merged through. IIUC there are memory leak fixes for
changes heading to v6.19, which speaks for drm-misc-fixes. But is it too
much at this stage? Up to drm and drm-misc maintainers I think.

BR,
Jani.


>
> [1] https://lore.kernel.org/dri-devel/cbe00ac4-a535-47d3-813a-e2eda7e9b991@amd.com/
> [2] https://lore.kernel.org/intel-gfx/20251203085211.3663374-1-uma.shankar@intel.com/
>
> v2:
>  - Re-arrange patches (Alex)
>  - Re-factor code to avoid repitition in pipeline creation (Suraj)
>
> v3:
>  - Add documentation only to function definition (Jani)
>  - s/nvl/xe3plpd (Suraj)
>
> Chaitanya Kumar Borah (13):
>   drm/i915/color: Place 3D LUT after CSC in plane color pipeline
>   drm/amd/display: Fix color pipeline enum name leak
>   drm/vkms: Fix color pipeline enum name leak
>   drm/i915/display: Fix color pipeline enum name leak
>   drm/colorop: Add destroy helper for colorop objects
>   drm: Allow driver-managed destruction of colorop objects
>   drm/amd/display: Hook up colorop destroy helper for plane pipelines
>   drm/vkms: Hook up colorop destroy helper for plane pipelines
>   drm/i915/display: Hook up intel_colorop_destroy
>   drm: Clean up colorop objects during mode_config cleanup
>   drm/vkms: Remove drm_colorop_pipeline_destroy() from vkms_destroy()
>   drm/colorop: Use destroy callback for color pipeline teardown
>   drm/i915/color: Add failure handling in plane color pipeline init
>
>  .../amd/display/amdgpu_dm/amdgpu_dm_colorop.c |  31 ++-
>  .../amd/display/amdgpu_dm/amdgpu_dm_plane.c   |  13 +-
>  drivers/gpu/drm/drm_colorop.c                 |  49 +++--
>  drivers/gpu/drm/drm_mode_config.c             |   6 +
>  .../drm/i915/display/intel_color_pipeline.c   | 179 +++++++++++++-----
>  drivers/gpu/drm/i915/display/intel_colorop.c  |   6 +
>  drivers/gpu/drm/i915/display/intel_colorop.h  |   1 +
>  drivers/gpu/drm/vkms/vkms_colorop.c           |  31 +--
>  drivers/gpu/drm/vkms/vkms_drv.c               |   1 -
>  include/drm/drm_colorop.h                     |  32 +++-
>  10 files changed, 259 insertions(+), 90 deletions(-)

-- 
Jani Nikula, Intel

Re: [PATCH v3 00/13] drm: Color pipeline teardown and follow-up fixes/improvements

[Maarten Lankhorst]

Hey,

Den 2026-01-21 kl. 14:51, skrev Jani Nikula:
> On Tue, 13 Jan 2026, Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com> wrote:
>> This series contains follow-up fixes and improvements for the DRM color
>> pipeline infrastructure that was introduced in v6.19.[1][2]
>>
>> The central handling of clean up of colorop from the mode_config list
>> is missing. While vkms calls drm_colorop_pipeline_destroy() in vkms_destroy(),
>> amd driver calls it only during failure of the init path and i915/xe driver
>> does not call it at all. This means amd and intel leaks these objects on
>> driver removal.
>>
>> This series adds the teardown of mode_config.colorop_list in drm_mode_config_cleanup().
>> Since, i915/xe sub-classes the drm_colorop within intel_colorop it was not enough
>> to just use drm_colorop_pipeline_destroy(). Therefore, this series
>>
>> - Introduces driver-managed destruction for drm_colorop objects and
>>   updates core helpers to use driver-provided destroy callbacks.
>> - Ensures all colorop objects are correctly torn down during
>>   mode_config cleanup and driver removal.
>>
>> In addition to that following changes are made in the series
>> - Fixes enum name lifetime leaks in color pipeline init in i915, amdgpu_dm, and vkms
>> - Corrects the ordering of the 3D LUT block in the i915 plane color pipeline
>> - Refactors i915 plane color pipeline initialization to reliably clean
>>   up partially constructed pipelines on failure.
>>
>> Thanks for taking a look. Feedback is welcome.
> 
> I did not do detailed review, but
> 
> Acked-by: Jani Nikula <jani.nikula@intel.com>
> 
> for merging via drm-misc.
> 
> Please coordinate with drm and drm-misc maintainers on which branch
> these should merged through. IIUC there are memory leak fixes for
> changes heading to v6.19, which speaks for drm-misc-fixes. But is it too
> much at this stage? Up to drm and drm-misc maintainers I think.

I pushed the first 4 patches from this series as they have to be
addressed most urgently. It's a leak affecting each system and
incorrect UAPI being presented for intel/display colorops.

We need to discuss on how to merge patches 5-13. Should we put
it in a topic branch or push it to drm-misc-next-fixes?

Kind regards,
~Maarten Lankhorst

Re: [PATCH v3 00/13] drm: Color pipeline teardown and follow-up fixes/improvements

[Borah, Chaitanya Kumar]

On 1/22/2026 3:27 PM, Maarten Lankhorst wrote:
> Hey,
> 
> Den 2026-01-21 kl. 14:51, skrev Jani Nikula:
>> On Tue, 13 Jan 2026, Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com> wrote:
>>> This series contains follow-up fixes and improvements for the DRM color
>>> pipeline infrastructure that was introduced in v6.19.[1][2]
>>>
>>> The central handling of clean up of colorop from the mode_config list
>>> is missing. While vkms calls drm_colorop_pipeline_destroy() in vkms_destroy(),
>>> amd driver calls it only during failure of the init path and i915/xe driver
>>> does not call it at all. This means amd and intel leaks these objects on
>>> driver removal.
>>>
>>> This series adds the teardown of mode_config.colorop_list in drm_mode_config_cleanup().
>>> Since, i915/xe sub-classes the drm_colorop within intel_colorop it was not enough
>>> to just use drm_colorop_pipeline_destroy(). Therefore, this series
>>>
>>> - Introduces driver-managed destruction for drm_colorop objects and
>>>    updates core helpers to use driver-provided destroy callbacks.
>>> - Ensures all colorop objects are correctly torn down during
>>>    mode_config cleanup and driver removal.
>>>
>>> In addition to that following changes are made in the series
>>> - Fixes enum name lifetime leaks in color pipeline init in i915, amdgpu_dm, and vkms
>>> - Corrects the ordering of the 3D LUT block in the i915 plane color pipeline
>>> - Refactors i915 plane color pipeline initialization to reliably clean
>>>    up partially constructed pipelines on failure.
>>>
>>> Thanks for taking a look. Feedback is welcome.
>>
>> I did not do detailed review, but
>>
>> Acked-by: Jani Nikula <jani.nikula@intel.com>
>>
>> for merging via drm-misc.
>>
>> Please coordinate with drm and drm-misc maintainers on which branch
>> these should merged through. IIUC there are memory leak fixes for
>> changes heading to v6.19, which speaks for drm-misc-fixes. But is it too
>> much at this stage? Up to drm and drm-misc maintainers I think.
> 
> I pushed the first 4 patches from this series as they have to be
> addressed most urgently. It's a leak affecting each system and
> incorrect UAPI being presented for intel/display colorops.
> 
> We need to discuss on how to merge patches 5-13. Should we put
> it in a topic branch or push it to drm-misc-next-fixes?
> 

Thank you very much, Maarten, for merging the patches.

In case the "Fixes" tag is mandatory to push through 
drm-misc-next-fixes/drm-misc-fixes, the best approach I see would be to 
add the following tags on the drm core patches. (i.e the first patches 
that added colorop for respective drivers)

Fixes: ef105316819d ("drm/i915/color: Create a transfer function color 
pipeline")
Fixes: c1e578bd08da ("drm/vkms: Add enumerated 1D curve colorop")
Fixes: 9ba25915efba ("drm/amd/display: Add support for sRGB EOTF in 
DEGAM block")

and then add the corresponding Fixes tag to the indivisual 
driver-specific patches.

This won't be totally accurate for vkms as it was already cleaning up 
the colorops but the patches are too intertwined to isolate it.

Please let me know if this sounds reasonable.

Thanks also to Jani for the ack.

==
Chaitanya

> Kind regards,
> ~Maarten Lankhorst