Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in
@ 2026-08-31 10:44 Mallesh Koujalagi
  2026-08-31 10:44 ` [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
                   ` (11 more replies)
  0 siblings, 12 replies; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:44 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

The patch series improves error handling in the System Controller mailbox
driver by making error propagation more consistent and using
xe_log_err() for structured error reporting.

It also removes a few redundant checks and simplifies helper
interfaces where no error path exists.

v2:
- Use -EOVERFLOW.
- Drop sysctrl in xe_log_err. (Michal)
- Drop redundant %pe.
- Print response->count.

v3:
- Rebase.

v4:
- Replace xe_warn to xe_log_err_info for. (Michal)
- Use xe_assert().
- Change sysctrl_wait_bit_* to return errno.
- Make sysctrl_write_frame return type as void.
- Log mismatched header info.

v5:
- Fix align issue. (Michal)
- Use %#x after REG_FIELD_GET change.
- Use REG_FIELD_GET instead of FIELD_GET.
- Modify logging message.
- Change Error as EXFULL.
- Update logging message.

v6:
- Move REG_FIELD_GET to different patch. (Michal)
- Update response message.
- Move all REG_FIELD_GET to single patch.
- Update log message. (Michal)
- Change correct errno. (Michal)

Mallesh Koujalagi (9):
  drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear()
  drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set()
  drm/xe/sysctrl: Make sysctrl_write_frame() void
  drm/xe/sysctrl: Make sysctrl_read_frame() void
  drm/xe/sysctrl: Use xe_assert() for payload size validation
  drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET()
  drm/xe/sysctrl: Improve firmware response error logging
  drm/xe/sysctrl: Log group and command ID on mailbox failure
  drm/xe/sysctrl: Report 'System Controller event' error using SIGID

 drivers/gpu/drm/xe/xe_sysctrl_event.c   | 14 ++--
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 96 +++++++++++--------------
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.h |  6 +-
 3 files changed, 54 insertions(+), 62 deletions(-)

-- 
2.48.1


^ permalink raw reply	[flat|nested] 24+ messages in thread

* [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear()
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
@ 2026-08-31 10:44 ` Mallesh Koujalagi
  2026-08-31 10:55   ` sashiko-bot
  2026-09-04  6:01   ` Tauro, Riana
  2026-08-31 10:44 ` [PATCH v6 2/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set() Mallesh Koujalagi
                   ` (10 subsequent siblings)
  11 siblings, 2 replies; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:44 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Make sysctrl_wait_bit_clear() return an error code rather than a bool
and update callers to use xe_log_err() with the propagated error code.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
v5:
- Fix align issue. (Michal)
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 28 ++++++++++++-------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index 72baf1aa4b3a..c36cac8c6288 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -11,6 +11,7 @@
 
 #include "regs/xe_sysctrl_regs.h"
 #include "xe_device.h"
+#include "xe_log.h"
 #include "xe_mmio.h"
 #include "xe_pm.h"
 #include "xe_printk.h"
@@ -34,15 +35,11 @@ struct xe_sysctrl_mailbox_msg_hdr {
 #define XE_SYSCTRL_HDR_RESULT(hdr) \
 	FIELD_GET(SYSCTRL_HDR_RESULT_MASK, le32_to_cpu((hdr)->data))
 
-static bool sysctrl_wait_bit_clear(struct xe_sysctrl *sc, u32 bit_mask,
-				   unsigned int timeout_ms)
+static int sysctrl_wait_bit_clear(struct xe_sysctrl *sc, u32 bit_mask,
+				  unsigned int timeout_ms)
 {
-	int ret;
-
-	ret = xe_mmio_wait32_not(sc->mmio, SYSCTRL_MB_CTRL, bit_mask, bit_mask,
-				 timeout_ms * 1000, NULL, false);
-
-	return ret == 0;
+	return xe_mmio_wait32_not(sc->mmio, SYSCTRL_MB_CTRL, bit_mask, bit_mask,
+				  timeout_ms * 1000, NULL, false);
 }
 
 static bool sysctrl_wait_bit_set(struct xe_sysctrl *sc, u32 bit_mask,
@@ -146,12 +143,14 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
 	u32 ctrl_reg, total_frames, frame;
 	size_t bytes_sent, frame_size;
 	bool phase;
+	int ret;
 
 	total_frames = DIV_ROUND_UP(cmd_size, XE_SYSCTRL_MB_FRAME_SIZE);
 
-	if (!sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms)) {
-		xe_err(xe, "sysctrl: Mailbox busy\n");
-		return -EBUSY;
+	ret = sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms);
+	if (ret) {
+		xe_log_err(xe, SYSCTRL, ret, "Mailbox busy\n");
+		return ret;
 	}
 
 	ctrl_reg = xe_mmio_read32(sc->mmio, SYSCTRL_MB_CTRL);
@@ -174,9 +173,10 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
 
 		xe_mmio_write32(sc->mmio, SYSCTRL_MB_CTRL, ctrl_reg);
 
-		if (!sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms)) {
-			xe_err(xe, "sysctrl: Frame %u acknowledgment timeout\n", frame);
-			return -ETIMEDOUT;
+		ret = sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms);
+		if (ret) {
+			xe_log_err(xe, SYSCTRL, ret, "Frame %u acknowledgment timeout\n", frame);
+			return ret;
 		}
 
 		bytes_sent += frame_size;
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 2/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set()
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
  2026-08-31 10:44 ` [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
@ 2026-08-31 10:44 ` Mallesh Koujalagi
  2026-08-31 10:59   ` sashiko-bot
  2026-08-31 10:45 ` [PATCH v6 3/9] drm/xe/sysctrl: Make sysctrl_write_frame() void Mallesh Koujalagi
                   ` (9 subsequent siblings)
  11 siblings, 1 reply; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:44 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Make sysctrl_wait_bit_set() return an error code rather than a bool
and update callers to use xe_log_err() with the propagated error code.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
v5:
- Fix align issue. (Michal)
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index c36cac8c6288..b3d1538c8990 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -42,15 +42,11 @@ static int sysctrl_wait_bit_clear(struct xe_sysctrl *sc, u32 bit_mask,
 				  timeout_ms * 1000, NULL, false);
 }
 
-static bool sysctrl_wait_bit_set(struct xe_sysctrl *sc, u32 bit_mask,
-				 unsigned int timeout_ms)
+static int sysctrl_wait_bit_set(struct xe_sysctrl *sc, u32 bit_mask,
+				unsigned int timeout_ms)
 {
-	int ret;
-
-	ret = xe_mmio_wait32(sc->mmio, SYSCTRL_MB_CTRL, bit_mask, bit_mask,
-			     timeout_ms * 1000, NULL, false);
-
-	return ret == 0;
+	return xe_mmio_wait32(sc->mmio, SYSCTRL_MB_CTRL, bit_mask, bit_mask,
+			      timeout_ms * 1000, NULL, false);
 }
 
 static int sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
@@ -193,9 +189,10 @@ static int sysctrl_process_frame(struct xe_sysctrl *sc, void *out,
 	struct xe_device *xe = sc_to_xe(sc);
 	int ret;
 
-	if (!sysctrl_wait_bit_set(sc, SYSCTRL_MB_CTRL_RUN_BUSY_OUT, timeout_ms)) {
-		xe_err(xe, "sysctrl: Response frame timeout\n");
-		return -ETIMEDOUT;
+	ret = sysctrl_wait_bit_set(sc, SYSCTRL_MB_CTRL_RUN_BUSY_OUT, timeout_ms);
+	if (ret) {
+		xe_log_err(xe, SYSCTRL, ret, "Response frame timeout\n");
+		return ret;
 	}
 
 	ctrl_reg = xe_mmio_read32(sc->mmio, SYSCTRL_MB_CTRL);
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 3/9] drm/xe/sysctrl: Make sysctrl_write_frame() void
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
  2026-08-31 10:44 ` [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
  2026-08-31 10:44 ` [PATCH v6 2/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set() Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-08-31 10:45 ` [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void Mallesh Koujalagi
                   ` (8 subsequent siblings)
  11 siblings, 0 replies; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

The function always returned 0, so the int return type is misleading.
Convert to void and drop the error handling in the caller.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index b3d1538c8990..3ef60067e226 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -49,8 +49,8 @@ static int sysctrl_wait_bit_set(struct xe_sysctrl *sc, u32 bit_mask,
 			      timeout_ms * 1000, NULL, false);
 }
 
-static int sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
-			       size_t len)
+static void sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
+				size_t len)
 {
 	static const struct xe_reg regs[] = {
 		SYSCTRL_MB_DATA0, SYSCTRL_MB_DATA1, SYSCTRL_MB_DATA2, SYSCTRL_MB_DATA3
@@ -66,8 +66,6 @@ static int sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
 
 	for (i = 0; i < dw; i++)
 		xe_mmio_write32(sc->mmio, regs[i], val[i]);
-
-	return 0;
 }
 
 static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
@@ -156,10 +154,7 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
 	for (frame = 0; frame < total_frames; frame++) {
 		frame_size = min_t(size_t, cmd_size - bytes_sent, XE_SYSCTRL_MB_FRAME_SIZE);
 
-		if (sysctrl_write_frame(sc, mbox_cmd + bytes_sent, frame_size)) {
-			xe_err(xe, "sysctrl: Failed to write frame %u\n", frame);
-			return -EIO;
-		}
+		sysctrl_write_frame(sc, mbox_cmd + bytes_sent, frame_size);
 
 		ctrl_reg = SYSCTRL_MB_CTRL_RUN_BUSY |
 			   REG_FIELD_PREP(SYSCTRL_FRAME_CURRENT_MASK, frame) |
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (2 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 3/9] drm/xe/sysctrl: Make sysctrl_write_frame() void Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-09-02 16:16   ` Michal Wajdeczko
  2026-08-31 10:45 ` [PATCH v6 5/9] drm/xe/sysctrl: Use xe_assert() for payload size validation Mallesh Koujalagi
                   ` (7 subsequent siblings)
  11 siblings, 1 reply; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Convert it to return void and remove the unnecessary error
handling in sysctrl_process_frame().

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index 3ef60067e226..4841dd7f2724 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -68,8 +68,8 @@ static void sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
 		xe_mmio_write32(sc->mmio, regs[i], val[i]);
 }
 
-static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
-			      size_t len)
+static void sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
+			       size_t len)
 {
 	static const struct xe_reg regs[] = {
 		SYSCTRL_MB_DATA0, SYSCTRL_MB_DATA1, SYSCTRL_MB_DATA2, SYSCTRL_MB_DATA3
@@ -85,8 +85,6 @@ static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
 		val[i] = xe_mmio_read32(sc->mmio, regs[i]);
 
 	memcpy(frame, val, len);
-
-	return 0;
 }
 
 static void sysctrl_clear_response(struct xe_sysctrl *sc)
@@ -194,10 +192,7 @@ static int sysctrl_process_frame(struct xe_sysctrl *sc, void *out,
 	total_frames = FIELD_GET(SYSCTRL_FRAME_TOTAL_MASK, ctrl_reg);
 	curr_frame = FIELD_GET(SYSCTRL_FRAME_CURRENT_MASK, ctrl_reg);
 
-	ret = sysctrl_read_frame(sc, out, frame_size);
-	if (ret)
-		return ret;
-
+	sysctrl_read_frame(sc, out, frame_size);
 	sysctrl_clear_response(sc);
 
 	if (curr_frame == total_frames)
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 5/9] drm/xe/sysctrl: Use xe_assert() for payload size validation
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (3 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-08-31 10:45 ` [PATCH v6 6/9] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET() Mallesh Koujalagi
                   ` (6 subsequent siblings)
  11 siblings, 0 replies; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Replace the runtime payload length check with xe_assert(),
as oversized messages should never reach code path.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index 4841dd7f2724..50bbdab072de 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -103,10 +103,7 @@ static int sysctrl_prepare_command(struct xe_device *xe,
 
 	xe_assert(xe, command <= SYSCTRL_HDR_COMMAND_MAX);
 
-	if (data_in_len > XE_SYSCTRL_MB_MAX_MESSAGE_SIZE - sizeof(*hdr)) {
-		xe_err(xe, "sysctrl: Input data too large: %zu bytes\n", data_in_len);
-		return -EINVAL;
-	}
+	xe_assert(xe, data_in_len <= XE_SYSCTRL_MB_MAX_MESSAGE_SIZE - sizeof(*hdr));
 
 	size = sizeof(*hdr) + data_in_len;
 
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 6/9] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET()
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (4 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 5/9] drm/xe/sysctrl: Use xe_assert() for payload size validation Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-08-31 11:07   ` sashiko-bot
  2026-08-31 10:45 ` [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging Mallesh Koujalagi
                   ` (5 subsequent siblings)
  11 siblings, 1 reply; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Use REG_FIELD_GET() when extracting fields from SysCtrl and
application headers. This better reflects that the masks
represent register-style bitfields and improves consistency.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 8 ++++----
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.h | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index 50bbdab072de..3aa868f60b38 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -24,16 +24,16 @@ struct xe_sysctrl_mailbox_msg_hdr {
 } __packed;
 
 #define XE_SYSCTRL_HDR_GROUP_ID(hdr) \
-	FIELD_GET(SYSCTRL_HDR_GROUP_ID_MASK, le32_to_cpu((hdr)->data))
+	REG_FIELD_GET(SYSCTRL_HDR_GROUP_ID_MASK, le32_to_cpu((hdr)->data))
 
 #define XE_SYSCTRL_HDR_COMMAND(hdr) \
-	FIELD_GET(SYSCTRL_HDR_COMMAND_MASK, le32_to_cpu((hdr)->data))
+	REG_FIELD_GET(SYSCTRL_HDR_COMMAND_MASK, le32_to_cpu((hdr)->data))
 
 #define XE_SYSCTRL_HDR_IS_RESPONSE(hdr) \
-	FIELD_GET(SYSCTRL_HDR_IS_RESPONSE, le32_to_cpu((hdr)->data))
+	REG_FIELD_GET(SYSCTRL_HDR_IS_RESPONSE, le32_to_cpu((hdr)->data))
 
 #define XE_SYSCTRL_HDR_RESULT(hdr) \
-	FIELD_GET(SYSCTRL_HDR_RESULT_MASK, le32_to_cpu((hdr)->data))
+	REG_FIELD_GET(SYSCTRL_HDR_RESULT_MASK, le32_to_cpu((hdr)->data))
 
 static int sysctrl_wait_bit_clear(struct xe_sysctrl *sc, u32 bit_mask,
 				  unsigned int timeout_ms)
diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h
index f0d5e3d7f5e3..5a3dd14e3a11 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h
@@ -15,13 +15,13 @@ struct xe_sysctrl;
 struct xe_sysctrl_mailbox_command;
 
 #define XE_SYSCTRL_APP_HDR_GROUP_ID(hdr) \
-	FIELD_GET(APP_HDR_GROUP_ID_MASK, (hdr)->data)
+	REG_FIELD_GET(APP_HDR_GROUP_ID_MASK, (hdr)->data)
 
 #define XE_SYSCTRL_APP_HDR_COMMAND(hdr) \
-	FIELD_GET(APP_HDR_COMMAND_MASK, (hdr)->data)
+	REG_FIELD_GET(APP_HDR_COMMAND_MASK, (hdr)->data)
 
 #define XE_SYSCTRL_APP_HDR_VERSION(hdr) \
-	FIELD_GET(APP_HDR_VERSION_MASK, (hdr)->data)
+	REG_FIELD_GET(APP_HDR_VERSION_MASK, (hdr)->data)
 
 void xe_sysctrl_create_command(struct xe_sysctrl_mailbox_command *command, u8 group_id, u8 cmd_id,
 			       void *request, size_t request_len, void *response,
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (5 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 6/9] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET() Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-08-31 11:00   ` sashiko-bot
  2026-09-02 16:22   ` Michal Wajdeczko
  2026-08-31 10:45 ` [PATCH v6 8/9] drm/xe/sysctrl: Log group and command ID on mailbox failure Mallesh Koujalagi
                   ` (4 subsequent siblings)
  11 siblings, 2 replies; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Use xe_log_err() helper and include additional details when a
response header mismatch occurs.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
---
v5:
- Use %#x after REG_FIELD_GET change. (Michal)

v6:
- Move REG_FIELD_GET to different patch. (Michal)
- Update response message.
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index 3aa868f60b38..87e656820279 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -233,13 +233,15 @@ static int sysctrl_receive_frames(struct xe_sysctrl *sc,
 	if (!XE_SYSCTRL_HDR_IS_RESPONSE(hdr) ||
 	    XE_SYSCTRL_HDR_GROUP_ID(hdr) != XE_SYSCTRL_HDR_GROUP_ID(req) ||
 	    XE_SYSCTRL_HDR_COMMAND(hdr) != XE_SYSCTRL_HDR_COMMAND(req)) {
-		xe_err(xe, "sysctrl: Response header mismatch\n");
+		xe_log_err(xe, SYSCTRL, -EPROTO,
+			   "Response header mismatch: %#x != %#x\n",
+			   le32_to_cpu(hdr->data), le32_to_cpu(req->data));
 		return -EPROTO;
 	}
 
 	if (XE_SYSCTRL_HDR_RESULT(hdr) != 0) {
-		xe_err(xe, "sysctrl: Firmware error: 0x%02lx\n",
-		       XE_SYSCTRL_HDR_RESULT(hdr));
+		xe_log_err(xe, SYSCTRL, -EIO, "Firmware error: %#04x\n",
+			   XE_SYSCTRL_HDR_RESULT(hdr));
 		return -EIO;
 	}
 
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 8/9] drm/xe/sysctrl: Log group and command ID on mailbox failure
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (6 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-08-31 11:00   ` sashiko-bot
  2026-08-31 10:45 ` [PATCH v6 9/9] drm/xe/sysctrl: Report 'System Controller event' error using SIGID Mallesh Koujalagi
                   ` (3 subsequent siblings)
  11 siblings, 1 reply; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Include the mailbox group ID and command code in error logs
for command preparation and execution failures.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
v5:
- Use REG_FIELD_GET instead of FIELD_GET. (Michal)
- Modify logging message.

v6:
- Move all REG_FIELD_GET to single patch.
- Update log message. (Michal)
---
 drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
index 87e656820279..d5e391c045c6 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
@@ -353,7 +353,8 @@ int xe_sysctrl_send_command(struct xe_sysctrl *sc,
 				      cmd->data_in, cmd->data_in_len,
 				      &mbox_cmd, &cmd_size);
 	if (ret) {
-		xe_err(xe, "sysctrl: Failed to prepare command: %pe\n", ERR_PTR(ret));
+		xe_log_err(xe, SYSCTRL, ret, "Failed to prepare command %#x.%#x\n",
+			   group_id, command_code);
 		return ret;
 	}
 
@@ -363,7 +364,8 @@ int xe_sysctrl_send_command(struct xe_sysctrl *sc,
 				   cmd->data_out, cmd->data_out_len, rdata_len,
 				   XE_SYSCTRL_MB_DEFAULT_TIMEOUT_MS);
 	if (ret)
-		xe_err(xe, "sysctrl: Mailbox command failed: %pe\n", ERR_PTR(ret));
+		xe_log_err(xe, SYSCTRL, ret, "Mailbox command failed to send %#x.%#x\n",
+			   group_id, command_code);
 
 	kfree(mbox_cmd);
 
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [PATCH v6 9/9] drm/xe/sysctrl: Report 'System Controller event' error using SIGID
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (7 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 8/9] drm/xe/sysctrl: Log group and command ID on mailbox failure Mallesh Koujalagi
@ 2026-08-31 10:45 ` Mallesh Koujalagi
  2026-08-31 11:00   ` sashiko-bot
  2026-08-31 14:37 ` ✓ CI.KUnit: success for drm/xe/sysctrl: Clean up error handling in (rev3) Patchwork
                   ` (2 subsequent siblings)
  11 siblings, 1 reply; 24+ messages in thread
From: Mallesh Koujalagi @ 2026-08-31 10:45 UTC (permalink / raw)
  To: intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, michal.wajdeczko,
	aravind.iddamsetty, umesh.nerlige.ramappa, dnyaneshwar.bhadane,
	anoop.c.vijay, Mallesh Koujalagi

Report 'System Controller event' error using xe_log() helpers.

Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
v5:
- Change Error as EXFULL. (Michal)
- Update logging message.

v6:
- Change correct errno. (Michal)
---
 drivers/gpu/drm/xe/xe_sysctrl_event.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_sysctrl_event.c b/drivers/gpu/drm/xe/xe_sysctrl_event.c
index 15ddd2d8bc6d..3239c110a66a 100644
--- a/drivers/gpu/drm/xe/xe_sysctrl_event.c
+++ b/drivers/gpu/drm/xe/xe_sysctrl_event.c
@@ -5,6 +5,7 @@
 
 #include "xe_device.h"
 #include "xe_irq.h"
+#include "xe_log.h"
 #include "xe_printk.h"
 #include "xe_ras.h"
 #include "xe_sysctrl.h"
@@ -25,23 +26,26 @@ static void get_pending_event(struct xe_sysctrl *sc, struct xe_sysctrl_mailbox_c
 
 		ret = xe_sysctrl_send_command(sc, command, &len);
 		if (ret) {
-			xe_err(xe, "sysctrl: failed to get pending event %d\n", ret);
+			xe_log_err(xe, SYSCTRL, ret, "Failed to get pending event\n");
 			return;
 		}
 
 		if (len != sizeof(*response)) {
-			xe_err(xe, "sysctrl: unexpected event response length %zu (expected %zu)\n",
-			       len, sizeof(*response));
+			xe_log_err(xe, SYSCTRL, -EPROTO,
+				   "Unexpected event response length %zu (expected %zu)\n",
+				   len, sizeof(*response));
 			return;
 		}
 
 		if (response->event == XE_SYSCTRL_EVENT_THRESHOLD_CROSSED)
 			xe_ras_counter_threshold_crossed(xe, response);
 		else
-			xe_warn(xe, "sysctrl: unexpected event %#x\n", response->event);
+			xe_log_err_info(xe, SYSCTRL, -ENOPKG,
+					"Unexpected event %#x\n", response->event);
 
 		if (!--count) {
-			xe_err(xe, "sysctrl: event flooding\n");
+			xe_log_err(xe, SYSCTRL, -EXFULL,
+				   "Ignored %u events due to flooding\n", response->count);
 			return;
 		}
 
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear()
  2026-08-31 10:44 ` [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
@ 2026-08-31 10:55   ` sashiko-bot
  2026-09-04  6:01   ` Tauro, Riana
  1 sibling, 0 replies; 24+ messages in thread
From: sashiko-bot @ 2026-08-31 10:55 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The patch introduces a call to `xe_log_err()` and includes `xe_log.h`, but neither the header nor the macro exist in the source tree, causing an immediate build failure.
--

commit 2a9731ca81fbb37bb5eeef908043568e8555c104
Author: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear()

This commit updates sysctrl_wait_bit_clear() to return an error code instead
of a boolean. It also modifies the callers to log failures using xe_log_err().

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 72baf1aa4b3ad..c36cac8c62887 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -11,6 +11,7 @@
>  
>  #include "regs/xe_sysctrl_regs.h"
>  #include "xe_device.h"
> +#include "xe_log.h"

[Severity: Low]
Does this header exist in the repository? It appears to be missing, which
causes a build failure.

[ ... ]

> @@ -146,12 +143,14 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
>  	u32 ctrl_reg, total_frames, frame;
>  	size_t bytes_sent, frame_size;
>  	bool phase;
> +	int ret;
>  
>  	total_frames = DIV_ROUND_UP(cmd_size, XE_SYSCTRL_MB_FRAME_SIZE);
>  
> -	if (!sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms)) {
> -		xe_err(xe, "sysctrl: Mailbox busy\n");
> -		return -EBUSY;
> +	ret = sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms);
> +	if (ret) {
> +		xe_log_err(xe, SYSCTRL, ret, "Mailbox busy\n");

[Severity: Low]
Is xe_log_err() defined anywhere in the tree? This introduces a call to an
undefined macro, resulting in an implicit declaration error during compilation
in sysctrl_send_frames().

[ ... ]

> @@ -174,9 +173,10 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
>  
>  		xe_mmio_write32(sc->mmio, SYSCTRL_MB_CTRL, ctrl_reg);
>  
> -		if (!sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms)) {
> -			xe_err(xe, "sysctrl: Frame %u acknowledgment timeout\n", frame);
> -			return -ETIMEDOUT;
> +		ret = sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms);
> +		if (ret) {
> +			xe_log_err(xe, SYSCTRL, ret, "Frame %u acknowledgment timeout\n", frame);

[Severity: Low]
This is another instance of using the undefined xe_log_err() macro, leading
to a compilation failure.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260831104456.1770605-11-mallesh.koujalagi@intel.com?part=1

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 2/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set()
  2026-08-31 10:44 ` [PATCH v6 2/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set() Mallesh Koujalagi
@ 2026-08-31 10:59   ` sashiko-bot
  0 siblings, 0 replies; 24+ messages in thread
From: sashiko-bot @ 2026-08-31 10:59 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Build failure due to missing `xe_log.h` dependency and undefined `xe_log_err` macro/function.
--

commit 857fe80daf658c4ae70ca1e5f7664ff0bfb076d5
Author: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set()

This patch changes sysctrl_wait_bit_set() to return an error code rather
than a boolean and updates callers to propagate and log the error code.

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index c36cac8c62887..b3d1538c89900 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
[ ... ]
> @@ -193,9 +189,10 @@ static int sysctrl_process_frame(struct xe_sysctrl *sc, void *out,
>  	struct xe_device *xe = sc_to_xe(sc);
>  	int ret;
>  
> -	if (!sysctrl_wait_bit_set(sc, SYSCTRL_MB_CTRL_RUN_BUSY_OUT, timeout_ms)) {
> -		xe_err(xe, "sysctrl: Response frame timeout\n");
> -		return -ETIMEDOUT;
> +	ret = sysctrl_wait_bit_set(sc, SYSCTRL_MB_CTRL_RUN_BUSY_OUT, timeout_ms);
> +	if (ret) {
> +		xe_log_err(xe, SYSCTRL, ret, "Response frame timeout\n");

[Severity: Low]
Will this result in a build failure? 

It appears that the xe_log_err macro and the SYSCTRL domain identifier are
not defined in this file, nor is there an include for the corresponding log
header. 

Are we missing a header inclusion, or does this patch depend on another
unmerged change that introduces this logging infrastructure?

> +		return ret;
>  	}
>  
>  	ctrl_reg = xe_mmio_read32(sc->mmio, SYSCTRL_MB_CTRL);

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260831104456.1770605-11-mallesh.koujalagi@intel.com?part=2

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 8/9] drm/xe/sysctrl: Log group and command ID on mailbox failure
  2026-08-31 10:45 ` [PATCH v6 8/9] drm/xe/sysctrl: Log group and command ID on mailbox failure Mallesh Koujalagi
@ 2026-08-31 11:00   ` sashiko-bot
  0 siblings, 0 replies; 24+ messages in thread
From: sashiko-bot @ 2026-08-31 11:00 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Missing xe_log.h header and xe_log_err definition causes a compilation failure.
--

commit 2afdd4e6d2ae348d6cec426e07181e76137fd2f6
Author: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

drm/xe/sysctrl: Log group and command ID on mailbox failure

This commit updates the error logging for mailbox command preparation and
execution failures to include the group ID and command code.

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 87e6568202795..d5e391c045c60 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -353,7 +353,8 @@ int xe_sysctrl_send_command(struct xe_sysctrl *sc,
>  				      cmd->data_in, cmd->data_in_len,
>  				      &mbox_cmd, &cmd_size);
>  	if (ret) {
> -		xe_err(xe, "sysctrl: Failed to prepare command: %pe\n", ERR_PTR(ret));
> +		xe_log_err(xe, SYSCTRL, ret, "Failed to prepare command %#x.%#x\n",
> +			   group_id, command_code);

[Severity: Low]
Does this cause a build failure? 

The xe_log_err macro and the SYSCTRL log category appear to be undeclared in
this file. Is it possible that a required include header is missing here, or
perhaps a prerequisite patch that defines these logging facilities was dropped
from the series?

>  		return ret;
>  	}
>  

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260831104456.1770605-11-mallesh.koujalagi@intel.com?part=8

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 9/9] drm/xe/sysctrl: Report 'System Controller event' error using SIGID
  2026-08-31 10:45 ` [PATCH v6 9/9] drm/xe/sysctrl: Report 'System Controller event' error using SIGID Mallesh Koujalagi
@ 2026-08-31 11:00   ` sashiko-bot
  0 siblings, 0 replies; 24+ messages in thread
From: sashiko-bot @ 2026-08-31 11:00 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The patch relies on a missing header file 'xe_log.h' and undefined logging macros and identifiers.
--

commit 6685ffdeb0695b8ea21292cfaa83d3bea319ad7d
Author: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

drm/xe/sysctrl: Report 'System Controller event' error using SIGID

This commit updates the System Controller event logging to use xe_log helpers
instead of the standard xe_err or xe_warn macros.

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_event.c b/drivers/gpu/drm/xe/xe_sysctrl_event.c
> index da395148ee9dc..5804d3649569c 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_event.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_event.c
> @@ -5,6 +5,7 @@
>  
>  #include "xe_device.h"
>  #include "xe_irq.h"
> +#include "xe_log.h"

[Severity: Low]
Is this header missing from the patch series? The xe_log.h file does not
appear to exist in the tree, which results in an immediate build failure
when applying this patch.

>  #include "xe_printk.h"
>  #include "xe_ras.h"
>  #include "xe_sysctrl.h"

[ ... ]

> @@ -25,23 +26,26 @@ static void get_pending_event(struct xe_sysctrl *sc, struct xe_sysctrl_mailbox_c
>  
>  		ret = xe_sysctrl_send_command(sc, command, &len);
>  		if (ret) {
> -			xe_err(xe, "sysctrl: failed to get pending event %d\n", ret);
> +			xe_log_err(xe, SYSCTRL, ret, "Failed to get pending event\n");

[Severity: Low]
Are xe_log_err and SYSCTRL defined in the missing xe_log.h header? Without
those definitions, the get_pending_event() function fails to compile due to
undefined identifiers.

>  			return;
>  		}

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260831104456.1770605-11-mallesh.koujalagi@intel.com?part=9

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging
  2026-08-31 10:45 ` [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging Mallesh Koujalagi
@ 2026-08-31 11:00   ` sashiko-bot
  2026-09-02 16:22   ` Michal Wajdeczko
  1 sibling, 0 replies; 24+ messages in thread
From: sashiko-bot @ 2026-08-31 11:00 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The patch introduces calls to an undefined function/macro `xe_log_err()` and relies on an undeclared identifier `SYSCTRL`. Furthermore, it depends on a header file (`xe_log.h`) that is missing from the repository tree, resulting in an immediate build failure.
--

commit d3ef42ba91500c22f5afaa468dbb619c680583f3
Author: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

drm/xe/sysctrl: Improve firmware response error logging

This commit replaces xe_err() with xe_log_err() to include additional details 
when a response header mismatch occurs.

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 3aa868f60b386..87e6568202795 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -233,13 +233,15 @@ static int sysctrl_receive_frames(struct xe_sysctrl *sc,
>  	if (!XE_SYSCTRL_HDR_IS_RESPONSE(hdr) ||
>  	    XE_SYSCTRL_HDR_GROUP_ID(hdr) != XE_SYSCTRL_HDR_GROUP_ID(req) ||
>  	    XE_SYSCTRL_HDR_COMMAND(hdr) != XE_SYSCTRL_HDR_COMMAND(req)) {
> -		xe_err(xe, "sysctrl: Response header mismatch\n");
> +		xe_log_err(xe, SYSCTRL, -EPROTO,
> +			   "Response header mismatch: %#x != %#x\n",
> +			   le32_to_cpu(hdr->data), le32_to_cpu(req->data));

[Severity: Low]
Does this code cause a build failure? It appears that xe_log_err() and SYSCTRL
are not defined, and the requisite header (such as xe_log.h) is missing from
the source tree.

>  		return -EPROTO;
>  	}
>  
>  	if (XE_SYSCTRL_HDR_RESULT(hdr) != 0) {
> -		xe_err(xe, "sysctrl: Firmware error: 0x%02lx\n",
> -		       XE_SYSCTRL_HDR_RESULT(hdr));
> +		xe_log_err(xe, SYSCTRL, -EIO, "Firmware error: %#04x\n",
> +			   XE_SYSCTRL_HDR_RESULT(hdr));

[Severity: Low]
Is this another location where the undefined identifiers will break the build?

>  		return -EIO;
>  	}
>

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260831104456.1770605-11-mallesh.koujalagi@intel.com?part=7

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 6/9] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET()
  2026-08-31 10:45 ` [PATCH v6 6/9] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET() Mallesh Koujalagi
@ 2026-08-31 11:07   ` sashiko-bot
  0 siblings, 0 replies; 24+ messages in thread
From: sashiko-bot @ 2026-08-31 11:07 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

Thank you for your contribution! Sashiko AI review found 3 potential issue(s) to consider:

New issues:
- [Low] Incomplete migration to REG_FIELD_PREP() leaves packing logic inconsistent.

Pre-existing issues:
- [High] Hardware mailbox state machine lockup on oversized firmware response.
- [High] Endianness corruption on Big Endian architectures due to memcpy followed by writel.
--

commit 44f13e369dbf9b94796ecb4d99fe19420e2a9d18
Author: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET()

Use REG_FIELD_GET() when extracting fields from SysCtrl and
application headers. This better reflects that the masks
represent register-style bitfields and improves consistency.

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 50bbdab072de6..3aa868f60b386 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -24,16 +24,16 @@ struct xe_sysctrl_mailbox_msg_hdr {
>  } __packed;
>  
>  #define XE_SYSCTRL_HDR_GROUP_ID(hdr) \
> -	FIELD_GET(SYSCTRL_HDR_GROUP_ID_MASK, le32_to_cpu((hdr)->data))
> +	REG_FIELD_GET(SYSCTRL_HDR_GROUP_ID_MASK, le32_to_cpu((hdr)->data))

[Severity: Low]
Does this migration leave the packing logic inconsistent?

In sysctrl_prepare_command(), the code continues to use FIELD_PREP() despite
the getters being migrated to REG_FIELD_GET():

	hdr->data = cpu_to_le32(FIELD_PREP(SYSCTRL_HDR_GROUP_ID_MASK, group_id) |
				     FIELD_PREP(SYSCTRL_HDR_COMMAND_MASK, command));

Should this be updated to REG_FIELD_PREP() for a complete migration?

[ ... ]

> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h
> index f0d5e3d7f5e31..5a3dd14e3a110 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.h
> @@ -15,13 +15,13 @@ struct xe_sysctrl;
>  struct xe_sysctrl_mailbox_command;
>  
>  #define XE_SYSCTRL_APP_HDR_GROUP_ID(hdr) \
> -	FIELD_GET(APP_HDR_GROUP_ID_MASK, (hdr)->data)
> +	REG_FIELD_GET(APP_HDR_GROUP_ID_MASK, (hdr)->data)

[Severity: Low]
Does this incomplete migration leave xe_sysctrl_create_command() inconsistent?

While the getters in the header are updated to REG_FIELD_GET(), the packing
logic in xe_sysctrl_create_command() continues to use FIELD_PREP() for these
same masks:

	header.data = FIELD_PREP(APP_HDR_GROUP_ID_MASK, group_id) |
		      FIELD_PREP(APP_HDR_COMMAND_MASK, cmd_id);

Should this be updated to REG_FIELD_PREP() as well?

[ ... ]

[Severity: High]
This is a pre-existing issue, but does sysctrl_receive_frames() leave the
hardware mailbox state machine wedged if the firmware response is oversized?

Looking at sysctrl_receive_frames() in drivers/gpu/drm/xe/xe_sysctrl_mailbox.c:

	while (!done && remain) {
		frame_size = min_t(size_t, remain, XE_SYSCTRL_MB_FRAME_SIZE);

		ret = sysctrl_process_frame(sc, out, frame_size, timeout_ms,
					    &done);
		if (ret)
			return ret;

		remain -= frame_size;
		out += frame_size;
	}

	hdr = (struct xe_sysctrl_mailbox_msg_hdr *)buffer;

	if (!XE_SYSCTRL_HDR_IS_RESPONSE(hdr) ||
	    XE_SYSCTRL_HDR_GROUP_ID(hdr) != XE_SYSCTRL_HDR_GROUP_ID(req) ||
	    XE_SYSCTRL_HDR_COMMAND(hdr) != XE_SYSCTRL_HDR_COMMAND(req)) {

If the firmware sends a response larger than expected, the loop exits when
remain reaches 0 even though the firmware indicates more frames are pending
(done == false).

The function then evaluates the header and returns without draining the
remaining frames from the hardware.

Does this leave the firmware stuck waiting for the host to clear
SYSCTRL_MB_CTRL_RUN_BUSY_OUT, preventing any subsequent mailbox communication?

[Severity: High]
This is a pre-existing issue, but can this cause endianness corruption on
Big Endian architectures?

In sysctrl_write_frame() in drivers/gpu/drm/xe/xe_sysctrl_mailbox.c:

	memcpy(val, frame, len);

	for (i = 0; i < dw; i++)
		xe_mmio_write32(sc->mmio, regs[i], val[i]);

A little-endian byte array is copied into a u32 array with memcpy() and then
written with an endian-swapping MMIO accessor.

On Big Endian platforms, the u32 value constructed via memcpy evaluates with
its bytes logically reversed.

When this is passed to xe_mmio_write32() (which performs a cpu_to_le32()
conversion), the value is byte-swapped again, writing the bytes in reversed
order.

The same issue exists in reverse in sysctrl_read_frame():

	for (i = 0; i < dw; i++)
		val[i] = xe_mmio_read32(sc->mmio, regs[i]);

	memcpy(frame, val, len);

Does this endian-swapped MMIO read being copied directly into a byte buffer
via memcpy corrupt the response byte order on Big Endian?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260831104456.1770605-11-mallesh.koujalagi@intel.com?part=6

^ permalink raw reply	[flat|nested] 24+ messages in thread

* ✓ CI.KUnit: success for drm/xe/sysctrl: Clean up error handling in (rev3)
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (8 preceding siblings ...)
  2026-08-31 10:45 ` [PATCH v6 9/9] drm/xe/sysctrl: Report 'System Controller event' error using SIGID Mallesh Koujalagi
@ 2026-08-31 14:37 ` Patchwork
  2026-08-31 15:26 ` ✓ Xe.CI.BAT: " Patchwork
  2026-08-31 17:52 ` ✗ Xe.CI.FULL: failure " Patchwork
  11 siblings, 0 replies; 24+ messages in thread
From: Patchwork @ 2026-08-31 14:37 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

== Series Details ==

Series: drm/xe/sysctrl: Clean up error handling in (rev3)
URL   : https://patchwork.freedesktop.org/series/172513/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[14:36:02] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[14:36: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
[14:36:39] Starting KUnit Kernel (1/1)...
[14:36:39] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[14:36:39] ============= refcount_interrupt (4 subtests) ==============
[14:36:39] [PASSED] test_single_irq_change
[14:36:39] [PASSED] test_nested_irq_change
[14:36:39] [PASSED] test_multiple_irq_change
[14:36:39] [PASSED] test_irq_save
[14:36:39] =============== [PASSED] refcount_interrupt ================
[14:36:39] ================== guc_buf (11 subtests) ===================
[14:36:39] [PASSED] test_smallest
[14:36:39] [PASSED] test_largest
[14:36:39] [PASSED] test_granular
[14:36:39] [PASSED] test_unique
[14:36:39] [PASSED] test_overlap
[14:36:39] [PASSED] test_reusable
[14:36:39] [PASSED] test_too_big
[14:36:39] [PASSED] test_flush
[14:36:39] [PASSED] test_lookup
[14:36:39] [PASSED] test_data
[14:36:39] [PASSED] test_class
[14:36:39] ===================== [PASSED] guc_buf =====================
[14:36:39] =================== guc_dbm (7 subtests) ===================
[14:36:39] [PASSED] test_empty
[14:36:39] [PASSED] test_default
[14:36:39] ======================== test_size  ========================
[14:36:39] [PASSED] 4
[14:36:39] [PASSED] 8
[14:36:39] [PASSED] 32
[14:36:39] [PASSED] 256
[14:36:39] ==================== [PASSED] test_size ====================
[14:36:39] ======================= test_reuse  ========================
[14:36:39] [PASSED] 4
[14:36:39] [PASSED] 8
[14:36:39] [PASSED] 32
[14:36:39] [PASSED] 256
[14:36:39] =================== [PASSED] test_reuse ====================
[14:36:39] =================== test_range_overlap  ====================
[14:36:39] [PASSED] 4
[14:36:39] [PASSED] 8
[14:36:39] [PASSED] 32
[14:36:39] [PASSED] 256
[14:36:39] =============== [PASSED] test_range_overlap ================
[14:36:39] =================== test_range_compact  ====================
[14:36:39] [PASSED] 4
[14:36:39] [PASSED] 8
[14:36:39] [PASSED] 32
[14:36:39] [PASSED] 256
[14:36:39] =============== [PASSED] test_range_compact ================
[14:36:39] ==================== test_range_spare  =====================
[14:36:39] [PASSED] 4
[14:36:39] [PASSED] 8
[14:36:39] [PASSED] 32
[14:36:39] [PASSED] 256
[14:36:39] ================ [PASSED] test_range_spare =================
[14:36:39] ===================== [PASSED] guc_dbm =====================
[14:36:39] =================== guc_idm (6 subtests) ===================
[14:36:39] [PASSED] bad_init
[14:36:39] [PASSED] no_init
[14:36:39] [PASSED] init_fini
[14:36:39] [PASSED] check_used
[14:36:39] [PASSED] check_quota
[14:36:39] [PASSED] check_all
[14:36:39] ===================== [PASSED] guc_idm =====================
[14:36:39] =============== guc_klv_helpers (9 subtests) ===============
[14:36:39] [PASSED] test_count
[14:36:39] [PASSED] test_encode_u32
[14:36:39] [PASSED] test_encode_u64
[14:36:39] [PASSED] test_encode_string
[14:36:39] [PASSED] test_encode_object_raw
[14:36:39] [PASSED] test_encode_object_klv
[14:36:39] [PASSED] test_encode_object_nested
[14:36:39] [PASSED] test_encode_object_basic
[14:36:39] [PASSED] test_print
[14:36:39] ================= [PASSED] guc_klv_helpers =================
[14:36:39] =================== xe_log (4 subtests) ====================
[14:36:39] [PASSED] demo_cper
[14:36:39] [PASSED] demo_dmesg
[14:36:39] ======================= test_dmesg  ========================
[14:36:39] [PASSED] test_fatal
[14:36:39] [PASSED] test_fatal_tile
[14:36:39] [PASSED] test_fatal_gt
[14:36:39] [PASSED] test_fatal_comp
[14:36:39] [PASSED] test_fatal_comp_tile
[14:36:39] [PASSED] test_fatal_comp_gt
[14:36:39] [PASSED] test_fatal_all
[14:36:39] [PASSED] test_recoverable
[14:36:39] [PASSED] test_recoverable_tile
[14:36:39] [PASSED] test_recoverable_gt
[14:36:39] [PASSED] test_recoverable_comp
[14:36:39] [PASSED] test_recoverable_comp_tile
[14:36:39] [PASSED] test_recoverable_comp_gt
[14:36:39] [PASSED] test_recoverable_all
[14:36:39] [PASSED] test_info
[14:36:39] [PASSED] test_info_tile
[14:36:39] [PASSED] test_info_gt
[14:36:39] [PASSED] test_info_err
[14:36:39] [PASSED] test_info_comp
[14:36:39] [PASSED] test_info_comp_tile
[14:36:39] [PASSED] test_info_comp_gt
[14:36:39] [PASSED] test_info_all
[14:36:39] [PASSED] test_hw_fatal
[14:36:39] [PASSED] test_hw_recoverable
[14:36:39] [PASSED] test_hw_corrected
[14:36:39] [PASSED] test_hw_informational
[14:36:39] =================== [PASSED] test_dmesg ====================
[14:36:39] ====================== test_invalid  =======================
[14:36:39] [SKIPPED] no-component no-location no-warn (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] reserved location (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] unknown location (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] nonzero-device-id location (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] invalid-tile-id location (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] invalid-gt-id location (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] unknown component class (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] unknown system component (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] unknown hardware component (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] [SKIPPED] unknown component and location (requires CONFIG_DRM_XE_DEBUG)
[14:36:39] ================== [SKIPPED] test_invalid ==================
[14:36:39] ===================== [PASSED] xe_log ======================
[14:36:39] ================== no_relay (3 subtests) ===================
[14:36:39] [PASSED] xe_drops_guc2pf_if_not_ready
[14:36:39] [PASSED] xe_drops_guc2vf_if_not_ready
[14:36:39] [PASSED] xe_rejects_send_if_not_ready
[14:36:39] ==================== [PASSED] no_relay =====================
[14:36:39] ================== pf_relay (14 subtests) ==================
[14:36:39] [PASSED] pf_rejects_guc2pf_too_short
[14:36:39] [PASSED] pf_rejects_guc2pf_too_long
[14:36:39] [PASSED] pf_rejects_guc2pf_no_payload
[14:36:39] [PASSED] pf_fails_no_payload
[14:36:39] [PASSED] pf_fails_bad_origin
[14:36:39] [PASSED] pf_fails_bad_type
[14:36:39] [PASSED] pf_txn_reports_error
[14:36:39] [PASSED] pf_txn_sends_pf2guc
[14:36:39] [PASSED] pf_sends_pf2guc
[14:36:39] [SKIPPED] pf_loopback_nop (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[14:36:39] [SKIPPED] pf_loopback_echo (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[14:36:39] [SKIPPED] pf_loopback_fail (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[14:36:39] [SKIPPED] pf_loopback_busy (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[14:36:39] [SKIPPED] pf_loopback_retry (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[14:36:39] ==================== [PASSED] pf_relay =====================
[14:36:39] ================== vf_relay (3 subtests) ===================
[14:36:39] [PASSED] vf_rejects_guc2vf_too_short
[14:36:39] [PASSED] vf_rejects_guc2vf_too_long
[14:36:39] [PASSED] vf_rejects_guc2vf_no_payload
[14:36:39] ==================== [PASSED] vf_relay =====================
[14:36:39] ================ pf_gt_config (9 subtests) =================
[14:36:39] [PASSED] fair_contexts_1vf
[14:36:39] [PASSED] fair_doorbells_1vf
[14:36:39] [PASSED] fair_ggtt_1vf
[14:36:39] ====================== fair_vram_1vf  ======================
[14:36:39] [PASSED] 3.50 GiB
[14:36:39] [PASSED] 11.5 GiB
[14:36:39] [PASSED] 15.5 GiB
[14:36:39] [PASSED] 31.5 GiB
[14:36:39] [PASSED] 63.5 GiB
[14:36:39] [PASSED] 1.91 GiB
[14:36:39] ================== [PASSED] fair_vram_1vf ==================
[14:36:39] ================ fair_vram_1vf_admin_only  =================
[14:36:39] [PASSED] 3.50 GiB
[14:36:39] [PASSED] 11.5 GiB
[14:36:39] [PASSED] 15.5 GiB
[14:36:39] [PASSED] 31.5 GiB
[14:36:39] [PASSED] 63.5 GiB
[14:36:39] [PASSED] 1.91 GiB
[14:36:39] ============ [PASSED] fair_vram_1vf_admin_only =============
[14:36:39] ====================== fair_contexts  ======================
[14:36:39] [PASSED] 1 VF
[14:36:39] [PASSED] 2 VFs
[14:36:39] [PASSED] 3 VFs
[14:36:39] [PASSED] 4 VFs
[14:36:39] [PASSED] 5 VFs
[14:36:39] [PASSED] 6 VFs
[14:36:39] [PASSED] 7 VFs
[14:36:39] [PASSED] 8 VFs
[14:36:39] [PASSED] 9 VFs
[14:36:39] [PASSED] 10 VFs
[14:36:39] [PASSED] 11 VFs
[14:36:39] [PASSED] 12 VFs
[14:36:39] [PASSED] 13 VFs
[14:36:39] [PASSED] 14 VFs
[14:36:39] [PASSED] 15 VFs
[14:36:39] [PASSED] 16 VFs
[14:36:39] [PASSED] 17 VFs
[14:36:39] [PASSED] 18 VFs
[14:36:39] [PASSED] 19 VFs
[14:36:39] [PASSED] 20 VFs
[14:36:39] [PASSED] 21 VFs
[14:36:39] [PASSED] 22 VFs
[14:36:39] [PASSED] 23 VFs
[14:36:39] [PASSED] 24 VFs
[14:36:39] [PASSED] 25 VFs
[14:36:39] [PASSED] 26 VFs
[14:36:39] [PASSED] 27 VFs
[14:36:39] [PASSED] 28 VFs
[14:36:39] [PASSED] 29 VFs
[14:36:39] [PASSED] 30 VFs
[14:36:39] [PASSED] 31 VFs
[14:36:39] [PASSED] 32 VFs
[14:36:39] [PASSED] 33 VFs
[14:36:39] [PASSED] 34 VFs
[14:36:39] [PASSED] 35 VFs
[14:36:39] [PASSED] 36 VFs
[14:36:39] [PASSED] 37 VFs
[14:36:39] [PASSED] 38 VFs
[14:36:39] [PASSED] 39 VFs
[14:36:39] [PASSED] 40 VFs
[14:36:39] [PASSED] 41 VFs
[14:36:39] [PASSED] 42 VFs
[14:36:39] [PASSED] 43 VFs
[14:36:39] [PASSED] 44 VFs
[14:36:39] [PASSED] 45 VFs
[14:36:39] [PASSED] 46 VFs
[14:36:39] [PASSED] 47 VFs
[14:36:39] [PASSED] 48 VFs
[14:36:39] [PASSED] 49 VFs
[14:36:39] [PASSED] 50 VFs
[14:36:39] [PASSED] 51 VFs
[14:36:39] [PASSED] 52 VFs
[14:36:39] [PASSED] 53 VFs
[14:36:39] [PASSED] 54 VFs
[14:36:39] [PASSED] 55 VFs
[14:36:39] [PASSED] 56 VFs
[14:36:39] [PASSED] 57 VFs
[14:36:39] [PASSED] 58 VFs
[14:36:39] [PASSED] 59 VFs
[14:36:39] [PASSED] 60 VFs
[14:36:39] [PASSED] 61 VFs
[14:36:39] [PASSED] 62 VFs
[14:36:39] [PASSED] 63 VFs
[14:36:39] ================== [PASSED] fair_contexts ==================
[14:36:39] ===================== fair_doorbells  ======================
[14:36:39] [PASSED] 1 VF
[14:36:39] [PASSED] 2 VFs
[14:36:39] [PASSED] 3 VFs
[14:36:39] [PASSED] 4 VFs
[14:36:39] [PASSED] 5 VFs
[14:36:39] [PASSED] 6 VFs
[14:36:39] [PASSED] 7 VFs
[14:36:39] [PASSED] 8 VFs
[14:36:39] [PASSED] 9 VFs
[14:36:39] [PASSED] 10 VFs
[14:36:39] [PASSED] 11 VFs
[14:36:39] [PASSED] 12 VFs
[14:36:39] [PASSED] 13 VFs
[14:36:39] [PASSED] 14 VFs
[14:36:39] [PASSED] 15 VFs
[14:36:39] [PASSED] 16 VFs
[14:36:39] [PASSED] 17 VFs
[14:36:39] [PASSED] 18 VFs
[14:36:39] [PASSED] 19 VFs
[14:36:39] [PASSED] 20 VFs
[14:36:39] [PASSED] 21 VFs
[14:36:39] [PASSED] 22 VFs
[14:36:39] [PASSED] 23 VFs
[14:36:39] [PASSED] 24 VFs
[14:36:39] [PASSED] 25 VFs
[14:36:39] [PASSED] 26 VFs
[14:36:39] [PASSED] 27 VFs
[14:36:39] [PASSED] 28 VFs
[14:36:39] [PASSED] 29 VFs
[14:36:39] [PASSED] 30 VFs
[14:36:39] [PASSED] 31 VFs
[14:36:39] [PASSED] 32 VFs
[14:36:39] [PASSED] 33 VFs
[14:36:39] [PASSED] 34 VFs
[14:36:39] [PASSED] 35 VFs
[14:36:39] [PASSED] 36 VFs
[14:36:39] [PASSED] 37 VFs
[14:36:39] [PASSED] 38 VFs
[14:36:39] [PASSED] 39 VFs
[14:36:39] [PASSED] 40 VFs
[14:36:39] [PASSED] 41 VFs
[14:36:39] [PASSED] 42 VFs
[14:36:39] [PASSED] 43 VFs
[14:36:39] [PASSED] 44 VFs
[14:36:39] [PASSED] 45 VFs
[14:36:39] [PASSED] 46 VFs
[14:36:39] [PASSED] 47 VFs
[14:36:39] [PASSED] 48 VFs
[14:36:39] [PASSED] 49 VFs
[14:36:39] [PASSED] 50 VFs
[14:36:39] [PASSED] 51 VFs
[14:36:39] [PASSED] 52 VFs
[14:36:39] [PASSED] 53 VFs
[14:36:39] [PASSED] 54 VFs
[14:36:39] [PASSED] 55 VFs
[14:36:39] [PASSED] 56 VFs
[14:36:39] [PASSED] 57 VFs
[14:36:39] [PASSED] 58 VFs
[14:36:39] [PASSED] 59 VFs
[14:36:39] [PASSED] 60 VFs
[14:36:39] [PASSED] 61 VFs
[14:36:39] [PASSED] 62 VFs
[14:36:39] [PASSED] 63 VFs
[14:36:39] ================= [PASSED] fair_doorbells ==================
[14:36:39] ======================== fair_ggtt  ========================
[14:36:39] [PASSED] 1 VF
[14:36:39] [PASSED] 2 VFs
[14:36:39] [PASSED] 3 VFs
[14:36:39] [PASSED] 4 VFs
[14:36:39] [PASSED] 5 VFs
[14:36:39] [PASSED] 6 VFs
[14:36:39] [PASSED] 7 VFs
[14:36:39] [PASSED] 8 VFs
[14:36:39] [PASSED] 9 VFs
[14:36:39] [PASSED] 10 VFs
[14:36:39] [PASSED] 11 VFs
[14:36:39] [PASSED] 12 VFs
[14:36:39] [PASSED] 13 VFs
[14:36:39] [PASSED] 14 VFs
[14:36:39] [PASSED] 15 VFs
[14:36:39] [PASSED] 16 VFs
[14:36:39] [PASSED] 17 VFs
[14:36:39] [PASSED] 18 VFs
[14:36:39] [PASSED] 19 VFs
[14:36:39] [PASSED] 20 VFs
[14:36:39] [PASSED] 21 VFs
[14:36:39] [PASSED] 22 VFs
[14:36:39] [PASSED] 23 VFs
[14:36:39] [PASSED] 24 VFs
[14:36:39] [PASSED] 25 VFs
[14:36:39] [PASSED] 26 VFs
[14:36:39] [PASSED] 27 VFs
[14:36:39] [PASSED] 28 VFs
[14:36:39] [PASSED] 29 VFs
[14:36:39] [PASSED] 30 VFs
[14:36:39] [PASSED] 31 VFs
[14:36:39] [PASSED] 32 VFs
[14:36:39] [PASSED] 33 VFs
[14:36:39] [PASSED] 34 VFs
[14:36:39] [PASSED] 35 VFs
[14:36:39] [PASSED] 36 VFs
[14:36:39] [PASSED] 37 VFs
[14:36:39] [PASSED] 38 VFs
[14:36:39] [PASSED] 39 VFs
[14:36:39] [PASSED] 40 VFs
[14:36:39] [PASSED] 41 VFs
[14:36:39] [PASSED] 42 VFs
[14:36:39] [PASSED] 43 VFs
[14:36:39] [PASSED] 44 VFs
[14:36:39] [PASSED] 45 VFs
[14:36:39] [PASSED] 46 VFs
[14:36:39] [PASSED] 47 VFs
[14:36:39] [PASSED] 48 VFs
[14:36:39] [PASSED] 49 VFs
[14:36:39] [PASSED] 50 VFs
[14:36:39] [PASSED] 51 VFs
[14:36:39] [PASSED] 52 VFs
[14:36:39] [PASSED] 53 VFs
[14:36:39] [PASSED] 54 VFs
[14:36:39] [PASSED] 55 VFs
[14:36:39] [PASSED] 56 VFs
[14:36:39] [PASSED] 57 VFs
[14:36:39] [PASSED] 58 VFs
[14:36:39] [PASSED] 59 VFs
[14:36:39] [PASSED] 60 VFs
[14:36:39] [PASSED] 61 VFs
[14:36:39] [PASSED] 62 VFs
[14:36:39] [PASSED] 63 VFs
[14:36:39] ==================== [PASSED] fair_ggtt ====================
[14:36:39] ======================== fair_vram  ========================
[14:36:39] [PASSED] 1 VF
[14:36:39] [PASSED] 2 VFs
[14:36:39] [PASSED] 3 VFs
[14:36:39] [PASSED] 4 VFs
[14:36:39] [PASSED] 5 VFs
[14:36:39] [PASSED] 6 VFs
[14:36:39] [PASSED] 7 VFs
[14:36:39] [PASSED] 8 VFs
[14:36:39] [PASSED] 9 VFs
[14:36:39] [PASSED] 10 VFs
[14:36:39] [PASSED] 11 VFs
[14:36:39] [PASSED] 12 VFs
[14:36:39] [PASSED] 13 VFs
[14:36:39] [PASSED] 14 VFs
[14:36:39] [PASSED] 15 VFs
[14:36:39] [PASSED] 16 VFs
[14:36:39] [PASSED] 17 VFs
[14:36:39] [PASSED] 18 VFs
[14:36:39] [PASSED] 19 VFs
[14:36:39] [PASSED] 20 VFs
[14:36:39] [PASSED] 21 VFs
[14:36:39] [PASSED] 22 VFs
[14:36:39] [PASSED] 23 VFs
[14:36:39] [PASSED] 24 VFs
[14:36:39] [PASSED] 25 VFs
[14:36:39] [PASSED] 26 VFs
[14:36:39] [PASSED] 27 VFs
[14:36:39] [PASSED] 28 VFs
[14:36:39] [PASSED] 29 VFs
[14:36:39] [PASSED] 30 VFs
[14:36:39] [PASSED] 31 VFs
[14:36:39] [PASSED] 32 VFs
[14:36:39] [PASSED] 33 VFs
[14:36:39] [PASSED] 34 VFs
[14:36:39] [PASSED] 35 VFs
[14:36:39] [PASSED] 36 VFs
[14:36:39] [PASSED] 37 VFs
[14:36:39] [PASSED] 38 VFs
[14:36:39] [PASSED] 39 VFs
[14:36:39] [PASSED] 40 VFs
[14:36:39] [PASSED] 41 VFs
[14:36:39] [PASSED] 42 VFs
[14:36:39] [PASSED] 43 VFs
[14:36:39] [PASSED] 44 VFs
[14:36:39] [PASSED] 45 VFs
[14:36:39] [PASSED] 46 VFs
[14:36:39] [PASSED] 47 VFs
[14:36:39] [PASSED] 48 VFs
[14:36:39] [PASSED] 49 VFs
[14:36:39] [PASSED] 50 VFs
[14:36:39] [PASSED] 51 VFs
[14:36:39] [PASSED] 52 VFs
[14:36:39] [PASSED] 53 VFs
[14:36:39] [PASSED] 54 VFs
[14:36:39] [PASSED] 55 VFs
[14:36:39] [PASSED] 56 VFs
[14:36:39] [PASSED] 57 VFs
[14:36:39] [PASSED] 58 VFs
[14:36:39] [PASSED] 59 VFs
[14:36:39] [PASSED] 60 VFs
[14:36:39] [PASSED] 61 VFs
[14:36:39] [PASSED] 62 VFs
[14:36:39] [PASSED] 63 VFs
[14:36:39] ==================== [PASSED] fair_vram ====================
[14:36:39] ================== [PASSED] pf_gt_config ===================
[14:36:39] ===================== lmtt (1 subtest) =====================
[14:36:39] ======================== test_ops  =========================
[14:36:39] [PASSED] 2-level
[14:36:39] [PASSED] multi-level
[14:36:39] ==================== [PASSED] test_ops =====================
[14:36:39] ====================== [PASSED] lmtt =======================
[14:36:39] ================= sriov_packet (1 subtest) =================
[14:36:39] [PASSED] test_descriptor_init
[14:36:39] ================== [PASSED] sriov_packet ===================
[14:36:39] ================= pf_service (11 subtests) =================
[14:36:39] [PASSED] pf_negotiate_any
[14:36:39] [PASSED] pf_negotiate_base_match
[14:36:39] [PASSED] pf_negotiate_base_newer
[14:36:39] [PASSED] pf_negotiate_base_next
[14:36:39] [SKIPPED] pf_negotiate_base_older (no older minor)
[14:36:39] [PASSED] pf_negotiate_base_prev
[14:36:39] [PASSED] pf_negotiate_latest_match
[14:36:39] [PASSED] pf_negotiate_latest_newer
[14:36:39] [PASSED] pf_negotiate_latest_next
[14:36:39] [SKIPPED] pf_negotiate_latest_older (no older minor)
[14:36:39] [SKIPPED] pf_negotiate_latest_prev (no prev major)
[14:36:39] =================== [PASSED] pf_service ====================
[14:36:39] ================= xe_guc_g2g (2 subtests) ==================
[14:36:39] ============== xe_live_guc_g2g_kunit_default  ==============
[14:36:39] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[14:36:39] ============== xe_live_guc_g2g_kunit_allmem  ===============
[14:36:39] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[14:36:39] =================== [SKIPPED] xe_guc_g2g ===================
[14:36:39] =================== xe_mocs (2 subtests) ===================
[14:36:39] ================ xe_live_mocs_kernel_kunit  ================
[14:36:39] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[14:36:39] ================ xe_live_mocs_reset_kunit  =================
[14:36:39] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[14:36:39] ==================== [SKIPPED] xe_mocs =====================
[14:36:39] ================= xe_migrate (2 subtests) ==================
[14:36:39] ================= xe_migrate_sanity_kunit  =================
[14:36:39] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[14:36:39] ================== xe_validate_ccs_kunit  ==================
[14:36:39] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[14:36:39] =================== [SKIPPED] xe_migrate ===================
[14:36:39] ================== xe_dma_buf (1 subtest) ==================
[14:36:39] ==================== xe_dma_buf_kunit  =====================
[14:36:39] ================ [SKIPPED] xe_dma_buf_kunit ================
[14:36:39] =================== [SKIPPED] xe_dma_buf ===================
[14:36:39] ================= xe_bo_shrink (1 subtest) =================
[14:36:39] =================== xe_bo_shrink_kunit  ====================
[14:36:39] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[14:36:39] ================== [SKIPPED] xe_bo_shrink ==================
[14:36:39] ==================== xe_bo (2 subtests) ====================
[14:36:39] ================== xe_ccs_migrate_kunit  ===================
[14:36:39] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[14:36:39] ==================== xe_bo_evict_kunit  ====================
[14:36:39] =============== [SKIPPED] xe_bo_evict_kunit ================
[14:36:39] ===================== [SKIPPED] xe_bo ======================
[14:36:39] =================== xe_any (9 subtests) ====================
[14:36:39] [PASSED] test_to_xe
[14:36:39] [PASSED] test_to_dev
[14:36:39] [PASSED] test_to_pdev
[14:36:39] [PASSED] test_to_drm
[14:36:39] [PASSED] test_if_pdev
[14:36:39] [PASSED] test_if_xe
[14:36:39] [PASSED] test_if_tile
[14:36:39] [PASSED] test_if_gt
[14:36:39] [PASSED] test_to_id
[14:36:39] ===================== [PASSED] xe_any ======================
[14:36:39] ==================== args (13 subtests) ====================
[14:36:39] [PASSED] count_args_test
[14:36:39] [PASSED] call_args_example
[14:36:39] [PASSED] call_args_test
[14:36:39] [PASSED] drop_first_arg_example
[14:36:39] [PASSED] drop_first_arg_test
[14:36:39] [PASSED] first_arg_example
[14:36:39] [PASSED] first_arg_test
[14:36:39] [PASSED] last_arg_example
[14:36:39] [PASSED] last_arg_test
[14:36:39] [PASSED] pick_arg_example
[14:36:39] [PASSED] if_args_example
[14:36:39] [PASSED] if_args_test
[14:36:39] [PASSED] sep_comma_example
[14:36:39] ====================== [PASSED] args =======================
[14:36:39] =================== xe_pci (3 subtests) ====================
[14:36:39] ==================== check_graphics_ip  ====================
[14:36:39] [PASSED] 12.00 Xe_LP
[14:36:39] [PASSED] 12.10 Xe_LP+
[14:36:39] [PASSED] 12.55 Xe_HPG
[14:36:39] [PASSED] 12.60 Xe_HPC
[14:36:39] [PASSED] 12.70 Xe_LPG
[14:36:39] [PASSED] 12.71 Xe_LPG
[14:36:39] [PASSED] 12.74 Xe_LPG+
[14:36:39] [PASSED] 20.01 Xe2_HPG
[14:36:39] [PASSED] 20.02 Xe2_HPG
[14:36:39] [PASSED] 20.04 Xe2_LPG
[14:36:39] [PASSED] 30.00 Xe3_LPG
[14:36:39] [PASSED] 30.01 Xe3_LPG
[14:36:39] [PASSED] 30.03 Xe3_LPG
[14:36:39] [PASSED] 30.04 Xe3_LPG
[14:36:39] [PASSED] 30.05 Xe3_LPG
[14:36:39] [PASSED] 35.10 Xe3p_LPG
[14:36:39] [PASSED] 35.11 Xe3p_XPC
[14:36:39] ================ [PASSED] check_graphics_ip ================
[14:36:39] ===================== check_media_ip  ======================
[14:36:39] [PASSED] 12.00 Xe_M
[14:36:39] [PASSED] 12.55 Xe_HPM
[14:36:39] [PASSED] 13.00 Xe_LPM+
[14:36:39] [PASSED] 13.01 Xe2_HPM
[14:36:39] [PASSED] 20.00 Xe2_LPM
[14:36:39] [PASSED] 30.00 Xe3_LPM
[14:36:39] [PASSED] 30.02 Xe3_LPM
[14:36:39] [PASSED] 35.00 Xe3p_LPM
[14:36:39] [PASSED] 35.03 Xe3p_HPM
[14:36:39] ================= [PASSED] check_media_ip ==================
[14:36:39] =================== check_platform_desc  ===================
[14:36:39] [PASSED] 0x9A60 (TIGERLAKE)
[14:36:39] [PASSED] 0x9A68 (TIGERLAKE)
[14:36:39] [PASSED] 0x9A70 (TIGERLAKE)
[14:36:39] [PASSED] 0x9A40 (TIGERLAKE)
[14:36:39] [PASSED] 0x9A49 (TIGERLAKE)
[14:36:39] [PASSED] 0x9A59 (TIGERLAKE)
[14:36:39] [PASSED] 0x9A78 (TIGERLAKE)
[14:36:39] [PASSED] 0x9AC0 (TIGERLAKE)
[14:36:39] [PASSED] 0x9AC9 (TIGERLAKE)
[14:36:39] [PASSED] 0x9AD9 (TIGERLAKE)
[14:36:39] [PASSED] 0x9AF8 (TIGERLAKE)
[14:36:39] [PASSED] 0x4C80 (ROCKETLAKE)
[14:36:39] [PASSED] 0x4C8A (ROCKETLAKE)
[14:36:39] [PASSED] 0x4C8B (ROCKETLAKE)
[14:36:39] [PASSED] 0x4C8C (ROCKETLAKE)
[14:36:39] [PASSED] 0x4C90 (ROCKETLAKE)
[14:36:39] [PASSED] 0x4C9A (ROCKETLAKE)
[14:36:39] [PASSED] 0x4680 (ALDERLAKE_S)
[14:36:39] [PASSED] 0x4682 (ALDERLAKE_S)
[14:36:39] [PASSED] 0x4688 (ALDERLAKE_S)
[14:36:39] [PASSED] 0x468A (ALDERLAKE_S)
[14:36:39] [PASSED] 0x468B (ALDERLAKE_S)
[14:36:39] [PASSED] 0x4690 (ALDERLAKE_S)
[14:36:39] [PASSED] 0x4692 (ALDERLAKE_S)
[14:36:39] [PASSED] 0x4693 (ALDERLAKE_S)
[14:36:39] [PASSED] 0x46A0 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46A1 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46A2 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46A3 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46A6 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46A8 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46AA (ALDERLAKE_P)
[14:36:39] [PASSED] 0x462A (ALDERLAKE_P)
[14:36:39] [PASSED] 0x4626 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x4628 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46B0 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46B1 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46B2 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46B3 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46C0 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46C1 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46C2 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46C3 (ALDERLAKE_P)
[14:36:39] [PASSED] 0x46D0 (ALDERLAKE_N)
[14:36:39] [PASSED] 0x46D1 (ALDERLAKE_N)
[14:36:39] [PASSED] 0x46D2 (ALDERLAKE_N)
[14:36:39] [PASSED] 0x46D3 (ALDERLAKE_N)
[14:36:39] [PASSED] 0x46D4 (ALDERLAKE_N)
[14:36:39] [PASSED] 0xA721 (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7A1 (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7A9 (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7AC (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7AD (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA720 (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7A0 (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7A8 (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7AA (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA7AB (ALDERLAKE_P)
[14:36:39] [PASSED] 0xA780 (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA781 (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA782 (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA783 (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA788 (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA789 (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA78A (ALDERLAKE_S)
[14:36:39] [PASSED] 0xA78B (ALDERLAKE_S)
[14:36:39] [PASSED] 0x4905 (DG1)
[14:36:39] [PASSED] 0x4906 (DG1)
[14:36:39] [PASSED] 0x4907 (DG1)
[14:36:39] [PASSED] 0x4908 (DG1)
[14:36:39] [PASSED] 0x4909 (DG1)
[14:36:39] [PASSED] 0x56C0 (DG2)
[14:36:39] [PASSED] 0x56C2 (DG2)
[14:36:39] [PASSED] 0x56C1 (DG2)
[14:36:39] [PASSED] 0x7D51 (METEORLAKE)
[14:36:39] [PASSED] 0x7DD1 (METEORLAKE)
[14:36:39] [PASSED] 0x7D41 (METEORLAKE)
[14:36:39] [PASSED] 0x7D67 (METEORLAKE)
[14:36:39] [PASSED] 0xB640 (METEORLAKE)
[14:36:39] [PASSED] 0x56A0 (DG2)
[14:36:39] [PASSED] 0x56A1 (DG2)
[14:36:39] [PASSED] 0x56A2 (DG2)
[14:36:39] [PASSED] 0x56BE (DG2)
[14:36:39] [PASSED] 0x56BF (DG2)
[14:36:39] [PASSED] 0x5690 (DG2)
[14:36:39] [PASSED] 0x5691 (DG2)
[14:36:39] [PASSED] 0x5692 (DG2)
[14:36:39] [PASSED] 0x56A5 (DG2)
[14:36:39] [PASSED] 0x56A6 (DG2)
[14:36:39] [PASSED] 0x56B0 (DG2)
[14:36:39] [PASSED] 0x56B1 (DG2)
[14:36:39] [PASSED] 0x56BA (DG2)
[14:36:39] [PASSED] 0x56BB (DG2)
[14:36:39] [PASSED] 0x56BC (DG2)
[14:36:39] [PASSED] 0x56BD (DG2)
[14:36:39] [PASSED] 0x5693 (DG2)
[14:36:39] [PASSED] 0x5694 (DG2)
[14:36:39] [PASSED] 0x5695 (DG2)
[14:36:39] [PASSED] 0x56A3 (DG2)
[14:36:39] [PASSED] 0x56A4 (DG2)
[14:36:39] [PASSED] 0x56B2 (DG2)
[14:36:39] [PASSED] 0x56B3 (DG2)
[14:36:39] [PASSED] 0x5696 (DG2)
[14:36:39] [PASSED] 0x5697 (DG2)
[14:36:39] [PASSED] 0xB69 (PVC)
[14:36:39] [PASSED] 0xB6E (PVC)
[14:36:39] [PASSED] 0xBD4 (PVC)
[14:36:39] [PASSED] 0xBD5 (PVC)
[14:36:39] [PASSED] 0xBD6 (PVC)
[14:36:39] [PASSED] 0xBD7 (PVC)
[14:36:39] [PASSED] 0xBD8 (PVC)
[14:36:39] [PASSED] 0xBD9 (PVC)
[14:36:39] [PASSED] 0xBDA (PVC)
[14:36:39] [PASSED] 0xBDB (PVC)
[14:36:39] [PASSED] 0xBE0 (PVC)
[14:36:39] [PASSED] 0xBE1 (PVC)
[14:36:39] [PASSED] 0xBE5 (PVC)
[14:36:39] [PASSED] 0x7D40 (METEORLAKE)
[14:36:39] [PASSED] 0x7D45 (METEORLAKE)
[14:36:39] [PASSED] 0x7D55 (METEORLAKE)
[14:36:39] [PASSED] 0x7D60 (METEORLAKE)
[14:36:39] [PASSED] 0x7DD5 (METEORLAKE)
[14:36:39] [PASSED] 0x6420 (LUNARLAKE)
[14:36:39] [PASSED] 0x64A0 (LUNARLAKE)
[14:36:39] [PASSED] 0x64B0 (LUNARLAKE)
[14:36:39] [PASSED] 0xE202 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE209 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE20B (BATTLEMAGE)
[14:36:39] [PASSED] 0xE20C (BATTLEMAGE)
[14:36:39] [PASSED] 0xE20D (BATTLEMAGE)
[14:36:39] [PASSED] 0xE210 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE211 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE212 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE216 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE220 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE221 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE222 (BATTLEMAGE)
[14:36:39] [PASSED] 0xE223 (BATTLEMAGE)
[14:36:39] [PASSED] 0xB080 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB081 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB082 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB083 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB084 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB085 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB086 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB087 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB08F (PANTHERLAKE)
[14:36:39] [PASSED] 0xB090 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB0A0 (PANTHERLAKE)
[14:36:39] [PASSED] 0xB0B0 (PANTHERLAKE)
[14:36:39] [PASSED] 0xFD80 (PANTHERLAKE)
[14:36:39] [PASSED] 0xFD81 (PANTHERLAKE)
[14:36:39] [PASSED] 0xD740 (NOVALAKE_S)
[14:36:39] [PASSED] 0xD741 (NOVALAKE_S)
[14:36:39] [PASSED] 0xD742 (NOVALAKE_S)
[14:36:39] [PASSED] 0xD743 (NOVALAKE_S)
[14:36:39] [PASSED] 0xD745 (NOVALAKE_S)
[14:36:39] [PASSED] 0xD74A (NOVALAKE_S)
[14:36:39] [PASSED] 0xD74B (NOVALAKE_S)
[14:36:39] [PASSED] 0x674C (CRESCENTISLAND)
[14:36:39] [PASSED] 0x674D (CRESCENTISLAND)
[14:36:39] [PASSED] 0x674E (CRESCENTISLAND)
[14:36:39] [PASSED] 0x674F (CRESCENTISLAND)
[14:36:39] [PASSED] 0x6750 (CRESCENTISLAND)
[14:36:39] [PASSED] 0xD750 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD751 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD752 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD753 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD754 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD755 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD756 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD757 (NOVALAKE_P)
[14:36:39] [PASSED] 0xD75F (NOVALAKE_P)
[14:36:39] =============== [PASSED] check_platform_desc ===============
[14:36:39] ===================== [PASSED] xe_pci ======================
[14:36:39] ============= xe_rtp_tables_test (5 subtests) ==============
[14:36:39] ================== xe_rtp_table_gt_test  ===================
[14:36:39] [PASSED] gt_was/14011060649
[14:36:39] [PASSED] gt_was/14011059788
[14:36:39] [PASSED] gt_was/14015795083
[14:36:39] [PASSED] gt_was/16021867713
[14:36:39] [PASSED] gt_was/14019449301
[14:36:39] [PASSED] gt_was/16028005424
[14:36:39] [PASSED] gt_was/14026578760
[14:36:39] [PASSED] gt_was/1409420604
[14:36:39] [PASSED] gt_was/1408615072
[14:36:39] [PASSED] gt_was/22010523718
[14:36:39] [PASSED] gt_was/14011006942
[14:36:39] [PASSED] gt_was/14014830051
[14:36:39] [PASSED] gt_was/18018781329
[14:36:39] [PASSED] gt_was/1509235366
[14:36:39] [PASSED] gt_was/18018781329
[14:36:39] [PASSED] gt_was/16016694945
[14:36:39] [PASSED] gt_was/14018575942
[14:36:39] [PASSED] gt_was/22016670082
[14:36:39] [PASSED] gt_was/22016670082
[14:36:39] [PASSED] gt_was/14017421178
[14:36:39] [PASSED] gt_was/16025250150
[14:36:39] [PASSED] gt_was/14021871409
[14:36:39] [PASSED] gt_was/16021865536
[14:36:39] [PASSED] gt_was/14021486841
[14:36:39] [PASSED] gt_was/14025160223
[14:36:39] [PASSED] gt_was/14026144927, 16029437861, 14026127056
[14:36:39] [PASSED] gt_was/14025635424
[14:36:39] [PASSED] gt_was/16028005424
[14:36:39] ============== [PASSED] xe_rtp_table_gt_test ===============
[14:36:39] ================== xe_rtp_table_gt_test  ===================
[14:36:39] [PASSED] gt_tunings/Tuning: Blend Fill Caching Optimization Disable
[14:36:39] [PASSED] gt_tunings/Tuning: 32B Access Enable
[14:36:39] [PASSED] gt_tunings/Tuning: L3 cache
[14:36:39] [PASSED] gt_tunings/Tuning: L3 cache - media
[14:36:39] [PASSED] gt_tunings/Tuning: Compression Overfetch
[14:36:39] [PASSED] gt_tunings/Tuning: Compression Overfetch - media
[14:36:39] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3
[14:36:39] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3 - media
[14:36:39] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only
[14:36:39] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only - media
[14:36:39] [PASSED] gt_tunings/Tuning: Stateless compression control
[14:36:39] [PASSED] gt_tunings/Tuning: Stateless compression control - media
[14:36:39] [PASSED] gt_tunings/Tuning: L3 RW flush all Cache
[14:36:39] [PASSED] gt_tunings/Tuning: L3 RW flush all cache - media
[14:36:39] [PASSED] gt_tunings/Tuning: Set STLB Bank Hash Mode to 4KB
[14:36:39] ============== [PASSED] xe_rtp_table_gt_test ===============
[14:36:39] ================== xe_rtp_table_oob_test  ==================
[14:36:39] [PASSED] oob_was/1607983814
[14:36:39] [PASSED] oob_was/16010904313
[14:36:39] [PASSED] oob_was/18022495364
[14:36:39] [PASSED] oob_was/22012773006
[14:36:39] [PASSED] oob_was/14014475959
[14:36:39] [PASSED] oob_was/22011391025
[14:36:39] [PASSED] oob_was/22012727170
[14:36:39] [PASSED] oob_was/22012727685
[14:36:39] [PASSED] oob_was/22016596838
[14:36:39] [PASSED] oob_was/18020744125
[14:36:39] [PASSED] oob_was/1409600907
[14:36:39] [PASSED] oob_was/22014953428
[14:36:39] [PASSED] oob_was/16017236439
[14:36:39] [PASSED] oob_was/14019821291
[14:36:39] [PASSED] oob_was/14015076503
[14:36:39] [PASSED] oob_was/14018913170
[14:36:39] [PASSED] oob_was/14018094691
[14:36:39] [PASSED] oob_was/18024947630
[14:36:39] [PASSED] oob_was/16022287689
[14:36:39] [PASSED] oob_was/13011645652
[14:36:39] [PASSED] oob_was/14022293748
[14:36:39] [PASSED] oob_was/22019794406
[14:36:39] [PASSED] oob_was/22019338487
[14:36:39] [PASSED] oob_was/16023588340
[14:36:39] [PASSED] oob_was/14019789679
[14:36:39] [PASSED] oob_was/14022866841
[14:36:39] [PASSED] oob_was/16021333562
[14:36:39] [PASSED] oob_was/14016712196
[14:36:39] [PASSED] oob_was/14015568240
[14:36:39] [PASSED] oob_was/18013179988
[14:36:39] [PASSED] oob_was/1508761755
[14:36:39] [PASSED] oob_was/16023105232
[14:36:39] [PASSED] oob_was/16026508708
[14:36:39] [PASSED] oob_was/14020001231
[14:36:39] [PASSED] oob_was/16023683509
[14:36:39] [PASSED] oob_was/14025515070
[14:36:39] [PASSED] oob_was/15015404425_disable
[14:36:39] [PASSED] oob_was/16026007364
[14:36:39] [PASSED] oob_was/14020316580
[14:36:39] [PASSED] oob_was/14025883347
[14:36:39] [PASSED] oob_was/16029380221
[14:36:39] [PASSED] oob_was/22022079272
[14:36:39] [PASSED] oob_was/16029897822
[14:36:39] [PASSED] oob_was/14027054324
[14:36:39] ============== [PASSED] xe_rtp_table_oob_test ==============
[14:36:39] ================ xe_rtp_table_dev_oob_test  ================
[14:36:39] [PASSED] device_oob_was/22010954014
[14:36:39] [PASSED] device_oob_was/15015404425
[14:36:39] [PASSED] device_oob_was/22019338487_display
[14:36:39] [PASSED] device_oob_was/14022085890
[14:36:39] [PASSED] device_oob_was/14026539277
[14:36:39] [PASSED] device_oob_was/14026633728
[14:36:39] [PASSED] device_oob_was/14026746987
[14:36:39] [PASSED] device_oob_was/14026779378
[14:36:39] ============ [PASSED] xe_rtp_table_dev_oob_test ============
[14:36:39] ========== xe_rtp_table_missing_upper_bound_test  ==========
[14:36:39] [PASSED] register_whitelist/WaAllowPMDepthAndInvocationCountAccessFromUMD, 1408556865
[14:36:39] [PASSED] register_whitelist/1508744258, 14012131227, 1808121037
[14:36:39] [PASSED] register_whitelist/1806527549
[14:36:39] [PASSED] register_whitelist/allow_read_ctx_timestamp
[14:36:39] [PASSED] register_whitelist/allow_read_queue_timestamp
[14:36:39] [PASSED] register_whitelist/16014440446
[14:36:39] [PASSED] register_whitelist/16017236439
[14:36:39] [PASSED] register_whitelist/16020183090
[14:36:39] [PASSED] register_whitelist/14024997852
[14:36:39] [PASSED] register_whitelist/14024997852
[14:36:39] ====== [PASSED] xe_rtp_table_missing_upper_bound_test ======
[14:36:39] =============== [PASSED] xe_rtp_tables_test ================
[14:36:39] =================== xe_rtp (3 subtests) ====================
[14:36:39] =================== xe_rtp_rules_tests  ====================
[14:36:39] [PASSED] no
[14:36:39] [PASSED] yes
[14:36:39] [PASSED] no-and-no
[14:36:39] [PASSED] no-and-yes
[14:36:39] [PASSED] yes-and-no
[14:36:39] [PASSED] yes-and-yes
[14:36:39] [PASSED] no-or-no
[14:36:39] [PASSED] no-or-yes
[14:36:39] [PASSED] yes-or-no
[14:36:39] [PASSED] yes-or-yes
[14:36:39] [PASSED] no-yes-or-yes-no
[14:36:39] [PASSED] no-yes-or-yes-yes
[14:36:39] [PASSED] yes-yes-or-no-yes
[14:36:39] [PASSED] yes-yes-or-yes-yes
[14:36:39] [PASSED] no-no-or-yes-or-no
[14:36:39] [PASSED] or
[14:36:39] [PASSED] or-yes
[14:36:39] [PASSED] or-no
[14:36:39] [PASSED] yes-or
[14:36:39] [PASSED] no-or
[14:36:39] [PASSED] no-or-or-yes
[14:36:39] [PASSED] yes-or-or-no
[14:36:39] [PASSED] no-or-or-no
[14:36:39] [PASSED] missing-context-engine-class
[14:36:39] [PASSED] missing-context-engine-class-or-yes
[14:36:39] [PASSED] missing-context-engine-class-or-or-yes
[14:36:39] =============== [PASSED] xe_rtp_rules_tests ================
[14:36:39] =============== xe_rtp_process_to_sr_tests  ================
[14:36:39] [PASSED] coalesce-same-reg
[14:36:39] [PASSED] coalesce-same-reg-literal-and-func
[14:36:39] [PASSED] no-match-no-add
[14:36:39] [PASSED] two-regs-two-entries
[14:36:39] [PASSED] clr-one-set-other
[14:36:39] [PASSED] set-field
[14:36:39] [PASSED] conflict-duplicate
[14:36:39] [PASSED] conflict-not-disjoint
[14:36:39] [PASSED] conflict-not-disjoint-literal-and-func
[14:36:39] [PASSED] conflict-reg-type
[14:36:39] [PASSED] bad-mcr-reg-forced-to-regular
[14:36:39] [PASSED] bad-regular-reg-forced-to-mcr
[14:36:39] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[14:36:39] ================== xe_rtp_process_tests  ===================
[14:36:39] [PASSED] active1
[14:36:39] [PASSED] active2
[14:36:39] [PASSED] active-inactive
[14:36:39] [PASSED] inactive-active
[14:36:39] [PASSED] inactive-active-inactive
[14:36:39] [PASSED] inactive-inactive-inactive
[14:36:39] ============== [PASSED] xe_rtp_process_tests ===============
[14:36:39] ===================== [PASSED] xe_rtp ======================
[14:36:39] ==================== xe_wa (1 subtest) =====================
[14:36:39] ======================== xe_wa_gt  =========================
[14:36:39] [PASSED] TIGERLAKE B0
[14:36:39] [PASSED] DG1 A0
[14:36:39] [PASSED] DG1 B0
[14:36:39] [PASSED] ALDERLAKE_S A0
[14:36:39] [PASSED] ALDERLAKE_S B0
[14:36:39] [PASSED] ALDERLAKE_S C0
[14:36:39] [PASSED] ALDERLAKE_S D0
[14:36:39] [PASSED] ALDERLAKE_P A0
[14:36:39] [PASSED] ALDERLAKE_P B0
[14:36:39] [PASSED] ALDERLAKE_P C0
[14:36:39] [PASSED] ALDERLAKE_S RPLS D0
[14:36:39] [PASSED] ALDERLAKE_P RPLU E0
[14:36:39] [PASSED] DG2 G10 C0
[14:36:39] [PASSED] DG2 G11 B1
[14:36:39] [PASSED] DG2 G12 A1
[14:36:39] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[14:36:39] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[14:36:39] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[14:36:39] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[14:36:39] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[14:36:39] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[14:36:39] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[14:36:39] ==================== [PASSED] xe_wa_gt =====================
[14:36:39] ====================== [PASSED] xe_wa ======================
[14:36:39] ============================================================
[14:36:39] Testing complete. Ran 793 tests: passed: 765, skipped: 28
[14:36:39] Elapsed time: 37.265s total, 4.434s configuring, 32.115s building, 0.688s running

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

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

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/dma-buf/.kunitconfig
[14:37:19] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[14:37:21] 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
[14:37:29] Starting KUnit Kernel (1/1)...
[14:37:29] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[14:37:30] ============= refcount_interrupt (4 subtests) ==============
[14:37:30] [PASSED] test_single_irq_change
[14:37:30] [PASSED] test_nested_irq_change
[14:37:30] [PASSED] test_multiple_irq_change
[14:37:30] [PASSED] test_irq_save
[14:37:30] =============== [PASSED] refcount_interrupt ================
[14:37:30] =============== dma-buf-fence (12 subtests) ================
[14:37:30] [PASSED] test_sanitycheck
[14:37:30] [PASSED] test_signaling
[14:37:30] [PASSED] test_add_callback
[14:37:30] [PASSED] test_late_add_callback
[14:37:30] [PASSED] test_rm_callback
[14:37:30] [PASSED] test_late_rm_callback
[14:37:30] [PASSED] test_status
[14:37:30] [PASSED] test_error
[14:37:30] [PASSED] test_wait
[14:37:30] [PASSED] test_wait_timeout
[14:37:30] [PASSED] test_stub
[14:37:30] [SKIPPED] test_race_signal_callback (requires at least 2 CPUs)
[14:37:30] ================== [PASSED] dma-buf-fence ==================
[14:37:30] ============ dma-buf-fence-chain (11 subtests) =============
[14:37:30] [PASSED] test_sanitycheck
[14:37:30] [PASSED] test_find_seqno
[14:37:30] [PASSED] test_find_signaled
[14:37:30] [PASSED] test_find_out_of_order
[14:37:35] [PASSED] test_find_gap
[14:37:35] [PASSED] test_find_race
[14:37:35] [PASSED] test_signal_forward
[14:37:35] [PASSED] test_signal_backward
[14:37:35] [PASSED] test_wait_forward
[14:37:35] [PASSED] test_wait_backward
[14:37:35] [PASSED] test_wait_random
[14:37:35] =============== [PASSED] dma-buf-fence-chain ===============
[14:37:35] ============ dma-buf-fence-unwrap (10 subtests) ============
[14:37:35] [PASSED] test_sanitycheck
[14:37:35] [PASSED] test_unwrap_array
[14:37:35] [PASSED] test_unwrap_chain
[14:37:35] [PASSED] test_unwrap_chain_array
[14:37:35] [PASSED] test_unwrap_merge
[14:37:35] [PASSED] test_unwrap_merge_duplicate
[14:37:35] [PASSED] test_unwrap_merge_seqno
[14:37:35] [PASSED] test_unwrap_merge_order
[14:37:35] [PASSED] test_unwrap_merge_complex
[14:37:35] [PASSED] test_unwrap_merge_complex_seqno
[14:37:35] ============== [PASSED] dma-buf-fence-unwrap ===============
[14:37:35] ================ dma-buf-resv (5 subtests) =================
[14:37:35] [PASSED] test_sanitycheck
[14:37:35] ===================== test_signaling  ======================
[14:37:35] [PASSED] kernel
[14:37:35] [PASSED] write
[14:37:35] [PASSED] read
[14:37:35] [PASSED] bookkeep
[14:37:35] ================= [PASSED] test_signaling ==================
[14:37:35] ====================== test_for_each  ======================
[14:37:35] [PASSED] kernel
[14:37:35] [PASSED] write
[14:37:35] [PASSED] read
[14:37:35] [PASSED] bookkeep
[14:37:35] ================== [PASSED] test_for_each ==================
[14:37:35] ================= test_for_each_unlocked  ==================
[14:37:35] [PASSED] kernel
[14:37:35] [PASSED] write
[14:37:35] [PASSED] read
[14:37:35] [PASSED] bookkeep
[14:37:35] ============= [PASSED] test_for_each_unlocked ==============
[14:37:35] ===================== test_get_fences  =====================
[14:37:35] [PASSED] kernel
[14:37:35] [PASSED] write
[14:37:35] [PASSED] read
[14:37:35] [PASSED] bookkeep
[14:37:35] ================= [PASSED] test_get_fences =================
[14:37:35] ================== [PASSED] dma-buf-resv ===================
[14:37:35] ============================================================
[14:37:35] Testing complete. Ran 54 tests: passed: 53, skipped: 1
[14:37:35] Elapsed time: 15.882s total, 1.822s configuring, 8.688s building, 5.345s running

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



^ permalink raw reply	[flat|nested] 24+ messages in thread

* ✓ Xe.CI.BAT: success for drm/xe/sysctrl: Clean up error handling in (rev3)
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (9 preceding siblings ...)
  2026-08-31 14:37 ` ✓ CI.KUnit: success for drm/xe/sysctrl: Clean up error handling in (rev3) Patchwork
@ 2026-08-31 15:26 ` Patchwork
  2026-08-31 17:52 ` ✗ Xe.CI.FULL: failure " Patchwork
  11 siblings, 0 replies; 24+ messages in thread
From: Patchwork @ 2026-08-31 15:26 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

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

== Series Details ==

Series: drm/xe/sysctrl: Clean up error handling in (rev3)
URL   : https://patchwork.freedesktop.org/series/172513/
State : success

== Summary ==

CI Bug Log - changes from xe-5662-aa389a514292b196ae89c6515568b3329abeec63_BAT -> xe-pw-172513v3_BAT
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

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

  Missing    (1): bat-nvls-1 


Changes
-------

  No changes found


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

  * Linux: xe-5662-aa389a514292b196ae89c6515568b3329abeec63 -> xe-pw-172513v3

  IGT_9078: 9078
  xe-5662-aa389a514292b196ae89c6515568b3329abeec63: aa389a514292b196ae89c6515568b3329abeec63
  xe-pw-172513v3: 172513v3

== Logs ==

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

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

^ permalink raw reply	[flat|nested] 24+ messages in thread

* ✗ Xe.CI.FULL: failure for drm/xe/sysctrl: Clean up error handling in (rev3)
  2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
                   ` (10 preceding siblings ...)
  2026-08-31 15:26 ` ✓ Xe.CI.BAT: " Patchwork
@ 2026-08-31 17:52 ` Patchwork
  11 siblings, 0 replies; 24+ messages in thread
From: Patchwork @ 2026-08-31 17:52 UTC (permalink / raw)
  To: Mallesh Koujalagi; +Cc: intel-xe

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

== Series Details ==

Series: drm/xe/sysctrl: Clean up error handling in (rev3)
URL   : https://patchwork.freedesktop.org/series/172513/
State : failure

== Summary ==

CI Bug Log - changes from xe-5662-aa389a514292b196ae89c6515568b3329abeec63_FULL -> xe-pw-172513v3_FULL
====================================================

Summary
-------

  **FAILURE**

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

  

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

  No changes in participating hosts

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@runner@aborted:
    - shard-lnl:          NOTRUN -> [FAIL][1]
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-lnl-1/igt@runner@aborted.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@kms_bw@linear-tiling-4-displays-target-2160x1440p:
    - shard-bmg:          NOTRUN -> [SKIP][2] ([Intel XE#367])
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-9/igt@kms_bw@linear-tiling-4-displays-target-2160x1440p.html

  * igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs:
    - shard-bmg:          [PASS][3] -> [INCOMPLETE][4] ([Intel XE#7084] / [Intel XE#8150]) +1 other test incomplete
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-7/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs.html
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-3/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs.html

  * igt@kms_flip@2x-flip-vs-absolute-wf_vblank@bc-dp2-hdmi-a3:
    - shard-bmg:          [PASS][5] -> [FAIL][6] ([Intel XE#6266]) +1 other test fail
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-7/igt@kms_flip@2x-flip-vs-absolute-wf_vblank@bc-dp2-hdmi-a3.html
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-6/igt@kms_flip@2x-flip-vs-absolute-wf_vblank@bc-dp2-hdmi-a3.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
    - shard-lnl:          [PASS][7] -> [FAIL][8] ([Intel XE#301]) +1 other test fail
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-lnl-5/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-lnl-6/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html

  * igt@kms_hdr@invalid-hdr:
    - shard-bmg:          [PASS][9] -> [SKIP][10] ([Intel XE#1503])
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-7/igt@kms_hdr@invalid-hdr.html
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-3/igt@kms_hdr@invalid-hdr.html

  * igt@kms_pm_dc@deep-pkgc:
    - shard-lnl:          [PASS][11] -> [FAIL][12] ([Intel XE#2029] / [Intel XE#7395])
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-lnl-3/igt@kms_pm_dc@deep-pkgc.html
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-lnl-4/igt@kms_pm_dc@deep-pkgc.html

  * igt@xe_pmu@engine-activity-accuracy-50@engine-drm_xe_engine_class_compute0:
    - shard-bmg:          [PASS][13] -> [FAIL][14] ([Intel XE#8555])
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-9/igt@xe_pmu@engine-activity-accuracy-50@engine-drm_xe_engine_class_compute0.html
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-2/igt@xe_pmu@engine-activity-accuracy-50@engine-drm_xe_engine_class_compute0.html

  
#### Possible fixes ####

  * igt@core_hotunplug@hotreplug-lateclose:
    - shard-bmg:          [ABORT][15] ([Intel XE#8007]) -> [PASS][16]
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-9/igt@core_hotunplug@hotreplug-lateclose.html
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-9/igt@core_hotunplug@hotreplug-lateclose.html

  * igt@kms_async_flips@alternate-sync-async-flip:
    - shard-bmg:          [FAIL][17] ([Intel XE#3718] / [Intel XE#6078]) -> [PASS][18]
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-1/igt@kms_async_flips@alternate-sync-async-flip.html
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-3/igt@kms_async_flips@alternate-sync-async-flip.html

  * igt@kms_async_flips@alternate-sync-async-flip@pipe-b-dp-2:
    - shard-bmg:          [FAIL][19] ([Intel XE#6078]) -> [PASS][20]
   [19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-1/igt@kms_async_flips@alternate-sync-async-flip@pipe-b-dp-2.html
   [20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-3/igt@kms_async_flips@alternate-sync-async-flip@pipe-b-dp-2.html

  * igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1:
    - shard-lnl:          [FAIL][21] ([Intel XE#5408] / [Intel XE#6266]) -> [PASS][22] +1 other test pass
   [21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-lnl-2/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html
   [22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-lnl-8/igt@kms_flip@plain-flip-fb-recreate-interruptible@b-edp1.html

  * igt@xe_exec_threads@threads-hang-fd-userptr-invalidate-race:
    - shard-bmg:          [FAIL][23] -> [PASS][24]
   [23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-bmg-2/igt@xe_exec_threads@threads-hang-fd-userptr-invalidate-race.html
   [24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-bmg-9/igt@xe_exec_threads@threads-hang-fd-userptr-invalidate-race.html

  
#### Warnings ####

  * igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions:
    - shard-lnl:          [SKIP][25] ([Intel XE#309] / [Intel XE#7343] / [Intel XE#7935]) -> [SKIP][26] ([Intel XE#309] / [Intel XE#7343])
   [25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-lnl-8/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html
   [26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-lnl-3/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html

  * igt@xe_wedged@basic-wedged:
    - shard-lnl:          [DMESG-WARN][27] ([Intel XE#8963]) -> [ABORT][28] ([Intel XE#8963])
   [27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5662-aa389a514292b196ae89c6515568b3329abeec63/shard-lnl-6/igt@xe_wedged@basic-wedged.html
   [28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172513v3/shard-lnl-3/igt@xe_wedged@basic-wedged.html

  
  [Intel XE#1503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1503
  [Intel XE#2029]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2029
  [Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
  [Intel XE#309]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/309
  [Intel XE#367]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/367
  [Intel XE#3718]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3718
  [Intel XE#5408]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5408
  [Intel XE#6078]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6078
  [Intel XE#6266]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6266
  [Intel XE#7084]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7084
  [Intel XE#7343]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7343
  [Intel XE#7395]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7395
  [Intel XE#7935]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7935
  [Intel XE#8007]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8007
  [Intel XE#8150]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8150
  [Intel XE#8555]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8555
  [Intel XE#8963]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8963


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

  * Linux: xe-5662-aa389a514292b196ae89c6515568b3329abeec63 -> xe-pw-172513v3

  IGT_9078: 9078
  xe-5662-aa389a514292b196ae89c6515568b3329abeec63: aa389a514292b196ae89c6515568b3329abeec63
  xe-pw-172513v3: 172513v3

== Logs ==

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

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

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void
  2026-08-31 10:45 ` [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void Mallesh Koujalagi
@ 2026-09-02 16:16   ` Michal Wajdeczko
  2026-09-03  4:45     ` Mallesh, Koujalagi
  0 siblings, 1 reply; 24+ messages in thread
From: Michal Wajdeczko @ 2026-09-02 16:16 UTC (permalink / raw)
  To: Mallesh Koujalagi, intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, aravind.iddamsetty,
	umesh.nerlige.ramappa, dnyaneshwar.bhadane, anoop.c.vijay



On 8/31/2026 12:45 PM, Mallesh Koujalagi wrote:
> Convert it to return void and remove the unnecessary error
> handling in sysctrl_process_frame().

... as there are no places where this function can fail.

> 
> Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>

(a few tags Suggested-by: wouldn't hurt either ;)

> ---
>  drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 11 +++--------
>  1 file changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 3ef60067e226..4841dd7f2724 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -68,8 +68,8 @@ static void sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
>  		xe_mmio_write32(sc->mmio, regs[i], val[i]);
>  }
>  
> -static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
> -			      size_t len)
> +static void sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
> +			       size_t len)
>  {
>  	static const struct xe_reg regs[] = {
>  		SYSCTRL_MB_DATA0, SYSCTRL_MB_DATA1, SYSCTRL_MB_DATA2, SYSCTRL_MB_DATA3
> @@ -85,8 +85,6 @@ static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
>  		val[i] = xe_mmio_read32(sc->mmio, regs[i]);
>  
>  	memcpy(frame, val, len);
> -
> -	return 0;
>  }
>  
>  static void sysctrl_clear_response(struct xe_sysctrl *sc)
> @@ -194,10 +192,7 @@ static int sysctrl_process_frame(struct xe_sysctrl *sc, void *out,
>  	total_frames = FIELD_GET(SYSCTRL_FRAME_TOTAL_MASK, ctrl_reg);
>  	curr_frame = FIELD_GET(SYSCTRL_FRAME_CURRENT_MASK, ctrl_reg);
>  
> -	ret = sysctrl_read_frame(sc, out, frame_size);
> -	if (ret)
> -		return ret;
> -
> +	sysctrl_read_frame(sc, out, frame_size);
>  	sysctrl_clear_response(sc);
>  
>  	if (curr_frame == total_frames)


^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging
  2026-08-31 10:45 ` [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging Mallesh Koujalagi
  2026-08-31 11:00   ` sashiko-bot
@ 2026-09-02 16:22   ` Michal Wajdeczko
  2026-09-03  4:59     ` Mallesh, Koujalagi
  1 sibling, 1 reply; 24+ messages in thread
From: Michal Wajdeczko @ 2026-09-02 16:22 UTC (permalink / raw)
  To: Mallesh Koujalagi, intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, aravind.iddamsetty,
	umesh.nerlige.ramappa, dnyaneshwar.bhadane, anoop.c.vijay



On 8/31/2026 12:45 PM, Mallesh Koujalagi wrote:
> Use xe_log_err() helper and include additional details when a
> response header mismatch occurs.
> 
> Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
> ---
> v5:
> - Use %#x after REG_FIELD_GET change. (Michal)
> 
> v6:
> - Move REG_FIELD_GET to different patch. (Michal)
> - Update response message.
> ---
>  drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 3aa868f60b38..87e656820279 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -233,13 +233,15 @@ static int sysctrl_receive_frames(struct xe_sysctrl *sc,
>  	if (!XE_SYSCTRL_HDR_IS_RESPONSE(hdr) ||
>  	    XE_SYSCTRL_HDR_GROUP_ID(hdr) != XE_SYSCTRL_HDR_GROUP_ID(req) ||
>  	    XE_SYSCTRL_HDR_COMMAND(hdr) != XE_SYSCTRL_HDR_COMMAND(req)) {
> -		xe_err(xe, "sysctrl: Response header mismatch\n");
> +		xe_log_err(xe, SYSCTRL, -EPROTO,
> +			   "Response header mismatch: %#x != %#x\n",
> +			   le32_to_cpu(hdr->data), le32_to_cpu(req->data));

nit: do we need this le32 conversions when we print both raw values?

>  		return -EPROTO;
>  	}
>  
>  	if (XE_SYSCTRL_HDR_RESULT(hdr) != 0) {
> -		xe_err(xe, "sysctrl: Firmware error: 0x%02lx\n",
> -		       XE_SYSCTRL_HDR_RESULT(hdr));
> +		xe_log_err(xe, SYSCTRL, -EIO, "Firmware error: %#04x\n",
> +			   XE_SYSCTRL_HDR_RESULT(hdr));
>  		return -EIO;
>  	}
>  

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>


^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void
  2026-09-02 16:16   ` Michal Wajdeczko
@ 2026-09-03  4:45     ` Mallesh, Koujalagi
  0 siblings, 0 replies; 24+ messages in thread
From: Mallesh, Koujalagi @ 2026-09-03  4:45 UTC (permalink / raw)
  To: Michal Wajdeczko, intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, aravind.iddamsetty,
	umesh.nerlige.ramappa, dnyaneshwar.bhadane, anoop.c.vijay


On 02-09-2026 09:46 pm, Michal Wajdeczko wrote:
>
> On 8/31/2026 12:45 PM, Mallesh Koujalagi wrote:
>> Convert it to return void and remove the unnecessary error
>> handling in sysctrl_process_frame().
> ... as there are no places where this function can fail.
>
>> Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
> Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Thanks Michal !!
>
> (a few tags Suggested-by: wouldn't hurt either ;)
Sure! will add.
>
>> ---
>>   drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 11 +++--------
>>   1 file changed, 3 insertions(+), 8 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
>> index 3ef60067e226..4841dd7f2724 100644
>> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
>> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
>> @@ -68,8 +68,8 @@ static void sysctrl_write_frame(struct xe_sysctrl *sc, const void *frame,
>>   		xe_mmio_write32(sc->mmio, regs[i], val[i]);
>>   }
>>   
>> -static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
>> -			      size_t len)
>> +static void sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
>> +			       size_t len)
>>   {
>>   	static const struct xe_reg regs[] = {
>>   		SYSCTRL_MB_DATA0, SYSCTRL_MB_DATA1, SYSCTRL_MB_DATA2, SYSCTRL_MB_DATA3
>> @@ -85,8 +85,6 @@ static int sysctrl_read_frame(struct xe_sysctrl *sc, void *frame,
>>   		val[i] = xe_mmio_read32(sc->mmio, regs[i]);
>>   
>>   	memcpy(frame, val, len);
>> -
>> -	return 0;
>>   }
>>   
>>   static void sysctrl_clear_response(struct xe_sysctrl *sc)
>> @@ -194,10 +192,7 @@ static int sysctrl_process_frame(struct xe_sysctrl *sc, void *out,
>>   	total_frames = FIELD_GET(SYSCTRL_FRAME_TOTAL_MASK, ctrl_reg);
>>   	curr_frame = FIELD_GET(SYSCTRL_FRAME_CURRENT_MASK, ctrl_reg);
>>   
>> -	ret = sysctrl_read_frame(sc, out, frame_size);
>> -	if (ret)
>> -		return ret;
>> -
>> +	sysctrl_read_frame(sc, out, frame_size);
>>   	sysctrl_clear_response(sc);
>>   
>>   	if (curr_frame == total_frames)

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging
  2026-09-02 16:22   ` Michal Wajdeczko
@ 2026-09-03  4:59     ` Mallesh, Koujalagi
  0 siblings, 0 replies; 24+ messages in thread
From: Mallesh, Koujalagi @ 2026-09-03  4:59 UTC (permalink / raw)
  To: Michal Wajdeczko, intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, riana.tauro,
	karthik.poosa, sk.anirban, raag.jadav, aravind.iddamsetty,
	umesh.nerlige.ramappa, dnyaneshwar.bhadane, anoop.c.vijay


On 02-09-2026 09:52 pm, Michal Wajdeczko wrote:
>
> On 8/31/2026 12:45 PM, Mallesh Koujalagi wrote:
>> Use xe_log_err() helper and include additional details when a
>> response header mismatch occurs.
>>
>> Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
>> ---
>> v5:
>> - Use %#x after REG_FIELD_GET change. (Michal)
>>
>> v6:
>> - Move REG_FIELD_GET to different patch. (Michal)
>> - Update response message.
>> ---
>>   drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 8 +++++---
>>   1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
>> index 3aa868f60b38..87e656820279 100644
>> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
>> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
>> @@ -233,13 +233,15 @@ static int sysctrl_receive_frames(struct xe_sysctrl *sc,
>>   	if (!XE_SYSCTRL_HDR_IS_RESPONSE(hdr) ||
>>   	    XE_SYSCTRL_HDR_GROUP_ID(hdr) != XE_SYSCTRL_HDR_GROUP_ID(req) ||
>>   	    XE_SYSCTRL_HDR_COMMAND(hdr) != XE_SYSCTRL_HDR_COMMAND(req)) {
>> -		xe_err(xe, "sysctrl: Response header mismatch\n");
>> +		xe_log_err(xe, SYSCTRL, -EPROTO,
>> +			   "Response header mismatch: %#x != %#x\n",
>> +			   le32_to_cpu(hdr->data), le32_to_cpu(req->data));
> nit: do we need this le32 conversions when we print both raw values?
Agreed!, if we do raw data comparison, it's make sense by removing le32.
>
>>   		return -EPROTO;
>>   	}
>>   
>>   	if (XE_SYSCTRL_HDR_RESULT(hdr) != 0) {
>> -		xe_err(xe, "sysctrl: Firmware error: 0x%02lx\n",
>> -		       XE_SYSCTRL_HDR_RESULT(hdr));
>> +		xe_log_err(xe, SYSCTRL, -EIO, "Firmware error: %#04x\n",
>> +			   XE_SYSCTRL_HDR_RESULT(hdr));
>>   		return -EIO;
>>   	}
>>   
> Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Thanks Michal !

^ permalink raw reply	[flat|nested] 24+ messages in thread

* Re: [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear()
  2026-08-31 10:44 ` [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
  2026-08-31 10:55   ` sashiko-bot
@ 2026-09-04  6:01   ` Tauro, Riana
  1 sibling, 0 replies; 24+ messages in thread
From: Tauro, Riana @ 2026-09-04  6:01 UTC (permalink / raw)
  To: Mallesh Koujalagi, intel-xe, rodrigo.vivi, matthew.brost
  Cc: anshuman.gupta, badal.nilawar, vinay.belgaumkar, karthik.poosa,
	sk.anirban, raag.jadav, michal.wajdeczko, aravind.iddamsetty,
	umesh.nerlige.ramappa, dnyaneshwar.bhadane, anoop.c.vijay

Hi Mallesh

On 31-08-2026 16:14, Mallesh Koujalagi wrote:
> Make sysctrl_wait_bit_clear() return an error code rather than a bool
> and update callers to use xe_log_err() with the propagated error code.
>
> Signed-off-by: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
> Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>

Can you please respond to all comments in previous revs before sending 
new revisions

Re: [PATCH v4 1/7] drm/xe/sysctrl: Return error codes from 
sysctrl_wait_bit_clear() - Tauro, Riana

<https://lore.kernel.org/intel-xe/292aaec1-662d-466a-83a6-dc820dace637@intel.com/>Thanks
Riana

> ---
> v5:
> - Fix align issue. (Michal)
> ---
>   drivers/gpu/drm/xe/xe_sysctrl_mailbox.c | 28 ++++++++++++-------------
>   1 file changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> index 72baf1aa4b3a..c36cac8c6288 100644
> --- a/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> +++ b/drivers/gpu/drm/xe/xe_sysctrl_mailbox.c
> @@ -11,6 +11,7 @@
>   
>   #include "regs/xe_sysctrl_regs.h"
>   #include "xe_device.h"
> +#include "xe_log.h"
>   #include "xe_mmio.h"
>   #include "xe_pm.h"
>   #include "xe_printk.h"
> @@ -34,15 +35,11 @@ struct xe_sysctrl_mailbox_msg_hdr {
>   #define XE_SYSCTRL_HDR_RESULT(hdr) \
>   	FIELD_GET(SYSCTRL_HDR_RESULT_MASK, le32_to_cpu((hdr)->data))
>   
> -static bool sysctrl_wait_bit_clear(struct xe_sysctrl *sc, u32 bit_mask,
> -				   unsigned int timeout_ms)
> +static int sysctrl_wait_bit_clear(struct xe_sysctrl *sc, u32 bit_mask,
> +				  unsigned int timeout_ms)
>   {
> -	int ret;
> -
> -	ret = xe_mmio_wait32_not(sc->mmio, SYSCTRL_MB_CTRL, bit_mask, bit_mask,
> -				 timeout_ms * 1000, NULL, false);
> -
> -	return ret == 0;
> +	return xe_mmio_wait32_not(sc->mmio, SYSCTRL_MB_CTRL, bit_mask, bit_mask,
> +				  timeout_ms * 1000, NULL, false);
>   }
>   
>   static bool sysctrl_wait_bit_set(struct xe_sysctrl *sc, u32 bit_mask,
> @@ -146,12 +143,14 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
>   	u32 ctrl_reg, total_frames, frame;
>   	size_t bytes_sent, frame_size;
>   	bool phase;
> +	int ret;
>   
>   	total_frames = DIV_ROUND_UP(cmd_size, XE_SYSCTRL_MB_FRAME_SIZE);
>   
> -	if (!sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms)) {
> -		xe_err(xe, "sysctrl: Mailbox busy\n");
> -		return -EBUSY;
> +	ret = sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms);
> +	if (ret) {
> +		xe_log_err(xe, SYSCTRL, ret, "Mailbox busy\n");
> +		return ret;
>   	}
>   
>   	ctrl_reg = xe_mmio_read32(sc->mmio, SYSCTRL_MB_CTRL);
> @@ -174,9 +173,10 @@ static int sysctrl_send_frames(struct xe_sysctrl *sc,
>   
>   		xe_mmio_write32(sc->mmio, SYSCTRL_MB_CTRL, ctrl_reg);
>   
> -		if (!sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms)) {
> -			xe_err(xe, "sysctrl: Frame %u acknowledgment timeout\n", frame);
> -			return -ETIMEDOUT;
> +		ret = sysctrl_wait_bit_clear(sc, SYSCTRL_MB_CTRL_RUN_BUSY, timeout_ms);
> +		if (ret) {
> +			xe_log_err(xe, SYSCTRL, ret, "Frame %u acknowledgment timeout\n", frame);
> +			return ret;
>   		}
>   
>   		bytes_sent += frame_size;

^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2026-09-04  6:03 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-31 10:44 [PATCH v6 0/9] drm/xe/sysctrl: Clean up error handling in Mallesh Koujalagi
2026-08-31 10:44 ` [PATCH v6 1/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_clear() Mallesh Koujalagi
2026-08-31 10:55   ` sashiko-bot
2026-09-04  6:01   ` Tauro, Riana
2026-08-31 10:44 ` [PATCH v6 2/9] drm/xe/sysctrl: Return error codes from sysctrl_wait_bit_set() Mallesh Koujalagi
2026-08-31 10:59   ` sashiko-bot
2026-08-31 10:45 ` [PATCH v6 3/9] drm/xe/sysctrl: Make sysctrl_write_frame() void Mallesh Koujalagi
2026-08-31 10:45 ` [PATCH v6 4/9] drm/xe/sysctrl: Make sysctrl_read_frame() void Mallesh Koujalagi
2026-09-02 16:16   ` Michal Wajdeczko
2026-09-03  4:45     ` Mallesh, Koujalagi
2026-08-31 10:45 ` [PATCH v6 5/9] drm/xe/sysctrl: Use xe_assert() for payload size validation Mallesh Koujalagi
2026-08-31 10:45 ` [PATCH v6 6/9] drm/xe/sysctrl: Replace FIELD_GET() with REG_FIELD_GET() Mallesh Koujalagi
2026-08-31 11:07   ` sashiko-bot
2026-08-31 10:45 ` [PATCH v6 7/9] drm/xe/sysctrl: Improve firmware response error logging Mallesh Koujalagi
2026-08-31 11:00   ` sashiko-bot
2026-09-02 16:22   ` Michal Wajdeczko
2026-09-03  4:59     ` Mallesh, Koujalagi
2026-08-31 10:45 ` [PATCH v6 8/9] drm/xe/sysctrl: Log group and command ID on mailbox failure Mallesh Koujalagi
2026-08-31 11:00   ` sashiko-bot
2026-08-31 10:45 ` [PATCH v6 9/9] drm/xe/sysctrl: Report 'System Controller event' error using SIGID Mallesh Koujalagi
2026-08-31 11:00   ` sashiko-bot
2026-08-31 14:37 ` ✓ CI.KUnit: success for drm/xe/sysctrl: Clean up error handling in (rev3) Patchwork
2026-08-31 15:26 ` ✓ Xe.CI.BAT: " Patchwork
2026-08-31 17:52 ` ✗ Xe.CI.FULL: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox