[PATCH v3 0/4] User readable error codes on atomic_ioctl failure

[PATCH v3 0/4] User readable error codes on atomic_ioctl failure

[Arun R Murthy]

The series focuses on providing a user readable error value on a failure
in drm_atomic_ioctl(). Usually -EINVAL is returned in most of the error
cases and it is difficult for the user to decode the error and get to
know the real cause for the error. If user gets to know the reason for
the error then corrective measurements can be taken up.

TODO: driver specific error codes are to be added and will be done in
the follow-up patches.

The IGT related changes are pushed for review @
https://patchwork.freedesktop.org/series/153330/

Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
---
Arun R Murthy (4):
      drm: Define user readable error codes for atomic ioctl
      drm/atomic: Add error_code element in atomic_state
      drm/atomic: Return user readable error in atomic_ioctl
      drm/i915/display: Error codes for async flip failures

 drivers/gpu/drm/drm_atomic.c                 |  6 +++
 drivers/gpu/drm/drm_atomic_uapi.c            | 60 +++++++++++++++++++++++-----
 drivers/gpu/drm/i915/display/intel_display.c |  4 ++
 include/drm/drm_atomic.h                     |  7 ++++
 include/uapi/drm/drm_mode.h                  | 42 +++++++++++++++++++
 5 files changed, 109 insertions(+), 10 deletions(-)
---
base-commit: cca87ca63e2f5b8a785dc59c23e526987530b27f
change-id: 20250728-atomic-c9713fd357e4

Best regards,
-- 
Arun R Murthy <arun.r.murthy@intel.com>

[PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Arun R Murthy]

There can be multiple reasons for a failure in atomic_ioctl. Most often
in these error conditions -EINVAL is returned. User/Compositor would
have to blindly take a call on failure of this ioctl so as to use
ALLOW_MODESET or any. It would be good if user/compositor gets a
readable error code on failure so they can take proper corrections in
the next commit.
The struct drm_mode_atomic is being passed by the user/compositor which
holds the properties for modeset/flip. Reusing the same struct for
returning the error code in case of failure can save by creating a new
uapi/interface for returning the error code.
The element 'reserved' in the struct drm_mode_atomic is used for
returning the user readable error code. This points to the struct
drm_mode_atomic_err_code. Failure reasons have been initialized in
DRM_MODE_ATOMIC_FAILURE_REASON.

Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
---
 include/uapi/drm/drm_mode.h | 42 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 42 insertions(+)

diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index a122bea2559387576150236e3a88f99c24ad3138..f0986a3fe9a7d61e57e9a9a5ec01a604343f6930 100644
--- a/include/uapi/drm/drm_mode.h
+++ b/include/uapi/drm/drm_mode.h
@@ -45,6 +45,7 @@ extern "C" {
 #define DRM_CONNECTOR_NAME_LEN	32
 #define DRM_DISPLAY_MODE_LEN	32
 #define DRM_PROP_NAME_LEN	32
+#define DRM_MODE_ATOMIC_FAILURE_STRING_LEN	64
 
 #define DRM_MODE_TYPE_BUILTIN	(1<<0) /* deprecated */
 #define DRM_MODE_TYPE_CLOCK_C	((1<<1) | DRM_MODE_TYPE_BUILTIN) /* deprecated */
@@ -1157,6 +1158,47 @@ struct drm_mode_destroy_dumb {
 		DRM_MODE_ATOMIC_NONBLOCK |\
 		DRM_MODE_ATOMIC_ALLOW_MODESET)
 
+#define DRM_MODE_ATOMIC_FAILURE_REASON \
+	FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
+
+#define FAILURE_REASON(flag, reason) flag,
+typedef enum {
+	DRM_MODE_ATOMIC_FAILURE_REASON
+} drm_mode_atomic_failure_flag;
+#undef FAILURE_REASON
+
+#define FAILURE_REASON(flag, reason) #reason,
+extern const char *drm_mode_atomic_failure_string[];
+#undef FAILURE_REASON
+
+/**
+ * drm_mode_atomic_err_code - struct to store the error code
+ *
+ * pointer to this struct will be stored in reserved variable of
+ * struct drm_mode_atomic to report the failure cause to the user.
+ *
+ * @failure_flags: error codes defined in drm_atomic_failure.failure_flag
+ * @failure_string_ptr: pointer to user readable error message drm_mode_failure.failure_string
+ * @failure_obj_ptr: pointer to the drm_object that caused error
+ * @reserved: reserved for future use
+ * @count_objs: count of drm_objects if multiple drm_objects caused error
+ */
+struct drm_mode_atomic_err_code {
+	__u64 failure_flags;
+	__u64 failure_objs_ptr;
+	__u64 reserved;
+	__u32 count_objs;
+	char failure_string[DRM_MODE_ATOMIC_FAILURE_STRING_LEN];
+};
+
 struct drm_mode_atomic {
 	__u32 flags;
 	__u32 count_objs;

-- 
2.25.1

[PATCH v3 2/4] drm/atomic: Add error_code element in atomic_state

[Arun R Murthy]

Now that a proper error code will be returned to the user on any failure
in atomic_ioctl() via struct drm_mode_atomic, add a new element
error_code in the struct drm_atomic_state so as to hold the error code
during the atomic_check() and atomic_commit() phases.

Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
---
 include/drm/drm_atomic.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index 38636a593c9d98cadda85ccd67326cb152f0dd27..d380001b24b4223baa54dae6c3c43e19dfb1958d 100644
--- a/include/drm/drm_atomic.h
+++ b/include/drm/drm_atomic.h
@@ -524,6 +524,13 @@ struct drm_atomic_state {
 	 * commit without blocking.
 	 */
 	struct work_struct commit_work;
+
+	/* @error_code
+	 *
+	 * struct to convey user readable error to the user.
+	 * Error codes defined in enum drm_mode_atomic_failure_flags
+	 */
+	struct drm_mode_atomic_err_code *error_code;
 };
 
 void __drm_crtc_commit_free(struct kref *kref);

-- 
2.25.1

[PATCH v3 3/4] drm/atomic: Return user readable error in atomic_ioctl

[Arun R Murthy]

Add user readable error codes for failure cases in drm_atomic_ioctl() so
that user can decode the error code and take corrective measurements.

Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
---
 drivers/gpu/drm/drm_atomic.c      |  6 ++++
 drivers/gpu/drm/drm_atomic_uapi.c | 60 ++++++++++++++++++++++++++++++++-------
 2 files changed, 56 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index cd15cf52f0c9144711da5879da57884674aea9e4..5f25e6d3cf6cf246f83a8c39450b410e97fe45bb 100644
--- a/drivers/gpu/drm/drm_atomic.c
+++ b/drivers/gpu/drm/drm_atomic.c
@@ -1513,6 +1513,9 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
 			if (drm_atomic_crtc_needs_modeset(new_crtc_state)) {
 				drm_dbg_atomic(dev, "[CRTC:%d:%s] requires full modeset\n",
 					       crtc->base.id, crtc->name);
+				state->error_code->failure_flags =
+					DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
+
 				return -EINVAL;
 			}
 		}
@@ -1537,8 +1540,11 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
 		drm_dbg_atomic(dev,
 			       "driver added CRTC to commit: requested 0x%x, affected 0x%0x\n",
 			       requested_crtc, affected_crtc);
+		state->error_code->failure_flags = DRM_MODE_ATOMIC_NEED_FULL_MODESET;
 		WARN(!state->allow_modeset, "adding CRTC not allowed without modesets: requested 0x%x, affected 0x%0x\n",
 		     requested_crtc, affected_crtc);
+
+		return -EINVAL;
 	}
 
 	return 0;
diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic_uapi.c
index ecc73d52bfae41a7ef455a7e13649ec56c690b90..94eaf9c98eb4ac2455799f1416010d366e1b5bbc 100644
--- a/drivers/gpu/drm/drm_atomic_uapi.c
+++ b/drivers/gpu/drm/drm_atomic_uapi.c
@@ -1058,6 +1058,9 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
 			ret = drm_atomic_crtc_get_property(crtc, crtc_state,
 							   prop, &old_val);
 			ret = drm_atomic_check_prop_changes(ret, old_val, prop_value, prop);
+			if (ret)
+				state->error_code->failure_flags =
+					DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED;
 			break;
 		}
 
@@ -1089,6 +1092,8 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
 
 			/* ask the driver if this non-primary plane is supported */
 			if (plane->type != DRM_PLANE_TYPE_PRIMARY) {
+				state->error_code->failure_flags =
+					DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE;
 				ret = -EINVAL;
 
 				if (plane_funcs && plane_funcs->atomic_async_check)
@@ -1380,6 +1385,13 @@ set_async_flip(struct drm_atomic_state *state)
 	}
 }
 
+#define FAILURE_REASON(flag, reason) #reason,
+const char *drm_mode_atomic_failure_string[] = {
+	DRM_MODE_ATOMIC_FAILURE_REASON
+};
+
+#undef FAILURE_REASON
+
 int drm_mode_atomic_ioctl(struct drm_device *dev,
 			  void *data, struct drm_file *file_priv)
 {
@@ -1389,9 +1401,11 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
 	uint32_t __user *props_ptr = (uint32_t __user *)(unsigned long)(arg->props_ptr);
 	uint64_t __user *prop_values_ptr = (uint64_t __user *)(unsigned long)(arg->prop_values_ptr);
 	unsigned int copied_objs, copied_props;
-	struct drm_atomic_state *state;
+	struct drm_atomic_state *state = NULL;
 	struct drm_modeset_acquire_ctx ctx;
 	struct drm_out_fence_state *fence_state;
+	struct drm_mode_atomic_err_code error_code;
+	struct drm_mode_atomic_err_code __user *error_code_ptr;
 	int ret = 0;
 	unsigned int i, j, num_fences;
 	bool async_flip = false;
@@ -1400,6 +1414,11 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
 	if (!drm_core_check_feature(dev, DRIVER_ATOMIC))
 		return -EOPNOTSUPP;
 
+	if (!arg->reserved)
+		drm_err(dev, "memory not allocated for drm_atomic error reporting\n");
+
+	memset(&error_code, 0, sizeof(struct drm_mode_atomic_err_code));
+
 	/* disallow for userspace that has not enabled atomic cap (even
 	 * though this may be a bit overkill, since legacy userspace
 	 * wouldn't know how to call this ioctl)
@@ -1407,24 +1426,25 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
 	if (!file_priv->atomic) {
 		drm_dbg_atomic(dev,
 			       "commit failed: atomic cap not enabled\n");
-		return -EINVAL;
+		error_code.failure_flags = DRM_MODE_ATOMIC_CAP_NOT_ENABLED;
+		ret = -EINVAL;
+		goto out;
 	}
 
 	if (arg->flags & ~DRM_MODE_ATOMIC_FLAGS) {
 		drm_dbg_atomic(dev, "commit failed: invalid flag\n");
-		return -EINVAL;
-	}
-
-	if (arg->reserved) {
-		drm_dbg_atomic(dev, "commit failed: reserved field set\n");
-		return -EINVAL;
+		error_code.failure_flags = DRM_MODE_ATOMIC_INVALID_FLAG;
+		ret = -EINVAL;
+		goto out;
 	}
 
 	if (arg->flags & DRM_MODE_PAGE_FLIP_ASYNC) {
 		if (!dev->mode_config.async_page_flip) {
 			drm_dbg_atomic(dev,
 				       "commit failed: DRM_MODE_PAGE_FLIP_ASYNC not supported\n");
-			return -EINVAL;
+			error_code.failure_flags = DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC;
+			ret = -EINVAL;
+			goto out;
 		}
 
 		async_flip = true;
@@ -1435,7 +1455,9 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
 			(arg->flags & DRM_MODE_PAGE_FLIP_EVENT)) {
 		drm_dbg_atomic(dev,
 			       "commit failed: page-flip event requested with test-only commit\n");
-		return -EINVAL;
+		error_code.failure_flags = DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY;
+		ret = -EINVAL;
+		goto out;
 	}
 
 	state = drm_atomic_state_alloc(dev);
@@ -1446,6 +1468,8 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
 	state->acquire_ctx = &ctx;
 	state->allow_modeset = !!(arg->flags & DRM_MODE_ATOMIC_ALLOW_MODESET);
 
+	state->error_code = &error_code;
+
 retry:
 	copied_objs = 0;
 	copied_props = 0;
@@ -1542,6 +1566,22 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
 	}
 
 out:
+	/* update the error code if any error to allow user handling it */
+	if (ret < 0 && arg->reserved) {
+		error_code_ptr = (struct drm_mode_atomic_err_code __user *)
+				 (unsigned long)arg->reserved;
+
+		strscpy_pad(error_code.failure_string,
+			    drm_mode_atomic_failure_string[error_code.failure_flags],
+			    sizeof(error_code.failure_string));
+
+		if (copy_to_user(error_code_ptr, &error_code, sizeof(error_code)))
+			return -EFAULT;
+	}
+
+	if (!state)
+		return ret;
+
 	complete_signaling(dev, state, fence_state, num_fences, !ret);
 
 	if (ret == -EDEADLK) {

-- 
2.25.1

[PATCH v3 4/4] drm/i915/display: Error codes for async flip failures

[Arun R Murthy]

For failures in async flip atomic check/commit path return user readable
error codes in struct drm_atomic_state.

Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
---
 drivers/gpu/drm/i915/display/intel_display.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index c1a3a95c65f0b66c24ddd64f47dfdc67bbde86c9..5e23f4fc747bd01fa05eba63661bf7279b083317 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -5950,6 +5950,7 @@ static int intel_async_flip_check_uapi(struct intel_atomic_state *state,
 		drm_dbg_kms(display->drm,
 			    "[CRTC:%d:%s] modeset required\n",
 			    crtc->base.base.id, crtc->base.name);
+		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
 		return -EINVAL;
 	}
 
@@ -6019,6 +6020,7 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
 		drm_dbg_kms(display->drm,
 			    "[CRTC:%d:%s] modeset required\n",
 			    crtc->base.base.id, crtc->base.name);
+		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
 		return -EINVAL;
 	}
 
@@ -6061,6 +6063,8 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
 				    plane->base.base.id, plane->base.name,
 				    &new_plane_state->hw.fb->format->format,
 				    new_plane_state->hw.fb->modifier);
+			state->base.error_code->failure_flags =
+				DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED;
 			return -EINVAL;
 		}
 

-- 
2.25.1

✗ CI.checkpatch: warning for User readable error codes on atomic_ioctl failure (rev2)

[Patchwork]

== Series Details ==

Series: User readable error codes on atomic_ioctl failure (rev2)
URL   : https://patchwork.freedesktop.org/series/152277/
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
553439844b6500767ce8aef522cfe9fbb7ece541
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 6157cc6938bb30d35d77ddfe3e937d84b4fe6e2c
Author: Arun R Murthy <arun.r.murthy@intel.com>
Date:   Fri Aug 22 12:30:04 2025 +0530

    drm/i915/display: Error codes for async flip failures
    
    For failures in async flip atomic check/commit path return user readable
    error codes in struct drm_atomic_state.
    
    Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
+ /mt/dim checkpatch cca87ca63e2f5b8a785dc59c23e526987530b27f drm-intel
75f970864f7d drm: Define user readable error codes for atomic ioctl
-:39: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#39: FILE: include/uapi/drm/drm_mode.h:1161:
+#define DRM_MODE_ATOMIC_FAILURE_REASON \
+	FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")

BUT SEE:

   do {} while (0) advice is over-stated in a few situations:

   The more obvious case is macros, like MODULE_PARM_DESC, invoked at
   file-scope, where C disallows code (it must be in functions).  See
   $exceptions if you have one to add by name.

   More troublesome is declarative macros used at top of new scope,
   like DECLARE_PER_CPU.  These might just compile with a do-while-0
   wrapper, but would be incorrect.  Most of these are handled by
   detecting struct,union,etc declaration primitives in $exceptions.

   Theres also macros called inside an if (block), which "return" an
   expression.  These cannot do-while, and need a ({}) wrapper.

   Enjoy this qualification while we work to improve our heuristics.

-:42: WARNING:LONG_LINE: line length of 123 exceeds 100 columns
#42: FILE: include/uapi/drm/drm_mode.h:1164:
+	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \

-:43: WARNING:LONG_LINE: line length of 112 exceeds 100 columns
#43: FILE: include/uapi/drm/drm_mode.h:1165:
+	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \

-:45: WARNING:LONG_LINE: line length of 108 exceeds 100 columns
#45: FILE: include/uapi/drm/drm_mode.h:1167:
+	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \

-:46: WARNING:LONG_LINE: line length of 103 exceeds 100 columns
#46: FILE: include/uapi/drm/drm_mode.h:1168:
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \

-:47: WARNING:LONG_LINE: line length of 126 exceeds 100 columns
#47: FILE: include/uapi/drm/drm_mode.h:1169:
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \

-:48: WARNING:LONG_LINE: line length of 115 exceeds 100 columns
#48: FILE: include/uapi/drm/drm_mode.h:1170:
+	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")

-:50: WARNING:MACRO_ARG_UNUSED: Argument 'reason' is not used in function-like macro
#50: FILE: include/uapi/drm/drm_mode.h:1172:
+#define FAILURE_REASON(flag, reason) flag,

-:51: WARNING:NEW_TYPEDEFS: do not add new typedefs
#51: FILE: include/uapi/drm/drm_mode.h:1173:
+typedef enum {

-:56: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#56: FILE: include/uapi/drm/drm_mode.h:1178:
+#define FAILURE_REASON(flag, reason) #reason,

BUT SEE:

   do {} while (0) advice is over-stated in a few situations:

   The more obvious case is macros, like MODULE_PARM_DESC, invoked at
   file-scope, where C disallows code (it must be in functions).  See
   $exceptions if you have one to add by name.

   More troublesome is declarative macros used at top of new scope,
   like DECLARE_PER_CPU.  These might just compile with a do-while-0
   wrapper, but would be incorrect.  Most of these are handled by
   detecting struct,union,etc declaration primitives in $exceptions.

   Theres also macros called inside an if (block), which "return" an
   expression.  These cannot do-while, and need a ({}) wrapper.

   Enjoy this qualification while we work to improve our heuristics.

-:56: WARNING:MACRO_ARG_UNUSED: Argument 'flag' is not used in function-like macro
#56: FILE: include/uapi/drm/drm_mode.h:1178:
+#define FAILURE_REASON(flag, reason) #reason,

total: 2 errors, 9 warnings, 0 checks, 54 lines checked
2de9aaf954be drm/atomic: Add error_code element in atomic_state
8df799dcfa6e drm/atomic: Return user readable error in atomic_ioctl
-:64: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#64: FILE: drivers/gpu/drm/drm_atomic_uapi.c:1388:
+#define FAILURE_REASON(flag, reason) #reason,

BUT SEE:

   do {} while (0) advice is over-stated in a few situations:

   The more obvious case is macros, like MODULE_PARM_DESC, invoked at
   file-scope, where C disallows code (it must be in functions).  See
   $exceptions if you have one to add by name.

   More troublesome is declarative macros used at top of new scope,
   like DECLARE_PER_CPU.  These might just compile with a do-while-0
   wrapper, but would be incorrect.  Most of these are handled by
   detecting struct,union,etc declaration primitives in $exceptions.

   Theres also macros called inside an if (block), which "return" an
   expression.  These cannot do-while, and need a ({}) wrapper.

   Enjoy this qualification while we work to improve our heuristics.

-:64: WARNING:MACRO_ARG_UNUSED: Argument 'flag' is not used in function-like macro
#64: FILE: drivers/gpu/drm/drm_atomic_uapi.c:1388:
+#define FAILURE_REASON(flag, reason) #reason,

total: 1 errors, 1 warnings, 0 checks, 146 lines checked
6157cc6938bb drm/i915/display: Error codes for async flip failures

✓ CI.KUnit: success for User readable error codes on atomic_ioctl failure (rev2)

[Patchwork]

== Series Details ==

Series: User readable error codes on atomic_ioctl failure (rev2)
URL   : https://patchwork.freedesktop.org/series/152277/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[07:09:05] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[07:09:09] 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=48
[07:09:38] Starting KUnit Kernel (1/1)...
[07:09:38] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[07:09:38] ================== guc_buf (11 subtests) ===================
[07:09:38] [PASSED] test_smallest
[07:09:38] [PASSED] test_largest
[07:09:38] [PASSED] test_granular
[07:09:38] [PASSED] test_unique
[07:09:38] [PASSED] test_overlap
[07:09:38] [PASSED] test_reusable
[07:09:38] [PASSED] test_too_big
[07:09:38] [PASSED] test_flush
[07:09:38] [PASSED] test_lookup
[07:09:38] [PASSED] test_data
[07:09:38] [PASSED] test_class
[07:09:38] ===================== [PASSED] guc_buf =====================
[07:09:38] =================== guc_dbm (7 subtests) ===================
[07:09:38] [PASSED] test_empty
[07:09:38] [PASSED] test_default
[07:09:38] ======================== test_size  ========================
[07:09:38] [PASSED] 4
[07:09:38] [PASSED] 8
[07:09:38] [PASSED] 32
[07:09:38] [PASSED] 256
[07:09:38] ==================== [PASSED] test_size ====================
[07:09:38] ======================= test_reuse  ========================
[07:09:38] [PASSED] 4
[07:09:38] [PASSED] 8
[07:09:38] [PASSED] 32
[07:09:38] [PASSED] 256
[07:09:38] =================== [PASSED] test_reuse ====================
[07:09:38] =================== test_range_overlap  ====================
[07:09:38] [PASSED] 4
[07:09:38] [PASSED] 8
[07:09:38] [PASSED] 32
[07:09:38] [PASSED] 256
[07:09:38] =============== [PASSED] test_range_overlap ================
[07:09:38] =================== test_range_compact  ====================
[07:09:38] [PASSED] 4
[07:09:38] [PASSED] 8
[07:09:38] [PASSED] 32
[07:09:38] [PASSED] 256
[07:09:38] =============== [PASSED] test_range_compact ================
[07:09:38] ==================== test_range_spare  =====================
[07:09:38] [PASSED] 4
[07:09:38] [PASSED] 8
[07:09:38] [PASSED] 32
[07:09:38] [PASSED] 256
[07:09:38] ================ [PASSED] test_range_spare =================
[07:09:38] ===================== [PASSED] guc_dbm =====================
[07:09:38] =================== guc_idm (6 subtests) ===================
[07:09:38] [PASSED] bad_init
[07:09:38] [PASSED] no_init
[07:09:38] [PASSED] init_fini
[07:09:38] [PASSED] check_used
[07:09:38] [PASSED] check_quota
[07:09:38] [PASSED] check_all
[07:09:38] ===================== [PASSED] guc_idm =====================
[07:09:38] ================== no_relay (3 subtests) ===================
[07:09:38] [PASSED] xe_drops_guc2pf_if_not_ready
[07:09:38] [PASSED] xe_drops_guc2vf_if_not_ready
[07:09:38] [PASSED] xe_rejects_send_if_not_ready
[07:09:38] ==================== [PASSED] no_relay =====================
[07:09:38] ================== pf_relay (14 subtests) ==================
[07:09:38] [PASSED] pf_rejects_guc2pf_too_short
[07:09:38] [PASSED] pf_rejects_guc2pf_too_long
[07:09:38] [PASSED] pf_rejects_guc2pf_no_payload
[07:09:38] [PASSED] pf_fails_no_payload
[07:09:38] [PASSED] pf_fails_bad_origin
[07:09:38] [PASSED] pf_fails_bad_type
[07:09:38] [PASSED] pf_txn_reports_error
[07:09:38] [PASSED] pf_txn_sends_pf2guc
[07:09:38] [PASSED] pf_sends_pf2guc
[07:09:38] [SKIPPED] pf_loopback_nop
[07:09:38] [SKIPPED] pf_loopback_echo
[07:09:38] [SKIPPED] pf_loopback_fail
[07:09:38] [SKIPPED] pf_loopback_busy
[07:09:38] [SKIPPED] pf_loopback_retry
[07:09:38] ==================== [PASSED] pf_relay =====================
[07:09:38] ================== vf_relay (3 subtests) ===================
[07:09:38] [PASSED] vf_rejects_guc2vf_too_short
[07:09:38] [PASSED] vf_rejects_guc2vf_too_long
[07:09:38] [PASSED] vf_rejects_guc2vf_no_payload
[07:09:38] ==================== [PASSED] vf_relay =====================
[07:09:38] ===================== lmtt (1 subtest) =====================
[07:09:38] ======================== test_ops  =========================
[07:09:38] [PASSED] 2-level
[07:09:38] [PASSED] multi-level
[07:09:38] ==================== [PASSED] test_ops =====================
[07:09:38] ====================== [PASSED] lmtt =======================
[07:09:38] ================= pf_service (11 subtests) =================
[07:09:38] [PASSED] pf_negotiate_any
[07:09:38] [PASSED] pf_negotiate_base_match
[07:09:38] [PASSED] pf_negotiate_base_newer
[07:09:38] [PASSED] pf_negotiate_base_next
[07:09:38] [SKIPPED] pf_negotiate_base_older
[07:09:38] [PASSED] pf_negotiate_base_prev
[07:09:38] [PASSED] pf_negotiate_latest_match
[07:09:38] [PASSED] pf_negotiate_latest_newer
[07:09:38] [PASSED] pf_negotiate_latest_next
[07:09:38] [SKIPPED] pf_negotiate_latest_older
[07:09:38] [SKIPPED] pf_negotiate_latest_prev
[07:09:38] =================== [PASSED] pf_service ====================
[07:09:38] =================== xe_mocs (2 subtests) ===================
[07:09:38] ================ xe_live_mocs_kernel_kunit  ================
[07:09:38] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[07:09:38] ================ xe_live_mocs_reset_kunit  =================
[07:09:38] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[07:09:38] ==================== [SKIPPED] xe_mocs =====================
[07:09:38] ================= xe_migrate (2 subtests) ==================
[07:09:38] ================= xe_migrate_sanity_kunit  =================
[07:09:38] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[07:09:38] ================== xe_validate_ccs_kunit  ==================
[07:09:38] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[07:09:38] =================== [SKIPPED] xe_migrate ===================
[07:09:38] ================== xe_dma_buf (1 subtest) ==================
[07:09:38] ==================== xe_dma_buf_kunit  =====================
[07:09:38] ================ [SKIPPED] xe_dma_buf_kunit ================
[07:09:38] =================== [SKIPPED] xe_dma_buf ===================
[07:09:38] ================= xe_bo_shrink (1 subtest) =================
[07:09:38] =================== xe_bo_shrink_kunit  ====================
[07:09:38] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[07:09:38] ================== [SKIPPED] xe_bo_shrink ==================
[07:09:38] ==================== xe_bo (2 subtests) ====================
[07:09:38] ================== xe_ccs_migrate_kunit  ===================
[07:09:38] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[07:09:38] ==================== xe_bo_evict_kunit  ====================
[07:09:38] =============== [SKIPPED] xe_bo_evict_kunit ================
[07:09:38] ===================== [SKIPPED] xe_bo ======================
[07:09:38] ==================== args (11 subtests) ====================
[07:09:38] [PASSED] count_args_test
[07:09:38] [PASSED] call_args_example
[07:09:38] [PASSED] call_args_test
[07:09:38] [PASSED] drop_first_arg_example
[07:09:38] [PASSED] drop_first_arg_test
[07:09:38] [PASSED] first_arg_example
[07:09:38] [PASSED] first_arg_test
[07:09:38] [PASSED] last_arg_example
[07:09:38] [PASSED] last_arg_test
[07:09:38] [PASSED] pick_arg_example
[07:09:38] [PASSED] sep_comma_example
[07:09:38] ====================== [PASSED] args =======================
[07:09:38] =================== xe_pci (3 subtests) ====================
[07:09:38] ==================== check_graphics_ip  ====================
[07:09:38] [PASSED] 12.70 Xe_LPG
[07:09:38] [PASSED] 12.71 Xe_LPG
[07:09:38] [PASSED] 12.74 Xe_LPG+
[07:09:38] [PASSED] 20.01 Xe2_HPG
[07:09:38] [PASSED] 20.02 Xe2_HPG
[07:09:38] [PASSED] 20.04 Xe2_LPG
[07:09:38] [PASSED] 30.00 Xe3_LPG
[07:09:38] [PASSED] 30.01 Xe3_LPG
[07:09:38] [PASSED] 30.03 Xe3_LPG
[07:09:38] ================ [PASSED] check_graphics_ip ================
[07:09:38] ===================== check_media_ip  ======================
[07:09:38] [PASSED] 13.00 Xe_LPM+
[07:09:38] [PASSED] 13.01 Xe2_HPM
[07:09:38] [PASSED] 20.00 Xe2_LPM
[07:09:38] [PASSED] 30.00 Xe3_LPM
[07:09:38] [PASSED] 30.02 Xe3_LPM
[07:09:38] ================= [PASSED] check_media_ip ==================
[07:09:38] ================= check_platform_gt_count  =================
[07:09:38] [PASSED] 0x9A60 (TIGERLAKE)
[07:09:38] [PASSED] 0x9A68 (TIGERLAKE)
[07:09:38] [PASSED] 0x9A70 (TIGERLAKE)
[07:09:38] [PASSED] 0x9A40 (TIGERLAKE)
[07:09:38] [PASSED] 0x9A49 (TIGERLAKE)
[07:09:38] [PASSED] 0x9A59 (TIGERLAKE)
[07:09:38] [PASSED] 0x9A78 (TIGERLAKE)
[07:09:38] [PASSED] 0x9AC0 (TIGERLAKE)
[07:09:38] [PASSED] 0x9AC9 (TIGERLAKE)
[07:09:38] [PASSED] 0x9AD9 (TIGERLAKE)
[07:09:38] [PASSED] 0x9AF8 (TIGERLAKE)
[07:09:38] [PASSED] 0x4C80 (ROCKETLAKE)
[07:09:38] [PASSED] 0x4C8A (ROCKETLAKE)
[07:09:38] [PASSED] 0x4C8B (ROCKETLAKE)
[07:09:38] [PASSED] 0x4C8C (ROCKETLAKE)
[07:09:38] [PASSED] 0x4C90 (ROCKETLAKE)
[07:09:38] [PASSED] 0x4C9A (ROCKETLAKE)
[07:09:38] [PASSED] 0x4680 (ALDERLAKE_S)
[07:09:38] [PASSED] 0x4682 (ALDERLAKE_S)
[07:09:38] [PASSED] 0x4688 (ALDERLAKE_S)
[07:09:38] [PASSED] 0x468A (ALDERLAKE_S)
[07:09:38] [PASSED] 0x468B (ALDERLAKE_S)
[07:09:38] [PASSED] 0x4690 (ALDERLAKE_S)
[07:09:38] [PASSED] 0x4692 (ALDERLAKE_S)
[07:09:38] [PASSED] 0x4693 (ALDERLAKE_S)
[07:09:38] [PASSED] 0x46A0 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46A1 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46A2 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46A3 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46A6 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46A8 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46AA (ALDERLAKE_P)
[07:09:38] [PASSED] 0x462A (ALDERLAKE_P)
[07:09:38] [PASSED] 0x4626 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x4628 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46B0 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46B1 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46B2 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46B3 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46C0 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46C1 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46C2 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46C3 (ALDERLAKE_P)
[07:09:38] [PASSED] 0x46D0 (ALDERLAKE_N)
[07:09:38] [PASSED] 0x46D1 (ALDERLAKE_N)
[07:09:38] [PASSED] 0x46D2 (ALDERLAKE_N)
[07:09:38] [PASSED] 0x46D3 (ALDERLAKE_N)
[07:09:38] [PASSED] 0x46D4 (ALDERLAKE_N)
[07:09:38] [PASSED] 0xA721 (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7A1 (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7A9 (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7AC (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7AD (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA720 (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7A0 (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7A8 (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7AA (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA7AB (ALDERLAKE_P)
[07:09:38] [PASSED] 0xA780 (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA781 (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA782 (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA783 (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA788 (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA789 (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA78A (ALDERLAKE_S)
[07:09:38] [PASSED] 0xA78B (ALDERLAKE_S)
[07:09:38] [PASSED] 0x4905 (DG1)
[07:09:38] [PASSED] 0x4906 (DG1)
[07:09:38] [PASSED] 0x4907 (DG1)
[07:09:38] [PASSED] 0x4908 (DG1)
[07:09:38] [PASSED] 0x4909 (DG1)
[07:09:38] [PASSED] 0x56C0 (DG2)
[07:09:38] [PASSED] 0x56C2 (DG2)
[07:09:38] [PASSED] 0x56C1 (DG2)
[07:09:38] [PASSED] 0x7D51 (METEORLAKE)
[07:09:38] [PASSED] 0x7DD1 (METEORLAKE)
[07:09:38] [PASSED] 0x7D41 (METEORLAKE)
[07:09:38] [PASSED] 0x7D67 (METEORLAKE)
[07:09:38] [PASSED] 0xB640 (METEORLAKE)
[07:09:38] [PASSED] 0x56A0 (DG2)
[07:09:38] [PASSED] 0x56A1 (DG2)
[07:09:38] [PASSED] 0x56A2 (DG2)
[07:09:38] [PASSED] 0x56BE (DG2)
[07:09:38] [PASSED] 0x56BF (DG2)
[07:09:38] [PASSED] 0x5690 (DG2)
[07:09:38] [PASSED] 0x5691 (DG2)
[07:09:38] [PASSED] 0x5692 (DG2)
[07:09:38] [PASSED] 0x56A5 (DG2)
[07:09:38] [PASSED] 0x56A6 (DG2)
[07:09:38] [PASSED] 0x56B0 (DG2)
[07:09:38] [PASSED] 0x56B1 (DG2)
[07:09:38] [PASSED] 0x56BA (DG2)
[07:09:38] [PASSED] 0x56BB (DG2)
[07:09:38] [PASSED] 0x56BC (DG2)
[07:09:38] [PASSED] 0x56BD (DG2)
[07:09:38] [PASSED] 0x5693 (DG2)
[07:09:38] [PASSED] 0x5694 (DG2)
[07:09:38] [PASSED] 0x5695 (DG2)
[07:09:38] [PASSED] 0x56A3 (DG2)
[07:09:38] [PASSED] 0x56A4 (DG2)
[07:09:38] [PASSED] 0x56B2 (DG2)
[07:09:38] [PASSED] 0x56B3 (DG2)
[07:09:38] [PASSED] 0x5696 (DG2)
[07:09:38] [PASSED] 0x5697 (DG2)
[07:09:38] [PASSED] 0xB69 (PVC)
[07:09:38] [PASSED] 0xB6E (PVC)
[07:09:38] [PASSED] 0xBD4 (PVC)
[07:09:38] [PASSED] 0xBD5 (PVC)
[07:09:38] [PASSED] 0xBD6 (PVC)
[07:09:38] [PASSED] 0xBD7 (PVC)
[07:09:38] [PASSED] 0xBD8 (PVC)
[07:09:38] [PASSED] 0xBD9 (PVC)
[07:09:38] [PASSED] 0xBDA (PVC)
[07:09:38] [PASSED] 0xBDB (PVC)
[07:09:38] [PASSED] 0xBE0 (PVC)
[07:09:38] [PASSED] 0xBE1 (PVC)
[07:09:38] [PASSED] 0xBE5 (PVC)
[07:09:38] [PASSED] 0x7D40 (METEORLAKE)
[07:09:38] [PASSED] 0x7D45 (METEORLAKE)
[07:09:38] [PASSED] 0x7D55 (METEORLAKE)
[07:09:38] [PASSED] 0x7D60 (METEORLAKE)
[07:09:38] [PASSED] 0x7DD5 (METEORLAKE)
[07:09:38] [PASSED] 0x6420 (LUNARLAKE)
[07:09:38] [PASSED] 0x64A0 (LUNARLAKE)
[07:09:38] [PASSED] 0x64B0 (LUNARLAKE)
[07:09:38] [PASSED] 0xE202 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE209 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE20B (BATTLEMAGE)
[07:09:38] [PASSED] 0xE20C (BATTLEMAGE)
[07:09:38] [PASSED] 0xE20D (BATTLEMAGE)
[07:09:38] [PASSED] 0xE210 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE211 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE212 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE216 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE220 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE221 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE222 (BATTLEMAGE)
[07:09:38] [PASSED] 0xE223 (BATTLEMAGE)
[07:09:38] [PASSED] 0xB080 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB081 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB082 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB083 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB084 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB085 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB086 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB087 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB08F (PANTHERLAKE)
[07:09:38] [PASSED] 0xB090 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB0A0 (PANTHERLAKE)
[07:09:38] [PASSED] 0xB0B0 (PANTHERLAKE)
[07:09:38] [PASSED] 0xFD80 (PANTHERLAKE)
[07:09:38] [PASSED] 0xFD81 (PANTHERLAKE)
[07:09:38] ============= [PASSED] check_platform_gt_count =============
[07:09:38] ===================== [PASSED] xe_pci ======================
[07:09:38] =================== xe_rtp (2 subtests) ====================
[07:09:38] =============== xe_rtp_process_to_sr_tests  ================
[07:09:38] [PASSED] coalesce-same-reg
[07:09:38] [PASSED] no-match-no-add
[07:09:38] [PASSED] match-or
[07:09:38] [PASSED] match-or-xfail
[07:09:38] [PASSED] no-match-no-add-multiple-rules
[07:09:38] [PASSED] two-regs-two-entries
[07:09:38] [PASSED] clr-one-set-other
[07:09:38] [PASSED] set-field
[07:09:38] [PASSED] conflict-duplicate
[07:09:38] [PASSED] conflict-not-disjoint
[07:09:38] [PASSED] conflict-reg-type
[07:09:38] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[07:09:38] ================== xe_rtp_process_tests  ===================
[07:09:38] [PASSED] active1
[07:09:38] [PASSED] active2
[07:09:38] [PASSED] active-inactive
[07:09:38] [PASSED] inactive-active
[07:09:38] [PASSED] inactive-1st_or_active-inactive
[07:09:38] [PASSED] inactive-2nd_or_active-inactive
[07:09:38] [PASSED] inactive-last_or_active-inactive
[07:09:38] [PASSED] inactive-no_or_active-inactive
[07:09:38] ============== [PASSED] xe_rtp_process_tests ===============
[07:09:38] ===================== [PASSED] xe_rtp ======================
[07:09:38] ==================== xe_wa (1 subtest) =====================
[07:09:38] ======================== xe_wa_gt  =========================
[07:09:38] [PASSED] TIGERLAKE (B0)
[07:09:38] [PASSED] DG1 (A0)
[07:09:38] [PASSED] DG1 (B0)
[07:09:39] [PASSED] ALDERLAKE_S (A0)
[07:09:39] [PASSED] ALDERLAKE_S (B0)
[07:09:39] [PASSED] ALDERLAKE_S (C0)
[07:09:39] [PASSED] ALDERLAKE_S (D0)
[07:09:39] [PASSED] ALDERLAKE_P (A0)
[07:09:39] [PASSED] ALDERLAKE_P (B0)
[07:09:39] [PASSED] ALDERLAKE_P (C0)
[07:09:39] [PASSED] ALDERLAKE_S_RPLS (D0)
[07:09:39] [PASSED] ALDERLAKE_P_RPLU (E0)
[07:09:39] [PASSED] DG2_G10 (C0)
[07:09:39] [PASSED] DG2_G11 (B1)
[07:09:39] [PASSED] DG2_G12 (A1)
[07:09:39] [PASSED] METEORLAKE (g:A0, m:A0)
[07:09:39] [PASSED] METEORLAKE (g:A0, m:A0)
[07:09:39] [PASSED] METEORLAKE (g:A0, m:A0)
[07:09:39] [PASSED] LUNARLAKE (g:A0, m:A0)
[07:09:39] [PASSED] LUNARLAKE (g:B0, m:A0)
stty: 'standard input': Inappropriate ioctl for device
[07:09:39] [PASSED] BATTLEMAGE (g:A0, m:A1)
[07:09:39] [PASSED] PANTHERLAKE (g:A0, m:A0)
[07:09:39] ==================== [PASSED] xe_wa_gt =====================
[07:09:39] ====================== [PASSED] xe_wa ======================
[07:09:39] ============================================================
[07:09:39] Testing complete. Ran 298 tests: passed: 282, skipped: 16
[07:09:39] Elapsed time: 33.374s total, 4.182s configuring, 28.825s building, 0.331s running

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

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

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

✗ CI.checksparse: warning for User readable error codes on atomic_ioctl failure (rev2)

[Patchwork]

== Series Details ==

Series: User readable error codes on atomic_ioctl failure (rev2)
URL   : https://patchwork.freedesktop.org/series/152277/
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 cca87ca63e2f5b8a785dc59c23e526987530b27f
Sparse version: 0.6.4 (Ubuntu: 0.6.4-4ubuntu3)
Fast mode used, each commit won't be checked separately.
-
+drivers/gpu/drm/drm_drv.c:449:6: warning: context imbalance in 'drm_dev_enter' - different lock contexts for basic block
+drivers/gpu/drm/drm_drv.c: note: in included file (through include/linux/notifier.h, arch/x86/include/asm/uprobes.h, include/linux/uprobes.h, include/linux/mm_types.h, include/linux/mmzone.h, include/linux/gfp.h, ...):
+drivers/gpu/drm/drm_plane.c:213:24: warning: Using plain integer as NULL pointer
+drivers/gpu/drm/i915/display/intel_alpm.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_cdclk.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_ddi.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2021:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2034:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2034:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_display_types.h:2034:24: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/display/intel_hdcp.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_hotplug.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_pps.c: note: in included file:
+drivers/gpu/drm/i915/display/intel_psr.c: note: in included file:
+drivers/gpu/drm/i915/gt/intel_reset.c:1572:12: warning: context imbalance in '_intel_gt_reset_lock' - different lock contexts for basic block
+drivers/gpu/drm/i915/gt/intel_sseu.c:598:17: error: too long token expansion
+drivers/gpu/drm/i915/i915_active.c:1062:16: warning: context imbalance in '__i915_active_fence_set' - different lock contexts for basic block
+drivers/gpu/drm/i915/i915_drm_client.c:92:9: error: incompatible types in comparison expression (different address spaces):
+drivers/gpu/drm/i915/i915_drm_client.c:92:9: error: incompatible types in comparison expression (different address spaces):
+drivers/gpu/drm/i915/i915_drm_client.c:92:9:    expected struct list_head const *list
+drivers/gpu/drm/i915/i915_drm_client.c:92:9:    got struct list_head [noderef] __rcu *pos
+drivers/gpu/drm/i915/i915_drm_client.c:92:9:    struct list_head *
+drivers/gpu/drm/i915/i915_drm_client.c:92:9:    struct list_head *
+drivers/gpu/drm/i915/i915_drm_client.c:92:9:    struct list_head [noderef] __rcu *
+drivers/gpu/drm/i915/i915_drm_client.c:92:9:    struct list_head [noderef] __rcu *
+drivers/gpu/drm/i915/i915_drm_client.c:92:9: warning: incorrect type in argument 1 (different address spaces)
+drivers/gpu/drm/i915/i915_irq.c:492:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:492:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:500:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:500:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:505:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:505:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:505:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:543:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:543:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:551:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:551:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:556:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:556:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:556:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:600:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:600:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:603:15: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:603:15: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:607:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:607:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:614:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:614:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:614:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:614:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/intel_uncore.c:1927:1: warning: context imbalance in 'fwtable_read8' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:1928:1: warning: context imbalance in 'fwtable_read16' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:1929:1: warning: context imbalance in 'fwtable_read32' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:1930:1: warning: context imbalance in 'fwtable_read64' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:1995:1: warning: context imbalance in 'gen6_write8' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:1996:1: warning: context imbalance in 'gen6_write16' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:1997:1: warning: context imbalance in 'gen6_write32' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:2017:1: warning: context imbalance in 'fwtable_write8' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:2018:1: warning: context imbalance in 'fwtable_write16' - unexpected unlock
+drivers/gpu/drm/i915/intel_uncore.c:2019:1: warning: context imbalance in 'fwtable_write32' - unexpected unlock
+drivers/gpu/drm/i915/intel_wakeref.c:146:19: warning: context imbalance in 'wakeref_auto_timeout' - unexpected unlock
+drivers/gpu/drm/ttm/ttm_bo.c:1199:31: warning: symbol 'ttm_swap_ops' was not declared. Should it be static?
+drivers/gpu/drm/ttm/ttm_bo_util.c:329:38:    expected void *virtual
+drivers/gpu/drm/ttm/ttm_bo_util.c:329:38:    got void [noderef] __iomem *
+drivers/gpu/drm/ttm/ttm_bo_util.c:329:38: warning: incorrect type in assignment (different address spaces)
+drivers/gpu/drm/ttm/ttm_bo_util.c:332:38:    expected void *virtual
+drivers/gpu/drm/ttm/ttm_bo_util.c:332:38:    got void [noderef] __iomem *
+drivers/gpu/drm/ttm/ttm_bo_util.c:332:38: warning: incorrect type in assignment (different address spaces)
+drivers/gpu/drm/ttm/ttm_bo_util.c:335:38:    expected void *virtual
+drivers/gpu/drm/ttm/ttm_bo_util.c:335:38:    got void [noderef] __iomem *
+drivers/gpu/drm/ttm/ttm_bo_util.c:335:38: warning: incorrect type in assignment (different address spaces)
+drivers/gpu/drm/ttm/ttm_bo_util.c:467:28:    expected void volatile [noderef] __iomem *addr
+drivers/gpu/drm/ttm/ttm_bo_util.c:467:28:    got void *virtual
+drivers/gpu/drm/ttm/ttm_bo_util.c:467:28: warning: incorrect type in argument 1 (different address spaces)
+./include/linux/srcu.h:373:9: warning: context imbalance in 'drm_dev_exit' - unexpected unlock

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

✗ Xe.CI.BAT: failure for User readable error codes on atomic_ioctl failure (rev2)

[Patchwork]

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

== Series Details ==

Series: User readable error codes on atomic_ioctl failure (rev2)
URL   : https://patchwork.freedesktop.org/series/152277/
State : failure

== Summary ==

CI Bug Log - changes from xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f_BAT -> xe-pw-152277v2_BAT
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with xe-pw-152277v2_BAT absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in xe-pw-152277v2_BAT, 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 (11 -> 11)
------------------------------

  No changes in participating hosts

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

  Here are the unknown changes that may have been introduced in xe-pw-152277v2_BAT:

### IGT changes ###

#### Possible regressions ####

  * igt@kms_cursor_legacy@basic-flip-after-cursor-atomic:
    - bat-dg2-oem2:       [PASS][1] -> [DMESG-WARN][2] +5 other tests dmesg-warn
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/bat-dg2-oem2/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/bat-dg2-oem2/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html

  * igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
    - bat-adlp-7:         [PASS][3] -> [DMESG-WARN][4] +5 other tests dmesg-warn
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/bat-adlp-7/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/bat-adlp-7/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
    - bat-bmg-1:          [PASS][5] -> [DMESG-WARN][6] +5 other tests dmesg-warn
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/bat-bmg-1/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/bat-bmg-1/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html

  * igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size:
    - bat-lnl-1:          [PASS][7] -> [DMESG-WARN][8] +5 other tests dmesg-warn
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/bat-lnl-1/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.html
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/bat-lnl-1/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@kms_cursor_legacy@basic-flip-after-cursor-atomic:
    - {bat-ptl-2}:        [PASS][9] -> [DMESG-WARN][10] +5 other tests dmesg-warn
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/bat-ptl-2/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/bat-ptl-2/igt@kms_cursor_legacy@basic-flip-after-cursor-atomic.html

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

  Here are the changes found in xe-pw-152277v2_BAT that come from known issues:

### IGT changes ###

  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

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


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

  * IGT: IGT_8503 -> IGT_8504
  * Linux: xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f -> xe-pw-152277v2

  IGT_8503: 8503
  IGT_8504: 8504
  xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f: cca87ca63e2f5b8a785dc59c23e526987530b27f
  xe-pw-152277v2: 152277v2

== Logs ==

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

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

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Jani Nikula]

On Fri, 22 Aug 2025, Arun R Murthy <arun.r.murthy@intel.com> wrote:
> There can be multiple reasons for a failure in atomic_ioctl. Most often
> in these error conditions -EINVAL is returned. User/Compositor would
> have to blindly take a call on failure of this ioctl so as to use
> ALLOW_MODESET or any. It would be good if user/compositor gets a
> readable error code on failure so they can take proper corrections in
> the next commit.
> The struct drm_mode_atomic is being passed by the user/compositor which
> holds the properties for modeset/flip. Reusing the same struct for
> returning the error code in case of failure can save by creating a new
> uapi/interface for returning the error code.
> The element 'reserved' in the struct drm_mode_atomic is used for
> returning the user readable error code. This points to the struct
> drm_mode_atomic_err_code. Failure reasons have been initialized in
> DRM_MODE_ATOMIC_FAILURE_REASON.

At the moment, I'm not (yet) convinced any of this will work, even as a
concept.

Even so, I'll comment on some of the choices in the series.

> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
> ---
>  include/uapi/drm/drm_mode.h | 42 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 42 insertions(+)
>
> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
> index a122bea2559387576150236e3a88f99c24ad3138..f0986a3fe9a7d61e57e9a9a5ec01a604343f6930 100644
> --- a/include/uapi/drm/drm_mode.h
> +++ b/include/uapi/drm/drm_mode.h
> @@ -45,6 +45,7 @@ extern "C" {
>  #define DRM_CONNECTOR_NAME_LEN	32
>  #define DRM_DISPLAY_MODE_LEN	32
>  #define DRM_PROP_NAME_LEN	32
> +#define DRM_MODE_ATOMIC_FAILURE_STRING_LEN	64
>  
>  #define DRM_MODE_TYPE_BUILTIN	(1<<0) /* deprecated */
>  #define DRM_MODE_TYPE_CLOCK_C	((1<<1) | DRM_MODE_TYPE_BUILTIN) /* deprecated */
> @@ -1157,6 +1158,47 @@ struct drm_mode_destroy_dumb {
>  		DRM_MODE_ATOMIC_NONBLOCK |\
>  		DRM_MODE_ATOMIC_ALLOW_MODESET)
>  
> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \

Please don't add macros that expect other macros in the context. They
become very confusing. Just pass in the other macro to use. See MACRO__
in include/drm/intel/pciids.h for an example.

> +	FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
> +
> +#define FAILURE_REASON(flag, reason) flag,
> +typedef enum {
> +	DRM_MODE_ATOMIC_FAILURE_REASON
> +} drm_mode_atomic_failure_flag;
> +#undef FAILURE_REASON

This is a uapi header. Do we really want all of that macro trickery here
in the first place? What's wrong with just a plain enum? (Or macros.)

> +
> +#define FAILURE_REASON(flag, reason) #reason,
> +extern const char *drm_mode_atomic_failure_string[];
> +#undef FAILURE_REASON

Data is not an interface. Don't expose arrays like this. Who is even
going to know what its size is? Again, uapi header, where does it point,
what address space is it?

> +
> +/**
> + * drm_mode_atomic_err_code - struct to store the error code
> + *
> + * pointer to this struct will be stored in reserved variable of
> + * struct drm_mode_atomic to report the failure cause to the user.
> + *
> + * @failure_flags: error codes defined in drm_atomic_failure.failure_flag

Flags? As in a mask of multiple things? Is 64 going to be enough for
everyone, all conditions in all drivers?

OTOH, what's the promise wrt multiple reasons? Practically all drivers
bail out at the sight of first issue, and it's usually pretty much
meaningless to continue to figure out *other* reasons after that.

And this brings us to one of my main concerns on making this fly:

You bail out on the first thing, but you can't expect all drivers to
check stuff in the same order. It's not practical. So different drivers
will likely return different reasons for virtually the same
condition. So how is userspace going to handle that?

> + * @failure_string_ptr: pointer to user readable error message drm_mode_failure.failure_string

Is the string part of uapi, and can never change? Doesn't sound
great. What's wrong with just a plain enum?

> + * @failure_obj_ptr: pointer to the drm_object that caused error
> + * @reserved: reserved for future use
> + * @count_objs: count of drm_objects if multiple drm_objects caused error
> + */
> +struct drm_mode_atomic_err_code {
> +	__u64 failure_flags;
> +	__u64 failure_objs_ptr;
> +	__u64 reserved;
> +	__u32 count_objs;
> +	char failure_string[DRM_MODE_ATOMIC_FAILURE_STRING_LEN];
> +};
> +
>  struct drm_mode_atomic {
>  	__u32 flags;
>  	__u32 count_objs;

-- 
Jani Nikula, Intel

Re: [PATCH v3 3/4] drm/atomic: Return user readable error in atomic_ioctl

[Jani Nikula]

On Fri, 22 Aug 2025, Arun R Murthy <arun.r.murthy@intel.com> wrote:
> Add user readable error codes for failure cases in drm_atomic_ioctl() so
> that user can decode the error code and take corrective measurements.
>
> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
> ---
>  drivers/gpu/drm/drm_atomic.c      |  6 ++++
>  drivers/gpu/drm/drm_atomic_uapi.c | 60 ++++++++++++++++++++++++++++++++-------
>  2 files changed, 56 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
> index cd15cf52f0c9144711da5879da57884674aea9e4..5f25e6d3cf6cf246f83a8c39450b410e97fe45bb 100644
> --- a/drivers/gpu/drm/drm_atomic.c
> +++ b/drivers/gpu/drm/drm_atomic.c
> @@ -1513,6 +1513,9 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
>  			if (drm_atomic_crtc_needs_modeset(new_crtc_state)) {
>  				drm_dbg_atomic(dev, "[CRTC:%d:%s] requires full modeset\n",
>  					       crtc->base.id, crtc->name);
> +				state->error_code->failure_flags =
> +					DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;

It says flags, implying multiple, but you're just adding one there
anyway. Just like it was a regular enum.

> +
>  				return -EINVAL;
>  			}
>  		}
> @@ -1537,8 +1540,11 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
>  		drm_dbg_atomic(dev,
>  			       "driver added CRTC to commit: requested 0x%x, affected 0x%0x\n",
>  			       requested_crtc, affected_crtc);
> +		state->error_code->failure_flags = DRM_MODE_ATOMIC_NEED_FULL_MODESET;
>  		WARN(!state->allow_modeset, "adding CRTC not allowed without modesets: requested 0x%x, affected 0x%0x\n",
>  		     requested_crtc, affected_crtc);
> +
> +		return -EINVAL;

This changes behaviour.

>  	}
>  
>  	return 0;
> diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic_uapi.c
> index ecc73d52bfae41a7ef455a7e13649ec56c690b90..94eaf9c98eb4ac2455799f1416010d366e1b5bbc 100644
> --- a/drivers/gpu/drm/drm_atomic_uapi.c
> +++ b/drivers/gpu/drm/drm_atomic_uapi.c
> @@ -1058,6 +1058,9 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
>  			ret = drm_atomic_crtc_get_property(crtc, crtc_state,
>  							   prop, &old_val);
>  			ret = drm_atomic_check_prop_changes(ret, old_val, prop_value, prop);
> +			if (ret)
> +				state->error_code->failure_flags =
> +					DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED;
>  			break;
>  		}
>  
> @@ -1089,6 +1092,8 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
>  
>  			/* ask the driver if this non-primary plane is supported */
>  			if (plane->type != DRM_PLANE_TYPE_PRIMARY) {
> +				state->error_code->failure_flags =
> +					DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE;
>  				ret = -EINVAL;
>  
>  				if (plane_funcs && plane_funcs->atomic_async_check)
> @@ -1380,6 +1385,13 @@ set_async_flip(struct drm_atomic_state *state)
>  	}
>  }
>  
> +#define FAILURE_REASON(flag, reason) #reason,
> +const char *drm_mode_atomic_failure_string[] = {
> +	DRM_MODE_ATOMIC_FAILURE_REASON
> +};
> +
> +#undef FAILURE_REASON
> +
>  int drm_mode_atomic_ioctl(struct drm_device *dev,
>  			  void *data, struct drm_file *file_priv)
>  {
> @@ -1389,9 +1401,11 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>  	uint32_t __user *props_ptr = (uint32_t __user *)(unsigned long)(arg->props_ptr);
>  	uint64_t __user *prop_values_ptr = (uint64_t __user *)(unsigned long)(arg->prop_values_ptr);
>  	unsigned int copied_objs, copied_props;
> -	struct drm_atomic_state *state;
> +	struct drm_atomic_state *state = NULL;
>  	struct drm_modeset_acquire_ctx ctx;
>  	struct drm_out_fence_state *fence_state;
> +	struct drm_mode_atomic_err_code error_code;
> +	struct drm_mode_atomic_err_code __user *error_code_ptr;
>  	int ret = 0;
>  	unsigned int i, j, num_fences;
>  	bool async_flip = false;
> @@ -1400,6 +1414,11 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>  	if (!drm_core_check_feature(dev, DRIVER_ATOMIC))
>  		return -EOPNOTSUPP;
>  
> +	if (!arg->reserved)
> +		drm_err(dev, "memory not allocated for drm_atomic error reporting\n");

This right here makes me suspect you never really tried this with your
regular desktop environment.

> +
> +	memset(&error_code, 0, sizeof(struct drm_mode_atomic_err_code));
> +
>  	/* disallow for userspace that has not enabled atomic cap (even
>  	 * though this may be a bit overkill, since legacy userspace
>  	 * wouldn't know how to call this ioctl)
> @@ -1407,24 +1426,25 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>  	if (!file_priv->atomic) {
>  		drm_dbg_atomic(dev,
>  			       "commit failed: atomic cap not enabled\n");
> -		return -EINVAL;
> +		error_code.failure_flags = DRM_MODE_ATOMIC_CAP_NOT_ENABLED;
> +		ret = -EINVAL;
> +		goto out;
>  	}
>  
>  	if (arg->flags & ~DRM_MODE_ATOMIC_FLAGS) {
>  		drm_dbg_atomic(dev, "commit failed: invalid flag\n");
> -		return -EINVAL;
> -	}
> -
> -	if (arg->reserved) {
> -		drm_dbg_atomic(dev, "commit failed: reserved field set\n");
> -		return -EINVAL;
> +		error_code.failure_flags = DRM_MODE_ATOMIC_INVALID_FLAG;
> +		ret = -EINVAL;
> +		goto out;
>  	}
>  
>  	if (arg->flags & DRM_MODE_PAGE_FLIP_ASYNC) {
>  		if (!dev->mode_config.async_page_flip) {
>  			drm_dbg_atomic(dev,
>  				       "commit failed: DRM_MODE_PAGE_FLIP_ASYNC not supported\n");
> -			return -EINVAL;
> +			error_code.failure_flags = DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC;
> +			ret = -EINVAL;
> +			goto out;
>  		}
>  
>  		async_flip = true;
> @@ -1435,7 +1455,9 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>  			(arg->flags & DRM_MODE_PAGE_FLIP_EVENT)) {
>  		drm_dbg_atomic(dev,
>  			       "commit failed: page-flip event requested with test-only commit\n");
> -		return -EINVAL;
> +		error_code.failure_flags = DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY;
> +		ret = -EINVAL;
> +		goto out;
>  	}
>  
>  	state = drm_atomic_state_alloc(dev);
> @@ -1446,6 +1468,8 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>  	state->acquire_ctx = &ctx;
>  	state->allow_modeset = !!(arg->flags & DRM_MODE_ATOMIC_ALLOW_MODESET);
>  
> +	state->error_code = &error_code;
> +
>  retry:
>  	copied_objs = 0;
>  	copied_props = 0;
> @@ -1542,6 +1566,22 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>  	}
>  
>  out:
> +	/* update the error code if any error to allow user handling it */
> +	if (ret < 0 && arg->reserved) {
> +		error_code_ptr = (struct drm_mode_atomic_err_code __user *)
> +				 (unsigned long)arg->reserved;
> +
> +		strscpy_pad(error_code.failure_string,
> +			    drm_mode_atomic_failure_string[error_code.failure_flags],
> +			    sizeof(error_code.failure_string));
> +
> +		if (copy_to_user(error_code_ptr, &error_code, sizeof(error_code)))
> +			return -EFAULT;
> +	}
> +
> +	if (!state)
> +		return ret;
> +
>  	complete_signaling(dev, state, fence_state, num_fences, !ret);
>  
>  	if (ret == -EDEADLK) {

-- 
Jani Nikula, Intel

Re: [PATCH v3 4/4] drm/i915/display: Error codes for async flip failures

[Maarten Lankhorst]

Hey,

I'm not entirely sold on the design, it's way more complicated than it should be, it should be trivial to add any amount of error messages.

Replace return -EINVAL; with return drm_atomic_error_einval(state, class, "string");
Where class may be an enum, but in a way more generic way than currently specified, for example "invalid use of api", "requires modeset", "invalid arguments", "driver limitations", "async flip not possible".

The drm_atomic_error_einval() would set class and str as appropriate, and then return -EINVAL.

That's probably all we should need here.

Kind regards,
~Maarten

Den 2025-08-22 kl. 09:00, skrev Arun R Murthy:
> For failures in async flip atomic check/commit path return user readable
> error codes in struct drm_atomic_state.
> 
> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_display.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index c1a3a95c65f0b66c24ddd64f47dfdc67bbde86c9..5e23f4fc747bd01fa05eba63661bf7279b083317 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -5950,6 +5950,7 @@ static int intel_async_flip_check_uapi(struct intel_atomic_state *state,
>  		drm_dbg_kms(display->drm,
>  			    "[CRTC:%d:%s] modeset required\n",
>  			    crtc->base.base.id, crtc->base.name);
> +		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
>  		return -EINVAL;
>  	}
>  
> @@ -6019,6 +6020,7 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
>  		drm_dbg_kms(display->drm,
>  			    "[CRTC:%d:%s] modeset required\n",
>  			    crtc->base.base.id, crtc->base.name);
> +		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
>  		return -EINVAL;
>  	}
>  
> @@ -6061,6 +6063,8 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
>  				    plane->base.base.id, plane->base.name,
>  				    &new_plane_state->hw.fb->format->format,
>  				    new_plane_state->hw.fb->modifier);
> +			state->base.error_code->failure_flags =
> +				DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED;
>  			return -EINVAL;
>  		}
>  
> 

Re: [PATCH v3 4/4] drm/i915/display: Error codes for async flip failures

[Jani Nikula]

On Fri, 22 Aug 2025, Maarten Lankhorst <maarten.lankhorst@linux.intel.com> wrote:
> Hey,
>
> I'm not entirely sold on the design, it's way more complicated than it
> should be, it should be trivial to add any amount of error messages.

If we add error messages, how do we ensure the error messages themselves
never become part of UAPI? It doesn't take much for someone in userspace
to start doing strcmp on them.

> Replace return -EINVAL; with return drm_atomic_error_einval(state, class, "string");

Yeah, much cleaner. And if we go this route, then maybe make it printf
style to avoid callers having to clumsily fill a buffer first.

> Where class may be an enum, but in a way more generic way than
> currently specified, for example "invalid use of api", "requires
> modeset", "invalid arguments", "driver limitations", "async flip not
> possible".

That's also easier for userspace to deal with than a plethora of
flags. I guess the goals were more ambitious wrt precision/granularity,
but is that achievable?

BR,
Jani.

> The drm_atomic_error_einval() would set class and str as appropriate,
> and then return -EINVAL.
>
> That's probably all we should need here.
>
> Kind regards,
> ~Maarten
>
> Den 2025-08-22 kl. 09:00, skrev Arun R Murthy:
>> For failures in async flip atomic check/commit path return user readable
>> error codes in struct drm_atomic_state.
>> 
>> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
>> ---
>>  drivers/gpu/drm/i915/display/intel_display.c | 4 ++++
>>  1 file changed, 4 insertions(+)
>> 
>> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
>> index c1a3a95c65f0b66c24ddd64f47dfdc67bbde86c9..5e23f4fc747bd01fa05eba63661bf7279b083317 100644
>> --- a/drivers/gpu/drm/i915/display/intel_display.c
>> +++ b/drivers/gpu/drm/i915/display/intel_display.c
>> @@ -5950,6 +5950,7 @@ static int intel_async_flip_check_uapi(struct intel_atomic_state *state,
>>  		drm_dbg_kms(display->drm,
>>  			    "[CRTC:%d:%s] modeset required\n",
>>  			    crtc->base.base.id, crtc->base.name);
>> +		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
>>  		return -EINVAL;
>>  	}
>>  
>> @@ -6019,6 +6020,7 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
>>  		drm_dbg_kms(display->drm,
>>  			    "[CRTC:%d:%s] modeset required\n",
>>  			    crtc->base.base.id, crtc->base.name);
>> +		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
>>  		return -EINVAL;
>>  	}
>>  
>> @@ -6061,6 +6063,8 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
>>  				    plane->base.base.id, plane->base.name,
>>  				    &new_plane_state->hw.fb->format->format,
>>  				    new_plane_state->hw.fb->modifier);
>> +			state->base.error_code->failure_flags =
>> +				DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED;
>>  			return -EINVAL;
>>  		}
>>  
>> 
>

-- 
Jani Nikula, Intel

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Xaver Hugl]

> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
As mentioned before, some of these errors are a bit too specific. We
don't need to have an enum value for every way the API can be used
wrongly - CAP_NOT_ENABLED, INVALID_FLAG, PAGE_FLIP_ASYNC and
MODIFIER_NOT_SUPPORTED should all just be one enum value for "invalid
API usage".
In general, there should only be enum values that the compositor
implementation can actually use on end-user systems. For further
information when debugging a broken compositor implementation, other
tools can be used instead, like drm debug logging or the returned
string.

> +#define FAILURE_REASON(flag, reason) flag,
> +typedef enum {
> +       DRM_MODE_ATOMIC_FAILURE_REASON
> +} drm_mode_atomic_failure_flag;
> +#undef FAILURE_REASON
> +
> +#define FAILURE_REASON(flag, reason) #reason,
> +extern const char *drm_mode_atomic_failure_string[];
> +#undef FAILURE_REASON
The intention for the string wasn't for the enum values to be paired
with a description of the enum - that belongs into documentation, not
uAPI.

The idea behind it was that drivers could add driver-specific
information in the string for compositors to log (only in commits
where failure isn't normally expected), so we have an easier time
debugging issues a user system experienced by looking at the
compositor logs. Sending the enum value again in string form isn't
useful.

- Xaver

✗ Xe.CI.Full: failure for User readable error codes on atomic_ioctl failure (rev2)

[Patchwork]

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

== Series Details ==

Series: User readable error codes on atomic_ioctl failure (rev2)
URL   : https://patchwork.freedesktop.org/series/152277/
State : failure

== Summary ==

CI Bug Log - changes from xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f_FULL -> xe-pw-152277v2_FULL
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with xe-pw-152277v2_FULL absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in xe-pw-152277v2_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 (4 -> 4)
------------------------------

  No changes in participating hosts

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

  Here are the unknown changes that may have been introduced in xe-pw-152277v2_FULL:

### IGT changes ###

#### Possible regressions ####

  * igt@kms_atomic@atomic-invalid-params:
    - shard-lnl:          [PASS][1] -> [DMESG-FAIL][2] +3 other tests dmesg-fail
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-5/igt@kms_atomic@atomic-invalid-params.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-7/igt@kms_atomic@atomic-invalid-params.html

  * igt@kms_atomic@atomic-invalid-params@pipe-a-dp-2:
    - shard-bmg:          [PASS][3] -> [DMESG-FAIL][4] +1 other test dmesg-fail
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-3/igt@kms_atomic@atomic-invalid-params@pipe-a-dp-2.html
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-1/igt@kms_atomic@atomic-invalid-params@pipe-a-dp-2.html

  * igt@kms_atomic@atomic-invalid-params@pipe-a-hdmi-a-6:
    - shard-dg2-set2:     [PASS][5] -> [DMESG-FAIL][6] +1 other test dmesg-fail
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-433/igt@kms_atomic@atomic-invalid-params@pipe-a-hdmi-a-6.html
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@kms_atomic@atomic-invalid-params@pipe-a-hdmi-a-6.html

  * igt@kms_atomic@crtc-invalid-params:
    - shard-lnl:          NOTRUN -> [DMESG-WARN][7] +23 other tests dmesg-warn
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_atomic@crtc-invalid-params.html

  * igt@kms_atomic_transition@modeset-transition-nonblocking-fencing:
    - shard-dg2-set2:     [PASS][8] -> [DMESG-WARN][9] +93 other tests dmesg-warn
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-466/igt@kms_atomic_transition@modeset-transition-nonblocking-fencing.html
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@kms_atomic_transition@modeset-transition-nonblocking-fencing.html

  * igt@kms_atomic_transition@plane-toggle-modeset-transition:
    - shard-adlp:         [PASS][10] -> [DMESG-FAIL][11] +3 other tests dmesg-fail
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-8/igt@kms_atomic_transition@plane-toggle-modeset-transition.html
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-4/igt@kms_atomic_transition@plane-toggle-modeset-transition.html

  * igt@kms_flip@basic-plain-flip@a-hdmi-a2:
    - shard-dg2-set2:     NOTRUN -> [ABORT][12] +5 other tests abort
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@kms_flip@basic-plain-flip@a-hdmi-a2.html

  * igt@kms_flip@flip-vs-panning@a-dp2:
    - shard-bmg:          [PASS][13] -> [ABORT][14] +20 other tests abort
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-8/igt@kms_flip@flip-vs-panning@a-dp2.html
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-8/igt@kms_flip@flip-vs-panning@a-dp2.html

  * igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary:
    - shard-dg2-set2:     [PASS][15] -> [ABORT][16] +28 other tests abort
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-464/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-464/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html

  * igt@kms_plane_cursor@primary@pipe-a-hdmi-a-6-size-256:
    - shard-dg2-set2:     NOTRUN -> [DMESG-FAIL][17] +2 other tests dmesg-fail
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-434/igt@kms_plane_cursor@primary@pipe-a-hdmi-a-6-size-256.html

  * igt@kms_plane_multiple@2x-tiling-4@pipe-a-dp-2-pipe-c-hdmi-a-3:
    - shard-bmg:          [PASS][18] -> [DMESG-WARN][19] +257 other tests dmesg-warn
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-7/igt@kms_plane_multiple@2x-tiling-4@pipe-a-dp-2-pipe-c-hdmi-a-3.html
   [19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_plane_multiple@2x-tiling-4@pipe-a-dp-2-pipe-c-hdmi-a-3.html

  * igt@kms_plane_multiple@2x-tiling-4@pipe-b-hdmi-a-6-pipe-d-dp-4:
    - shard-dg2-set2:     NOTRUN -> [DMESG-WARN][20] +45 other tests dmesg-warn
   [20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-436/igt@kms_plane_multiple@2x-tiling-4@pipe-b-hdmi-a-6-pipe-d-dp-4.html

  * igt@kms_plane_multiple@tiling-none@pipe-c-hdmi-a-3:
    - shard-bmg:          NOTRUN -> [DMESG-WARN][21] +41 other tests dmesg-warn
   [21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-2/igt@kms_plane_multiple@tiling-none@pipe-c-hdmi-a-3.html

  * igt@kms_plane_scaling@planes-downscale-factor-0-75-upscale-factor-0-25@pipe-b:
    - shard-adlp:         NOTRUN -> [DMESG-WARN][22] +31 other tests dmesg-warn
   [22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_plane_scaling@planes-downscale-factor-0-75-upscale-factor-0-25@pipe-b.html

  * igt@kms_plane_scaling@planes-scaler-unity-scaling@pipe-b:
    - shard-lnl:          [PASS][23] -> [DMESG-WARN][24] +777 other tests dmesg-warn
   [23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-8/igt@kms_plane_scaling@planes-scaler-unity-scaling@pipe-b.html
   [24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_plane_scaling@planes-scaler-unity-scaling@pipe-b.html

  * igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1:
    - shard-adlp:         [PASS][25] -> [DMESG-WARN][26] +803 other tests dmesg-warn
   [25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-1/igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1.html
   [26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-3/igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1.html

  
#### Warnings ####

  * igt@kms_async_flips@async-flip-with-page-flip-events-linear-atomic@pipe-c-edp-1:
    - shard-lnl:          [FAIL][27] ([Intel XE#911]) -> [DMESG-FAIL][28] +3 other tests dmesg-fail
   [27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-2/igt@kms_async_flips@async-flip-with-page-flip-events-linear-atomic@pipe-c-edp-1.html
   [28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-8/igt@kms_async_flips@async-flip-with-page-flip-events-linear-atomic@pipe-c-edp-1.html

  * igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip:
    - shard-adlp:         [DMESG-FAIL][29] ([Intel XE#4543]) -> [DMESG-WARN][30] +3 other tests dmesg-warn
   [29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-4/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
   [30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip.html

  * igt@kms_flip_scaled_crc@flip-32bpp-xtile-to-64bpp-xtile-upscaling:
    - shard-lnl:          [FAIL][31] ([Intel XE#4683]) -> [DMESG-FAIL][32] +3 other tests dmesg-fail
   [31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-7/igt@kms_flip_scaled_crc@flip-32bpp-xtile-to-64bpp-xtile-upscaling.html
   [32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-7/igt@kms_flip_scaled_crc@flip-32bpp-xtile-to-64bpp-xtile-upscaling.html

  * igt@kms_flip_scaled_crc@flip-64bpp-xtile-to-16bpp-xtile-upscaling@pipe-a-default-mode:
    - shard-lnl:          [FAIL][33] ([Intel XE#3106] / [Intel XE#4683]) -> [DMESG-FAIL][34] +1 other test dmesg-fail
   [33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-1/igt@kms_flip_scaled_crc@flip-64bpp-xtile-to-16bpp-xtile-upscaling@pipe-a-default-mode.html
   [34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_flip_scaled_crc@flip-64bpp-xtile-to-16bpp-xtile-upscaling@pipe-a-default-mode.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-plflip-blt:
    - shard-bmg:          [SKIP][35] ([Intel XE#2312]) -> [ABORT][36]
   [35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-plflip-blt.html
   [36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-plflip-blt.html

  * igt@kms_frontbuffer_tracking@fbc-tiling-linear:
    - shard-bmg:          [SKIP][37] ([Intel XE#5390]) -> [ABORT][38] +4 other tests abort
   [37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-3/igt@kms_frontbuffer_tracking@fbc-tiling-linear.html
   [38]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_frontbuffer_tracking@fbc-tiling-linear.html

  * igt@kms_plane@pixel-format-source-clamping@pipe-a-plane-0:
    - shard-lnl:          [FAIL][39] ([Intel XE#5195]) -> [DMESG-FAIL][40] +2 other tests dmesg-fail
   [39]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-3/igt@kms_plane@pixel-format-source-clamping@pipe-a-plane-0.html
   [40]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_plane@pixel-format-source-clamping@pipe-a-plane-0.html

  * igt@kms_plane@pixel-format-source-clamping@pipe-b-plane-0:
    - shard-adlp:         [FAIL][41] ([Intel XE#5195]) -> [DMESG-FAIL][42] +4 other tests dmesg-fail
   [41]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-2/igt@kms_plane@pixel-format-source-clamping@pipe-b-plane-0.html
   [42]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@kms_plane@pixel-format-source-clamping@pipe-b-plane-0.html

  * igt@kms_plane_cursor@primary:
    - shard-dg2-set2:     [FAIL][43] ([Intel XE#616]) -> [DMESG-FAIL][44]
   [43]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-432/igt@kms_plane_cursor@primary.html
   [44]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-434/igt@kms_plane_cursor@primary.html

  * igt@kms_plane_multiple@tiling-x@pipe-b-edp-1:
    - shard-lnl:          [FAIL][45] ([Intel XE#4658]) -> [DMESG-FAIL][46] +3 other tests dmesg-fail
   [45]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-5/igt@kms_plane_multiple@tiling-x@pipe-b-edp-1.html
   [46]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_plane_multiple@tiling-x@pipe-b-edp-1.html

  * igt@kms_pm_dc@deep-pkgc:
    - shard-lnl:          [FAIL][47] ([Intel XE#2029]) -> [DMESG-FAIL][48]
   [47]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-5/igt@kms_pm_dc@deep-pkgc.html
   [48]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-8/igt@kms_pm_dc@deep-pkgc.html

  * igt@xe_pm@d3cold-mmap-system:
    - shard-dg2-set2:     [SKIP][49] ([Intel XE#2284] / [Intel XE#366]) -> [ABORT][50]
   [49]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-466/igt@xe_pm@d3cold-mmap-system.html
   [50]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-464/igt@xe_pm@d3cold-mmap-system.html

  * igt@xe_pm@d3cold-mocs:
    - shard-bmg:          [SKIP][51] ([Intel XE#2284]) -> [ABORT][52] +1 other test abort
   [51]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-8/igt@xe_pm@d3cold-mocs.html
   [52]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-1/igt@xe_pm@d3cold-mocs.html
    - shard-dg2-set2:     [SKIP][53] ([Intel XE#2284]) -> [ABORT][54]
   [53]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-434/igt@xe_pm@d3cold-mocs.html
   [54]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@xe_pm@d3cold-mocs.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * {igt@kms_async_flips@async-flip-hang}:
    - shard-adlp:         [DMESG-WARN][55] ([Intel XE#4543]) -> [DMESG-WARN][56] +2 other tests dmesg-warn
   [55]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-4/igt@kms_async_flips@async-flip-hang.html
   [56]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_async_flips@async-flip-hang.html

  * {igt@kms_async_flips@basic-modeset-with-all-modifiers-formats@pipe-a-edp-1-linear-rgb565}:
    - shard-lnl:          [PASS][57] -> [DMESG-WARN][58] +48 other tests dmesg-warn
   [57]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-4/igt@kms_async_flips@basic-modeset-with-all-modifiers-formats@pipe-a-edp-1-linear-rgb565.html
   [58]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_async_flips@basic-modeset-with-all-modifiers-formats@pipe-a-edp-1-linear-rgb565.html

  * {igt@kms_async_flips@basic-modeset-with-all-modifiers-formats@pipe-a-hdmi-a-1-linear-xrgb16161616f}:
    - shard-adlp:         [PASS][59] -> [DMESG-WARN][60] +34 other tests dmesg-warn
   [59]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-1/igt@kms_async_flips@basic-modeset-with-all-modifiers-formats@pipe-a-hdmi-a-1-linear-xrgb16161616f.html
   [60]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_async_flips@basic-modeset-with-all-modifiers-formats@pipe-a-hdmi-a-1-linear-xrgb16161616f.html

  
New tests
---------

  New tests have been introduced between xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f_FULL and xe-pw-152277v2_FULL:

### New IGT tests (1) ###

  * igt@kms_hdmi_inject:
    - Statuses :
    - Exec time: [None] s

  

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

  Here are the changes found in xe-pw-152277v2_FULL that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
    - shard-lnl:          NOTRUN -> [SKIP][61] ([Intel XE#1466])
   [61]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html

  * igt@kms_big_fb@4-tiled-addfb-size-overflow:
    - shard-adlp:         NOTRUN -> [SKIP][62] ([Intel XE#610])
   [62]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-6/igt@kms_big_fb@4-tiled-addfb-size-overflow.html

  * igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-180-hflip:
    - shard-adlp:         NOTRUN -> [SKIP][63] ([Intel XE#1124])
   [63]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-8/igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-180-hflip.html

  * igt@kms_big_fb@x-tiled-32bpp-rotate-90:
    - shard-bmg:          NOTRUN -> [SKIP][64] ([Intel XE#2327])
   [64]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-2/igt@kms_big_fb@x-tiled-32bpp-rotate-90.html
    - shard-lnl:          NOTRUN -> [SKIP][65] ([Intel XE#1407]) +1 other test skip
   [65]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_big_fb@x-tiled-32bpp-rotate-90.html

  * igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-0-async-flip:
    - shard-adlp:         NOTRUN -> [DMESG-FAIL][66] ([Intel XE#4543])
   [66]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-4/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-0-async-flip.html

  * igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip:
    - shard-adlp:         [PASS][67] -> [DMESG-FAIL][68] ([Intel XE#4543]) +3 other tests dmesg-fail
   [67]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-4/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip.html
   [68]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-8/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip.html

  * igt@kms_big_fb@yf-tiled-16bpp-rotate-180:
    - shard-lnl:          NOTRUN -> [SKIP][69] ([Intel XE#1124]) +1 other test skip
   [69]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@kms_big_fb@yf-tiled-16bpp-rotate-180.html

  * igt@kms_big_fb@yf-tiled-8bpp-rotate-180:
    - shard-bmg:          NOTRUN -> [SKIP][70] ([Intel XE#1124])
   [70]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_big_fb@yf-tiled-8bpp-rotate-180.html

  * igt@kms_bw@connected-linear-tiling-3-displays-2160x1440p:
    - shard-adlp:         NOTRUN -> [SKIP][71] ([Intel XE#2191])
   [71]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-3/igt@kms_bw@connected-linear-tiling-3-displays-2160x1440p.html
    - shard-lnl:          NOTRUN -> [SKIP][72] ([Intel XE#2191])
   [72]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_bw@connected-linear-tiling-3-displays-2160x1440p.html

  * igt@kms_ccs@bad-rotation-90-4-tiled-mtl-rc-ccs-cc:
    - shard-lnl:          NOTRUN -> [SKIP][73] ([Intel XE#2887]) +1 other test skip
   [73]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_ccs@bad-rotation-90-4-tiled-mtl-rc-ccs-cc.html

  * igt@kms_ccs@bad-rotation-90-y-tiled-gen12-rc-ccs@pipe-d-dp-2:
    - shard-dg2-set2:     NOTRUN -> [SKIP][74] ([Intel XE#455] / [Intel XE#787]) +5 other tests skip
   [74]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@kms_ccs@bad-rotation-90-y-tiled-gen12-rc-ccs@pipe-d-dp-2.html

  * igt@kms_ccs@crc-primary-basic-4-tiled-mtl-mc-ccs:
    - shard-adlp:         NOTRUN -> [SKIP][75] ([Intel XE#455] / [Intel XE#787]) +5 other tests skip
   [75]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@kms_ccs@crc-primary-basic-4-tiled-mtl-mc-ccs.html
    - shard-bmg:          NOTRUN -> [SKIP][76] ([Intel XE#2887])
   [76]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-2/igt@kms_ccs@crc-primary-basic-4-tiled-mtl-mc-ccs.html

  * igt@kms_ccs@crc-primary-basic-4-tiled-mtl-mc-ccs@pipe-c-hdmi-a-1:
    - shard-adlp:         NOTRUN -> [SKIP][77] ([Intel XE#787]) +8 other tests skip
   [77]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@kms_ccs@crc-primary-basic-4-tiled-mtl-mc-ccs@pipe-c-hdmi-a-1.html

  * igt@kms_ccs@crc-primary-suspend-y-tiled-ccs:
    - shard-bmg:          NOTRUN -> [SKIP][78] ([Intel XE#3432])
   [78]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_ccs@crc-primary-suspend-y-tiled-ccs.html
    - shard-lnl:          NOTRUN -> [SKIP][79] ([Intel XE#3432])
   [79]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@kms_ccs@crc-primary-suspend-y-tiled-ccs.html

  * igt@kms_ccs@crc-sprite-planes-basic-4-tiled-mtl-mc-ccs@pipe-a-hdmi-a-2:
    - shard-dg2-set2:     NOTRUN -> [SKIP][80] ([Intel XE#787]) +34 other tests skip
   [80]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@kms_ccs@crc-sprite-planes-basic-4-tiled-mtl-mc-ccs@pipe-a-hdmi-a-2.html

  * igt@kms_ccs@random-ccs-data-4-tiled-bmg-ccs:
    - shard-adlp:         NOTRUN -> [SKIP][81] ([Intel XE#2907])
   [81]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-4/igt@kms_ccs@random-ccs-data-4-tiled-bmg-ccs.html
    - shard-lnl:          NOTRUN -> [SKIP][82] ([Intel XE#2669]) +3 other tests skip
   [82]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-4/igt@kms_ccs@random-ccs-data-4-tiled-bmg-ccs.html

  * igt@kms_cdclk@plane-scaling:
    - shard-lnl:          NOTRUN -> [SKIP][83] ([Intel XE#4416]) +3 other tests skip
   [83]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_cdclk@plane-scaling.html
    - shard-adlp:         NOTRUN -> [SKIP][84] ([Intel XE#4416] / [Intel XE#455])
   [84]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_cdclk@plane-scaling.html

  * igt@kms_cdclk@plane-scaling@pipe-a-hdmi-a-1:
    - shard-adlp:         NOTRUN -> [SKIP][85] ([Intel XE#4416]) +2 other tests skip
   [85]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_cdclk@plane-scaling@pipe-a-hdmi-a-1.html

  * igt@kms_chamelium_frames@hdmi-crc-fast:
    - shard-dg2-set2:     NOTRUN -> [SKIP][86] ([Intel XE#373])
   [86]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@kms_chamelium_frames@hdmi-crc-fast.html

  * igt@kms_chamelium_hpd@dp-hpd-storm:
    - shard-adlp:         NOTRUN -> [SKIP][87] ([Intel XE#373]) +2 other tests skip
   [87]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-3/igt@kms_chamelium_hpd@dp-hpd-storm.html
    - shard-bmg:          NOTRUN -> [SKIP][88] ([Intel XE#2252]) +2 other tests skip
   [88]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_chamelium_hpd@dp-hpd-storm.html

  * igt@kms_chamelium_hpd@vga-hpd-with-enabled-mode:
    - shard-lnl:          NOTRUN -> [SKIP][89] ([Intel XE#373]) +3 other tests skip
   [89]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_chamelium_hpd@vga-hpd-with-enabled-mode.html

  * igt@kms_cursor_crc@cursor-offscreen-64x21:
    - shard-lnl:          NOTRUN -> [SKIP][90] ([Intel XE#1424]) +1 other test skip
   [90]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_cursor_crc@cursor-offscreen-64x21.html

  * igt@kms_cursor_crc@cursor-rapid-movement-512x512:
    - shard-dg2-set2:     NOTRUN -> [SKIP][91] ([Intel XE#308])
   [91]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@kms_cursor_crc@cursor-rapid-movement-512x512.html
    - shard-bmg:          NOTRUN -> [SKIP][92] ([Intel XE#2321])
   [92]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_cursor_crc@cursor-rapid-movement-512x512.html

  * igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions-varying-size:
    - shard-bmg:          [PASS][93] -> [SKIP][94] ([Intel XE#2291]) +5 other tests skip
   [93]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-3/igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions-varying-size.html
   [94]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions-varying-size.html

  * igt@kms_dither@fb-8bpc-vs-panel-6bpc:
    - shard-bmg:          [PASS][95] -> [SKIP][96] ([Intel XE#1340])
   [95]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-8/igt@kms_dither@fb-8bpc-vs-panel-6bpc.html
   [96]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_dither@fb-8bpc-vs-panel-6bpc.html

  * igt@kms_fbc_dirty_rect@fbc-dirty-rectangle-out-visible-area:
    - shard-adlp:         NOTRUN -> [SKIP][97] ([Intel XE#4422])
   [97]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@kms_fbc_dirty_rect@fbc-dirty-rectangle-out-visible-area.html
    - shard-lnl:          NOTRUN -> [SKIP][98] ([Intel XE#4422])
   [98]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@kms_fbc_dirty_rect@fbc-dirty-rectangle-out-visible-area.html

  * igt@kms_flip@2x-flip-vs-dpms-on-nop:
    - shard-lnl:          NOTRUN -> [SKIP][99] ([Intel XE#1421]) +1 other test skip
   [99]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_flip@2x-flip-vs-dpms-on-nop.html

  * igt@kms_flip@2x-flip-vs-dpms-on-nop-interruptible:
    - shard-bmg:          [PASS][100] -> [SKIP][101] ([Intel XE#2316]) +4 other tests skip
   [100]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-1/igt@kms_flip@2x-flip-vs-dpms-on-nop-interruptible.html
   [101]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_flip@2x-flip-vs-dpms-on-nop-interruptible.html

  * igt@kms_flip@2x-flip-vs-wf_vblank:
    - shard-adlp:         NOTRUN -> [SKIP][102] ([Intel XE#310])
   [102]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-8/igt@kms_flip@2x-flip-vs-wf_vblank.html

  * igt@kms_flip@flip-vs-expired-vblank@a-edp1:
    - shard-lnl:          [PASS][103] -> [FAIL][104] ([Intel XE#301]) +1 other test fail
   [103]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-3/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
   [104]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-4/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html

  * igt@kms_flip@plain-flip-interruptible@b-hdmi-a1:
    - shard-adlp:         [PASS][105] -> [DMESG-WARN][106] ([Intel XE#4543]) +14 other tests dmesg-warn
   [105]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-2/igt@kms_flip@plain-flip-interruptible@b-hdmi-a1.html
   [106]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_flip@plain-flip-interruptible@b-hdmi-a1.html

  * igt@kms_flip_scaled_crc@flip-32bpp-yftile-to-32bpp-yftileccs-upscaling:
    - shard-adlp:         NOTRUN -> [SKIP][107] ([Intel XE#455]) +3 other tests skip
   [107]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_flip_scaled_crc@flip-32bpp-yftile-to-32bpp-yftileccs-upscaling.html
    - shard-lnl:          NOTRUN -> [SKIP][108] ([Intel XE#1401] / [Intel XE#1745])
   [108]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_flip_scaled_crc@flip-32bpp-yftile-to-32bpp-yftileccs-upscaling.html

  * igt@kms_flip_scaled_crc@flip-32bpp-yftile-to-32bpp-yftileccs-upscaling@pipe-a-default-mode:
    - shard-lnl:          NOTRUN -> [SKIP][109] ([Intel XE#1401])
   [109]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_flip_scaled_crc@flip-32bpp-yftile-to-32bpp-yftileccs-upscaling@pipe-a-default-mode.html

  * igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling:
    - shard-lnl:          NOTRUN -> [SKIP][110] ([Intel XE#1397] / [Intel XE#1745])
   [110]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-4/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling.html

  * igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode:
    - shard-lnl:          NOTRUN -> [SKIP][111] ([Intel XE#1397])
   [111]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-4/igt@kms_flip_scaled_crc@flip-64bpp-linear-to-32bpp-linear-downscaling@pipe-a-default-mode.html

  * igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-pri-shrfb-draw-mmap-wc:
    - shard-adlp:         NOTRUN -> [SKIP][112] ([Intel XE#656]) +5 other tests skip
   [112]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-pri-shrfb-draw-mmap-wc.html

  * igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-spr-indfb-move:
    - shard-bmg:          NOTRUN -> [SKIP][113] ([Intel XE#2311])
   [113]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-spr-indfb-move.html

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-render:
    - shard-lnl:          NOTRUN -> [SKIP][114] ([Intel XE#656]) +8 other tests skip
   [114]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-render.html

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-wc:
    - shard-bmg:          NOTRUN -> [SKIP][115] ([Intel XE#2312]) +2 other tests skip
   [115]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-mmap-wc.html

  * igt@kms_frontbuffer_tracking@fbcdrrs-1p-primscrn-cur-indfb-draw-mmap-wc:
    - shard-lnl:          NOTRUN -> [SKIP][116] ([Intel XE#651])
   [116]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@kms_frontbuffer_tracking@fbcdrrs-1p-primscrn-cur-indfb-draw-mmap-wc.html

  * igt@kms_frontbuffer_tracking@fbcdrrs-1p-primscrn-cur-indfb-onoff:
    - shard-dg2-set2:     NOTRUN -> [SKIP][117] ([Intel XE#651]) +1 other test skip
   [117]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-434/igt@kms_frontbuffer_tracking@fbcdrrs-1p-primscrn-cur-indfb-onoff.html

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render:
    - shard-adlp:         NOTRUN -> [SKIP][118] ([Intel XE#653]) +1 other test skip
   [118]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html

  * igt@kms_frontbuffer_tracking@plane-fbc-rte:
    - shard-adlp:         NOTRUN -> [SKIP][119] ([Intel XE#1158])
   [119]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-3/igt@kms_frontbuffer_tracking@plane-fbc-rte.html

  * igt@kms_hdmi_inject@inject-audio:
    - shard-lnl:          NOTRUN -> [SKIP][120] ([Intel XE#1470] / [Intel XE#2853])
   [120]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-8/igt@kms_hdmi_inject@inject-audio.html

  * igt@kms_hdr@static-toggle-suspend:
    - shard-bmg:          [PASS][121] -> [SKIP][122] ([Intel XE#1503])
   [121]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-5/igt@kms_hdr@static-toggle-suspend.html
   [122]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_hdr@static-toggle-suspend.html

  * igt@kms_joiner@invalid-modeset-force-big-joiner:
    - shard-bmg:          [PASS][123] -> [SKIP][124] ([Intel XE#3012])
   [123]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-7/igt@kms_joiner@invalid-modeset-force-big-joiner.html
   [124]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_joiner@invalid-modeset-force-big-joiner.html

  * igt@kms_pipe_crc_basic@disable-crc-after-crtc:
    - shard-adlp:         [PASS][125] -> [DMESG-WARN][126] ([Intel XE#2953] / [Intel XE#4173]) +3 other tests dmesg-warn
   [125]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-8/igt@kms_pipe_crc_basic@disable-crc-after-crtc.html
   [126]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@kms_pipe_crc_basic@disable-crc-after-crtc.html

  * igt@kms_plane_lowres@tiling-none:
    - shard-bmg:          [PASS][127] -> [DMESG-WARN][128] ([Intel XE#5681])
   [127]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-7/igt@kms_plane_lowres@tiling-none.html
   [128]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_plane_lowres@tiling-none.html

  * igt@kms_plane_multiple@2x-tiling-4:
    - shard-bmg:          [PASS][129] -> [DMESG-WARN][130] ([Intel XE#5175]) +2 other tests dmesg-warn
   [129]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-7/igt@kms_plane_multiple@2x-tiling-4.html
   [130]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_plane_multiple@2x-tiling-4.html
    - shard-dg2-set2:     [PASS][131] -> [DMESG-WARN][132] ([Intel XE#5175]) +2 other tests dmesg-warn
   [131]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-432/igt@kms_plane_multiple@2x-tiling-4.html
   [132]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-436/igt@kms_plane_multiple@2x-tiling-4.html

  * igt@kms_plane_multiple@tiling-none:
    - shard-bmg:          NOTRUN -> [DMESG-WARN][133] ([Intel XE#5175])
   [133]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-2/igt@kms_plane_multiple@tiling-none.html
    - shard-dg2-set2:     NOTRUN -> [DMESG-WARN][134] ([Intel XE#5175])
   [134]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-436/igt@kms_plane_multiple@tiling-none.html

  * igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5:
    - shard-bmg:          NOTRUN -> [SKIP][135] ([Intel XE#2763]) +4 other tests skip
   [135]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5.html

  * igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-c:
    - shard-lnl:          NOTRUN -> [SKIP][136] ([Intel XE#2763]) +3 other tests skip
   [136]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-5@pipe-c.html

  * igt@kms_pm_dc@dc6-dpms:
    - shard-lnl:          [PASS][137] -> [FAIL][138] ([Intel XE#718])
   [137]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-1/igt@kms_pm_dc@dc6-dpms.html
   [138]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@kms_pm_dc@dc6-dpms.html

  * igt@kms_pm_rpm@dpms-mode-unset-non-lpsp:
    - shard-adlp:         NOTRUN -> [SKIP][139] ([Intel XE#836])
   [139]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@kms_pm_rpm@dpms-mode-unset-non-lpsp.html
    - shard-lnl:          NOTRUN -> [SKIP][140] ([Intel XE#1439] / [Intel XE#836])
   [140]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_pm_rpm@dpms-mode-unset-non-lpsp.html

  * igt@kms_psr2_sf@fbc-pr-overlay-primary-update-sf-dmg-area:
    - shard-dg2-set2:     NOTRUN -> [SKIP][141] ([Intel XE#1489] / [Intel XE#5899])
   [141]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-466/igt@kms_psr2_sf@fbc-pr-overlay-primary-update-sf-dmg-area.html
    - shard-lnl:          NOTRUN -> [SKIP][142] ([Intel XE#2893] / [Intel XE#5899])
   [142]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_psr2_sf@fbc-pr-overlay-primary-update-sf-dmg-area.html

  * igt@kms_psr2_sf@psr2-cursor-plane-move-continuous-exceed-fully-sf:
    - shard-adlp:         NOTRUN -> [SKIP][143] ([Intel XE#1489] / [Intel XE#5899]) +1 other test skip
   [143]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@kms_psr2_sf@psr2-cursor-plane-move-continuous-exceed-fully-sf.html
    - shard-bmg:          NOTRUN -> [SKIP][144] ([Intel XE#1489] / [Intel XE#5899]) +1 other test skip
   [144]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_psr2_sf@psr2-cursor-plane-move-continuous-exceed-fully-sf.html

  * igt@kms_psr@fbc-pr-dpms:
    - shard-lnl:          NOTRUN -> [SKIP][145] ([Intel XE#1406] / [Intel XE#5899])
   [145]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@kms_psr@fbc-pr-dpms.html

  * igt@kms_psr@pr-cursor-blt:
    - shard-adlp:         NOTRUN -> [SKIP][146] ([Intel XE#2850] / [Intel XE#5899] / [Intel XE#929]) +2 other tests skip
   [146]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-6/igt@kms_psr@pr-cursor-blt.html
    - shard-lnl:          NOTRUN -> [SKIP][147] ([Intel XE#5784] / [Intel XE#5899])
   [147]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-8/igt@kms_psr@pr-cursor-blt.html

  * igt@kms_psr@psr2-sprite-plane-move:
    - shard-bmg:          NOTRUN -> [SKIP][148] ([Intel XE#2234] / [Intel XE#2850] / [Intel XE#5899])
   [148]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_psr@psr2-sprite-plane-move.html

  * igt@kms_setmode@clone-exclusive-crtc:
    - shard-bmg:          [PASS][149] -> [SKIP][150] ([Intel XE#1435]) +1 other test skip
   [149]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-1/igt@kms_setmode@clone-exclusive-crtc.html
   [150]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_setmode@clone-exclusive-crtc.html

  * igt@xe_eudebug@basic-vm-access-userptr:
    - shard-adlp:         NOTRUN -> [SKIP][151] ([Intel XE#4837] / [Intel XE#5565])
   [151]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-3/igt@xe_eudebug@basic-vm-access-userptr.html
    - shard-bmg:          NOTRUN -> [SKIP][152] ([Intel XE#4837])
   [152]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@xe_eudebug@basic-vm-access-userptr.html

  * igt@xe_eudebug_online@basic-breakpoint:
    - shard-lnl:          NOTRUN -> [SKIP][153] ([Intel XE#4837]) +2 other tests skip
   [153]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-7/igt@xe_eudebug_online@basic-breakpoint.html

  * igt@xe_evict@evict-beng-large:
    - shard-adlp:         NOTRUN -> [SKIP][154] ([Intel XE#261] / [Intel XE#5564])
   [154]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@xe_evict@evict-beng-large.html
    - shard-lnl:          NOTRUN -> [SKIP][155] ([Intel XE#688])
   [155]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@xe_evict@evict-beng-large.html

  * igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-userptr-rebind:
    - shard-lnl:          NOTRUN -> [SKIP][156] ([Intel XE#1392]) +1 other test skip
   [156]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-userptr-rebind.html

  * igt@xe_exec_basic@multigpu-once-basic-defer-bind:
    - shard-adlp:         NOTRUN -> [SKIP][157] ([Intel XE#1392] / [Intel XE#5575]) +1 other test skip
   [157]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-3/igt@xe_exec_basic@multigpu-once-basic-defer-bind.html
    - shard-bmg:          NOTRUN -> [SKIP][158] ([Intel XE#2322]) +1 other test skip
   [158]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@xe_exec_basic@multigpu-once-basic-defer-bind.html

  * igt@xe_exec_fault_mode@many-execqueues-bindexecqueue-userptr-rebind-imm:
    - shard-dg2-set2:     NOTRUN -> [SKIP][159] ([Intel XE#288]) +2 other tests skip
   [159]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@xe_exec_fault_mode@many-execqueues-bindexecqueue-userptr-rebind-imm.html

  * igt@xe_exec_fault_mode@many-execqueues-rebind:
    - shard-adlp:         NOTRUN -> [SKIP][160] ([Intel XE#288] / [Intel XE#5561]) +4 other tests skip
   [160]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@xe_exec_fault_mode@many-execqueues-rebind.html

  * igt@xe_exec_mix_modes@exec-spinner-interrupted-lr:
    - shard-adlp:         NOTRUN -> [SKIP][161] ([Intel XE#2360] / [Intel XE#5573])
   [161]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@xe_exec_mix_modes@exec-spinner-interrupted-lr.html

  * igt@xe_exec_reset@gt-reset-stress:
    - shard-adlp:         NOTRUN -> [ABORT][162] ([Intel XE#5729])
   [162]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-8/igt@xe_exec_reset@gt-reset-stress.html

  * igt@xe_exec_reset@parallel-gt-reset:
    - shard-adlp:         [PASS][163] -> [DMESG-WARN][164] ([Intel XE#3876])
   [163]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-8/igt@xe_exec_reset@parallel-gt-reset.html
   [164]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@xe_exec_reset@parallel-gt-reset.html

  * igt@xe_exec_system_allocator@many-execqueues-mmap-free:
    - shard-dg2-set2:     NOTRUN -> [SKIP][165] ([Intel XE#4915]) +12 other tests skip
   [165]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-433/igt@xe_exec_system_allocator@many-execqueues-mmap-free.html

  * igt@xe_exec_system_allocator@many-stride-mmap-huge-nomemset:
    - shard-lnl:          NOTRUN -> [SKIP][166] ([Intel XE#4943]) +6 other tests skip
   [166]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@xe_exec_system_allocator@many-stride-mmap-huge-nomemset.html

  * igt@xe_exec_system_allocator@once-large-mmap-huge-nomemset:
    - shard-bmg:          NOTRUN -> [SKIP][167] ([Intel XE#4943]) +2 other tests skip
   [167]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@xe_exec_system_allocator@once-large-mmap-huge-nomemset.html

  * igt@xe_exec_system_allocator@threads-shared-vm-many-mmap-shared-remap:
    - shard-adlp:         NOTRUN -> [SKIP][168] ([Intel XE#4915]) +39 other tests skip
   [168]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-4/igt@xe_exec_system_allocator@threads-shared-vm-many-mmap-shared-remap.html

  * igt@xe_exec_threads@threads-mixed-fd-rebind:
    - shard-adlp:         [PASS][169] -> [DMESG-FAIL][170] ([Intel XE#3876])
   [169]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-6/igt@xe_exec_threads@threads-mixed-fd-rebind.html
   [170]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@xe_exec_threads@threads-mixed-fd-rebind.html

  * igt@xe_live_ktest@xe_bo:
    - shard-adlp:         NOTRUN -> [SKIP][171] ([Intel XE#2229] / [Intel XE#455]) +1 other test skip
   [171]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@xe_live_ktest@xe_bo.html

  * igt@xe_live_ktest@xe_bo@xe_bo_evict_kunit:
    - shard-lnl:          NOTRUN -> [SKIP][172] ([Intel XE#2229])
   [172]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-1/igt@xe_live_ktest@xe_bo@xe_bo_evict_kunit.html

  * igt@xe_live_ktest@xe_bo@xe_ccs_migrate_kunit:
    - shard-adlp:         NOTRUN -> [SKIP][173] ([Intel XE#2229])
   [173]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-1/igt@xe_live_ktest@xe_bo@xe_ccs_migrate_kunit.html

  * igt@xe_oa@oa-regs-whitelisted:
    - shard-adlp:         NOTRUN -> [SKIP][174] ([Intel XE#3573])
   [174]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-4/igt@xe_oa@oa-regs-whitelisted.html

  * igt@xe_pat@pat-index-xehpc:
    - shard-adlp:         NOTRUN -> [SKIP][175] ([Intel XE#2838] / [Intel XE#979])
   [175]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-4/igt@xe_pat@pat-index-xehpc.html
    - shard-lnl:          NOTRUN -> [SKIP][176] ([Intel XE#1420] / [Intel XE#2838])
   [176]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-7/igt@xe_pat@pat-index-xehpc.html

  * igt@xe_pm@s3-d3cold-basic-exec:
    - shard-adlp:         NOTRUN -> [SKIP][177] ([Intel XE#2284] / [Intel XE#366])
   [177]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@xe_pm@s3-d3cold-basic-exec.html
    - shard-lnl:          NOTRUN -> [SKIP][178] ([Intel XE#2284] / [Intel XE#366])
   [178]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@xe_pm@s3-d3cold-basic-exec.html

  * igt@xe_pxp@pxp-stale-bo-exec-post-rpm:
    - shard-bmg:          NOTRUN -> [SKIP][179] ([Intel XE#4733])
   [179]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-3/igt@xe_pxp@pxp-stale-bo-exec-post-rpm.html

  * igt@xe_query@multigpu-query-mem-usage:
    - shard-bmg:          NOTRUN -> [SKIP][180] ([Intel XE#944])
   [180]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@xe_query@multigpu-query-mem-usage.html
    - shard-dg2-set2:     NOTRUN -> [SKIP][181] ([Intel XE#944])
   [181]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@xe_query@multigpu-query-mem-usage.html
    - shard-lnl:          NOTRUN -> [SKIP][182] ([Intel XE#944])
   [182]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-5/igt@xe_query@multigpu-query-mem-usage.html
    - shard-adlp:         NOTRUN -> [SKIP][183] ([Intel XE#944])
   [183]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-8/igt@xe_query@multigpu-query-mem-usage.html

  
#### Possible fixes ####

  * igt@kms_flip@flip-vs-panning-interruptible:
    - shard-adlp:         [DMESG-WARN][184] ([Intel XE#4543] / [Intel XE#5208]) -> [PASS][185]
   [184]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-8/igt@kms_flip@flip-vs-panning-interruptible.html
   [185]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@kms_flip@flip-vs-panning-interruptible.html

  * igt@kms_flip@flip-vs-panning-interruptible@b-hdmi-a1:
    - shard-adlp:         [DMESG-WARN][186] ([Intel XE#4543]) -> [PASS][187] +16 other tests pass
   [186]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-8/igt@kms_flip@flip-vs-panning-interruptible@b-hdmi-a1.html
   [187]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-9/igt@kms_flip@flip-vs-panning-interruptible@b-hdmi-a1.html

  * igt@kms_frontbuffer_tracking@fbc-shrfb-scaledprimary:
    - shard-adlp:         [DMESG-FAIL][188] ([Intel XE#4543]) -> [PASS][189] +3 other tests pass
   [188]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-adlp-4/igt@kms_frontbuffer_tracking@fbc-shrfb-scaledprimary.html
   [189]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-adlp-2/igt@kms_frontbuffer_tracking@fbc-shrfb-scaledprimary.html

  * igt@xe_module_load@load:
    - shard-dg2-set2:     ([PASS][190], [PASS][191], [PASS][192], [PASS][193], [PASS][194], [PASS][195], [PASS][196], [PASS][197], [PASS][198], [PASS][199], [PASS][200], [PASS][201], [PASS][202], [PASS][203], [PASS][204], [PASS][205], [PASS][206], [PASS][207], [SKIP][208], [PASS][209], [PASS][210], [PASS][211], [PASS][212], [PASS][213], [PASS][214], [PASS][215]) ([Intel XE#378]) -> ([PASS][216], [PASS][217], [PASS][218], [PASS][219], [PASS][220], [PASS][221], [PASS][222], [PASS][223], [PASS][224], [PASS][225], [PASS][226], [PASS][227], [PASS][228], [PASS][229], [PASS][230], [PASS][231], [PASS][232], [PASS][233], [PASS][234], [PASS][235], [PASS][236], [PASS][237], [PASS][238], [PASS][239], [PASS][240])
   [190]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-463/igt@xe_module_load@load.html
   [191]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-463/igt@xe_module_load@load.html
   [192]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-463/igt@xe_module_load@load.html
   [193]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-466/igt@xe_module_load@load.html
   [194]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-432/igt@xe_module_load@load.html
   [195]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-435/igt@xe_module_load@load.html
   [196]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-436/igt@xe_module_load@load.html
   [197]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-436/igt@xe_module_load@load.html
   [198]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-436/igt@xe_module_load@load.html
   [199]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-434/igt@xe_module_load@load.html
   [200]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-434/igt@xe_module_load@load.html
   [201]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-463/igt@xe_module_load@load.html
   [202]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-464/igt@xe_module_load@load.html
   [203]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-435/igt@xe_module_load@load.html
   [204]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-464/igt@xe_module_load@load.html
   [205]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-466/igt@xe_module_load@load.html
   [206]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-464/igt@xe_module_load@load.html
   [207]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-435/igt@xe_module_load@load.html
   [208]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-463/igt@xe_module_load@load.html
   [209]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-435/igt@xe_module_load@load.html
   [210]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-432/igt@xe_module_load@load.html
   [211]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-434/igt@xe_module_load@load.html
   [212]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-433/igt@xe_module_load@load.html
   [213]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-433/igt@xe_module_load@load.html
   [214]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-433/igt@xe_module_load@load.html
   [215]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-466/igt@xe_module_load@load.html
   [216]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@xe_module_load@load.html
   [217]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@xe_module_load@load.html
   [218]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@xe_module_load@load.html
   [219]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-466/igt@xe_module_load@load.html
   [220]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-434/igt@xe_module_load@load.html
   [221]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-433/igt@xe_module_load@load.html
   [222]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-434/igt@xe_module_load@load.html
   [223]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-434/igt@xe_module_load@load.html
   [224]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@xe_module_load@load.html
   [225]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@xe_module_load@load.html
   [226]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-433/igt@xe_module_load@load.html
   [227]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-436/igt@xe_module_load@load.html
   [228]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-436/igt@xe_module_load@load.html
   [229]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-436/igt@xe_module_load@load.html
   [230]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-466/igt@xe_module_load@load.html
   [231]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@xe_module_load@load.html
   [232]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-466/igt@xe_module_load@load.html
   [233]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@xe_module_load@load.html
   [234]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@xe_module_load@load.html
   [235]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-435/igt@xe_module_load@load.html
   [236]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-433/igt@xe_module_load@load.html
   [237]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-464/igt@xe_module_load@load.html
   [238]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-464/igt@xe_module_load@load.html
   [239]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-432/igt@xe_module_load@load.html
   [240]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-464/igt@xe_module_load@load.html

  * igt@xe_pmu@gt-frequency:
    - shard-dg2-set2:     [FAIL][241] ([Intel XE#4819]) -> [PASS][242] +1 other test pass
   [241]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-dg2-435/igt@xe_pmu@gt-frequency.html
   [242]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-dg2-463/igt@xe_pmu@gt-frequency.html
    - shard-lnl:          [FAIL][243] ([Intel XE#5166]) -> [PASS][244] +1 other test pass
   [243]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-7/igt@xe_pmu@gt-frequency.html
   [244]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-7/igt@xe_pmu@gt-frequency.html

  
#### Warnings ####

  * igt@kms_content_protection@srm:
    - shard-bmg:          [FAIL][245] ([Intel XE#1178]) -> [SKIP][246] ([Intel XE#2341])
   [245]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-1/igt@kms_content_protection@srm.html
   [246]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_content_protection@srm.html

  * igt@kms_flip@bo-too-big-interruptible@a-edp1:
    - shard-lnl:          [TIMEOUT][247] ([Intel XE#1504] / [Intel XE#5737]) -> [TIMEOUT][248] ([Intel XE#1504]) +1 other test timeout
   [247]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-2/igt@kms_flip@bo-too-big-interruptible@a-edp1.html
   [248]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-2/igt@kms_flip@bo-too-big-interruptible@a-edp1.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-render:
    - shard-bmg:          [SKIP][249] ([Intel XE#5390]) -> [SKIP][250] ([Intel XE#2312]) +4 other tests skip
   [249]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-render.html
   [250]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-render.html

  * igt@kms_frontbuffer_tracking@fbcdrrs-2p-primscrn-indfb-pgflip-blt:
    - shard-bmg:          [SKIP][251] ([Intel XE#2311]) -> [SKIP][252] ([Intel XE#2312]) +10 other tests skip
   [251]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-4/igt@kms_frontbuffer_tracking@fbcdrrs-2p-primscrn-indfb-pgflip-blt.html
   [252]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_frontbuffer_tracking@fbcdrrs-2p-primscrn-indfb-pgflip-blt.html

  * igt@kms_frontbuffer_tracking@fbcdrrs-2p-primscrn-pri-indfb-draw-blt:
    - shard-bmg:          [SKIP][253] ([Intel XE#2312]) -> [SKIP][254] ([Intel XE#2311]) +1 other test skip
   [253]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-6/igt@kms_frontbuffer_tracking@fbcdrrs-2p-primscrn-pri-indfb-draw-blt.html
   [254]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-5/igt@kms_frontbuffer_tracking@fbcdrrs-2p-primscrn-pri-indfb-draw-blt.html

  * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-shrfb-pgflip-blt:
    - shard-bmg:          [SKIP][255] ([Intel XE#2313]) -> [SKIP][256] ([Intel XE#2312]) +13 other tests skip
   [255]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-bmg-1/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-shrfb-pgflip-blt.html
   [256]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-bmg-6/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-shrfb-pgflip-blt.html

  * igt@kms_rotation_crc@primary-x-tiled-reflect-x-180:
    - shard-lnl:          [FAIL][257] ([Intel XE#4689]) -> [DMESG-FAIL][258] ([Intel XE#4689]) +1 other test dmesg-fail
   [257]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f/shard-lnl-8/igt@kms_rotation_crc@primary-x-tiled-reflect-x-180.html
   [258]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-152277v2/shard-lnl-3/igt@kms_rotation_crc@primary-x-tiled-reflect-x-180.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
  [Intel XE#1158]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1158
  [Intel XE#1178]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1178
  [Intel XE#1340]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1340
  [Intel XE#1392]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1392
  [Intel XE#1397]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1397
  [Intel XE#1401]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1401
  [Intel XE#1406]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1406
  [Intel XE#1407]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1407
  [Intel XE#1420]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1420
  [Intel XE#1421]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1421
  [Intel XE#1424]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1424
  [Intel XE#1435]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1435
  [Intel XE#1439]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1439
  [Intel XE#1466]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1466
  [Intel XE#1470]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1470
  [Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
  [Intel XE#1503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1503
  [Intel XE#1504]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1504
  [Intel XE#1745]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1745
  [Intel XE#2029]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2029
  [Intel XE#2191]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2191
  [Intel XE#2229]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2229
  [Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
  [Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
  [Intel XE#2284]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2284
  [Intel XE#2291]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2291
  [Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
  [Intel XE#2312]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2312
  [Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
  [Intel XE#2316]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2316
  [Intel XE#2321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2321
  [Intel XE#2322]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2322
  [Intel XE#2327]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2327
  [Intel XE#2341]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2341
  [Intel XE#2360]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2360
  [Intel XE#261]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/261
  [Intel XE#2669]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2669
  [Intel XE#2763]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2763
  [Intel XE#2838]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2838
  [Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
  [Intel XE#2853]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2853
  [Intel XE#288]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/288
  [Intel XE#2887]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2887
  [Intel XE#2893]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2893
  [Intel XE#2907]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2907
  [Intel XE#2953]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2953
  [Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
  [Intel XE#3012]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3012
  [Intel XE#308]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/308
  [Intel XE#310]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/310
  [Intel XE#3106]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3106
  [Intel XE#3432]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3432
  [Intel XE#3573]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3573
  [Intel XE#366]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/366
  [Intel XE#373]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/373
  [Intel XE#378]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/378
  [Intel XE#3876]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3876
  [Intel XE#4173]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4173
  [Intel XE#4416]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4416
  [Intel XE#4422]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4422
  [Intel XE#4543]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4543
  [Intel XE#455]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/455
  [Intel XE#4658]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4658
  [Intel XE#4683]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4683
  [Intel XE#4689]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4689
  [Intel XE#4733]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4733
  [Intel XE#4819]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4819
  [Intel XE#4837]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4837
  [Intel XE#4915]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4915
  [Intel XE#4943]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4943
  [Intel XE#5166]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5166
  [Intel XE#5175]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5175
  [Intel XE#5195]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5195
  [Intel XE#5208]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5208
  [Intel XE#5390]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5390
  [Intel XE#5561]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5561
  [Intel XE#5564]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5564
  [Intel XE#5565]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5565
  [Intel XE#5573]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5573
  [Intel XE#5575]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5575
  [Intel XE#5681]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5681
  [Intel XE#5729]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5729
  [Intel XE#5737]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5737
  [Intel XE#5784]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5784
  [Intel XE#5899]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5899
  [Intel XE#610]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/610
  [Intel XE#616]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/616
  [Intel XE#651]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/651
  [Intel XE#653]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/653
  [Intel XE#656]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/656
  [Intel XE#688]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/688
  [Intel XE#718]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/718
  [Intel XE#787]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/787
  [Intel XE#836]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/836
  [Intel XE#911]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/911
  [Intel XE#929]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/929
  [Intel XE#944]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/944
  [Intel XE#979]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/979


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

  * IGT: IGT_8503 -> IGT_8504
  * Linux: xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f -> xe-pw-152277v2

  IGT_8503: 8503
  IGT_8504: 8504
  xe-3597-cca87ca63e2f5b8a785dc59c23e526987530b27f: cca87ca63e2f5b8a785dc59c23e526987530b27f
  xe-pw-152277v2: 152277v2

== Logs ==

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

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

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Murthy, Arun R]

On 22-08-2025 16:07, Jani Nikula wrote:
> On Fri, 22 Aug 2025, Arun R Murthy <arun.r.murthy@intel.com> wrote:
>> There can be multiple reasons for a failure in atomic_ioctl. Most often
>> in these error conditions -EINVAL is returned. User/Compositor would
>> have to blindly take a call on failure of this ioctl so as to use
>> ALLOW_MODESET or any. It would be good if user/compositor gets a
>> readable error code on failure so they can take proper corrections in
>> the next commit.
>> The struct drm_mode_atomic is being passed by the user/compositor which
>> holds the properties for modeset/flip. Reusing the same struct for
>> returning the error code in case of failure can save by creating a new
>> uapi/interface for returning the error code.
>> The element 'reserved' in the struct drm_mode_atomic is used for
>> returning the user readable error code. This points to the struct
>> drm_mode_atomic_err_code. Failure reasons have been initialized in
>> DRM_MODE_ATOMIC_FAILURE_REASON.
> At the moment, I'm not (yet) convinced any of this will work, even as a
> concept.
km_flip test in IGT has been run and is working fine with these patch 
series. Also
the existing kms_atomic test with atomic_invalid_test has been modified to
test this error code(https://patchwork.freedesktop.org/series/153330/)

Am I missing anything over here?

> Even so, I'll comment on some of the choices in the series.
>
>> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
>> ---
>>   include/uapi/drm/drm_mode.h | 42 ++++++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 42 insertions(+)
>>
>> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
>> index a122bea2559387576150236e3a88f99c24ad3138..f0986a3fe9a7d61e57e9a9a5ec01a604343f6930 100644
>> --- a/include/uapi/drm/drm_mode.h
>> +++ b/include/uapi/drm/drm_mode.h
>> @@ -45,6 +45,7 @@ extern "C" {
>>   #define DRM_CONNECTOR_NAME_LEN	32
>>   #define DRM_DISPLAY_MODE_LEN	32
>>   #define DRM_PROP_NAME_LEN	32
>> +#define DRM_MODE_ATOMIC_FAILURE_STRING_LEN	64
>>   
>>   #define DRM_MODE_TYPE_BUILTIN	(1<<0) /* deprecated */
>>   #define DRM_MODE_TYPE_CLOCK_C	((1<<1) | DRM_MODE_TYPE_BUILTIN) /* deprecated */
>> @@ -1157,6 +1158,47 @@ struct drm_mode_destroy_dumb {
>>   		DRM_MODE_ATOMIC_NONBLOCK |\
>>   		DRM_MODE_ATOMIC_ALLOW_MODESET)
>>   
>> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
> Please don't add macros that expect other macros in the context. They
> become very confusing. Just pass in the other macro to use. See MACRO__
> in include/drm/intel/pciids.h for an example.
Here we are mapping an error_code defined as enum with a corresponding 
string
using the X macros.

Anyway will have a look at the macro in include/drm/intel/pciids.h


>> +	FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
>> +
>> +#define FAILURE_REASON(flag, reason) flag,
>> +typedef enum {
>> +	DRM_MODE_ATOMIC_FAILURE_REASON
>> +} drm_mode_atomic_failure_flag;
>> +#undef FAILURE_REASON
> This is a uapi header. Do we really want all of that macro trickery here
> in the first place? What's wrong with just a plain enum? (Or macros.)
This is defined as enum. We have two separate list one for enum and other
for error message(string) but  upon user adding a new element/error_code 
in the
enum, there can be a possibility of missing adding the error string. So 
have linked
enum with the corresponding error message/string so both are at a single 
place
and upon adding new entry will be easy for the user.
>> +
>> +#define FAILURE_REASON(flag, reason) #reason,
>> +extern const char *drm_mode_atomic_failure_string[];
>> +#undef FAILURE_REASON
> Data is not an interface. Don't expose arrays like this. Who is even
> going to know what its size is? Again, uapi header, where does it point,
> what address space is it?
>
Sorry missed adding documentation for this. Will add in my next series.
>> +
>> +/**
>> + * drm_mode_atomic_err_code - struct to store the error code
>> + *
>> + * pointer to this struct will be stored in reserved variable of
>> + * struct drm_mode_atomic to report the failure cause to the user.
>> + *
>> + * @failure_flags: error codes defined in drm_atomic_failure.failure_flag
> Flags? As in a mask of multiple things? Is 64 going to be enough for
> everyone, all conditions in all drivers?
Should be more that sufficient, this is an enum.
> OTOH, what's the promise wrt multiple reasons? Practically all drivers
> bail out at the sight of first issue, and it's usually pretty much
> meaningless to continue to figure out *other* reasons after that.
>
> And this brings us to one of my main concerns on making this fly:
>
> You bail out on the first thing, but you can't expect all drivers to
> check stuff in the same order. It's not practical. So different drivers
> will likely return different reasons for virtually the same
> condition. So how is userspace going to handle that?
Most of the error check is in the drm-core and these are common across 
the drivers.
Upon drivers sending driver specific error code, the compositor may not 
be able to
handle the error code. But in cases required compositor changes can be 
added so
as to handle them. In general the generic error conditions that are met 
by compositor
and for which compositor can take corrective measurements are listed 
down first.
>> + * @failure_string_ptr: pointer to user readable error message drm_mode_failure.failure_string
> Is the string part of uapi, and can never change? Doesn't sound
> great. What's wrong with just a plain enum?
enum is sufficient enough and this string is added so that this error 
message

can be printed out in the compositor logs for the user.

Thanks and Regards,
Arun R Murthy
-------------------

>> + * @failure_obj_ptr: pointer to the drm_object that caused error
>> + * @reserved: reserved for future use
>> + * @count_objs: count of drm_objects if multiple drm_objects caused error
>> + */
>> +struct drm_mode_atomic_err_code {
>> +	__u64 failure_flags;
>> +	__u64 failure_objs_ptr;
>> +	__u64 reserved;
>> +	__u32 count_objs;
>> +	char failure_string[DRM_MODE_ATOMIC_FAILURE_STRING_LEN];
>> +};
>> +
>>   struct drm_mode_atomic {
>>   	__u32 flags;
>>   	__u32 count_objs;

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Murthy, Arun R]

On 22-08-2025 21:44, Xaver Hugl wrote:
>> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
> As mentioned before, some of these errors are a bit too specific. We
> don't need to have an enum value for every way the API can be used
> wrongly - CAP_NOT_ENABLED, INVALID_FLAG, PAGE_FLIP_ASYNC and
> MODIFIER_NOT_SUPPORTED should all just be one enum value for "invalid
> API usage".
> In general, there should only be enum values that the compositor
> implementation can actually use on end-user systems. For further
> information when debugging a broken compositor implementation, other
> tools can be used instead, like drm debug logging or the returned
> string.
I have considered your comment in the last series and have removed 
driver specific errors.
Anyway will have a look again on this and will get back.
>> +#define FAILURE_REASON(flag, reason) flag,
>> +typedef enum {
>> +       DRM_MODE_ATOMIC_FAILURE_REASON
>> +} drm_mode_atomic_failure_flag;
>> +#undef FAILURE_REASON
>> +
>> +#define FAILURE_REASON(flag, reason) #reason,
>> +extern const char *drm_mode_atomic_failure_string[];
>> +#undef FAILURE_REASON
> The intention for the string wasn't for the enum values to be paired
> with a description of the enum - that belongs into documentation, not
> uAPI.
>
> The idea behind it was that drivers could add driver-specific
> information in the string for compositors to log (only in commits
> where failure isn't normally expected), so we have an easier time
> debugging issues a user system experienced by looking at the
> compositor logs. Sending the enum value again in string form isn't
> useful.

We are not sending enum value in string. Its just a single place where 
we have both enum and string. Upon user adding new error codes if both 
enum and string are at a single place it would be easy for the user. 
Hence adding both in a single place using X macros.

Its not mandatory to have a string for every enum, the string can be 
left empty if not required, or later in the driver user can overwrite 
the string as well.

Thanks and Regards,
Arun R Murthy
--------------------

> - Xaver

Re: [PATCH v3 3/4] drm/atomic: Return user readable error in atomic_ioctl

[Murthy, Arun R]

On 22-08-2025 16:20, Jani Nikula wrote:
> On Fri, 22 Aug 2025, Arun R Murthy <arun.r.murthy@intel.com> wrote:
>> Add user readable error codes for failure cases in drm_atomic_ioctl() so
>> that user can decode the error code and take corrective measurements.
>>
>> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
>> ---
>>   drivers/gpu/drm/drm_atomic.c      |  6 ++++
>>   drivers/gpu/drm/drm_atomic_uapi.c | 60 ++++++++++++++++++++++++++++++++-------
>>   2 files changed, 56 insertions(+), 10 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
>> index cd15cf52f0c9144711da5879da57884674aea9e4..5f25e6d3cf6cf246f83a8c39450b410e97fe45bb 100644
>> --- a/drivers/gpu/drm/drm_atomic.c
>> +++ b/drivers/gpu/drm/drm_atomic.c
>> @@ -1513,6 +1513,9 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
>>   			if (drm_atomic_crtc_needs_modeset(new_crtc_state)) {
>>   				drm_dbg_atomic(dev, "[CRTC:%d:%s] requires full modeset\n",
>>   					       crtc->base.id, crtc->name);
>> +				state->error_code->failure_flags =
>> +					DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
> It says flags, implying multiple, but you're just adding one there
> anyway. Just like it was a regular enum.
Yes its a enum!
>
>> +
>>   				return -EINVAL;
>>   			}
>>   		}
>> @@ -1537,8 +1540,11 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
>>   		drm_dbg_atomic(dev,
>>   			       "driver added CRTC to commit: requested 0x%x, affected 0x%0x\n",
>>   			       requested_crtc, affected_crtc);
>> +		state->error_code->failure_flags = DRM_MODE_ATOMIC_NEED_FULL_MODESET;
>>   		WARN(!state->allow_modeset, "adding CRTC not allowed without modesets: requested 0x%x, affected 0x%0x\n",
>>   		     requested_crtc, affected_crtc);
>> +
>> +		return -EINVAL;
> This changes behaviour.
Will get it corrected in the next version!
>
>>   	}
>>   
>>   	return 0;
>> diff --git a/drivers/gpu/drm/drm_atomic_uapi.c b/drivers/gpu/drm/drm_atomic_uapi.c
>> index ecc73d52bfae41a7ef455a7e13649ec56c690b90..94eaf9c98eb4ac2455799f1416010d366e1b5bbc 100644
>> --- a/drivers/gpu/drm/drm_atomic_uapi.c
>> +++ b/drivers/gpu/drm/drm_atomic_uapi.c
>> @@ -1058,6 +1058,9 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
>>   			ret = drm_atomic_crtc_get_property(crtc, crtc_state,
>>   							   prop, &old_val);
>>   			ret = drm_atomic_check_prop_changes(ret, old_val, prop_value, prop);
>> +			if (ret)
>> +				state->error_code->failure_flags =
>> +					DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED;
>>   			break;
>>   		}
>>   
>> @@ -1089,6 +1092,8 @@ int drm_atomic_set_property(struct drm_atomic_state *state,
>>   
>>   			/* ask the driver if this non-primary plane is supported */
>>   			if (plane->type != DRM_PLANE_TYPE_PRIMARY) {
>> +				state->error_code->failure_flags =
>> +					DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE;
>>   				ret = -EINVAL;
>>   
>>   				if (plane_funcs && plane_funcs->atomic_async_check)
>> @@ -1380,6 +1385,13 @@ set_async_flip(struct drm_atomic_state *state)
>>   	}
>>   }
>>   
>> +#define FAILURE_REASON(flag, reason) #reason,
>> +const char *drm_mode_atomic_failure_string[] = {
>> +	DRM_MODE_ATOMIC_FAILURE_REASON
>> +};
>> +
>> +#undef FAILURE_REASON
>> +
>>   int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   			  void *data, struct drm_file *file_priv)
>>   {
>> @@ -1389,9 +1401,11 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   	uint32_t __user *props_ptr = (uint32_t __user *)(unsigned long)(arg->props_ptr);
>>   	uint64_t __user *prop_values_ptr = (uint64_t __user *)(unsigned long)(arg->prop_values_ptr);
>>   	unsigned int copied_objs, copied_props;
>> -	struct drm_atomic_state *state;
>> +	struct drm_atomic_state *state = NULL;
>>   	struct drm_modeset_acquire_ctx ctx;
>>   	struct drm_out_fence_state *fence_state;
>> +	struct drm_mode_atomic_err_code error_code;
>> +	struct drm_mode_atomic_err_code __user *error_code_ptr;
>>   	int ret = 0;
>>   	unsigned int i, j, num_fences;
>>   	bool async_flip = false;
>> @@ -1400,6 +1414,11 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   	if (!drm_core_check_feature(dev, DRIVER_ATOMIC))
>>   		return -EOPNOTSUPP;
>>   
>> +	if (!arg->reserved)
>> +		drm_err(dev, "memory not allocated for drm_atomic error reporting\n");
> This right here makes me suspect you never really tried this with your
> regular desktop environment.
>
>> +
>> +	memset(&error_code, 0, sizeof(struct drm_mode_atomic_err_code));
>> +
>>   	/* disallow for userspace that has not enabled atomic cap (even
>>   	 * though this may be a bit overkill, since legacy userspace
>>   	 * wouldn't know how to call this ioctl)
>> @@ -1407,24 +1426,25 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   	if (!file_priv->atomic) {
>>   		drm_dbg_atomic(dev,
>>   			       "commit failed: atomic cap not enabled\n");
>> -		return -EINVAL;
>> +		error_code.failure_flags = DRM_MODE_ATOMIC_CAP_NOT_ENABLED;
>> +		ret = -EINVAL;
>> +		goto out;
>>   	}
>>   
>>   	if (arg->flags & ~DRM_MODE_ATOMIC_FLAGS) {
>>   		drm_dbg_atomic(dev, "commit failed: invalid flag\n");
>> -		return -EINVAL;
>> -	}
>> -
>> -	if (arg->reserved) {
>> -		drm_dbg_atomic(dev, "commit failed: reserved field set\n");
>> -		return -EINVAL;
>> +		error_code.failure_flags = DRM_MODE_ATOMIC_INVALID_FLAG;
>> +		ret = -EINVAL;
>> +		goto out;
>>   	}
>>   
>>   	if (arg->flags & DRM_MODE_PAGE_FLIP_ASYNC) {
>>   		if (!dev->mode_config.async_page_flip) {
>>   			drm_dbg_atomic(dev,
>>   				       "commit failed: DRM_MODE_PAGE_FLIP_ASYNC not supported\n");
>> -			return -EINVAL;
>> +			error_code.failure_flags = DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC;
>> +			ret = -EINVAL;
>> +			goto out;
>>   		}
>>   
>>   		async_flip = true;
>> @@ -1435,7 +1455,9 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   			(arg->flags & DRM_MODE_PAGE_FLIP_EVENT)) {
>>   		drm_dbg_atomic(dev,
>>   			       "commit failed: page-flip event requested with test-only commit\n");
>> -		return -EINVAL;
>> +		error_code.failure_flags = DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY;
>> +		ret = -EINVAL;
>> +		goto out;
>>   	}
>>   
>>   	state = drm_atomic_state_alloc(dev);
>> @@ -1446,6 +1468,8 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   	state->acquire_ctx = &ctx;
>>   	state->allow_modeset = !!(arg->flags & DRM_MODE_ATOMIC_ALLOW_MODESET);
>>   
>> +	state->error_code = &error_code;
>> +
>>   retry:
>>   	copied_objs = 0;
>>   	copied_props = 0;
>> @@ -1542,6 +1566,22 @@ int drm_mode_atomic_ioctl(struct drm_device *dev,
>>   	}
>>   
>>   out:
>> +	/* update the error code if any error to allow user handling it */
>> +	if (ret < 0 && arg->reserved) {
>> +		error_code_ptr = (struct drm_mode_atomic_err_code __user *)
>> +				 (unsigned long)arg->reserved;
>> +
>> +		strscpy_pad(error_code.failure_string,
>> +			    drm_mode_atomic_failure_string[error_code.failure_flags],
>> +			    sizeof(error_code.failure_string));
>> +
>> +		if (copy_to_user(error_code_ptr, &error_code, sizeof(error_code)))
>> +			return -EFAULT;
>> +	}
>> +
>> +	if (!state)
>> +		return ret;
>> +
>>   	complete_signaling(dev, state, fence_state, num_fences, !ret);
>>   
>>   	if (ret == -EDEADLK) {

Re: [PATCH v3 4/4] drm/i915/display: Error codes for async flip failures

[Murthy, Arun R]

On 22-08-2025 17:01, Maarten Lankhorst wrote:
> Hey,
>
> I'm not entirely sold on the design, it's way more complicated than it should be, it should be trivial to add any amount of error messages.
>
> Replace return -EINVAL; with return drm_atomic_error_einval(state, class, "string");
> Where class may be an enum, but in a way more generic way than currently specified, for example "invalid use of api", "requires modeset", "invalid arguments", "driver limitations", "async flip not possible".
>
> The drm_atomic_error_einval() would set class and str as appropriate, and then return -EINVAL.
Looks good to me, but again this state, string will have to be filled to 
the struct and the address of the struct to be sent in the reserved 
field to the user.
So background for choosing this struct [1]

> That's probably all we should need here.
>
> Kind regards,
> ~Maarten
>
> Den 2025-08-22 kl. 09:00, skrev Arun R Murthy:
>> For failures in async flip atomic check/commit path return user readable
>> error codes in struct drm_atomic_state.
>>
>> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
>> ---
>>   drivers/gpu/drm/i915/display/intel_display.c | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
>> index c1a3a95c65f0b66c24ddd64f47dfdc67bbde86c9..5e23f4fc747bd01fa05eba63661bf7279b083317 100644
>> --- a/drivers/gpu/drm/i915/display/intel_display.c
>> +++ b/drivers/gpu/drm/i915/display/intel_display.c
>> @@ -5950,6 +5950,7 @@ static int intel_async_flip_check_uapi(struct intel_atomic_state *state,
>>   		drm_dbg_kms(display->drm,
>>   			    "[CRTC:%d:%s] modeset required\n",
>>   			    crtc->base.base.id, crtc->base.name);
>> +		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
>>   		return -EINVAL;
>>   	}
>>   
>> @@ -6019,6 +6020,7 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
>>   		drm_dbg_kms(display->drm,
>>   			    "[CRTC:%d:%s] modeset required\n",
>>   			    crtc->base.base.id, crtc->base.name);
>> +		state->base.error_code->failure_flags = DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET;
>>   		return -EINVAL;
>>   	}
>>   
>> @@ -6061,6 +6063,8 @@ static int intel_async_flip_check_hw(struct intel_atomic_state *state, struct in
>>   				    plane->base.base.id, plane->base.name,
>>   				    &new_plane_state->hw.fb->format->format,
>>   				    new_plane_state->hw.fb->modifier);
>> +			state->base.error_code->failure_flags =
>> +				DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED;
>>   			return -EINVAL;
>>   		}
>>   
>>
Thanks and Regards,
Arun R Murthy
-------------------
[1] https://hackmd.io/f3bDn3kyRUalLn4LbMfCVQ#Commit-Failure-Feedback

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Jani Nikula]

On Sat, 23 Aug 2025, "Murthy, Arun R" <arun.r.murthy@intel.com> wrote:
> On 22-08-2025 16:07, Jani Nikula wrote:
>> On Fri, 22 Aug 2025, Arun R Murthy <arun.r.murthy@intel.com> wrote:
>>> There can be multiple reasons for a failure in atomic_ioctl. Most often
>>> in these error conditions -EINVAL is returned. User/Compositor would
>>> have to blindly take a call on failure of this ioctl so as to use
>>> ALLOW_MODESET or any. It would be good if user/compositor gets a
>>> readable error code on failure so they can take proper corrections in
>>> the next commit.
>>> The struct drm_mode_atomic is being passed by the user/compositor which
>>> holds the properties for modeset/flip. Reusing the same struct for
>>> returning the error code in case of failure can save by creating a new
>>> uapi/interface for returning the error code.
>>> The element 'reserved' in the struct drm_mode_atomic is used for
>>> returning the user readable error code. This points to the struct
>>> drm_mode_atomic_err_code. Failure reasons have been initialized in
>>> DRM_MODE_ATOMIC_FAILURE_REASON.
>> At the moment, I'm not (yet) convinced any of this will work, even as a
>> concept.
> km_flip test in IGT has been run and is working fine with these patch 
> series. Also
> the existing kms_atomic test with atomic_invalid_test has been modified to
> test this error code(https://patchwork.freedesktop.org/series/153330/)
>
> Am I missing anything over here?

Having a few tests in IGT does not mean a non-trivial userspace
(compositor) can use the information in a useful way.

>> Even so, I'll comment on some of the choices in the series.
>>
>>> Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com>
>>> ---
>>>   include/uapi/drm/drm_mode.h | 42 ++++++++++++++++++++++++++++++++++++++++++
>>>   1 file changed, 42 insertions(+)
>>>
>>> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
>>> index a122bea2559387576150236e3a88f99c24ad3138..f0986a3fe9a7d61e57e9a9a5ec01a604343f6930 100644
>>> --- a/include/uapi/drm/drm_mode.h
>>> +++ b/include/uapi/drm/drm_mode.h
>>> @@ -45,6 +45,7 @@ extern "C" {
>>>   #define DRM_CONNECTOR_NAME_LEN	32
>>>   #define DRM_DISPLAY_MODE_LEN	32
>>>   #define DRM_PROP_NAME_LEN	32
>>> +#define DRM_MODE_ATOMIC_FAILURE_STRING_LEN	64
>>>   
>>>   #define DRM_MODE_TYPE_BUILTIN	(1<<0) /* deprecated */
>>>   #define DRM_MODE_TYPE_CLOCK_C	((1<<1) | DRM_MODE_TYPE_BUILTIN) /* deprecated */
>>> @@ -1157,6 +1158,47 @@ struct drm_mode_destroy_dumb {
>>>   		DRM_MODE_ATOMIC_NONBLOCK |\
>>>   		DRM_MODE_ATOMIC_ALLOW_MODESET)
>>>   
>>> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
>> Please don't add macros that expect other macros in the context. They
>> become very confusing. Just pass in the other macro to use. See MACRO__
>> in include/drm/intel/pciids.h for an example.
> Here we are mapping an error_code defined as enum with a corresponding 
> string
> using the X macros.
>
> Anyway will have a look at the macro in include/drm/intel/pciids.h
>
>
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
>>> +
>>> +#define FAILURE_REASON(flag, reason) flag,
>>> +typedef enum {
>>> +	DRM_MODE_ATOMIC_FAILURE_REASON
>>> +} drm_mode_atomic_failure_flag;
>>> +#undef FAILURE_REASON
>> This is a uapi header. Do we really want all of that macro trickery here
>> in the first place? What's wrong with just a plain enum? (Or macros.)
> This is defined as enum. We have two separate list one for enum and
> other for error message(string) but  upon user adding a new
> element/error_code in the enum, there can be a possibility of missing
> adding the error string. So have linked enum with the corresponding
> error message/string so both are at a single place and upon adding new
> entry will be easy for the user.

I think the overall point is this:

a) If the messages are fixed for each enum, there is no point in passing
   that message across the ioctl. Userspace can map the enums and
   messages directly.

b) If the messages are not fixed, I see no point in having the above
   fixed messages at all.

c) If the messages are not fixed, but you want to have default messages,
   the messages do not belong in the uapi header at all. Hide them in
   the kernel.

>>> +
>>> +#define FAILURE_REASON(flag, reason) #reason,
>>> +extern const char *drm_mode_atomic_failure_string[];
>>> +#undef FAILURE_REASON
>> Data is not an interface. Don't expose arrays like this. Who is even
>> going to know what its size is? Again, uapi header, where does it point,
>> what address space is it?
>>
> Sorry missed adding documentation for this. Will add in my next series.

Documentation does not fix the issue that data is not an interface.

>>> +
>>> +/**
>>> + * drm_mode_atomic_err_code - struct to store the error code
>>> + *
>>> + * pointer to this struct will be stored in reserved variable of
>>> + * struct drm_mode_atomic to report the failure cause to the user.
>>> + *
>>> + * @failure_flags: error codes defined in drm_atomic_failure.failure_flag
>> Flags? As in a mask of multiple things? Is 64 going to be enough for
>> everyone, all conditions in all drivers?
> Should be more that sufficient, this is an enum.

Then don't call it "flags".

>> OTOH, what's the promise wrt multiple reasons? Practically all drivers
>> bail out at the sight of first issue, and it's usually pretty much
>> meaningless to continue to figure out *other* reasons after that.
>>
>> And this brings us to one of my main concerns on making this fly:
>>
>> You bail out on the first thing, but you can't expect all drivers to
>> check stuff in the same order. It's not practical. So different drivers
>> will likely return different reasons for virtually the same
>> condition. So how is userspace going to handle that?
> Most of the error check is in the drm-core and these are common across
> the drivers.  Upon drivers sending driver specific error code, the
> compositor may not be able to handle the error code. But in cases
> required compositor changes can be added so as to handle them. In
> general the generic error conditions that are met by compositor and
> for which compositor can take corrective measurements are listed down
> first.

I think we're going to need a handful of error codes for starters, along
with compositor changes that make meaningful and benefitial usage of the
error codes.

>>> + * @failure_string_ptr: pointer to user readable error message drm_mode_failure.failure_string
>> Is the string part of uapi, and can never change? Doesn't sound
>> great. What's wrong with just a plain enum?
> enum is sufficient enough and this string is added so that this error 
> message
>
> can be printed out in the compositor logs for the user.
>
> Thanks and Regards,
> Arun R Murthy
> -------------------
>
>>> + * @failure_obj_ptr: pointer to the drm_object that caused error
>>> + * @reserved: reserved for future use
>>> + * @count_objs: count of drm_objects if multiple drm_objects caused error
>>> + */
>>> +struct drm_mode_atomic_err_code {
>>> +	__u64 failure_flags;
>>> +	__u64 failure_objs_ptr;
>>> +	__u64 reserved;
>>> +	__u32 count_objs;
>>> +	char failure_string[DRM_MODE_ATOMIC_FAILURE_STRING_LEN];
>>> +};
>>> +
>>>   struct drm_mode_atomic {
>>>   	__u32 flags;
>>>   	__u32 count_objs;

-- 
Jani Nikula, Intel

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Jani Nikula]

On Sat, 23 Aug 2025, "Murthy, Arun R" <arun.r.murthy@intel.com> wrote:
> On 22-08-2025 21:44, Xaver Hugl wrote:
>>> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
>> As mentioned before, some of these errors are a bit too specific. We
>> don't need to have an enum value for every way the API can be used
>> wrongly - CAP_NOT_ENABLED, INVALID_FLAG, PAGE_FLIP_ASYNC and
>> MODIFIER_NOT_SUPPORTED should all just be one enum value for "invalid
>> API usage".
>> In general, there should only be enum values that the compositor
>> implementation can actually use on end-user systems. For further
>> information when debugging a broken compositor implementation, other
>> tools can be used instead, like drm debug logging or the returned
>> string.
> I have considered your comment in the last series and have removed 
> driver specific errors.
> Anyway will have a look again on this and will get back.
>>> +#define FAILURE_REASON(flag, reason) flag,
>>> +typedef enum {
>>> +       DRM_MODE_ATOMIC_FAILURE_REASON
>>> +} drm_mode_atomic_failure_flag;
>>> +#undef FAILURE_REASON
>>> +
>>> +#define FAILURE_REASON(flag, reason) #reason,
>>> +extern const char *drm_mode_atomic_failure_string[];
>>> +#undef FAILURE_REASON
>> The intention for the string wasn't for the enum values to be paired
>> with a description of the enum - that belongs into documentation, not
>> uAPI.
>>
>> The idea behind it was that drivers could add driver-specific
>> information in the string for compositors to log (only in commits
>> where failure isn't normally expected), so we have an easier time
>> debugging issues a user system experienced by looking at the
>> compositor logs. Sending the enum value again in string form isn't
>> useful.
>
> We are not sending enum value in string. Its just a single place where 
> we have both enum and string. Upon user adding new error codes if both 
> enum and string are at a single place it would be easy for the user. 
> Hence adding both in a single place using X macros.
>
> Its not mandatory to have a string for every enum, the string can be 
> left empty if not required, or later in the driver user can overwrite 
> the string as well.

See my reply [1] about fixed vs. non-fixed error messages.

I believe Xaver is also saying we don't want the fixed error messages,
and especially not in a uapi header.

BR,
Jani.


[1] https://lore.kernel.org/r/419591dda7158b3d56c40aac0df86ca499202676@intel.com

-- 
Jani Nikula, Intel

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Murthy, Arun R]

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

On 25-08-2025 15:14, Jani Nikula wrote:
> On Sat, 23 Aug 2025, "Murthy, Arun R"<arun.r.murthy@intel.com> wrote:
>> On 22-08-2025 16:07, Jani Nikula wrote:
>>> On Fri, 22 Aug 2025, Arun R Murthy<arun.r.murthy@intel.com> wrote:
>>>> There can be multiple reasons for a failure in atomic_ioctl. Most often
>>>> in these error conditions -EINVAL is returned. User/Compositor would
>>>> have to blindly take a call on failure of this ioctl so as to use
>>>> ALLOW_MODESET or any. It would be good if user/compositor gets a
>>>> readable error code on failure so they can take proper corrections in
>>>> the next commit.
>>>> The struct drm_mode_atomic is being passed by the user/compositor which
>>>> holds the properties for modeset/flip. Reusing the same struct for
>>>> returning the error code in case of failure can save by creating a new
>>>> uapi/interface for returning the error code.
>>>> The element 'reserved' in the struct drm_mode_atomic is used for
>>>> returning the user readable error code. This points to the struct
>>>> drm_mode_atomic_err_code. Failure reasons have been initialized in
>>>> DRM_MODE_ATOMIC_FAILURE_REASON.
>>> At the moment, I'm not (yet) convinced any of this will work, even as a
>>> concept.
>> km_flip test in IGT has been run and is working fine with these patch
>> series. Also
>> the existing kms_atomic test with atomic_invalid_test has been modified to
>> test this error code(https://patchwork.freedesktop.org/series/153330/)
>>
>> Am I missing anything over here?
> Having a few tests in IGT does not mean a non-trivial userspace
> (compositor) can use the information in a useful way.
>
>>> Even so, I'll comment on some of the choices in the series.
>>>
>>>> Signed-off-by: Arun R Murthy<arun.r.murthy@intel.com>
>>>> ---
>>>>    include/uapi/drm/drm_mode.h | 42 ++++++++++++++++++++++++++++++++++++++++++
>>>>    1 file changed, 42 insertions(+)
>>>>
>>>> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
>>>> index a122bea2559387576150236e3a88f99c24ad3138..f0986a3fe9a7d61e57e9a9a5ec01a604343f6930 100644
>>>> --- a/include/uapi/drm/drm_mode.h
>>>> +++ b/include/uapi/drm/drm_mode.h
>>>> @@ -45,6 +45,7 @@ extern "C" {
>>>>    #define DRM_CONNECTOR_NAME_LEN	32
>>>>    #define DRM_DISPLAY_MODE_LEN	32
>>>>    #define DRM_PROP_NAME_LEN	32
>>>> +#define DRM_MODE_ATOMIC_FAILURE_STRING_LEN	64
>>>>    
>>>>    #define DRM_MODE_TYPE_BUILTIN	(1<<0) /* deprecated */
>>>>    #define DRM_MODE_TYPE_CLOCK_C	((1<<1) | DRM_MODE_TYPE_BUILTIN) /* deprecated */
>>>> @@ -1157,6 +1158,47 @@ struct drm_mode_destroy_dumb {
>>>>    		DRM_MODE_ATOMIC_NONBLOCK |\
>>>>    		DRM_MODE_ATOMIC_ALLOW_MODESET)
>>>>    
>>>> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
>>> Please don't add macros that expect other macros in the context. They
>>> become very confusing. Just pass in the other macro to use. See MACRO__
>>> in include/drm/intel/pciids.h for an example.
>> Here we are mapping an error_code defined as enum with a corresponding
>> string
>> using the X macros.
>>
>> Anyway will have a look at the macro in include/drm/intel/pciids.h
>>
>>
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
>>>> +	FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
>>>> +
>>>> +#define FAILURE_REASON(flag, reason) flag,
>>>> +typedef enum {
>>>> +	DRM_MODE_ATOMIC_FAILURE_REASON
>>>> +} drm_mode_atomic_failure_flag;
>>>> +#undef FAILURE_REASON
>>> This is a uapi header. Do we really want all of that macro trickery here
>>> in the first place? What's wrong with just a plain enum? (Or macros.)
>> This is defined as enum. We have two separate list one for enum and
>> other for error message(string) but  upon user adding a new
>> element/error_code in the enum, there can be a possibility of missing
>> adding the error string. So have linked enum with the corresponding
>> error message/string so both are at a single place and upon adding new
>> entry will be easy for the user.
> I think the overall point is this:
>
> a) If the messages are fixed for each enum, there is no point in passing
>     that message across the ioctl. Userspace can map the enums and
>     messages directly.
>
> b) If the messages are not fixed, I see no point in having the above
>     fixed messages at all.
>
> c) If the messages are not fixed, but you want to have default messages,
>     the messages do not belong in the uapi header at all. Hide them in
>     the kernel.
My intention was option C, wherein have a default message and the driver 
is free
to overwrite the default message string if required.
I agree in that case doesn't make sense having it on uapi, will move this
inside the driver.
>>>> +
>>>> +#define FAILURE_REASON(flag, reason) #reason,
>>>> +extern const char *drm_mode_atomic_failure_string[];
>>>> +#undef FAILURE_REASON
>>> Data is not an interface. Don't expose arrays like this. Who is even
>>> going to know what its size is? Again, uapi header, where does it point,
>>> what address space is it?
>>>
>> Sorry missed adding documentation for this. Will add in my next series.
> Documentation does not fix the issue that data is not an interface.
As pointed out above will move to option C, and will get rid of this 
failure messages
in the uapi.
>>>> +
>>>> +/**
>>>> + * drm_mode_atomic_err_code - struct to store the error code
>>>> + *
>>>> + * pointer to this struct will be stored in reserved variable of
>>>> + * struct drm_mode_atomic to report the failure cause to the user.
>>>> + *
>>>> + * @failure_flags: error codes defined in drm_atomic_failure.failure_flag
>>> Flags? As in a mask of multiple things? Is 64 going to be enough for
>>> everyone, all conditions in all drivers?
>> Should be more that sufficient, this is an enum.
> Then don't call it "flags".
Done!
>>> OTOH, what's the promise wrt multiple reasons? Practically all drivers
>>> bail out at the sight of first issue, and it's usually pretty much
>>> meaningless to continue to figure out *other* reasons after that.
>>>
>>> And this brings us to one of my main concerns on making this fly:
>>>
>>> You bail out on the first thing, but you can't expect all drivers to
>>> check stuff in the same order. It's not practical. So different drivers
>>> will likely return different reasons for virtually the same
>>> condition. So how is userspace going to handle that?
>> Most of the error check is in the drm-core and these are common across
>> the drivers.  Upon drivers sending driver specific error code, the
>> compositor may not be able to handle the error code. But in cases
>> required compositor changes can be added so as to handle them. In
>> general the generic error conditions that are met by compositor and
>> for which compositor can take corrective measurements are listed down
>> first.
> I think we're going to need a handful of error codes for starters, along
> with compositor changes that make meaningful and benefitial usage of the
> error codes.

Sure will narrow down the list in my next version.

Thanks and Regards,
Arun R Murthy
-------------------

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

Re: [PATCH v3 1/4] drm: Define user readable error codes for atomic ioctl

[Murthy, Arun R]

On 25-08-2025 15:17, Jani Nikula wrote:
> On Sat, 23 Aug 2025, "Murthy, Arun R" <arun.r.murthy@intel.com> wrote:
>> On 22-08-2025 21:44, Xaver Hugl wrote:
>>>> +#define DRM_MODE_ATOMIC_FAILURE_REASON \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_CAP_NOT_ENABLED, "DRM_ATOMIC capability not enabled") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_INVALID_FLAG, "invalid flag") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_PAGE_FLIP_ASYNC, "Legacy DRM_MODE_PAGE_FLIP_ASYNC not to be used in atomic ioctl") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_FLIP_EVENT_WITH_CHECKONLY, "requesting page-flip event with TEST_ONLY") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_CRTC_NEED_FULL_MODESET, "Need full modeset on this crtc") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_NEED_FULL_MODESET, "Need full modeset on all the connected crtc's") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_NOT_SUP_PLANE, "Async flip not supported on this plane") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_MODIFIER_NOT_SUPPORTED, "Modifier not supported on this plane with async flip") \
>>>> +       FAILURE_REASON(DRM_MODE_ATOMIC_ASYNC_PROP_CHANGED, "No property change allowed when async flip is enabled")
>>> As mentioned before, some of these errors are a bit too specific. We
>>> don't need to have an enum value for every way the API can be used
>>> wrongly - CAP_NOT_ENABLED, INVALID_FLAG, PAGE_FLIP_ASYNC and
>>> MODIFIER_NOT_SUPPORTED should all just be one enum value for "invalid
>>> API usage".
>>> In general, there should only be enum values that the compositor
>>> implementation can actually use on end-user systems. For further
>>> information when debugging a broken compositor implementation, other
>>> tools can be used instead, like drm debug logging or the returned
>>> string.
>> I have considered your comment in the last series and have removed
>> driver specific errors.
>> Anyway will have a look again on this and will get back.
>>>> +#define FAILURE_REASON(flag, reason) flag,
>>>> +typedef enum {
>>>> +       DRM_MODE_ATOMIC_FAILURE_REASON
>>>> +} drm_mode_atomic_failure_flag;
>>>> +#undef FAILURE_REASON
>>>> +
>>>> +#define FAILURE_REASON(flag, reason) #reason,
>>>> +extern const char *drm_mode_atomic_failure_string[];
>>>> +#undef FAILURE_REASON
>>> The intention for the string wasn't for the enum values to be paired
>>> with a description of the enum - that belongs into documentation, not
>>> uAPI.
>>>
>>> The idea behind it was that drivers could add driver-specific
>>> information in the string for compositors to log (only in commits
>>> where failure isn't normally expected), so we have an easier time
>>> debugging issues a user system experienced by looking at the
>>> compositor logs. Sending the enum value again in string form isn't
>>> useful.
>> We are not sending enum value in string. Its just a single place where
>> we have both enum and string. Upon user adding new error codes if both
>> enum and string are at a single place it would be easy for the user.
>> Hence adding both in a single place using X macros.
>>
>> Its not mandatory to have a string for every enum, the string can be
>> left empty if not required, or later in the driver user can overwrite
>> the string as well.
> See my reply [1] about fixed vs. non-fixed error messages.
>
> I believe Xaver is also saying we don't want the fixed error messages,
> and especially not in a uapi header.
>
> BR,
> Jani.
>
>
> [1] https://lore.kernel.org/r/419591dda7158b3d56c40aac0df86ca499202676@intel.com
As pointed out in option C of [2] I also feel better to have a default 
message so that
  compositor can use it for logging.

Thanks and Regards,
Arun R Murthy
--------------------

[2] 
https://lore.kernel.org/all/419591dda7158b3d56c40aac0df86ca499202676@intel.com/