[PATCH v3 00/11] Support run ticks for multi-queue use case

[PATCH v3 00/11] Support run ticks for multi-queue use case

[Umesh Nerlige Ramappa]

In single queue use cases, the CTX TIMESTAMP can be used to track
context run ticks. In multi-queue scenarios the CTX_TIMESTAMP represents
run ticks of all the queues. To determine individual queue run ticks, we
need to use QUEUE TIMESTAMP.

The series adds support to read out QUEUE TIMESTAMP for multi-queue use
cases.

v2: Include review comments from Matt, Stuart, Niranjana
v3: Include review comments from Niranjana

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>

Matthew Brost (1):
  drm/xe: Add timestamp_ms to LRC snapshot

Umesh Nerlige Ramappa (10):
  drm/xe/lrc: Use 64 bit ctx timestamp in the LRC snapshot
  drm/xe/lrc: Refactor xe_lrc_timestamp to simplify logic
  drm/xe/multi_queue: Refactor check for multi queue support for engine
    class
  drm/xe/multi_queue: Store primary LRC and position info in LRC
  drm/xe/multi_queue: Add helpers to access CS QUEUE TIMESTAMP from lrc
  drm/xe/lrc: Refactor out engine id to hwe conversion
  drm/xe/multi_queue: Capture queue run times for active queues
  drm/xe/multi_queue: Add trace event for the multi queue timestamp
  drm/xe/multi_queue: Use QUEUE_TIMESTAMP as job timestamp for
    multi-queue
  drm/xe/multi_queue: Whitelist QUEUE_TIMESTAMP register

 drivers/gpu/drm/xe/regs/xe_engine_regs.h |   4 +
 drivers/gpu/drm/xe/regs/xe_lrc_layout.h  |   3 +
 drivers/gpu/drm/xe/xe_exec_queue.c       |  12 +-
 drivers/gpu/drm/xe/xe_gt.c               |  14 ++
 drivers/gpu/drm/xe/xe_gt.h               |   3 +
 drivers/gpu/drm/xe/xe_lrc.c              | 222 ++++++++++++++++++-----
 drivers/gpu/drm/xe/xe_lrc.h              |  10 +-
 drivers/gpu/drm/xe/xe_lrc_types.h        |  11 ++
 drivers/gpu/drm/xe/xe_reg_whitelist.c    |  14 ++
 drivers/gpu/drm/xe/xe_ring_ops.c         |   8 +-
 drivers/gpu/drm/xe/xe_trace_lrc.h        |  29 +++
 11 files changed, 273 insertions(+), 57 deletions(-)

-- 
2.51.0

[PATCH v3 01/11] drm/xe/lrc: Use 64 bit ctx timestamp in the LRC snapshot

[Umesh Nerlige Ramappa]

Use the 64 bit value when available for the context timestamp in the LRC
snapshot.

Suggested-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
---
 drivers/gpu/drm/xe/xe_lrc.c | 4 ++--
 drivers/gpu/drm/xe/xe_lrc.h | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index fdfe2ed5f683..570ff9f55849 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -2462,7 +2462,7 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
 	snapshot->replay_offset = 0;
 	snapshot->replay_size = lrc->replay_size;
 	snapshot->lrc_snapshot = NULL;
-	snapshot->ctx_timestamp = lower_32_bits(xe_lrc_ctx_timestamp(lrc));
+	snapshot->ctx_timestamp = xe_lrc_ctx_timestamp(lrc);
 	snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);
 	return snapshot;
 }
@@ -2515,7 +2515,7 @@ void xe_lrc_snapshot_print(struct xe_lrc_snapshot *snapshot, struct drm_printer
 	drm_printf(p, "\tRing start: (memory) 0x%08x\n", snapshot->start);
 	drm_printf(p, "\tStart seqno: (memory) %d\n", snapshot->start_seqno);
 	drm_printf(p, "\tSeqno: (memory) %d\n", snapshot->seqno);
-	drm_printf(p, "\tTimestamp: 0x%08x\n", snapshot->ctx_timestamp);
+	drm_printf(p, "\tTimestamp: 0x%016llx\n", snapshot->ctx_timestamp);
 	drm_printf(p, "\tJob Timestamp: 0x%08x\n", snapshot->ctx_job_timestamp);
 
 	if (!snapshot->lrc_snapshot)
diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
index 5440663183f6..bef7af9a578b 100644
--- a/drivers/gpu/drm/xe/xe_lrc.h
+++ b/drivers/gpu/drm/xe/xe_lrc.h
@@ -37,7 +37,7 @@ struct xe_lrc_snapshot {
 	} tail;
 	u32 start_seqno;
 	u32 seqno;
-	u32 ctx_timestamp;
+	u64 ctx_timestamp;
 	u32 ctx_job_timestamp;
 };
 
-- 
2.51.0

[PATCH v3 02/11] drm/xe: Add timestamp_ms to LRC snapshot

[Umesh Nerlige Ramappa]

From: Matthew Brost <matthew.brost@intel.com>

Add a timestamp in milliseconds to the LRC snapshot to make it easier to
reason about how long the LRC has been running and the average duration
of each job.

Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
v2: Group u32s to avoid padding (Niranjana)
---
 drivers/gpu/drm/xe/xe_lrc.c | 4 ++++
 drivers/gpu/drm/xe/xe_lrc.h | 1 +
 2 files changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index 570ff9f55849..66682a678cc2 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -23,6 +23,7 @@
 #include "xe_drm_client.h"
 #include "xe_exec_queue_types.h"
 #include "xe_gt.h"
+#include "xe_gt_clock.h"
 #include "xe_gt_printk.h"
 #include "xe_hw_fence.h"
 #include "xe_map.h"
@@ -2463,6 +2464,8 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
 	snapshot->replay_size = lrc->replay_size;
 	snapshot->lrc_snapshot = NULL;
 	snapshot->ctx_timestamp = xe_lrc_ctx_timestamp(lrc);
+	snapshot->ctx_timestamp_ms =
+		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_ctx_timestamp(lrc));
 	snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);
 	return snapshot;
 }
@@ -2516,6 +2519,7 @@ void xe_lrc_snapshot_print(struct xe_lrc_snapshot *snapshot, struct drm_printer
 	drm_printf(p, "\tStart seqno: (memory) %d\n", snapshot->start_seqno);
 	drm_printf(p, "\tSeqno: (memory) %d\n", snapshot->seqno);
 	drm_printf(p, "\tTimestamp: 0x%016llx\n", snapshot->ctx_timestamp);
+	drm_printf(p, "\tTimestamp ms: %llu\n", snapshot->ctx_timestamp_ms);
 	drm_printf(p, "\tJob Timestamp: 0x%08x\n", snapshot->ctx_job_timestamp);
 
 	if (!snapshot->lrc_snapshot)
diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
index bef7af9a578b..d280e6572398 100644
--- a/drivers/gpu/drm/xe/xe_lrc.h
+++ b/drivers/gpu/drm/xe/xe_lrc.h
@@ -38,6 +38,7 @@ struct xe_lrc_snapshot {
 	u32 start_seqno;
 	u32 seqno;
 	u64 ctx_timestamp;
+	u64 ctx_timestamp_ms;
 	u32 ctx_job_timestamp;
 };
 
-- 
2.51.0

[PATCH v3 03/11] drm/xe/lrc: Refactor xe_lrc_timestamp to simplify logic

[Umesh Nerlige Ramappa]

Use a context_active() helper and simplify the timestamp logic.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
 drivers/gpu/drm/xe/xe_lrc.c | 35 +++++++++++++----------------------
 1 file changed, 13 insertions(+), 22 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index 66682a678cc2..3b6990d68a5a 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -2585,6 +2585,11 @@ static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
 	return 0;
 }
 
+static bool context_active(struct xe_lrc *lrc)
+{
+	return xe_lrc_ctx_timestamp(lrc) == CONTEXT_ACTIVE;
+}
+
 /**
  * xe_lrc_timestamp() - Current ctx timestamp
  * @lrc: Pointer to the lrc.
@@ -2597,33 +2602,19 @@ static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
  */
 u64 xe_lrc_timestamp(struct xe_lrc *lrc)
 {
-	u64 lrc_ts, reg_ts, new_ts = lrc->ctx_timestamp;
-	u32 engine_id;
+	u64 reg_ts, new_ts = lrc->ctx_timestamp;
 
-	lrc_ts = xe_lrc_ctx_timestamp(lrc);
 	/* CTX_TIMESTAMP mmio read is invalid on VF, so return the LRC value */
-	if (IS_SRIOV_VF(lrc_to_xe(lrc))) {
-		new_ts = lrc_ts;
-		goto done;
-	}
+	if (IS_SRIOV_VF(lrc_to_xe(lrc)))
+		return xe_lrc_ctx_timestamp(lrc);
 
-	if (lrc_ts == CONTEXT_ACTIVE) {
-		engine_id = xe_lrc_engine_id(lrc);
-		if (!get_ctx_timestamp(lrc, engine_id, &reg_ts))
-			new_ts = reg_ts;
+	if (context_active(lrc) &&
+	    !get_ctx_timestamp(lrc, xe_lrc_engine_id(lrc), &reg_ts))
+		new_ts = reg_ts;
 
-		/* read lrc again to ensure context is still active */
-		lrc_ts = xe_lrc_ctx_timestamp(lrc);
-	}
-
-	/*
-	 * If context switched out, just use the lrc_ts. Note that this needs to
-	 * be a separate if condition.
-	 */
-	if (lrc_ts != CONTEXT_ACTIVE)
-		new_ts = lrc_ts;
+	if (!context_active(lrc))
+		return xe_lrc_ctx_timestamp(lrc);
 
-done:
 	return new_ts;
 }
 
-- 
2.51.0

[PATCH v3 04/11] drm/xe/multi_queue: Refactor check for multi queue support for engine class

[Umesh Nerlige Ramappa]

xe exec queue code is using a check to see if a class of engines support
multi queue. This check is also needed by other code, so move it to
xe_gt and export it for others.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
 drivers/gpu/drm/xe/xe_exec_queue.c |  7 +------
 drivers/gpu/drm/xe/xe_gt.c         | 14 ++++++++++++++
 drivers/gpu/drm/xe/xe_gt.h         |  3 +++
 3 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_exec_queue.c b/drivers/gpu/drm/xe/xe_exec_queue.c
index 071b8c41df43..9b36ff5b09a6 100644
--- a/drivers/gpu/drm/xe/xe_exec_queue.c
+++ b/drivers/gpu/drm/xe/xe_exec_queue.c
@@ -852,11 +852,6 @@ static int xe_exec_queue_group_init(struct xe_device *xe, struct xe_exec_queue *
 	return 0;
 }
 
-static inline bool xe_exec_queue_supports_multi_queue(struct xe_exec_queue *q)
-{
-	return q->gt->info.multi_queue_engine_class_mask & BIT(q->class);
-}
-
 static int xe_exec_queue_group_validate(struct xe_device *xe, struct xe_exec_queue *q,
 					u32 primary_id)
 {
@@ -931,7 +926,7 @@ static void xe_exec_queue_group_delete(struct xe_device *xe, struct xe_exec_queu
 static int exec_queue_set_multi_group(struct xe_device *xe, struct xe_exec_queue *q,
 				      u64 value)
 {
-	if (XE_IOCTL_DBG(xe, !xe_exec_queue_supports_multi_queue(q)))
+	if (XE_IOCTL_DBG(xe, !xe_gt_hwe_class_supports_multi_queue(q->gt, q->class)))
 		return -ENODEV;
 
 	if (XE_IOCTL_DBG(xe, !xe_device_uc_enabled(xe)))
diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
index cdc678d1ae1f..0b49252e2001 100644
--- a/drivers/gpu/drm/xe/xe_gt.c
+++ b/drivers/gpu/drm/xe/xe_gt.c
@@ -1170,6 +1170,20 @@ struct xe_hw_engine *xe_gt_any_hw_engine(struct xe_gt *gt)
 	return NULL;
 }
 
+/**
+ * xe_gt_hwe_class_supports_multi_queue() - Check hwe class supports multi queue.
+ *
+ * @gt: the GT object
+ * @class: hwe class type
+ *
+ * Return: true if the hw engine class supports multi queue, else false
+ */
+bool xe_gt_hwe_class_supports_multi_queue(const struct xe_gt *gt,
+					  enum xe_engine_class class)
+{
+	return gt->info.multi_queue_engine_class_mask & BIT(class);
+}
+
 /**
  * xe_gt_declare_wedged() - Declare GT wedged
  * @gt: the GT object
diff --git a/drivers/gpu/drm/xe/xe_gt.h b/drivers/gpu/drm/xe/xe_gt.h
index de7e47763411..1972ab01e716 100644
--- a/drivers/gpu/drm/xe/xe_gt.h
+++ b/drivers/gpu/drm/xe/xe_gt.h
@@ -155,4 +155,7 @@ static inline bool xe_gt_recovery_pending(struct xe_gt *gt)
 		xe_gt_sriov_vf_recovery_pending(gt);
 }
 
+bool xe_gt_hwe_class_supports_multi_queue(const struct xe_gt *gt,
+					  enum xe_engine_class class);
+
 #endif
-- 
2.51.0

[PATCH v3 05/11] drm/xe/multi_queue: Store primary LRC and position info in LRC

[Umesh Nerlige Ramappa]

Given an LRC belonging to the secondary queue, in order to check if its
context group is active, we need to check the LRC of the primary queue.
In addition to that we want to compare the secondary queue position to
CSMQDEBUG register to check if the queue itself is active.

To do so, store primary LRC and position information in the LRC as well
as take a reference to the primary LRC from each LRC in the queue group.

A note on references involved:

- In general the Queue takes a ref on its LRC.
- In addition, for multi-queue,
a. Primary Queue takes a ref for each Secondary LRC.
b. Each Secondary Queue takes a ref to the Primary Queue

In the current patch, each LRC in the queue group is storing a pointer
to Primary LRC. Both Primary and secondary LRCs are freed only when
primary queue is destroyed. At this time there all secondary queues are
already destroyed, so there is no one using secondary LRCs. We should be
good without taking any additional references.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
v2:
- Store primary LRC instead of primary queue (Niranjana)
- Drop the valid flag and check if primary_lrc is NULL (Niranjana)
- Document/Revisit references (Matt/Umesh)

v3:
- Drop the reference logic since it's not needed (Niranjana)
- Move lrc->multi_queue initialization to a later point (Niranjana)
---
 drivers/gpu/drm/xe/xe_exec_queue.c | 5 +++++
 drivers/gpu/drm/xe/xe_lrc.h        | 5 +++++
 drivers/gpu/drm/xe/xe_lrc_types.h  | 8 ++++++++
 3 files changed, 18 insertions(+)

diff --git a/drivers/gpu/drm/xe/xe_exec_queue.c b/drivers/gpu/drm/xe/xe_exec_queue.c
index 9b36ff5b09a6..bc8110a9b054 100644
--- a/drivers/gpu/drm/xe/xe_exec_queue.c
+++ b/drivers/gpu/drm/xe/xe_exec_queue.c
@@ -1389,6 +1389,11 @@ int xe_exec_queue_create_ioctl(struct drm_device *dev, void *data,
 				goto put_exec_queue;
 		}
 
+		if (xe_exec_queue_is_multi_queue(q)) {
+			q->lrc[0]->multi_queue.pos = q->multi_queue.pos;
+			q->lrc[0]->multi_queue.primary_lrc = q->multi_queue.group->primary->lrc[0];
+		}
+
 		if (xe_vm_in_preempt_fence_mode(vm)) {
 			q->lr.context = dma_fence_context_alloc(1);
 
diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
index d280e6572398..557dce004d48 100644
--- a/drivers/gpu/drm/xe/xe_lrc.h
+++ b/drivers/gpu/drm/xe/xe_lrc.h
@@ -91,6 +91,11 @@ static inline size_t xe_lrc_ring_size(void)
 	return SZ_16K;
 }
 
+static inline bool xe_lrc_is_multi_queue(struct xe_lrc *lrc)
+{
+	return lrc->multi_queue.primary_lrc;
+}
+
 size_t xe_gt_lrc_hang_replay_size(struct xe_gt *gt, enum xe_engine_class class);
 size_t xe_gt_lrc_size(struct xe_gt *gt, enum xe_engine_class class);
 u32 xe_lrc_pphwsp_offset(struct xe_lrc *lrc);
diff --git a/drivers/gpu/drm/xe/xe_lrc_types.h b/drivers/gpu/drm/xe/xe_lrc_types.h
index 5a718f759ed6..0a5c13ec2ad7 100644
--- a/drivers/gpu/drm/xe/xe_lrc_types.h
+++ b/drivers/gpu/drm/xe/xe_lrc_types.h
@@ -63,6 +63,14 @@ struct xe_lrc {
 
 	/** @ctx_timestamp: readout value of CTX_TIMESTAMP on last update */
 	u64 ctx_timestamp;
+
+	/** @multi_queue: Multi queue LRC related information */
+	struct {
+		/** @multi_queue.primary_lrc: Primary lrc of this multi-queue group*/
+		struct xe_lrc *primary_lrc;
+		/** @multi_queue.pos: Position of LRC within the multi-queue group */
+		u8 pos;
+	} multi_queue;
 };
 
 struct xe_lrc_snapshot;
-- 
2.51.0

[PATCH v3 06/11] drm/xe/multi_queue: Add helpers to access CS QUEUE TIMESTAMP from lrc

[Umesh Nerlige Ramappa]

In secondary queue LRCs, the QUEUE TIMESTAMP register is saved and
restored allowing us to view the individual queue run times. Add helpers
to read this value from the LRC.

BSpec: 73988

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
v2: (Matt)
- Add BSpec reference
- Make queue_timestamp snapshot 64 bit
- Add a snapshot of queue timestamp in ms

v3:
- Replace xe_lrc_is_multi_queue check in lrc init (Niranajana)
- Use assert instead of check in xe_lrc_queue_timestamp
---
 drivers/gpu/drm/xe/regs/xe_lrc_layout.h |  3 ++
 drivers/gpu/drm/xe/xe_lrc.c             | 46 +++++++++++++++++++++++++
 drivers/gpu/drm/xe/xe_lrc.h             |  2 ++
 drivers/gpu/drm/xe/xe_lrc_types.h       |  3 ++
 4 files changed, 54 insertions(+)

diff --git a/drivers/gpu/drm/xe/regs/xe_lrc_layout.h b/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
index b5eff383902c..4ab86fc369fd 100644
--- a/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
+++ b/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
@@ -34,6 +34,9 @@
 #define CTX_CS_INT_VEC_REG		0x5a
 #define CTX_CS_INT_VEC_DATA		(CTX_CS_INT_VEC_REG + 1)
 
+#define CTX_QUEUE_TIMESTAMP		(0xd0 + 1)
+#define CTX_QUEUE_TIMESTAMP_UDW		(0xd2 + 1)
+
 #define INDIRECT_CTX_RING_HEAD		(0x02 + 1)
 #define INDIRECT_CTX_RING_TAIL		(0x04 + 1)
 #define INDIRECT_CTX_RING_START		(0x06 + 1)
diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index 3b6990d68a5a..127a2ebc3679 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -777,6 +777,16 @@ static u32 __xe_lrc_ctx_timestamp_udw_offset(struct xe_lrc *lrc)
 	return __xe_lrc_regs_offset(lrc) + CTX_TIMESTAMP_UDW * sizeof(u32);
 }
 
+static u32 __xe_lrc_queue_timestamp_offset(struct xe_lrc *lrc)
+{
+	return __xe_lrc_regs_offset(lrc) + CTX_QUEUE_TIMESTAMP * sizeof(u32);
+}
+
+static u32 __xe_lrc_queue_timestamp_udw_offset(struct xe_lrc *lrc)
+{
+	return __xe_lrc_regs_offset(lrc) + CTX_QUEUE_TIMESTAMP_UDW * sizeof(u32);
+}
+
 static inline u32 __xe_lrc_indirect_ring_offset(struct xe_lrc *lrc)
 {
 	u32 offset = xe_bo_size(lrc->bo) - LRC_WA_BB_SIZE -
@@ -826,6 +836,8 @@ DECL_MAP_ADDR_HELPERS(ctx_timestamp_udw, lrc->bo)
 DECL_MAP_ADDR_HELPERS(parallel, lrc->bo)
 DECL_MAP_ADDR_HELPERS(indirect_ring, lrc->bo)
 DECL_MAP_ADDR_HELPERS(engine_id, lrc->bo)
+DECL_MAP_ADDR_HELPERS(queue_timestamp, lrc->bo)
+DECL_MAP_ADDR_HELPERS(queue_timestamp_udw, lrc->bo)
 
 #undef DECL_MAP_ADDR_HELPERS
 
@@ -874,6 +886,29 @@ static u64 xe_lrc_ctx_timestamp(struct xe_lrc *lrc)
 	return (u64)udw << 32 | ldw;
 }
 
+/**
+ * xe_lrc_queue_timestamp() - Read queue timestamp value
+ * @lrc: Pointer to the lrc.
+ *
+ * Returns: queue timestamp value
+ */
+static u64 xe_lrc_queue_timestamp(struct xe_lrc *lrc)
+{
+	struct xe_device *xe = lrc_to_xe(lrc);
+	struct iosys_map map;
+	u32 ldw, udw = 0;
+
+	xe_assert(xe, xe_lrc_is_multi_queue(lrc));
+
+	map = __xe_lrc_queue_timestamp_map(lrc);
+	ldw = xe_map_read32(xe, &map);
+
+	map = __xe_lrc_queue_timestamp_udw_map(lrc);
+	udw = xe_map_read32(xe, &map);
+
+	return (u64)udw << 32 | ldw;
+}
+
 /**
  * xe_lrc_ctx_job_timestamp_ggtt_addr() - Get ctx job timestamp GGTT address
  * @lrc: Pointer to the lrc.
@@ -1538,6 +1573,12 @@ static int xe_lrc_ctx_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe, struct
 	if (lrc_to_xe(lrc)->info.has_64bit_timestamp)
 		xe_lrc_write_ctx_reg(lrc, CTX_TIMESTAMP_UDW, 0);
 
+	if (xe_gt_hwe_class_supports_multi_queue(gt, hwe->class)) {
+		lrc->queue_timestamp = 0;
+		xe_lrc_write_ctx_reg(lrc, CTX_QUEUE_TIMESTAMP, 0);
+		xe_lrc_write_ctx_reg(lrc, CTX_QUEUE_TIMESTAMP_UDW, 0);
+	}
+
 	if (xe->info.has_asid && vm)
 		xe_lrc_write_ctx_reg(lrc, CTX_ASID, vm->usm.asid);
 
@@ -2466,6 +2507,9 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
 	snapshot->ctx_timestamp = xe_lrc_ctx_timestamp(lrc);
 	snapshot->ctx_timestamp_ms =
 		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_ctx_timestamp(lrc));
+	snapshot->queue_timestamp = xe_lrc_queue_timestamp(lrc);
+	snapshot->queue_timestamp_ms =
+		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_queue_timestamp(lrc));
 	snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);
 	return snapshot;
 }
@@ -2520,6 +2564,8 @@ void xe_lrc_snapshot_print(struct xe_lrc_snapshot *snapshot, struct drm_printer
 	drm_printf(p, "\tSeqno: (memory) %d\n", snapshot->seqno);
 	drm_printf(p, "\tTimestamp: 0x%016llx\n", snapshot->ctx_timestamp);
 	drm_printf(p, "\tTimestamp ms: %llu\n", snapshot->ctx_timestamp_ms);
+	drm_printf(p, "\tQueue Timestamp: 0x%016llx\n", snapshot->queue_timestamp);
+	drm_printf(p, "\tQueue Timestamp ms: %llu\n", snapshot->queue_timestamp_ms);
 	drm_printf(p, "\tJob Timestamp: 0x%08x\n", snapshot->ctx_job_timestamp);
 
 	if (!snapshot->lrc_snapshot)
diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
index 557dce004d48..0a3a611391ee 100644
--- a/drivers/gpu/drm/xe/xe_lrc.h
+++ b/drivers/gpu/drm/xe/xe_lrc.h
@@ -39,6 +39,8 @@ struct xe_lrc_snapshot {
 	u32 seqno;
 	u64 ctx_timestamp;
 	u64 ctx_timestamp_ms;
+	u64 queue_timestamp;
+	u64 queue_timestamp_ms;
 	u32 ctx_job_timestamp;
 };
 
diff --git a/drivers/gpu/drm/xe/xe_lrc_types.h b/drivers/gpu/drm/xe/xe_lrc_types.h
index 0a5c13ec2ad7..53ef48feebfc 100644
--- a/drivers/gpu/drm/xe/xe_lrc_types.h
+++ b/drivers/gpu/drm/xe/xe_lrc_types.h
@@ -64,6 +64,9 @@ struct xe_lrc {
 	/** @ctx_timestamp: readout value of CTX_TIMESTAMP on last update */
 	u64 ctx_timestamp;
 
+	/** @queue_timestamp: value of QUEUE_TIMESTAMP on last update */
+	u64 queue_timestamp;
+
 	/** @multi_queue: Multi queue LRC related information */
 	struct {
 		/** @multi_queue.primary_lrc: Primary lrc of this multi-queue group*/
-- 
2.51.0

[PATCH v3 07/11] drm/xe/lrc: Refactor out engine id to hwe conversion

[Umesh Nerlige Ramappa]

We need to define more helpers that read engine ID specific register, so
move that logic outside of get_ctx_timestamp().

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
---
 drivers/gpu/drm/xe/xe_lrc.c | 20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index 127a2ebc3679..e7f7f0efcf2d 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -2606,17 +2606,27 @@ void xe_lrc_snapshot_free(struct xe_lrc_snapshot *snapshot)
 	kfree(snapshot);
 }
 
-static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
+static struct xe_hw_engine *engine_id_to_hwe(struct xe_gt *gt, u32 engine_id)
 {
 	u16 class = REG_FIELD_GET(ENGINE_CLASS_ID, engine_id);
 	u16 instance = REG_FIELD_GET(ENGINE_INSTANCE_ID, engine_id);
+	struct xe_hw_engine *hwe = xe_gt_hw_engine(gt, class, instance, false);
+
+	if (xe_gt_WARN_ONCE(gt, !hwe || xe_hw_engine_is_reserved(hwe),
+			    "Unexpected engine class:instance %d:%d for utilization\n",
+			    class, instance))
+		return NULL;
+
+	return hwe;
+}
+
+static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
+{
 	struct xe_hw_engine *hwe;
 	u64 val;
 
-	hwe = xe_gt_hw_engine(lrc->gt, class, instance, false);
-	if (xe_gt_WARN_ONCE(lrc->gt, !hwe || xe_hw_engine_is_reserved(hwe),
-			    "Unexpected engine class:instance %d:%d for context utilization\n",
-			    class, instance))
+	hwe = engine_id_to_hwe(lrc->gt, engine_id);
+	if (!hwe)
 		return -1;
 
 	if (lrc_to_xe(lrc)->info.has_64bit_timestamp)
-- 
2.51.0

[PATCH v3 08/11] drm/xe/multi_queue: Capture queue run times for active queues

[Umesh Nerlige Ramappa]

If a queue is currently active on the CS, query the QUEUE TIMESTAMP
register to get an up to date value of the runtime. To do so, ensure
that the primary queue is active and then check if the secondary queue
is executing on the CS.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
v2:
- Move trace to a separate patch (Stuart)
- Refactor multi queue timestamp logic (Matt/Niranjana)

v3: (Niranjana)
- s/get_queue_id/get_multi_queue_active_queue_id/
- assert for primary_lrc in xe_lrc_multi_queue_timestamp
- don't use the name single_queue
---
 drivers/gpu/drm/xe/regs/xe_engine_regs.h |   4 +
 drivers/gpu/drm/xe/xe_lrc.c              | 113 +++++++++++++++++++----
 2 files changed, 97 insertions(+), 20 deletions(-)

diff --git a/drivers/gpu/drm/xe/regs/xe_engine_regs.h b/drivers/gpu/drm/xe/regs/xe_engine_regs.h
index c4c879a9e555..94033982e694 100644
--- a/drivers/gpu/drm/xe/regs/xe_engine_regs.h
+++ b/drivers/gpu/drm/xe/regs/xe_engine_regs.h
@@ -170,6 +170,10 @@
 #define   GFX_DISABLE_LEGACY_MODE		REG_BIT(3)
 
 #define RING_CSMQDEBUG(base)			XE_REG((base) + 0x2b0)
+#define   CURRENT_ACTIVE_QUEUE_ID_MASK		REG_GENMASK(7, 0)
+
+#define RING_QUEUE_TIMESTAMP(base)		XE_REG((base) + 0x4c0)
+#define RING_QUEUE_TIMESTAMP_UDW(base)		XE_REG((base) + 0x4c0 + 4)
 
 #define RING_TIMESTAMP(base)			XE_REG((base) + 0x358)
 
diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index e7f7f0efcf2d..28060a3d4f11 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -21,6 +21,7 @@
 #include "xe_configfs.h"
 #include "xe_device.h"
 #include "xe_drm_client.h"
+#include "xe_exec_queue.h"
 #include "xe_exec_queue_types.h"
 #include "xe_gt.h"
 #include "xe_gt_clock.h"
@@ -2641,22 +2642,68 @@ static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
 	return 0;
 }
 
+static u64 get_queue_timestamp(struct xe_hw_engine *hwe)
+{
+	return xe_mmio_read64_2x32(&hwe->gt->mmio,
+				   RING_QUEUE_TIMESTAMP(hwe->mmio_base));
+}
+
+static u32 get_multi_queue_active_queue_id(struct xe_hw_engine *hwe)
+{
+	u32 val = xe_mmio_read32(&hwe->gt->mmio,
+				 RING_CSMQDEBUG(hwe->mmio_base));
+
+	return REG_FIELD_GET(CURRENT_ACTIVE_QUEUE_ID_MASK, val);
+}
+
 static bool context_active(struct xe_lrc *lrc)
 {
 	return xe_lrc_ctx_timestamp(lrc) == CONTEXT_ACTIVE;
 }
 
-/**
- * xe_lrc_timestamp() - Current ctx timestamp
- * @lrc: Pointer to the lrc.
- *
- * Return latest ctx timestamp. With support for active contexts, the
- * calculation may be slightly racy, so follow a read-again logic to ensure that
- * the context is still active before returning the right timestamp.
- *
- * Returns: New ctx timestamp value
- */
-u64 xe_lrc_timestamp(struct xe_lrc *lrc)
+static u64 xe_lrc_multi_queue_timestamp(struct xe_lrc *lrc)
+{
+	struct xe_device *xe = lrc_to_xe(lrc);
+	struct xe_lrc *primary_lrc = lrc->multi_queue.primary_lrc;
+	struct xe_hw_engine *hwe;
+	u64 reg_queue_ts = lrc->queue_timestamp;
+
+	if (IS_SRIOV_VF(xe))
+		return xe_lrc_queue_timestamp(lrc);
+
+	xe_assert(xe, primary_lrc);
+
+	if (!context_active(primary_lrc))
+		return xe_lrc_queue_timestamp(lrc);
+
+	/* WA BB populates engine id in PPHWSP of primary context only */
+	hwe = engine_id_to_hwe(primary_lrc->gt, xe_lrc_engine_id(primary_lrc));
+	if (!hwe)
+		return xe_lrc_queue_timestamp(lrc);
+
+	if (get_multi_queue_active_queue_id(hwe) != lrc->multi_queue.pos)
+		return xe_lrc_queue_timestamp(lrc);
+
+	/* queue is active, so store the queue timestamp register */
+	reg_queue_ts = get_queue_timestamp(hwe);
+
+	/* double check queue and primary queue are both still active */
+	if (get_multi_queue_active_queue_id(hwe) != lrc->multi_queue.pos ||
+	    !context_active(primary_lrc))
+		return xe_lrc_queue_timestamp(lrc);
+
+	return reg_queue_ts;
+}
+
+static u64 xe_lrc_update_multi_queue_timestamp(struct xe_lrc *lrc, u64 *old_ts)
+{
+	*old_ts = lrc->queue_timestamp;
+	lrc->queue_timestamp = xe_lrc_multi_queue_timestamp(lrc);
+
+	return lrc->queue_timestamp;
+}
+
+static u64 xe_lrc_context_timestamp(struct xe_lrc *lrc)
 {
 	u64 reg_ts, new_ts = lrc->ctx_timestamp;
 
@@ -2674,24 +2721,50 @@ u64 xe_lrc_timestamp(struct xe_lrc *lrc)
 	return new_ts;
 }
 
+static u64 xe_lrc_update_context_timestamp(struct xe_lrc *lrc, u64 *old_ts)
+{
+	*old_ts = lrc->ctx_timestamp;
+	lrc->ctx_timestamp = xe_lrc_context_timestamp(lrc);
+
+	trace_xe_lrc_update_timestamp(lrc, *old_ts);
+
+	return lrc->ctx_timestamp;
+}
+
+/**
+ * xe_lrc_timestamp() - Current lrc timestamp
+ * @lrc: Pointer to the lrc.
+ *
+ * Return latest lrc timestamp. With support for active contexts/queues, the
+ * calculation may be slightly racy, so follow a read-again logic to ensure that
+ * the context/queue is still active before returning the right timestamp.
+ *
+ * Returns: New lrc timestamp value
+ */
+u64 xe_lrc_timestamp(struct xe_lrc *lrc)
+{
+	if (xe_lrc_is_multi_queue(lrc))
+		return xe_lrc_multi_queue_timestamp(lrc);
+	else
+		return xe_lrc_context_timestamp(lrc);
+}
+
 /**
- * xe_lrc_update_timestamp() - Update ctx timestamp
+ * xe_lrc_update_timestamp() - Update lrc timestamp
  * @lrc: Pointer to the lrc.
  * @old_ts: Old timestamp value
  *
- * Populate @old_ts current saved ctx timestamp, read new ctx timestamp and
+ * Populate @old_ts with current saved lrc timestamp, read new lrc timestamp and
  * update saved value.
  *
- * Returns: New ctx timestamp value
+ * Returns: New lrc timestamp value
  */
 u64 xe_lrc_update_timestamp(struct xe_lrc *lrc, u64 *old_ts)
 {
-	*old_ts = lrc->ctx_timestamp;
-	lrc->ctx_timestamp = xe_lrc_timestamp(lrc);
-
-	trace_xe_lrc_update_timestamp(lrc, *old_ts);
-
-	return lrc->ctx_timestamp;
+	if (xe_lrc_is_multi_queue(lrc))
+		return xe_lrc_update_multi_queue_timestamp(lrc, old_ts);
+	else
+		return xe_lrc_update_context_timestamp(lrc, old_ts);
 }
 
 /**
-- 
2.51.0

[PATCH v3 09/11] drm/xe/multi_queue: Add trace event for the multi queue timestamp

[Umesh Nerlige Ramappa]

Add a trace event for multi queue timestamp capture.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
v2:
- Split traces from original patch (Stuart)
- Print primary lrc in the trace (Niranjana)

v3:
- Include primary_lrc as well as lrc in the trace (Niranjana)
---
 drivers/gpu/drm/xe/xe_lrc.c       |  2 ++
 drivers/gpu/drm/xe/xe_trace_lrc.h | 29 +++++++++++++++++++++++++++++
 2 files changed, 31 insertions(+)

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index 28060a3d4f11..ea65160a582a 100644
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -2700,6 +2700,8 @@ static u64 xe_lrc_update_multi_queue_timestamp(struct xe_lrc *lrc, u64 *old_ts)
 	*old_ts = lrc->queue_timestamp;
 	lrc->queue_timestamp = xe_lrc_multi_queue_timestamp(lrc);
 
+	trace_xe_lrc_update_queue_timestamp(lrc, *old_ts);
+
 	return lrc->queue_timestamp;
 }
 
diff --git a/drivers/gpu/drm/xe/xe_trace_lrc.h b/drivers/gpu/drm/xe/xe_trace_lrc.h
index d525cbee1e34..4f3c1a0de573 100644
--- a/drivers/gpu/drm/xe/xe_trace_lrc.h
+++ b/drivers/gpu/drm/xe/xe_trace_lrc.h
@@ -12,6 +12,7 @@
 #include <linux/tracepoint.h>
 #include <linux/types.h>
 
+#include "xe_exec_queue_types.h"
 #include "xe_gt_types.h"
 #include "xe_lrc.h"
 #include "xe_lrc_types.h"
@@ -42,6 +43,34 @@ TRACE_EVENT(xe_lrc_update_timestamp,
 		      __get_str(device_id))
 );
 
+TRACE_EVENT(xe_lrc_update_queue_timestamp,
+	    TP_PROTO(struct xe_lrc *lrc, uint64_t old),
+	    TP_ARGS(lrc, old),
+	    TP_STRUCT__entry(
+		     __field(struct xe_lrc *, lrc)
+		     __field(struct xe_lrc *, primary_lrc)
+		     __field(u8, pos)
+		     __field(u64, old)
+		     __field(u64, new)
+		     __string(name, lrc->fence_ctx.name)
+		     __string(device_id, __dev_name_lrc(lrc))
+		     ),
+
+	    TP_fast_assign(
+		   __entry->lrc = lrc;
+		   __entry->primary_lrc = lrc->multi_queue.primary_lrc;
+		   __entry->pos = lrc->multi_queue.pos;
+		   __entry->old = old;
+		   __entry->new = lrc->queue_timestamp;
+		   __assign_str(name);
+		   __assign_str(device_id);
+		   ),
+	    TP_printk("lrc=:%p primary_lrc:%p pos=%d lrc->name=%s old=%llu new=%llu device_id:%s",
+		      __entry->lrc, __entry->primary_lrc, __entry->pos, __get_str(name),
+		      __entry->old, __entry->new,
+		      __get_str(device_id))
+);
+
 #endif
 
 /* This part must be outside protection */
-- 
2.51.0

[PATCH v3 10/11] drm/xe/multi_queue: Use QUEUE_TIMESTAMP as job timestamp for multi-queue

[Umesh Nerlige Ramappa]

Each queue in a multi queue group has a dedicated timestamp counter. Use
this QUEUE TIMESTAMP register to capture the start timestamp for the
job.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
---
v2: Use xe_lrc_is_multi_queue for check (Niranjana)
---
 drivers/gpu/drm/xe/xe_ring_ops.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_ring_ops.c b/drivers/gpu/drm/xe/xe_ring_ops.c
index cfeb4fc7d217..39a670e91ba7 100644
--- a/drivers/gpu/drm/xe/xe_ring_ops.c
+++ b/drivers/gpu/drm/xe/xe_ring_ops.c
@@ -269,8 +269,12 @@ static u32 get_ppgtt_flag(struct xe_sched_job *job)
 static int emit_copy_timestamp(struct xe_device *xe, struct xe_lrc *lrc,
 			       u32 *dw, int i)
 {
+	const struct xe_reg reg = xe_lrc_is_multi_queue(lrc) ?
+				   RING_QUEUE_TIMESTAMP(0) :
+				   RING_CTX_TIMESTAMP(0);
+
 	dw[i++] = MI_STORE_REGISTER_MEM | MI_SRM_USE_GGTT | MI_SRM_ADD_CS_OFFSET;
-	dw[i++] = RING_CTX_TIMESTAMP(0).addr;
+	dw[i++] = reg.addr;
 	dw[i++] = xe_lrc_ctx_job_timestamp_ggtt_addr(lrc);
 	dw[i++] = 0;
 
@@ -281,7 +285,7 @@ static int emit_copy_timestamp(struct xe_device *xe, struct xe_lrc *lrc,
 	if (IS_SRIOV_VF(xe)) {
 		dw[i++] = MI_STORE_REGISTER_MEM | MI_SRM_USE_GGTT |
 			MI_SRM_ADD_CS_OFFSET;
-		dw[i++] = RING_CTX_TIMESTAMP(0).addr;
+		dw[i++] = reg.addr;
 		dw[i++] = xe_lrc_ctx_timestamp_ggtt_addr(lrc);
 		dw[i++] = 0;
 	}
-- 
2.51.0

[PATCH v3 11/11] drm/xe/multi_queue: Whitelist QUEUE_TIMESTAMP register

[Umesh Nerlige Ramappa]

In a multi-queue use case, when a job is running on the secondary queue,
the CTX_TIMESTAMP does not reflect the queues run ticks. Instead, we use
the QUEUE TIMESTAMP to check how long the job ran. For user space to see
the run ticks for a secondary queue, whitelist the QUEUE_TIMESTAMP
register.

Compute PR: https://github.com/intel/compute-runtime/pull/923

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
---
v2: Whitelist QUEUE_TIMESTAMP only for copy and compute engines (Niranjana)
v3: (Niranjana)
- s/multiq/multi_queue/
- Base the condition on multi_queue_engine_class_mask
---
 drivers/gpu/drm/xe/xe_reg_whitelist.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/drivers/gpu/drm/xe/xe_reg_whitelist.c b/drivers/gpu/drm/xe/xe_reg_whitelist.c
index 8cc313182968..6f96bf84df66 100644
--- a/drivers/gpu/drm/xe/xe_reg_whitelist.c
+++ b/drivers/gpu/drm/xe/xe_reg_whitelist.c
@@ -9,6 +9,7 @@
 #include "regs/xe_gt_regs.h"
 #include "regs/xe_oa_regs.h"
 #include "xe_device.h"
+#include "xe_gt.h"
 #include "xe_gt_types.h"
 #include "xe_gt_printk.h"
 #include "xe_platform_types.h"
@@ -33,6 +34,13 @@ static bool match_has_mert(const struct xe_device *xe,
 	return xe_device_has_mert((struct xe_device *)xe);
 }
 
+static bool match_multi_queue_class(const struct xe_device *xe,
+				    const struct xe_gt *gt,
+				    const struct xe_hw_engine *hwe)
+{
+	return xe_gt_hwe_class_supports_multi_queue(gt, hwe->class);
+}
+
 static const struct xe_rtp_entry_sr register_whitelist[] = {
 	{ XE_RTP_NAME("WaAllowPMDepthAndInvocationCountAccessFromUMD, 1408556865"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(1200, 1210), ENGINE_CLASS(RENDER)),
@@ -54,6 +62,12 @@ static const struct xe_rtp_entry_sr register_whitelist[] = {
 				RING_FORCE_TO_NONPRIV_ACCESS_RD,
 				XE_RTP_ACTION_FLAG(ENGINE_BASE)))
 	},
+	{ XE_RTP_NAME("allow_read_queue_timestamp"),
+	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3500, 3511), FUNC(match_multi_queue_class)),
+	  XE_RTP_ACTIONS(WHITELIST(RING_QUEUE_TIMESTAMP(0),
+				   RING_FORCE_TO_NONPRIV_ACCESS_RD,
+				   XE_RTP_ACTION_FLAG(ENGINE_BASE)))
+	},
 	{ XE_RTP_NAME("16014440446"),
 	  XE_RTP_RULES(PLATFORM(PVC)),
 	  XE_RTP_ACTIONS(WHITELIST(XE_REG(0x4400),
-- 
2.51.0

✗ CI.checkpatch: warning for Support run ticks for multi-queue use case (rev3)

[Patchwork]

== Series Details ==

Series: Support run ticks for multi-queue use case (rev3)
URL   : https://patchwork.freedesktop.org/series/164654/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
c8c12e558adaef7a4d125d83b6e1f8824bc13b82
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 49b32612190e93bdd2c71e637e613616e3405922
Author: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Date:   Tue May 5 16:44:20 2026 -0700

    drm/xe/multi_queue: Whitelist QUEUE_TIMESTAMP register
    
    In a multi-queue use case, when a job is running on the secondary queue,
    the CTX_TIMESTAMP does not reflect the queues run ticks. Instead, we use
    the QUEUE TIMESTAMP to check how long the job ran. For user space to see
    the run ticks for a secondary queue, whitelist the QUEUE_TIMESTAMP
    register.
    
    Compute PR: https://github.com/intel/compute-runtime/pull/923
    
    Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
+ /mt/dim checkpatch f95f99492a2826e042e114c91504a3216c42adbd drm-intel
bff14686f275 drm/xe/lrc: Use 64 bit ctx timestamp in the LRC snapshot
988de792d635 drm/xe: Add timestamp_ms to LRC snapshot
4303ee05999c drm/xe/lrc: Refactor xe_lrc_timestamp to simplify logic
c4cb94f36075 drm/xe/multi_queue: Refactor check for multi queue support for engine class
59747380d02a drm/xe/multi_queue: Store primary LRC and position info in LRC
690f2e503e4f drm/xe/multi_queue: Add helpers to access CS QUEUE TIMESTAMP from lrc
e1dc3424d7ef drm/xe/lrc: Refactor out engine id to hwe conversion
30ccdc788606 drm/xe/multi_queue: Capture queue run times for active queues
1a65d24a2374 drm/xe/multi_queue: Add trace event for the multi queue timestamp
-:43: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#43: FILE: drivers/gpu/drm/xe/xe_trace_lrc.h:49:
+	    TP_STRUCT__entry(

-:53: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#53: FILE: drivers/gpu/drm/xe/xe_trace_lrc.h:59:
+	    TP_fast_assign(

total: 0 errors, 0 warnings, 2 checks, 49 lines checked
4fe3dbad2802 drm/xe/multi_queue: Use QUEUE_TIMESTAMP as job timestamp for multi-queue
49b32612190e drm/xe/multi_queue: Whitelist QUEUE_TIMESTAMP register

✓ CI.KUnit: success for Support run ticks for multi-queue use case (rev3)

[Patchwork]

== Series Details ==

Series: Support run ticks for multi-queue use case (rev3)
URL   : https://patchwork.freedesktop.org/series/164654/
State : success

== Summary ==

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

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

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

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

Re: [PATCH v3 02/11] drm/xe: Add timestamp_ms to LRC snapshot

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:11PM -0700, Umesh Nerlige Ramappa wrote:
>From: Matthew Brost <matthew.brost@intel.com>
>
>Add a timestamp in milliseconds to the LRC snapshot to make it easier to
>reason about how long the LRC has been running and the average duration
>of each job.
>
>Signed-off-by: Matthew Brost <matthew.brost@intel.com>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>

LGTM.
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>

>---
>v2: Group u32s to avoid padding (Niranjana)
>---
> drivers/gpu/drm/xe/xe_lrc.c | 4 ++++
> drivers/gpu/drm/xe/xe_lrc.h | 1 +
> 2 files changed, 5 insertions(+)
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>index 570ff9f55849..66682a678cc2 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.c
>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>@@ -23,6 +23,7 @@
> #include "xe_drm_client.h"
> #include "xe_exec_queue_types.h"
> #include "xe_gt.h"
>+#include "xe_gt_clock.h"
> #include "xe_gt_printk.h"
> #include "xe_hw_fence.h"
> #include "xe_map.h"
>@@ -2463,6 +2464,8 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
> 	snapshot->replay_size = lrc->replay_size;
> 	snapshot->lrc_snapshot = NULL;
> 	snapshot->ctx_timestamp = xe_lrc_ctx_timestamp(lrc);
>+	snapshot->ctx_timestamp_ms =
>+		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_ctx_timestamp(lrc));
> 	snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);
> 	return snapshot;
> }
>@@ -2516,6 +2519,7 @@ void xe_lrc_snapshot_print(struct xe_lrc_snapshot *snapshot, struct drm_printer
> 	drm_printf(p, "\tStart seqno: (memory) %d\n", snapshot->start_seqno);
> 	drm_printf(p, "\tSeqno: (memory) %d\n", snapshot->seqno);
> 	drm_printf(p, "\tTimestamp: 0x%016llx\n", snapshot->ctx_timestamp);
>+	drm_printf(p, "\tTimestamp ms: %llu\n", snapshot->ctx_timestamp_ms);
> 	drm_printf(p, "\tJob Timestamp: 0x%08x\n", snapshot->ctx_job_timestamp);
>
> 	if (!snapshot->lrc_snapshot)
>diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
>index bef7af9a578b..d280e6572398 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.h
>+++ b/drivers/gpu/drm/xe/xe_lrc.h
>@@ -38,6 +38,7 @@ struct xe_lrc_snapshot {
> 	u32 start_seqno;
> 	u32 seqno;
> 	u64 ctx_timestamp;
>+	u64 ctx_timestamp_ms;
> 	u32 ctx_job_timestamp;
> };
>
>-- 
>2.51.0
>

Re: [PATCH v3 03/11] drm/xe/lrc: Refactor xe_lrc_timestamp to simplify logic

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:12PM -0700, Umesh Nerlige Ramappa wrote:
>Use a context_active() helper and simplify the timestamp logic.
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
> drivers/gpu/drm/xe/xe_lrc.c | 35 +++++++++++++----------------------
> 1 file changed, 13 insertions(+), 22 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>index 66682a678cc2..3b6990d68a5a 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.c
>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>@@ -2585,6 +2585,11 @@ static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
> 	return 0;
> }
>
>+static bool context_active(struct xe_lrc *lrc)
>+{
>+	return xe_lrc_ctx_timestamp(lrc) == CONTEXT_ACTIVE;
>+}
>+
> /**
>  * xe_lrc_timestamp() - Current ctx timestamp
>  * @lrc: Pointer to the lrc.
>@@ -2597,33 +2602,19 @@ static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
>  */
> u64 xe_lrc_timestamp(struct xe_lrc *lrc)
> {
>-	u64 lrc_ts, reg_ts, new_ts = lrc->ctx_timestamp;
>-	u32 engine_id;
>+	u64 reg_ts, new_ts = lrc->ctx_timestamp;
>
>-	lrc_ts = xe_lrc_ctx_timestamp(lrc);
> 	/* CTX_TIMESTAMP mmio read is invalid on VF, so return the LRC value */
>-	if (IS_SRIOV_VF(lrc_to_xe(lrc))) {
>-		new_ts = lrc_ts;
>-		goto done;
>-	}
>+	if (IS_SRIOV_VF(lrc_to_xe(lrc)))
>+		return xe_lrc_ctx_timestamp(lrc);
>
>-	if (lrc_ts == CONTEXT_ACTIVE) {
>-		engine_id = xe_lrc_engine_id(lrc);
>-		if (!get_ctx_timestamp(lrc, engine_id, &reg_ts))
>-			new_ts = reg_ts;
>+	if (context_active(lrc) &&
>+	    !get_ctx_timestamp(lrc, xe_lrc_engine_id(lrc), &reg_ts))
>+		new_ts = reg_ts;
>
>-		/* read lrc again to ensure context is still active */
>-		lrc_ts = xe_lrc_ctx_timestamp(lrc);
>-	}
>-
>-	/*
>-	 * If context switched out, just use the lrc_ts. Note that this needs to
>-	 * be a separate if condition.
>-	 */
>-	if (lrc_ts != CONTEXT_ACTIVE)
>-		new_ts = lrc_ts;
>+	if (!context_active(lrc))
>+		return xe_lrc_ctx_timestamp(lrc);

NIT....may be it is good to keep a comment here like the one removed above?
Either way, lgtm.
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>

>
>-done:
> 	return new_ts;
> }
>
>-- 
>2.51.0
>

Re: [PATCH v3 04/11] drm/xe/multi_queue: Refactor check for multi queue support for engine class

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:13PM -0700, Umesh Nerlige Ramappa wrote:
>xe exec queue code is using a check to see if a class of engines support
>multi queue. This check is also needed by other code, so move it to
>xe_gt and export it for others.
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
> drivers/gpu/drm/xe/xe_exec_queue.c |  7 +------
> drivers/gpu/drm/xe/xe_gt.c         | 14 ++++++++++++++
> drivers/gpu/drm/xe/xe_gt.h         |  3 +++
> 3 files changed, 18 insertions(+), 6 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_exec_queue.c b/drivers/gpu/drm/xe/xe_exec_queue.c
>index 071b8c41df43..9b36ff5b09a6 100644
>--- a/drivers/gpu/drm/xe/xe_exec_queue.c
>+++ b/drivers/gpu/drm/xe/xe_exec_queue.c
>@@ -852,11 +852,6 @@ static int xe_exec_queue_group_init(struct xe_device *xe, struct xe_exec_queue *
> 	return 0;
> }
>
>-static inline bool xe_exec_queue_supports_multi_queue(struct xe_exec_queue *q)
>-{
>-	return q->gt->info.multi_queue_engine_class_mask & BIT(q->class);
>-}
>-
> static int xe_exec_queue_group_validate(struct xe_device *xe, struct xe_exec_queue *q,
> 					u32 primary_id)
> {
>@@ -931,7 +926,7 @@ static void xe_exec_queue_group_delete(struct xe_device *xe, struct xe_exec_queu
> static int exec_queue_set_multi_group(struct xe_device *xe, struct xe_exec_queue *q,
> 				      u64 value)
> {
>-	if (XE_IOCTL_DBG(xe, !xe_exec_queue_supports_multi_queue(q)))
>+	if (XE_IOCTL_DBG(xe, !xe_gt_hwe_class_supports_multi_queue(q->gt, q->class)))
> 		return -ENODEV;
>
> 	if (XE_IOCTL_DBG(xe, !xe_device_uc_enabled(xe)))
>diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
>index cdc678d1ae1f..0b49252e2001 100644
>--- a/drivers/gpu/drm/xe/xe_gt.c
>+++ b/drivers/gpu/drm/xe/xe_gt.c
>@@ -1170,6 +1170,20 @@ struct xe_hw_engine *xe_gt_any_hw_engine(struct xe_gt *gt)
> 	return NULL;
> }
>
>+/**
>+ * xe_gt_hwe_class_supports_multi_queue() - Check hwe class supports multi queue.

"Check if gt supports multi queue for the specified engine class"

>+ *
>+ * @gt: the GT object
>+ * @class: hwe class type
>+ *
>+ * Return: true if the hw engine class supports multi queue, else false
>+ */
>+bool xe_gt_hwe_class_supports_multi_queue(const struct xe_gt *gt,
>+					  enum xe_engine_class class)
>+{
>+	return gt->info.multi_queue_engine_class_mask & BIT(class);
>+}

We can make it 'static inline' and move it to xe_gt.h (no need to be here).
Also, may be xe_gt_supports_multi_queue(gt, class) would be a good simple
name (don't need hwe_class in the name).

Niranjana

>+
> /**
>  * xe_gt_declare_wedged() - Declare GT wedged
>  * @gt: the GT object
>diff --git a/drivers/gpu/drm/xe/xe_gt.h b/drivers/gpu/drm/xe/xe_gt.h
>index de7e47763411..1972ab01e716 100644
>--- a/drivers/gpu/drm/xe/xe_gt.h
>+++ b/drivers/gpu/drm/xe/xe_gt.h
>@@ -155,4 +155,7 @@ static inline bool xe_gt_recovery_pending(struct xe_gt *gt)
> 		xe_gt_sriov_vf_recovery_pending(gt);
> }
>
>+bool xe_gt_hwe_class_supports_multi_queue(const struct xe_gt *gt,
>+					  enum xe_engine_class class);
>+
> #endif
>-- 
>2.51.0
>

Re: [PATCH v3 06/11] drm/xe/multi_queue: Add helpers to access CS QUEUE TIMESTAMP from lrc

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:15PM -0700, Umesh Nerlige Ramappa wrote:
>In secondary queue LRCs, the QUEUE TIMESTAMP register is saved and
>restored allowing us to view the individual queue run times. Add helpers
>to read this value from the LRC.
>
>BSpec: 73988
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
>v2: (Matt)
>- Add BSpec reference
>- Make queue_timestamp snapshot 64 bit
>- Add a snapshot of queue timestamp in ms
>
>v3:
>- Replace xe_lrc_is_multi_queue check in lrc init (Niranajana)
>- Use assert instead of check in xe_lrc_queue_timestamp
>---
> drivers/gpu/drm/xe/regs/xe_lrc_layout.h |  3 ++
> drivers/gpu/drm/xe/xe_lrc.c             | 46 +++++++++++++++++++++++++
> drivers/gpu/drm/xe/xe_lrc.h             |  2 ++
> drivers/gpu/drm/xe/xe_lrc_types.h       |  3 ++
> 4 files changed, 54 insertions(+)
>
>diff --git a/drivers/gpu/drm/xe/regs/xe_lrc_layout.h b/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
>index b5eff383902c..4ab86fc369fd 100644
>--- a/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
>+++ b/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
>@@ -34,6 +34,9 @@
> #define CTX_CS_INT_VEC_REG		0x5a
> #define CTX_CS_INT_VEC_DATA		(CTX_CS_INT_VEC_REG + 1)
>
>+#define CTX_QUEUE_TIMESTAMP		(0xd0 + 1)
>+#define CTX_QUEUE_TIMESTAMP_UDW		(0xd2 + 1)
>+
> #define INDIRECT_CTX_RING_HEAD		(0x02 + 1)
> #define INDIRECT_CTX_RING_TAIL		(0x04 + 1)
> #define INDIRECT_CTX_RING_START		(0x06 + 1)
>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>index 3b6990d68a5a..127a2ebc3679 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.c
>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>@@ -777,6 +777,16 @@ static u32 __xe_lrc_ctx_timestamp_udw_offset(struct xe_lrc *lrc)
> 	return __xe_lrc_regs_offset(lrc) + CTX_TIMESTAMP_UDW * sizeof(u32);
> }
>
>+static u32 __xe_lrc_queue_timestamp_offset(struct xe_lrc *lrc)
>+{
>+	return __xe_lrc_regs_offset(lrc) + CTX_QUEUE_TIMESTAMP * sizeof(u32);
>+}
>+
>+static u32 __xe_lrc_queue_timestamp_udw_offset(struct xe_lrc *lrc)
>+{
>+	return __xe_lrc_regs_offset(lrc) + CTX_QUEUE_TIMESTAMP_UDW * sizeof(u32);
>+}
>+
> static inline u32 __xe_lrc_indirect_ring_offset(struct xe_lrc *lrc)
> {
> 	u32 offset = xe_bo_size(lrc->bo) - LRC_WA_BB_SIZE -
>@@ -826,6 +836,8 @@ DECL_MAP_ADDR_HELPERS(ctx_timestamp_udw, lrc->bo)
> DECL_MAP_ADDR_HELPERS(parallel, lrc->bo)
> DECL_MAP_ADDR_HELPERS(indirect_ring, lrc->bo)
> DECL_MAP_ADDR_HELPERS(engine_id, lrc->bo)
>+DECL_MAP_ADDR_HELPERS(queue_timestamp, lrc->bo)
>+DECL_MAP_ADDR_HELPERS(queue_timestamp_udw, lrc->bo)
>
> #undef DECL_MAP_ADDR_HELPERS
>
>@@ -874,6 +886,29 @@ static u64 xe_lrc_ctx_timestamp(struct xe_lrc *lrc)
> 	return (u64)udw << 32 | ldw;
> }
>
>+/**
>+ * xe_lrc_queue_timestamp() - Read queue timestamp value
>+ * @lrc: Pointer to the lrc.
>+ *
>+ * Returns: queue timestamp value
>+ */
>+static u64 xe_lrc_queue_timestamp(struct xe_lrc *lrc)
>+{
>+	struct xe_device *xe = lrc_to_xe(lrc);
>+	struct iosys_map map;
>+	u32 ldw, udw = 0;
>+
>+	xe_assert(xe, xe_lrc_is_multi_queue(lrc));
>+
>+	map = __xe_lrc_queue_timestamp_map(lrc);
>+	ldw = xe_map_read32(xe, &map);
>+
>+	map = __xe_lrc_queue_timestamp_udw_map(lrc);
>+	udw = xe_map_read32(xe, &map);
>+
>+	return (u64)udw << 32 | ldw;
>+}
>+
> /**
>  * xe_lrc_ctx_job_timestamp_ggtt_addr() - Get ctx job timestamp GGTT address
>  * @lrc: Pointer to the lrc.
>@@ -1538,6 +1573,12 @@ static int xe_lrc_ctx_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe, struct
> 	if (lrc_to_xe(lrc)->info.has_64bit_timestamp)
> 		xe_lrc_write_ctx_reg(lrc, CTX_TIMESTAMP_UDW, 0);
>
>+	if (xe_gt_hwe_class_supports_multi_queue(gt, hwe->class)) {
>+		lrc->queue_timestamp = 0;
>+		xe_lrc_write_ctx_reg(lrc, CTX_QUEUE_TIMESTAMP, 0);
>+		xe_lrc_write_ctx_reg(lrc, CTX_QUEUE_TIMESTAMP_UDW, 0);
>+	}

xe_gt_hwe_class_supports_multi_queue() doesn't mean given 'lrc' is multi-queue.
The 'lrc' is multi-queue only if xe_gt_hwe_class_supports_multi_queue() is true
and user has opened that exec queue as a multi-queue.

But yah, we don't know whether the given 'lrc' is a multi-queue lrc here.
Perhaps, it is ok to keep this code as is, as it doesn't hurt. May be add
a comment above explaining it.

>+
> 	if (xe->info.has_asid && vm)
> 		xe_lrc_write_ctx_reg(lrc, CTX_ASID, vm->usm.asid);
>
>@@ -2466,6 +2507,9 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
> 	snapshot->ctx_timestamp = xe_lrc_ctx_timestamp(lrc);
> 	snapshot->ctx_timestamp_ms =
> 		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_ctx_timestamp(lrc));
>+	snapshot->queue_timestamp = xe_lrc_queue_timestamp(lrc);
>+	snapshot->queue_timestamp_ms =
>+		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_queue_timestamp(lrc));

queue_timestamp is only supported if lrc is multi-queue.
As we are calling this for all cases here, xe_lrc_queue_timestamp() will assert.

I think we should do,
if (xe_lrc_is_multi_queue(lrc)) {
    snapshot->queue_timestamp = ...;
    snapshot->queue_timestamp_ms = ...;
} else {
    snapshot->queue_timestamp = 0
    snapshot->queue_timestamp_ms = 0
}

Niranjana

> 	snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);
> 	return snapshot;
> }
>@@ -2520,6 +2564,8 @@ void xe_lrc_snapshot_print(struct xe_lrc_snapshot *snapshot, struct drm_printer
> 	drm_printf(p, "\tSeqno: (memory) %d\n", snapshot->seqno);
> 	drm_printf(p, "\tTimestamp: 0x%016llx\n", snapshot->ctx_timestamp);
> 	drm_printf(p, "\tTimestamp ms: %llu\n", snapshot->ctx_timestamp_ms);
>+	drm_printf(p, "\tQueue Timestamp: 0x%016llx\n", snapshot->queue_timestamp);
>+	drm_printf(p, "\tQueue Timestamp ms: %llu\n", snapshot->queue_timestamp_ms);
> 	drm_printf(p, "\tJob Timestamp: 0x%08x\n", snapshot->ctx_job_timestamp);
>
> 	if (!snapshot->lrc_snapshot)
>diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
>index 557dce004d48..0a3a611391ee 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.h
>+++ b/drivers/gpu/drm/xe/xe_lrc.h
>@@ -39,6 +39,8 @@ struct xe_lrc_snapshot {
> 	u32 seqno;
> 	u64 ctx_timestamp;
> 	u64 ctx_timestamp_ms;
>+	u64 queue_timestamp;
>+	u64 queue_timestamp_ms;
> 	u32 ctx_job_timestamp;
> };
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc_types.h b/drivers/gpu/drm/xe/xe_lrc_types.h
>index 0a5c13ec2ad7..53ef48feebfc 100644
>--- a/drivers/gpu/drm/xe/xe_lrc_types.h
>+++ b/drivers/gpu/drm/xe/xe_lrc_types.h
>@@ -64,6 +64,9 @@ struct xe_lrc {
> 	/** @ctx_timestamp: readout value of CTX_TIMESTAMP on last update */
> 	u64 ctx_timestamp;
>
>+	/** @queue_timestamp: value of QUEUE_TIMESTAMP on last update */
>+	u64 queue_timestamp;
>+
> 	/** @multi_queue: Multi queue LRC related information */
> 	struct {
> 		/** @multi_queue.primary_lrc: Primary lrc of this multi-queue group*/
>-- 
>2.51.0
>

Re: [PATCH v3 08/11] drm/xe/multi_queue: Capture queue run times for active queues

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:17PM -0700, Umesh Nerlige Ramappa wrote:
>If a queue is currently active on the CS, query the QUEUE TIMESTAMP
>register to get an up to date value of the runtime. To do so, ensure
>that the primary queue is active and then check if the secondary queue
>is executing on the CS.
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
>v2:
>- Move trace to a separate patch (Stuart)
>- Refactor multi queue timestamp logic (Matt/Niranjana)
>
>v3: (Niranjana)
>- s/get_queue_id/get_multi_queue_active_queue_id/
>- assert for primary_lrc in xe_lrc_multi_queue_timestamp
>- don't use the name single_queue
>---
> drivers/gpu/drm/xe/regs/xe_engine_regs.h |   4 +
> drivers/gpu/drm/xe/xe_lrc.c              | 113 +++++++++++++++++++----
> 2 files changed, 97 insertions(+), 20 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/regs/xe_engine_regs.h b/drivers/gpu/drm/xe/regs/xe_engine_regs.h
>index c4c879a9e555..94033982e694 100644
>--- a/drivers/gpu/drm/xe/regs/xe_engine_regs.h
>+++ b/drivers/gpu/drm/xe/regs/xe_engine_regs.h
>@@ -170,6 +170,10 @@
> #define   GFX_DISABLE_LEGACY_MODE		REG_BIT(3)
>
> #define RING_CSMQDEBUG(base)			XE_REG((base) + 0x2b0)
>+#define   CURRENT_ACTIVE_QUEUE_ID_MASK		REG_GENMASK(7, 0)
>+
>+#define RING_QUEUE_TIMESTAMP(base)		XE_REG((base) + 0x4c0)
>+#define RING_QUEUE_TIMESTAMP_UDW(base)		XE_REG((base) + 0x4c0 + 4)
>
> #define RING_TIMESTAMP(base)			XE_REG((base) + 0x358)
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>index e7f7f0efcf2d..28060a3d4f11 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.c
>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>@@ -21,6 +21,7 @@
> #include "xe_configfs.h"
> #include "xe_device.h"
> #include "xe_drm_client.h"
>+#include "xe_exec_queue.h"
> #include "xe_exec_queue_types.h"
> #include "xe_gt.h"
> #include "xe_gt_clock.h"
>@@ -2641,22 +2642,68 @@ static int get_ctx_timestamp(struct xe_lrc *lrc, u32 engine_id, u64 *reg_ctx_ts)
> 	return 0;
> }
>
>+static u64 get_queue_timestamp(struct xe_hw_engine *hwe)
>+{
>+	return xe_mmio_read64_2x32(&hwe->gt->mmio,
>+				   RING_QUEUE_TIMESTAMP(hwe->mmio_base));
>+}
>+
>+static u32 get_multi_queue_active_queue_id(struct xe_hw_engine *hwe)
>+{
>+	u32 val = xe_mmio_read32(&hwe->gt->mmio,
>+				 RING_CSMQDEBUG(hwe->mmio_base));
>+
>+	return REG_FIELD_GET(CURRENT_ACTIVE_QUEUE_ID_MASK, val);
>+}
>+
> static bool context_active(struct xe_lrc *lrc)
> {
> 	return xe_lrc_ctx_timestamp(lrc) == CONTEXT_ACTIVE;
> }
>
>-/**
>- * xe_lrc_timestamp() - Current ctx timestamp
>- * @lrc: Pointer to the lrc.
>- *
>- * Return latest ctx timestamp. With support for active contexts, the
>- * calculation may be slightly racy, so follow a read-again logic to ensure that
>- * the context is still active before returning the right timestamp.
>- *
>- * Returns: New ctx timestamp value
>- */
>-u64 xe_lrc_timestamp(struct xe_lrc *lrc)
>+static u64 xe_lrc_multi_queue_timestamp(struct xe_lrc *lrc)
>+{
>+	struct xe_device *xe = lrc_to_xe(lrc);
>+	struct xe_lrc *primary_lrc = lrc->multi_queue.primary_lrc;
>+	struct xe_hw_engine *hwe;
>+	u64 reg_queue_ts = lrc->queue_timestamp;
>+
>+	if (IS_SRIOV_VF(xe))
>+		return xe_lrc_queue_timestamp(lrc);
>+
>+	xe_assert(xe, primary_lrc);
>+
>+	if (!context_active(primary_lrc))
>+		return xe_lrc_queue_timestamp(lrc);

NIT...May be here too we need a comment like below that WA BB only stores
context_active cookie only for primary lrc?
Either way,
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>

>+
>+	/* WA BB populates engine id in PPHWSP of primary context only */
>+	hwe = engine_id_to_hwe(primary_lrc->gt, xe_lrc_engine_id(primary_lrc));
>+	if (!hwe)
>+		return xe_lrc_queue_timestamp(lrc);
>+
>+	if (get_multi_queue_active_queue_id(hwe) != lrc->multi_queue.pos)
>+		return xe_lrc_queue_timestamp(lrc);
>+
>+	/* queue is active, so store the queue timestamp register */
>+	reg_queue_ts = get_queue_timestamp(hwe);
>+
>+	/* double check queue and primary queue are both still active */
>+	if (get_multi_queue_active_queue_id(hwe) != lrc->multi_queue.pos ||
>+	    !context_active(primary_lrc))
>+		return xe_lrc_queue_timestamp(lrc);
>+
>+	return reg_queue_ts;
>+}
>+
>+static u64 xe_lrc_update_multi_queue_timestamp(struct xe_lrc *lrc, u64 *old_ts)
>+{
>+	*old_ts = lrc->queue_timestamp;
>+	lrc->queue_timestamp = xe_lrc_multi_queue_timestamp(lrc);
>+
>+	return lrc->queue_timestamp;
>+}
>+
>+static u64 xe_lrc_context_timestamp(struct xe_lrc *lrc)
> {
> 	u64 reg_ts, new_ts = lrc->ctx_timestamp;
>
>@@ -2674,24 +2721,50 @@ u64 xe_lrc_timestamp(struct xe_lrc *lrc)
> 	return new_ts;
> }
>
>+static u64 xe_lrc_update_context_timestamp(struct xe_lrc *lrc, u64 *old_ts)
>+{
>+	*old_ts = lrc->ctx_timestamp;
>+	lrc->ctx_timestamp = xe_lrc_context_timestamp(lrc);
>+
>+	trace_xe_lrc_update_timestamp(lrc, *old_ts);
>+
>+	return lrc->ctx_timestamp;
>+}
>+
>+/**
>+ * xe_lrc_timestamp() - Current lrc timestamp
>+ * @lrc: Pointer to the lrc.
>+ *
>+ * Return latest lrc timestamp. With support for active contexts/queues, the
>+ * calculation may be slightly racy, so follow a read-again logic to ensure that
>+ * the context/queue is still active before returning the right timestamp.
>+ *
>+ * Returns: New lrc timestamp value
>+ */
>+u64 xe_lrc_timestamp(struct xe_lrc *lrc)
>+{
>+	if (xe_lrc_is_multi_queue(lrc))
>+		return xe_lrc_multi_queue_timestamp(lrc);
>+	else
>+		return xe_lrc_context_timestamp(lrc);
>+}
>+
> /**
>- * xe_lrc_update_timestamp() - Update ctx timestamp
>+ * xe_lrc_update_timestamp() - Update lrc timestamp
>  * @lrc: Pointer to the lrc.
>  * @old_ts: Old timestamp value
>  *
>- * Populate @old_ts current saved ctx timestamp, read new ctx timestamp and
>+ * Populate @old_ts with current saved lrc timestamp, read new lrc timestamp and
>  * update saved value.
>  *
>- * Returns: New ctx timestamp value
>+ * Returns: New lrc timestamp value
>  */
> u64 xe_lrc_update_timestamp(struct xe_lrc *lrc, u64 *old_ts)
> {
>-	*old_ts = lrc->ctx_timestamp;
>-	lrc->ctx_timestamp = xe_lrc_timestamp(lrc);
>-
>-	trace_xe_lrc_update_timestamp(lrc, *old_ts);
>-
>-	return lrc->ctx_timestamp;
>+	if (xe_lrc_is_multi_queue(lrc))
>+		return xe_lrc_update_multi_queue_timestamp(lrc, old_ts);
>+	else
>+		return xe_lrc_update_context_timestamp(lrc, old_ts);
> }
>
> /**
>-- 
>2.51.0
>

Re: [PATCH v3 11/11] drm/xe/multi_queue: Whitelist QUEUE_TIMESTAMP register

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:20PM -0700, Umesh Nerlige Ramappa wrote:
>In a multi-queue use case, when a job is running on the secondary queue,
>the CTX_TIMESTAMP does not reflect the queues run ticks. Instead, we use
>the QUEUE TIMESTAMP to check how long the job ran. For user space to see
>the run ticks for a secondary queue, whitelist the QUEUE_TIMESTAMP
>register.
>
>Compute PR: https://github.com/intel/compute-runtime/pull/923
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>

LGTM.
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>

>---
>v2: Whitelist QUEUE_TIMESTAMP only for copy and compute engines (Niranjana)
>v3: (Niranjana)
>- s/multiq/multi_queue/
>- Base the condition on multi_queue_engine_class_mask
>---
> drivers/gpu/drm/xe/xe_reg_whitelist.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
>diff --git a/drivers/gpu/drm/xe/xe_reg_whitelist.c b/drivers/gpu/drm/xe/xe_reg_whitelist.c
>index 8cc313182968..6f96bf84df66 100644
>--- a/drivers/gpu/drm/xe/xe_reg_whitelist.c
>+++ b/drivers/gpu/drm/xe/xe_reg_whitelist.c
>@@ -9,6 +9,7 @@
> #include "regs/xe_gt_regs.h"
> #include "regs/xe_oa_regs.h"
> #include "xe_device.h"
>+#include "xe_gt.h"
> #include "xe_gt_types.h"
> #include "xe_gt_printk.h"
> #include "xe_platform_types.h"
>@@ -33,6 +34,13 @@ static bool match_has_mert(const struct xe_device *xe,
> 	return xe_device_has_mert((struct xe_device *)xe);
> }
>
>+static bool match_multi_queue_class(const struct xe_device *xe,
>+				    const struct xe_gt *gt,
>+				    const struct xe_hw_engine *hwe)
>+{
>+	return xe_gt_hwe_class_supports_multi_queue(gt, hwe->class);
>+}
>+
> static const struct xe_rtp_entry_sr register_whitelist[] = {
> 	{ XE_RTP_NAME("WaAllowPMDepthAndInvocationCountAccessFromUMD, 1408556865"),
> 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(1200, 1210), ENGINE_CLASS(RENDER)),
>@@ -54,6 +62,12 @@ static const struct xe_rtp_entry_sr register_whitelist[] = {
> 				RING_FORCE_TO_NONPRIV_ACCESS_RD,
> 				XE_RTP_ACTION_FLAG(ENGINE_BASE)))
> 	},
>+	{ XE_RTP_NAME("allow_read_queue_timestamp"),
>+	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3500, 3511), FUNC(match_multi_queue_class)),
>+	  XE_RTP_ACTIONS(WHITELIST(RING_QUEUE_TIMESTAMP(0),
>+				   RING_FORCE_TO_NONPRIV_ACCESS_RD,
>+				   XE_RTP_ACTION_FLAG(ENGINE_BASE)))
>+	},
> 	{ XE_RTP_NAME("16014440446"),
> 	  XE_RTP_RULES(PLATFORM(PVC)),
> 	  XE_RTP_ACTIONS(WHITELIST(XE_REG(0x4400),
>-- 
>2.51.0
>

Re: [PATCH v3 05/11] drm/xe/multi_queue: Store primary LRC and position info in LRC

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:14PM -0700, Umesh Nerlige Ramappa wrote:
>Given an LRC belonging to the secondary queue, in order to check if its
>context group is active, we need to check the LRC of the primary queue.
>In addition to that we want to compare the secondary queue position to
>CSMQDEBUG register to check if the queue itself is active.
>
>To do so, store primary LRC and position information in the LRC as well
>as take a reference to the primary LRC from each LRC in the queue group.
>
>A note on references involved:
>
>- In general the Queue takes a ref on its LRC.
>- In addition, for multi-queue,
>a. Primary Queue takes a ref for each Secondary LRC.
>b. Each Secondary Queue takes a ref to the Primary Queue
>
>In the current patch, each LRC in the queue group is storing a pointer
>to Primary LRC. Both Primary and secondary LRCs are freed only when
>primary queue is destroyed. At this time there all secondary queues are
>already destroyed, so there is no one using secondary LRCs. We should be
>good without taking any additional references.
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
>v2:
>- Store primary LRC instead of primary queue (Niranjana)
>- Drop the valid flag and check if primary_lrc is NULL (Niranjana)
>- Document/Revisit references (Matt/Umesh)
>
>v3:
>- Drop the reference logic since it's not needed (Niranjana)
>- Move lrc->multi_queue initialization to a later point (Niranjana)
>---
> drivers/gpu/drm/xe/xe_exec_queue.c | 5 +++++
> drivers/gpu/drm/xe/xe_lrc.h        | 5 +++++
> drivers/gpu/drm/xe/xe_lrc_types.h  | 8 ++++++++
> 3 files changed, 18 insertions(+)
>
>diff --git a/drivers/gpu/drm/xe/xe_exec_queue.c b/drivers/gpu/drm/xe/xe_exec_queue.c
>index 9b36ff5b09a6..bc8110a9b054 100644
>--- a/drivers/gpu/drm/xe/xe_exec_queue.c
>+++ b/drivers/gpu/drm/xe/xe_exec_queue.c
>@@ -1389,6 +1389,11 @@ int xe_exec_queue_create_ioctl(struct drm_device *dev, void *data,
> 				goto put_exec_queue;
> 		}
>
>+		if (xe_exec_queue_is_multi_queue(q)) {
>+			q->lrc[0]->multi_queue.pos = q->multi_queue.pos;
>+			q->lrc[0]->multi_queue.primary_lrc = q->multi_queue.group->primary->lrc[0];
>+		}

This is fine, but looks bit odd without proper comment in the code.
I can think of two better ways to do it,

1. Do it here as below

/* Update lrc multi-queue information after secondary queues is added to group */
if (xe_exec_queue_is_multi_queue(q))
      xe_exec_queue_set_multi_queue_lrc(q);

Yes, xe_exec_queue_set_multi_queue_lrc() will be new static function.

2. Remove the code here, and...
    a. set q->lrc[0].multi_queue.primary_lrc in xe_exec_queue_set_lrc() if queue is
       multi-queue, so that it gets set for both primary and secondary.
    b. set q->lrc[0].multi_queue.pos in xe_exec_queue_group_add() for secondary queues,
       (for primary, pos is 0 anyway).

Niranjana

>+
> 		if (xe_vm_in_preempt_fence_mode(vm)) {
> 			q->lr.context = dma_fence_context_alloc(1);
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
>index d280e6572398..557dce004d48 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.h
>+++ b/drivers/gpu/drm/xe/xe_lrc.h
>@@ -91,6 +91,11 @@ static inline size_t xe_lrc_ring_size(void)
> 	return SZ_16K;
> }
>
>+static inline bool xe_lrc_is_multi_queue(struct xe_lrc *lrc)
>+{
>+	return lrc->multi_queue.primary_lrc;
>+}
>+
> size_t xe_gt_lrc_hang_replay_size(struct xe_gt *gt, enum xe_engine_class class);
> size_t xe_gt_lrc_size(struct xe_gt *gt, enum xe_engine_class class);
> u32 xe_lrc_pphwsp_offset(struct xe_lrc *lrc);
>diff --git a/drivers/gpu/drm/xe/xe_lrc_types.h b/drivers/gpu/drm/xe/xe_lrc_types.h
>index 5a718f759ed6..0a5c13ec2ad7 100644
>--- a/drivers/gpu/drm/xe/xe_lrc_types.h
>+++ b/drivers/gpu/drm/xe/xe_lrc_types.h
>@@ -63,6 +63,14 @@ struct xe_lrc {
>
> 	/** @ctx_timestamp: readout value of CTX_TIMESTAMP on last update */
> 	u64 ctx_timestamp;
>+
>+	/** @multi_queue: Multi queue LRC related information */
>+	struct {
>+		/** @multi_queue.primary_lrc: Primary lrc of this multi-queue group*/
>+		struct xe_lrc *primary_lrc;
>+		/** @multi_queue.pos: Position of LRC within the multi-queue group */
>+		u8 pos;
>+	} multi_queue;
> };
>
> struct xe_lrc_snapshot;
>-- 
>2.51.0
>

Re: [PATCH v3 09/11] drm/xe/multi_queue: Add trace event for the multi queue timestamp

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:18PM -0700, Umesh Nerlige Ramappa wrote:
>Add a trace event for multi queue timestamp capture.
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
>v2:
>- Split traces from original patch (Stuart)
>- Print primary lrc in the trace (Niranjana)
>
>v3:
>- Include primary_lrc as well as lrc in the trace (Niranjana)
>---
> drivers/gpu/drm/xe/xe_lrc.c       |  2 ++
> drivers/gpu/drm/xe/xe_trace_lrc.h | 29 +++++++++++++++++++++++++++++
> 2 files changed, 31 insertions(+)
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>index 28060a3d4f11..ea65160a582a 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.c
>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>@@ -2700,6 +2700,8 @@ static u64 xe_lrc_update_multi_queue_timestamp(struct xe_lrc *lrc, u64 *old_ts)
> 	*old_ts = lrc->queue_timestamp;
> 	lrc->queue_timestamp = xe_lrc_multi_queue_timestamp(lrc);
>
>+	trace_xe_lrc_update_queue_timestamp(lrc, *old_ts);
>+
> 	return lrc->queue_timestamp;
> }
>
>diff --git a/drivers/gpu/drm/xe/xe_trace_lrc.h b/drivers/gpu/drm/xe/xe_trace_lrc.h
>index d525cbee1e34..4f3c1a0de573 100644
>--- a/drivers/gpu/drm/xe/xe_trace_lrc.h
>+++ b/drivers/gpu/drm/xe/xe_trace_lrc.h
>@@ -12,6 +12,7 @@
> #include <linux/tracepoint.h>
> #include <linux/types.h>
>
>+#include "xe_exec_queue_types.h"
> #include "xe_gt_types.h"
> #include "xe_lrc.h"
> #include "xe_lrc_types.h"
>@@ -42,6 +43,34 @@ TRACE_EVENT(xe_lrc_update_timestamp,
> 		      __get_str(device_id))
> );
>
>+TRACE_EVENT(xe_lrc_update_queue_timestamp,
>+	    TP_PROTO(struct xe_lrc *lrc, uint64_t old),
>+	    TP_ARGS(lrc, old),
>+	    TP_STRUCT__entry(
>+		     __field(struct xe_lrc *, lrc)
>+		     __field(struct xe_lrc *, primary_lrc)
>+		     __field(u8, pos)
>+		     __field(u64, old)
>+		     __field(u64, new)
>+		     __string(name, lrc->fence_ctx.name)
>+		     __string(device_id, __dev_name_lrc(lrc))
>+		     ),
>+
>+	    TP_fast_assign(
>+		   __entry->lrc = lrc;
>+		   __entry->primary_lrc = lrc->multi_queue.primary_lrc;
>+		   __entry->pos = lrc->multi_queue.pos;
>+		   __entry->old = old;
>+		   __entry->new = lrc->queue_timestamp;
>+		   __assign_str(name);
>+		   __assign_str(device_id);
>+		   ),
>+	    TP_printk("lrc=:%p primary_lrc:%p pos=%d lrc->name=%s old=%llu new=%llu device_id:%s",
>+		      __entry->lrc, __entry->primary_lrc, __entry->pos, __get_str(name),
>+		      __entry->old, __entry->new,
>+		      __get_str(device_id))

I still think printing 'pos' here does not add much value (we are not printing it in
exec queue trace event also).

BTW, why do we need ':'s before %p? I see it is there in xe_lrc_update_timestamp(),
but that seems to be a mistake (I don't see other trace messages having it while
printing pointer values).

Niranjana

>+);
>+
> #endif
>
> /* This part must be outside protection */
>-- 
>2.51.0
>

Re: [PATCH v3 06/11] drm/xe/multi_queue: Add helpers to access CS QUEUE TIMESTAMP from lrc

[Niranjana Vishwanathapura]

On Tue, May 05, 2026 at 04:44:15PM -0700, Umesh Nerlige Ramappa wrote:
>In secondary queue LRCs, the QUEUE TIMESTAMP register is saved and
>restored allowing us to view the individual queue run times. Add helpers
>to read this value from the LRC.
>
>BSpec: 73988
>
>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>---
>v2: (Matt)
>- Add BSpec reference
>- Make queue_timestamp snapshot 64 bit
>- Add a snapshot of queue timestamp in ms
>
>v3:
>- Replace xe_lrc_is_multi_queue check in lrc init (Niranajana)
>- Use assert instead of check in xe_lrc_queue_timestamp
>---
> drivers/gpu/drm/xe/regs/xe_lrc_layout.h |  3 ++
> drivers/gpu/drm/xe/xe_lrc.c             | 46 +++++++++++++++++++++++++
> drivers/gpu/drm/xe/xe_lrc.h             |  2 ++
> drivers/gpu/drm/xe/xe_lrc_types.h       |  3 ++
> 4 files changed, 54 insertions(+)
>
>diff --git a/drivers/gpu/drm/xe/regs/xe_lrc_layout.h b/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
>index b5eff383902c..4ab86fc369fd 100644
>--- a/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
>+++ b/drivers/gpu/drm/xe/regs/xe_lrc_layout.h
>@@ -34,6 +34,9 @@
> #define CTX_CS_INT_VEC_REG		0x5a
> #define CTX_CS_INT_VEC_DATA		(CTX_CS_INT_VEC_REG + 1)
>
>+#define CTX_QUEUE_TIMESTAMP		(0xd0 + 1)
>+#define CTX_QUEUE_TIMESTAMP_UDW		(0xd2 + 1)
>+
> #define INDIRECT_CTX_RING_HEAD		(0x02 + 1)
> #define INDIRECT_CTX_RING_TAIL		(0x04 + 1)
> #define INDIRECT_CTX_RING_START		(0x06 + 1)
>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>index 3b6990d68a5a..127a2ebc3679 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.c
>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>@@ -777,6 +777,16 @@ static u32 __xe_lrc_ctx_timestamp_udw_offset(struct xe_lrc *lrc)
> 	return __xe_lrc_regs_offset(lrc) + CTX_TIMESTAMP_UDW * sizeof(u32);
> }
>
>+static u32 __xe_lrc_queue_timestamp_offset(struct xe_lrc *lrc)
>+{
>+	return __xe_lrc_regs_offset(lrc) + CTX_QUEUE_TIMESTAMP * sizeof(u32);
>+}
>+
>+static u32 __xe_lrc_queue_timestamp_udw_offset(struct xe_lrc *lrc)
>+{
>+	return __xe_lrc_regs_offset(lrc) + CTX_QUEUE_TIMESTAMP_UDW * sizeof(u32);
>+}
>+
> static inline u32 __xe_lrc_indirect_ring_offset(struct xe_lrc *lrc)
> {
> 	u32 offset = xe_bo_size(lrc->bo) - LRC_WA_BB_SIZE -
>@@ -826,6 +836,8 @@ DECL_MAP_ADDR_HELPERS(ctx_timestamp_udw, lrc->bo)
> DECL_MAP_ADDR_HELPERS(parallel, lrc->bo)
> DECL_MAP_ADDR_HELPERS(indirect_ring, lrc->bo)
> DECL_MAP_ADDR_HELPERS(engine_id, lrc->bo)
>+DECL_MAP_ADDR_HELPERS(queue_timestamp, lrc->bo)
>+DECL_MAP_ADDR_HELPERS(queue_timestamp_udw, lrc->bo)
>
> #undef DECL_MAP_ADDR_HELPERS
>
>@@ -874,6 +886,29 @@ static u64 xe_lrc_ctx_timestamp(struct xe_lrc *lrc)
> 	return (u64)udw << 32 | ldw;
> }
>
>+/**
>+ * xe_lrc_queue_timestamp() - Read queue timestamp value
>+ * @lrc: Pointer to the lrc.
>+ *
>+ * Returns: queue timestamp value
>+ */
>+static u64 xe_lrc_queue_timestamp(struct xe_lrc *lrc)
>+{
>+	struct xe_device *xe = lrc_to_xe(lrc);
>+	struct iosys_map map;
>+	u32 ldw, udw = 0;
>+
>+	xe_assert(xe, xe_lrc_is_multi_queue(lrc));
>+
>+	map = __xe_lrc_queue_timestamp_map(lrc);
>+	ldw = xe_map_read32(xe, &map);
>+
>+	map = __xe_lrc_queue_timestamp_udw_map(lrc);
>+	udw = xe_map_read32(xe, &map);
>+
>+	return (u64)udw << 32 | ldw;
>+}
>+
> /**
>  * xe_lrc_ctx_job_timestamp_ggtt_addr() - Get ctx job timestamp GGTT address
>  * @lrc: Pointer to the lrc.
>@@ -1538,6 +1573,12 @@ static int xe_lrc_ctx_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe, struct
> 	if (lrc_to_xe(lrc)->info.has_64bit_timestamp)
> 		xe_lrc_write_ctx_reg(lrc, CTX_TIMESTAMP_UDW, 0);
>
>+	if (xe_gt_hwe_class_supports_multi_queue(gt, hwe->class)) {
>+		lrc->queue_timestamp = 0;
>+		xe_lrc_write_ctx_reg(lrc, CTX_QUEUE_TIMESTAMP, 0);
>+		xe_lrc_write_ctx_reg(lrc, CTX_QUEUE_TIMESTAMP_UDW, 0);
>+	}
>+
> 	if (xe->info.has_asid && vm)
> 		xe_lrc_write_ctx_reg(lrc, CTX_ASID, vm->usm.asid);
>
>@@ -2466,6 +2507,9 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
> 	snapshot->ctx_timestamp = xe_lrc_ctx_timestamp(lrc);
> 	snapshot->ctx_timestamp_ms =
> 		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_ctx_timestamp(lrc));
>+	snapshot->queue_timestamp = xe_lrc_queue_timestamp(lrc);
>+	snapshot->queue_timestamp_ms =
>+		xe_gt_clock_interval_to_ms(lrc->gt, xe_lrc_queue_timestamp(lrc));

Maybe in patch 5, we also dump primary_lrc and pos in snapshot capture.
We do that in exec_queue snapshot capture.

Niranjana

> 	snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);
> 	return snapshot;
> }
>@@ -2520,6 +2564,8 @@ void xe_lrc_snapshot_print(struct xe_lrc_snapshot *snapshot, struct drm_printer
> 	drm_printf(p, "\tSeqno: (memory) %d\n", snapshot->seqno);
> 	drm_printf(p, "\tTimestamp: 0x%016llx\n", snapshot->ctx_timestamp);
> 	drm_printf(p, "\tTimestamp ms: %llu\n", snapshot->ctx_timestamp_ms);
>+	drm_printf(p, "\tQueue Timestamp: 0x%016llx\n", snapshot->queue_timestamp);
>+	drm_printf(p, "\tQueue Timestamp ms: %llu\n", snapshot->queue_timestamp_ms);
> 	drm_printf(p, "\tJob Timestamp: 0x%08x\n", snapshot->ctx_job_timestamp);
>
> 	if (!snapshot->lrc_snapshot)
>diff --git a/drivers/gpu/drm/xe/xe_lrc.h b/drivers/gpu/drm/xe/xe_lrc.h
>index 557dce004d48..0a3a611391ee 100644
>--- a/drivers/gpu/drm/xe/xe_lrc.h
>+++ b/drivers/gpu/drm/xe/xe_lrc.h
>@@ -39,6 +39,8 @@ struct xe_lrc_snapshot {
> 	u32 seqno;
> 	u64 ctx_timestamp;
> 	u64 ctx_timestamp_ms;
>+	u64 queue_timestamp;
>+	u64 queue_timestamp_ms;
> 	u32 ctx_job_timestamp;
> };
>
>diff --git a/drivers/gpu/drm/xe/xe_lrc_types.h b/drivers/gpu/drm/xe/xe_lrc_types.h
>index 0a5c13ec2ad7..53ef48feebfc 100644
>--- a/drivers/gpu/drm/xe/xe_lrc_types.h
>+++ b/drivers/gpu/drm/xe/xe_lrc_types.h
>@@ -64,6 +64,9 @@ struct xe_lrc {
> 	/** @ctx_timestamp: readout value of CTX_TIMESTAMP on last update */
> 	u64 ctx_timestamp;
>
>+	/** @queue_timestamp: value of QUEUE_TIMESTAMP on last update */
>+	u64 queue_timestamp;
>+
> 	/** @multi_queue: Multi queue LRC related information */
> 	struct {
> 		/** @multi_queue.primary_lrc: Primary lrc of this multi-queue group*/
>-- 
>2.51.0
>

Re: [PATCH v3 09/11] drm/xe/multi_queue: Add trace event for the multi queue timestamp

[Umesh Nerlige Ramappa]

On Tue, May 05, 2026 at 10:03:26PM -0700, Niranjana Vishwanathapura wrote:
>On Tue, May 05, 2026 at 04:44:18PM -0700, Umesh Nerlige Ramappa wrote:
>>Add a trace event for multi queue timestamp capture.
>>
>>Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>>---
>>v2:
>>- Split traces from original patch (Stuart)
>>- Print primary lrc in the trace (Niranjana)
>>
>>v3:
>>- Include primary_lrc as well as lrc in the trace (Niranjana)
>>---
>>drivers/gpu/drm/xe/xe_lrc.c       |  2 ++
>>drivers/gpu/drm/xe/xe_trace_lrc.h | 29 +++++++++++++++++++++++++++++
>>2 files changed, 31 insertions(+)
>>
>>diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
>>index 28060a3d4f11..ea65160a582a 100644
>>--- a/drivers/gpu/drm/xe/xe_lrc.c
>>+++ b/drivers/gpu/drm/xe/xe_lrc.c
>>@@ -2700,6 +2700,8 @@ static u64 xe_lrc_update_multi_queue_timestamp(struct xe_lrc *lrc, u64 *old_ts)
>>	*old_ts = lrc->queue_timestamp;
>>	lrc->queue_timestamp = xe_lrc_multi_queue_timestamp(lrc);
>>
>>+	trace_xe_lrc_update_queue_timestamp(lrc, *old_ts);
>>+
>>	return lrc->queue_timestamp;
>>}
>>
>>diff --git a/drivers/gpu/drm/xe/xe_trace_lrc.h b/drivers/gpu/drm/xe/xe_trace_lrc.h
>>index d525cbee1e34..4f3c1a0de573 100644
>>--- a/drivers/gpu/drm/xe/xe_trace_lrc.h
>>+++ b/drivers/gpu/drm/xe/xe_trace_lrc.h
>>@@ -12,6 +12,7 @@
>>#include <linux/tracepoint.h>
>>#include <linux/types.h>
>>
>>+#include "xe_exec_queue_types.h"
>>#include "xe_gt_types.h"
>>#include "xe_lrc.h"
>>#include "xe_lrc_types.h"
>>@@ -42,6 +43,34 @@ TRACE_EVENT(xe_lrc_update_timestamp,
>>		      __get_str(device_id))
>>);
>>
>>+TRACE_EVENT(xe_lrc_update_queue_timestamp,
>>+	    TP_PROTO(struct xe_lrc *lrc, uint64_t old),
>>+	    TP_ARGS(lrc, old),
>>+	    TP_STRUCT__entry(
>>+		     __field(struct xe_lrc *, lrc)
>>+		     __field(struct xe_lrc *, primary_lrc)
>>+		     __field(u8, pos)
>>+		     __field(u64, old)
>>+		     __field(u64, new)
>>+		     __string(name, lrc->fence_ctx.name)
>>+		     __string(device_id, __dev_name_lrc(lrc))
>>+		     ),
>>+
>>+	    TP_fast_assign(
>>+		   __entry->lrc = lrc;
>>+		   __entry->primary_lrc = lrc->multi_queue.primary_lrc;
>>+		   __entry->pos = lrc->multi_queue.pos;
>>+		   __entry->old = old;
>>+		   __entry->new = lrc->queue_timestamp;
>>+		   __assign_str(name);
>>+		   __assign_str(device_id);
>>+		   ),
>>+	    TP_printk("lrc=:%p primary_lrc:%p pos=%d lrc->name=%s old=%llu new=%llu device_id:%s",
>>+		      __entry->lrc, __entry->primary_lrc, __entry->pos, __get_str(name),
>>+		      __entry->old, __entry->new,
>>+		      __get_str(device_id))
>
>I still think printing 'pos' here does not add much value (we are not printing it in
>exec queue trace event also).

ok, dropped pos

>
>BTW, why do we need ':'s before %p? I see it is there in xe_lrc_update_timestamp(),
>but that seems to be a mistake (I don't see other trace messages having it while
>printing pointer values).

Not sure why. I have removed it from xe_lrc_update_queue_timestamp. I 
don't think I can remove it from xe_lrc_update_timestamp since that may 
already be uabi.

Umesh
>
>Niranjana
>
>>+);
>>+
>>#endif
>>
>>/* This part must be outside protection */
>>-- 
>>2.51.0
>>