* [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
@ 2026-08-18 18:38 Jagmeet Randhawa
2026-08-18 18:45 ` ✓ CI.KUnit: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
` (5 more replies)
0 siblings, 6 replies; 11+ messages in thread
From: Jagmeet Randhawa @ 2026-08-18 18:38 UTC (permalink / raw)
To: intel-xe; +Cc: niranjana.vishwanathapura, Jagmeet Randhawa
q->guc->suspend_pending is accessed without any common lock.
__suspend_fence_signal(), called from guc_exec_queue_kill() and the
suspend-timeout ban path, clears the flag asynchronously. Meanwhile
handle_sched_done(), guc_exec_queue_stop() and
__guc_exec_queue_process_msg_suspend() check the flag and then call
suspend_fence_signal(), which asserts that it is still set.
As the check and suspend_fence_signal() are not atomic, the clear can
land in between and trip the xe_gt_assert(q->guc->suspend_pending).
The flag is already set and read under the per-queue msg_lock
(xe_sched_msg_lock()) on the suspend and resume paths. Extend that same
lock to the clear paths (kill and ban) and to the three check-then-act
sites so the check and the signal are atomic with respect to the clear.
In __guc_exec_queue_process_msg_suspend() only the non-sleeping branch is
wrapped, since the other branch waits. In handle_sched_done() the flag is
snapshotted under the lock and deregister_exec_queue() is kept outside it.
v2: Document that sched->msg_lock also protects
guc->suspend_pending, which indicates a suspend message is in
flight, in addition to the sched->msgs list (Niranjana)
Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
---
drivers/gpu/drm/xe/xe_gpu_scheduler_types.h | 5 +++-
drivers/gpu/drm/xe/xe_guc_exec_queue_types.h | 5 +++-
drivers/gpu/drm/xe/xe_guc_submit.c | 29 ++++++++++++++++----
3 files changed, 32 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
index 63d9bf92583c..78ef2e8ded4f 100644
--- a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
+++ b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
@@ -47,7 +47,10 @@ struct xe_gpu_scheduler {
const struct xe_sched_backend_ops *ops;
/** @msgs: list of messages to be processed in @work_process_msg */
struct list_head msgs;
- /** @msg_lock: Message lock */
+ /**
+ * @msg_lock: Protects @msgs and guc->suspend_pending (indicates a
+ * suspend message is in flight) of exec queues on this scheduler.
+ */
spinlock_t msg_lock;
/** @work_process_msg: processes messages */
struct work_struct work_process_msg;
diff --git a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
index d27826b36649..74b711abe257 100644
--- a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
+++ b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
@@ -52,7 +52,10 @@ struct xe_guc_exec_queue {
u16 id;
/** @suspend_wait: wait queue used to wait on pending suspends */
wait_queue_head_t suspend_wait;
- /** @suspend_pending: a suspend of the exec_queue is pending */
+ /**
+ * @suspend_pending: a suspend of the exec_queue is pending.
+ * Protected by @sched.msg_lock.
+ */
bool suspend_pending;
/**
* @suspend_count: Reference count of active suspend requests. The
diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
index 9036f89dff7d..c565c1d32d3a 100644
--- a/drivers/gpu/drm/xe/xe_guc_submit.c
+++ b/drivers/gpu/drm/xe/xe_guc_submit.c
@@ -1928,9 +1928,13 @@ static void __guc_exec_queue_process_msg_suspend(struct xe_sched_msg *msg)
set_exec_queue_suspended(q);
disable_scheduling(q, false);
}
- } else if (q->guc->suspend_pending) {
- set_exec_queue_suspended(q);
- suspend_fence_signal(q);
+ } else {
+ xe_sched_msg_lock(&q->guc->sched);
+ if (q->guc->suspend_pending) {
+ set_exec_queue_suspended(q);
+ suspend_fence_signal(q);
+ }
+ xe_sched_msg_unlock(&q->guc->sched);
}
}
@@ -2130,7 +2134,9 @@ static void guc_exec_queue_kill(struct xe_exec_queue *q)
{
trace_xe_exec_queue_kill(q);
set_exec_queue_killed(q);
+ xe_sched_msg_lock(&q->guc->sched);
__suspend_fence_signal(q);
+ xe_sched_msg_unlock(&q->guc->sched);
xe_guc_exec_queue_trigger_cleanup(q);
}
@@ -2392,11 +2398,15 @@ static void guc_exec_queue_suspend_timeout_ban(struct xe_exec_queue *q)
*/
if (xe_exec_queue_is_multi_queue(q)) {
set_exec_queue_group_banned(q);
+ xe_sched_msg_lock(&q->guc->sched);
__suspend_fence_signal(q);
+ xe_sched_msg_unlock(&q->guc->sched);
xe_guc_exec_queue_group_trigger_cleanup(q);
} else {
set_exec_queue_banned(q);
+ xe_sched_msg_lock(&q->guc->sched);
__suspend_fence_signal(q);
+ xe_sched_msg_unlock(&q->guc->sched);
xe_guc_exec_queue_trigger_cleanup(q);
}
}
@@ -2614,10 +2624,12 @@ static void guc_exec_queue_stop(struct xe_guc *guc, struct xe_exec_queue *q)
if (exec_queue_destroyed(q))
do_destroy = true;
}
+ xe_sched_msg_lock(sched);
if (q->guc->suspend_pending) {
set_exec_queue_suspended(q);
suspend_fence_signal(q);
}
+ xe_sched_msg_unlock(sched);
atomic_and(EXEC_QUEUE_STATE_WEDGED | EXEC_QUEUE_STATE_BANNED |
EXEC_QUEUE_STATE_KILLED | EXEC_QUEUE_STATE_DESTROYED |
EXEC_QUEUE_STATE_SUSPENDED,
@@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
smp_wmb();
wake_up_all(&guc->ct.wq);
} else {
+ bool was_pending;
+
xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
- if (q->guc->suspend_pending) {
+ xe_sched_msg_lock(&q->guc->sched);
+ was_pending = q->guc->suspend_pending;
+ if (was_pending) {
clear_exec_queue_pending_disable(q);
suspend_fence_signal(q);
- } else {
+ }
+ xe_sched_msg_unlock(&q->guc->sched);
+
+ if (!was_pending) {
if (exec_queue_banned(q)) {
smp_wmb();
wake_up_all(&guc->ct.wq);
--
2.53.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* ✓ CI.KUnit: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2)
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
@ 2026-08-18 18:45 ` Patchwork
2026-08-18 18:51 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access sashiko-bot
` (4 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2026-08-18 18:45 UTC (permalink / raw)
To: Jagmeet Randhawa; +Cc: intel-xe
== Series Details ==
Series: drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2)
URL : https://patchwork.freedesktop.org/series/172176/
State : success
== Summary ==
+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[18:44:22] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[18:44:26] 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
[18:44:58] Starting KUnit Kernel (1/1)...
[18:44:58] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[18:44:58] ================== guc_buf (11 subtests) ===================
[18:44:58] [PASSED] test_smallest
[18:44:58] [PASSED] test_largest
[18:44:58] [PASSED] test_granular
[18:44:58] [PASSED] test_unique
[18:44:58] [PASSED] test_overlap
[18:44:58] [PASSED] test_reusable
[18:44:58] [PASSED] test_too_big
[18:44:58] [PASSED] test_flush
[18:44:58] [PASSED] test_lookup
[18:44:58] [PASSED] test_data
[18:44:58] [PASSED] test_class
[18:44:58] ===================== [PASSED] guc_buf =====================
[18:44:58] =================== guc_dbm (7 subtests) ===================
[18:44:58] [PASSED] test_empty
[18:44:58] [PASSED] test_default
[18:44:58] ======================== test_size ========================
[18:44:58] [PASSED] 4
[18:44:58] [PASSED] 8
[18:44:58] [PASSED] 32
[18:44:58] [PASSED] 256
[18:44:58] ==================== [PASSED] test_size ====================
[18:44:58] ======================= test_reuse ========================
[18:44:58] [PASSED] 4
[18:44:58] [PASSED] 8
[18:44:58] [PASSED] 32
[18:44:58] [PASSED] 256
[18:44:58] =================== [PASSED] test_reuse ====================
[18:44:58] =================== test_range_overlap ====================
[18:44:58] [PASSED] 4
[18:44:58] [PASSED] 8
[18:44:58] [PASSED] 32
[18:44:58] [PASSED] 256
[18:44:58] =============== [PASSED] test_range_overlap ================
[18:44:58] =================== test_range_compact ====================
[18:44:58] [PASSED] 4
[18:44:58] [PASSED] 8
[18:44:58] [PASSED] 32
[18:44:58] [PASSED] 256
[18:44:58] =============== [PASSED] test_range_compact ================
[18:44:58] ==================== test_range_spare =====================
[18:44:58] [PASSED] 4
[18:44:58] [PASSED] 8
[18:44:58] [PASSED] 32
[18:44:58] [PASSED] 256
[18:44:58] ================ [PASSED] test_range_spare =================
[18:44:58] ===================== [PASSED] guc_dbm =====================
[18:44:58] =================== guc_idm (6 subtests) ===================
[18:44:58] [PASSED] bad_init
[18:44:58] [PASSED] no_init
[18:44:58] [PASSED] init_fini
[18:44:58] [PASSED] check_used
[18:44:58] [PASSED] check_quota
[18:44:58] [PASSED] check_all
[18:44:58] ===================== [PASSED] guc_idm =====================
[18:44:58] =============== guc_klv_helpers (9 subtests) ===============
[18:44:58] [PASSED] test_count
[18:44:58] [PASSED] test_encode_u32
[18:44:58] [PASSED] test_encode_u64
[18:44:58] [PASSED] test_encode_string
[18:44:58] [PASSED] test_encode_object_raw
[18:44:58] [PASSED] test_encode_object_klv
[18:44:58] [PASSED] test_encode_object_nested
[18:44:58] [PASSED] test_encode_object_basic
[18:44:58] [PASSED] test_print
[18:44:58] ================= [PASSED] guc_klv_helpers =================
[18:44:58] =================== xe_log (4 subtests) ====================
[18:44:58] [PASSED] demo_cper
[18:44:58] [PASSED] demo_dmesg
[18:44:58] ======================= test_dmesg ========================
[18:44:58] [PASSED] test_fatal
[18:44:58] [PASSED] test_fatal_tile
[18:44:58] [PASSED] test_fatal_gt
[18:44:58] [PASSED] test_fatal_comp
[18:44:58] [PASSED] test_fatal_comp_tile
[18:44:58] [PASSED] test_fatal_comp_gt
[18:44:58] [PASSED] test_fatal_all
[18:44:58] [PASSED] test_recoverable
[18:44:58] [PASSED] test_recoverable_tile
[18:44:58] [PASSED] test_recoverable_gt
[18:44:58] [PASSED] test_recoverable_comp
[18:44:58] [PASSED] test_recoverable_comp_tile
[18:44:58] [PASSED] test_recoverable_comp_gt
[18:44:58] [PASSED] test_recoverable_all
[18:44:58] [PASSED] test_info
[18:44:58] [PASSED] test_info_tile
[18:44:58] [PASSED] test_info_gt
[18:44:58] [PASSED] test_info_err
[18:44:58] [PASSED] test_info_comp
[18:44:58] [PASSED] test_info_comp_tile
[18:44:58] [PASSED] test_info_comp_gt
[18:44:58] [PASSED] test_info_all
[18:44:58] [PASSED] test_hw_fatal
[18:44:58] [PASSED] test_hw_recoverable
[18:44:58] [PASSED] test_hw_corrected
[18:44:58] [PASSED] test_hw_informational
[18:44:58] =================== [PASSED] test_dmesg ====================
[18:44:58] ====================== test_invalid =======================
[18:44:58] [SKIPPED] no-component no-location no-warn (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] reserved location (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] unknown location (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] nonzero-device-id location (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] invalid-tile-id location (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] invalid-gt-id location (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] unknown component class (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] unknown system component (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] unknown hardware component (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] [SKIPPED] unknown component and location (requires CONFIG_DRM_XE_DEBUG)
[18:44:58] ================== [SKIPPED] test_invalid ==================
[18:44:58] ===================== [PASSED] xe_log ======================
[18:44:58] ================== no_relay (3 subtests) ===================
[18:44:58] [PASSED] xe_drops_guc2pf_if_not_ready
[18:44:58] [PASSED] xe_drops_guc2vf_if_not_ready
[18:44:58] [PASSED] xe_rejects_send_if_not_ready
[18:44:58] ==================== [PASSED] no_relay =====================
[18:44:58] ================== pf_relay (14 subtests) ==================
[18:44:58] [PASSED] pf_rejects_guc2pf_too_short
[18:44:58] [PASSED] pf_rejects_guc2pf_too_long
[18:44:58] [PASSED] pf_rejects_guc2pf_no_payload
[18:44:58] [PASSED] pf_fails_no_payload
[18:44:58] [PASSED] pf_fails_bad_origin
[18:44:58] [PASSED] pf_fails_bad_type
[18:44:58] [PASSED] pf_txn_reports_error
[18:44:58] [PASSED] pf_txn_sends_pf2guc
[18:44:58] [PASSED] pf_sends_pf2guc
[18:44:58] [SKIPPED] pf_loopback_nop (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[18:44:58] [SKIPPED] pf_loopback_echo (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[18:44:58] [SKIPPED] pf_loopback_fail (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[18:44:58] [SKIPPED] pf_loopback_busy (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[18:44:58] [SKIPPED] pf_loopback_retry (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[18:44:58] ==================== [PASSED] pf_relay =====================
[18:44:58] ================== vf_relay (3 subtests) ===================
[18:44:58] [PASSED] vf_rejects_guc2vf_too_short
[18:44:58] [PASSED] vf_rejects_guc2vf_too_long
[18:44:58] [PASSED] vf_rejects_guc2vf_no_payload
[18:44:58] ==================== [PASSED] vf_relay =====================
[18:44:58] ================ pf_gt_config (9 subtests) =================
[18:44:58] [PASSED] fair_contexts_1vf
[18:44:58] [PASSED] fair_doorbells_1vf
[18:44:58] [PASSED] fair_ggtt_1vf
[18:44:58] ====================== fair_vram_1vf ======================
[18:44:58] [PASSED] 3.50 GiB
[18:44:58] [PASSED] 11.5 GiB
[18:44:58] [PASSED] 15.5 GiB
[18:44:58] [PASSED] 31.5 GiB
[18:44:58] [PASSED] 63.5 GiB
[18:44:58] [PASSED] 1.91 GiB
[18:44:58] ================== [PASSED] fair_vram_1vf ==================
[18:44:58] ================ fair_vram_1vf_admin_only =================
[18:44:58] [PASSED] 3.50 GiB
[18:44:58] [PASSED] 11.5 GiB
[18:44:58] [PASSED] 15.5 GiB
[18:44:58] [PASSED] 31.5 GiB
[18:44:58] [PASSED] 63.5 GiB
[18:44:58] [PASSED] 1.91 GiB
[18:44:58] ============ [PASSED] fair_vram_1vf_admin_only =============
[18:44:58] ====================== fair_contexts ======================
[18:44:58] [PASSED] 1 VF
[18:44:58] [PASSED] 2 VFs
[18:44:58] [PASSED] 3 VFs
[18:44:58] [PASSED] 4 VFs
[18:44:58] [PASSED] 5 VFs
[18:44:58] [PASSED] 6 VFs
[18:44:58] [PASSED] 7 VFs
[18:44:58] [PASSED] 8 VFs
[18:44:58] [PASSED] 9 VFs
[18:44:58] [PASSED] 10 VFs
[18:44:58] [PASSED] 11 VFs
[18:44:58] [PASSED] 12 VFs
[18:44:58] [PASSED] 13 VFs
[18:44:58] [PASSED] 14 VFs
[18:44:58] [PASSED] 15 VFs
[18:44:58] [PASSED] 16 VFs
[18:44:58] [PASSED] 17 VFs
[18:44:58] [PASSED] 18 VFs
[18:44:58] [PASSED] 19 VFs
[18:44:58] [PASSED] 20 VFs
[18:44:58] [PASSED] 21 VFs
[18:44:58] [PASSED] 22 VFs
[18:44:58] [PASSED] 23 VFs
[18:44:58] [PASSED] 24 VFs
[18:44:58] [PASSED] 25 VFs
[18:44:58] [PASSED] 26 VFs
[18:44:58] [PASSED] 27 VFs
[18:44:58] [PASSED] 28 VFs
[18:44:58] [PASSED] 29 VFs
[18:44:58] [PASSED] 30 VFs
[18:44:58] [PASSED] 31 VFs
[18:44:58] [PASSED] 32 VFs
[18:44:58] [PASSED] 33 VFs
[18:44:58] [PASSED] 34 VFs
[18:44:58] [PASSED] 35 VFs
[18:44:58] [PASSED] 36 VFs
[18:44:58] [PASSED] 37 VFs
[18:44:58] [PASSED] 38 VFs
[18:44:58] [PASSED] 39 VFs
[18:44:58] [PASSED] 40 VFs
[18:44:58] [PASSED] 41 VFs
[18:44:58] [PASSED] 42 VFs
[18:44:58] [PASSED] 43 VFs
[18:44:58] [PASSED] 44 VFs
[18:44:58] [PASSED] 45 VFs
[18:44:58] [PASSED] 46 VFs
[18:44:58] [PASSED] 47 VFs
[18:44:58] [PASSED] 48 VFs
[18:44:58] [PASSED] 49 VFs
[18:44:58] [PASSED] 50 VFs
[18:44:58] [PASSED] 51 VFs
[18:44:58] [PASSED] 52 VFs
[18:44:58] [PASSED] 53 VFs
[18:44:58] [PASSED] 54 VFs
[18:44:58] [PASSED] 55 VFs
[18:44:58] [PASSED] 56 VFs
[18:44:58] [PASSED] 57 VFs
[18:44:58] [PASSED] 58 VFs
[18:44:58] [PASSED] 59 VFs
[18:44:58] [PASSED] 60 VFs
[18:44:58] [PASSED] 61 VFs
[18:44:58] [PASSED] 62 VFs
[18:44:58] [PASSED] 63 VFs
[18:44:58] ================== [PASSED] fair_contexts ==================
[18:44:58] ===================== fair_doorbells ======================
[18:44:58] [PASSED] 1 VF
[18:44:58] [PASSED] 2 VFs
[18:44:58] [PASSED] 3 VFs
[18:44:58] [PASSED] 4 VFs
[18:44:58] [PASSED] 5 VFs
[18:44:58] [PASSED] 6 VFs
[18:44:58] [PASSED] 7 VFs
[18:44:58] [PASSED] 8 VFs
[18:44:58] [PASSED] 9 VFs
[18:44:58] [PASSED] 10 VFs
[18:44:58] [PASSED] 11 VFs
[18:44:58] [PASSED] 12 VFs
[18:44:58] [PASSED] 13 VFs
[18:44:58] [PASSED] 14 VFs
[18:44:58] [PASSED] 15 VFs
[18:44:58] [PASSED] 16 VFs
[18:44:58] [PASSED] 17 VFs
[18:44:58] [PASSED] 18 VFs
[18:44:58] [PASSED] 19 VFs
[18:44:58] [PASSED] 20 VFs
[18:44:58] [PASSED] 21 VFs
[18:44:58] [PASSED] 22 VFs
[18:44:58] [PASSED] 23 VFs
[18:44:58] [PASSED] 24 VFs
[18:44:58] [PASSED] 25 VFs
[18:44:58] [PASSED] 26 VFs
[18:44:58] [PASSED] 27 VFs
[18:44:58] [PASSED] 28 VFs
[18:44:58] [PASSED] 29 VFs
[18:44:58] [PASSED] 30 VFs
[18:44:58] [PASSED] 31 VFs
[18:44:58] [PASSED] 32 VFs
[18:44:58] [PASSED] 33 VFs
[18:44:58] [PASSED] 34 VFs
[18:44:58] [PASSED] 35 VFs
[18:44:58] [PASSED] 36 VFs
[18:44:58] [PASSED] 37 VFs
[18:44:58] [PASSED] 38 VFs
[18:44:58] [PASSED] 39 VFs
[18:44:58] [PASSED] 40 VFs
[18:44:58] [PASSED] 41 VFs
[18:44:58] [PASSED] 42 VFs
[18:44:58] [PASSED] 43 VFs
[18:44:58] [PASSED] 44 VFs
[18:44:58] [PASSED] 45 VFs
[18:44:58] [PASSED] 46 VFs
[18:44:58] [PASSED] 47 VFs
[18:44:58] [PASSED] 48 VFs
[18:44:58] [PASSED] 49 VFs
[18:44:58] [PASSED] 50 VFs
[18:44:58] [PASSED] 51 VFs
[18:44:58] [PASSED] 52 VFs
[18:44:58] [PASSED] 53 VFs
[18:44:58] [PASSED] 54 VFs
[18:44:58] [PASSED] 55 VFs
[18:44:58] [PASSED] 56 VFs
[18:44:58] [PASSED] 57 VFs
[18:44:58] [PASSED] 58 VFs
[18:44:58] [PASSED] 59 VFs
[18:44:58] [PASSED] 60 VFs
[18:44:58] [PASSED] 61 VFs
[18:44:58] [PASSED] 62 VFs
[18:44:58] [PASSED] 63 VFs
[18:44:58] ================= [PASSED] fair_doorbells ==================
[18:44:58] ======================== fair_ggtt ========================
[18:44:58] [PASSED] 1 VF
[18:44:58] [PASSED] 2 VFs
[18:44:58] [PASSED] 3 VFs
[18:44:58] [PASSED] 4 VFs
[18:44:58] [PASSED] 5 VFs
[18:44:58] [PASSED] 6 VFs
[18:44:58] [PASSED] 7 VFs
[18:44:58] [PASSED] 8 VFs
[18:44:58] [PASSED] 9 VFs
[18:44:58] [PASSED] 10 VFs
[18:44:58] [PASSED] 11 VFs
[18:44:58] [PASSED] 12 VFs
[18:44:58] [PASSED] 13 VFs
[18:44:58] [PASSED] 14 VFs
[18:44:58] [PASSED] 15 VFs
[18:44:58] [PASSED] 16 VFs
[18:44:58] [PASSED] 17 VFs
[18:44:58] [PASSED] 18 VFs
[18:44:58] [PASSED] 19 VFs
[18:44:58] [PASSED] 20 VFs
[18:44:58] [PASSED] 21 VFs
[18:44:58] [PASSED] 22 VFs
[18:44:58] [PASSED] 23 VFs
[18:44:58] [PASSED] 24 VFs
[18:44:58] [PASSED] 25 VFs
[18:44:58] [PASSED] 26 VFs
[18:44:58] [PASSED] 27 VFs
[18:44:58] [PASSED] 28 VFs
[18:44:58] [PASSED] 29 VFs
[18:44:58] [PASSED] 30 VFs
[18:44:58] [PASSED] 31 VFs
[18:44:58] [PASSED] 32 VFs
[18:44:58] [PASSED] 33 VFs
[18:44:58] [PASSED] 34 VFs
[18:44:58] [PASSED] 35 VFs
[18:44:58] [PASSED] 36 VFs
[18:44:58] [PASSED] 37 VFs
[18:44:58] [PASSED] 38 VFs
[18:44:58] [PASSED] 39 VFs
[18:44:58] [PASSED] 40 VFs
[18:44:58] [PASSED] 41 VFs
[18:44:58] [PASSED] 42 VFs
[18:44:58] [PASSED] 43 VFs
[18:44:58] [PASSED] 44 VFs
[18:44:58] [PASSED] 45 VFs
[18:44:58] [PASSED] 46 VFs
[18:44:58] [PASSED] 47 VFs
[18:44:58] [PASSED] 48 VFs
[18:44:58] [PASSED] 49 VFs
[18:44:58] [PASSED] 50 VFs
[18:44:58] [PASSED] 51 VFs
[18:44:58] [PASSED] 52 VFs
[18:44:58] [PASSED] 53 VFs
[18:44:58] [PASSED] 54 VFs
[18:44:58] [PASSED] 55 VFs
[18:44:58] [PASSED] 56 VFs
[18:44:58] [PASSED] 57 VFs
[18:44:58] [PASSED] 58 VFs
[18:44:58] [PASSED] 59 VFs
[18:44:58] [PASSED] 60 VFs
[18:44:58] [PASSED] 61 VFs
[18:44:58] [PASSED] 62 VFs
[18:44:58] [PASSED] 63 VFs
[18:44:58] ==================== [PASSED] fair_ggtt ====================
[18:44:58] ======================== fair_vram ========================
[18:44:58] [PASSED] 1 VF
[18:44:58] [PASSED] 2 VFs
[18:44:58] [PASSED] 3 VFs
[18:44:58] [PASSED] 4 VFs
[18:44:58] [PASSED] 5 VFs
[18:44:58] [PASSED] 6 VFs
[18:44:58] [PASSED] 7 VFs
[18:44:58] [PASSED] 8 VFs
[18:44:58] [PASSED] 9 VFs
[18:44:58] [PASSED] 10 VFs
[18:44:58] [PASSED] 11 VFs
[18:44:58] [PASSED] 12 VFs
[18:44:58] [PASSED] 13 VFs
[18:44:58] [PASSED] 14 VFs
[18:44:58] [PASSED] 15 VFs
[18:44:58] [PASSED] 16 VFs
[18:44:58] [PASSED] 17 VFs
[18:44:58] [PASSED] 18 VFs
[18:44:58] [PASSED] 19 VFs
[18:44:58] [PASSED] 20 VFs
[18:44:58] [PASSED] 21 VFs
[18:44:58] [PASSED] 22 VFs
[18:44:58] [PASSED] 23 VFs
[18:44:58] [PASSED] 24 VFs
[18:44:58] [PASSED] 25 VFs
[18:44:58] [PASSED] 26 VFs
[18:44:58] [PASSED] 27 VFs
[18:44:58] [PASSED] 28 VFs
[18:44:58] [PASSED] 29 VFs
[18:44:58] [PASSED] 30 VFs
[18:44:58] [PASSED] 31 VFs
[18:44:58] [PASSED] 32 VFs
[18:44:58] [PASSED] 33 VFs
[18:44:58] [PASSED] 34 VFs
[18:44:58] [PASSED] 35 VFs
[18:44:58] [PASSED] 36 VFs
[18:44:58] [PASSED] 37 VFs
[18:44:58] [PASSED] 38 VFs
[18:44:58] [PASSED] 39 VFs
[18:44:58] [PASSED] 40 VFs
[18:44:58] [PASSED] 41 VFs
[18:44:58] [PASSED] 42 VFs
[18:44:58] [PASSED] 43 VFs
[18:44:58] [PASSED] 44 VFs
[18:44:58] [PASSED] 45 VFs
[18:44:58] [PASSED] 46 VFs
[18:44:58] [PASSED] 47 VFs
[18:44:58] [PASSED] 48 VFs
[18:44:58] [PASSED] 49 VFs
[18:44:58] [PASSED] 50 VFs
[18:44:58] [PASSED] 51 VFs
[18:44:58] [PASSED] 52 VFs
[18:44:58] [PASSED] 53 VFs
[18:44:58] [PASSED] 54 VFs
[18:44:58] [PASSED] 55 VFs
[18:44:58] [PASSED] 56 VFs
[18:44:58] [PASSED] 57 VFs
[18:44:58] [PASSED] 58 VFs
[18:44:58] [PASSED] 59 VFs
[18:44:58] [PASSED] 60 VFs
[18:44:58] [PASSED] 61 VFs
[18:44:58] [PASSED] 62 VFs
[18:44:58] [PASSED] 63 VFs
[18:44:58] ==================== [PASSED] fair_vram ====================
[18:44:58] ================== [PASSED] pf_gt_config ===================
[18:44:58] ===================== lmtt (1 subtest) =====================
[18:44:58] ======================== test_ops =========================
[18:44:58] [PASSED] 2-level
[18:44:58] [PASSED] multi-level
[18:44:58] ==================== [PASSED] test_ops =====================
[18:44:58] ====================== [PASSED] lmtt =======================
[18:44:58] ================= sriov_packet (1 subtest) =================
[18:44:58] [PASSED] test_descriptor_init
[18:44:58] ================== [PASSED] sriov_packet ===================
[18:44:58] ================= pf_service (11 subtests) =================
[18:44:58] [PASSED] pf_negotiate_any
[18:44:58] [PASSED] pf_negotiate_base_match
[18:44:58] [PASSED] pf_negotiate_base_newer
[18:44:58] [PASSED] pf_negotiate_base_next
[18:44:58] [SKIPPED] pf_negotiate_base_older (no older minor)
[18:44:58] [PASSED] pf_negotiate_base_prev
[18:44:58] [PASSED] pf_negotiate_latest_match
[18:44:58] [PASSED] pf_negotiate_latest_newer
[18:44:58] [PASSED] pf_negotiate_latest_next
[18:44:58] [SKIPPED] pf_negotiate_latest_older (no older minor)
[18:44:58] [SKIPPED] pf_negotiate_latest_prev (no prev major)
[18:44:58] =================== [PASSED] pf_service ====================
[18:44:58] ================= xe_guc_g2g (2 subtests) ==================
[18:44:58] ============== xe_live_guc_g2g_kunit_default ==============
[18:44:58] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[18:44:58] ============== xe_live_guc_g2g_kunit_allmem ===============
[18:44:58] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[18:44:58] =================== [SKIPPED] xe_guc_g2g ===================
[18:44:58] =================== xe_mocs (2 subtests) ===================
[18:44:58] ================ xe_live_mocs_kernel_kunit ================
[18:44:58] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[18:44:58] ================ xe_live_mocs_reset_kunit =================
[18:44:58] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[18:44:58] ==================== [SKIPPED] xe_mocs =====================
[18:44:58] ================= xe_migrate (2 subtests) ==================
[18:44:58] ================= xe_migrate_sanity_kunit =================
[18:44:58] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[18:44:58] ================== xe_validate_ccs_kunit ==================
[18:44:58] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[18:44:58] =================== [SKIPPED] xe_migrate ===================
[18:44:58] ================== xe_dma_buf (1 subtest) ==================
[18:44:58] ==================== xe_dma_buf_kunit =====================
[18:44:58] ================ [SKIPPED] xe_dma_buf_kunit ================
[18:44:58] =================== [SKIPPED] xe_dma_buf ===================
[18:44:58] ================= xe_bo_shrink (1 subtest) =================
[18:44:58] =================== xe_bo_shrink_kunit ====================
[18:44:58] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[18:44:58] ================== [SKIPPED] xe_bo_shrink ==================
[18:44:58] ==================== xe_bo (2 subtests) ====================
[18:44:58] ================== xe_ccs_migrate_kunit ===================
[18:44:58] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[18:44:58] ==================== xe_bo_evict_kunit ====================
[18:44:58] =============== [SKIPPED] xe_bo_evict_kunit ================
[18:44:58] ===================== [SKIPPED] xe_bo ======================
[18:44:58] =================== xe_any (9 subtests) ====================
[18:44:58] [PASSED] test_to_xe
[18:44:58] [PASSED] test_to_dev
[18:44:58] [PASSED] test_to_pdev
[18:44:58] [PASSED] test_to_drm
[18:44:58] [PASSED] test_if_pdev
[18:44:58] [PASSED] test_if_xe
[18:44:58] [PASSED] test_if_tile
[18:44:58] [PASSED] test_if_gt
[18:44:58] [PASSED] test_to_id
[18:44:58] ===================== [PASSED] xe_any ======================
[18:44:58] ==================== args (13 subtests) ====================
[18:44:58] [PASSED] count_args_test
[18:44:58] [PASSED] call_args_example
[18:44:58] [PASSED] call_args_test
[18:44:58] [PASSED] drop_first_arg_example
[18:44:58] [PASSED] drop_first_arg_test
[18:44:58] [PASSED] first_arg_example
[18:44:58] [PASSED] first_arg_test
[18:44:58] [PASSED] last_arg_example
[18:44:58] [PASSED] last_arg_test
[18:44:58] [PASSED] pick_arg_example
[18:44:58] [PASSED] if_args_example
[18:44:58] [PASSED] if_args_test
[18:44:58] [PASSED] sep_comma_example
[18:44:58] ====================== [PASSED] args =======================
[18:44:58] =================== xe_pci (3 subtests) ====================
[18:44:58] ==================== check_graphics_ip ====================
[18:44:58] [PASSED] 12.00 Xe_LP
[18:44:58] [PASSED] 12.10 Xe_LP+
[18:44:58] [PASSED] 12.55 Xe_HPG
[18:44:58] [PASSED] 12.60 Xe_HPC
[18:44:58] [PASSED] 12.70 Xe_LPG
[18:44:58] [PASSED] 12.71 Xe_LPG
[18:44:58] [PASSED] 12.74 Xe_LPG+
[18:44:58] [PASSED] 20.01 Xe2_HPG
[18:44:58] [PASSED] 20.02 Xe2_HPG
[18:44:58] [PASSED] 20.04 Xe2_LPG
[18:44:58] [PASSED] 30.00 Xe3_LPG
[18:44:58] [PASSED] 30.01 Xe3_LPG
[18:44:58] [PASSED] 30.03 Xe3_LPG
[18:44:58] [PASSED] 30.04 Xe3_LPG
[18:44:58] [PASSED] 30.05 Xe3_LPG
[18:44:58] [PASSED] 35.10 Xe3p_LPG
[18:44:58] [PASSED] 35.11 Xe3p_XPC
[18:44:58] ================ [PASSED] check_graphics_ip ================
[18:44:58] ===================== check_media_ip ======================
[18:44:58] [PASSED] 12.00 Xe_M
[18:44:58] [PASSED] 12.55 Xe_HPM
[18:44:58] [PASSED] 13.00 Xe_LPM+
[18:44:58] [PASSED] 13.01 Xe2_HPM
[18:44:58] [PASSED] 20.00 Xe2_LPM
[18:44:58] [PASSED] 30.00 Xe3_LPM
[18:44:58] [PASSED] 30.02 Xe3_LPM
[18:44:58] [PASSED] 35.00 Xe3p_LPM
[18:44:58] [PASSED] 35.03 Xe3p_HPM
[18:44:58] ================= [PASSED] check_media_ip ==================
[18:44:58] =================== check_platform_desc ===================
[18:44:58] [PASSED] 0x9A60 (TIGERLAKE)
[18:44:58] [PASSED] 0x9A68 (TIGERLAKE)
[18:44:58] [PASSED] 0x9A70 (TIGERLAKE)
[18:44:58] [PASSED] 0x9A40 (TIGERLAKE)
[18:44:58] [PASSED] 0x9A49 (TIGERLAKE)
[18:44:58] [PASSED] 0x9A59 (TIGERLAKE)
[18:44:58] [PASSED] 0x9A78 (TIGERLAKE)
[18:44:58] [PASSED] 0x9AC0 (TIGERLAKE)
[18:44:58] [PASSED] 0x9AC9 (TIGERLAKE)
[18:44:58] [PASSED] 0x9AD9 (TIGERLAKE)
[18:44:58] [PASSED] 0x9AF8 (TIGERLAKE)
[18:44:58] [PASSED] 0x4C80 (ROCKETLAKE)
[18:44:58] [PASSED] 0x4C8A (ROCKETLAKE)
[18:44:58] [PASSED] 0x4C8B (ROCKETLAKE)
[18:44:58] [PASSED] 0x4C8C (ROCKETLAKE)
[18:44:58] [PASSED] 0x4C90 (ROCKETLAKE)
[18:44:58] [PASSED] 0x4C9A (ROCKETLAKE)
[18:44:58] [PASSED] 0x4680 (ALDERLAKE_S)
[18:44:58] [PASSED] 0x4682 (ALDERLAKE_S)
[18:44:58] [PASSED] 0x4688 (ALDERLAKE_S)
[18:44:58] [PASSED] 0x468A (ALDERLAKE_S)
[18:44:58] [PASSED] 0x468B (ALDERLAKE_S)
[18:44:58] [PASSED] 0x4690 (ALDERLAKE_S)
[18:44:58] [PASSED] 0x4692 (ALDERLAKE_S)
[18:44:58] [PASSED] 0x4693 (ALDERLAKE_S)
[18:44:58] [PASSED] 0x46A0 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46A1 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46A2 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46A3 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46A6 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46A8 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46AA (ALDERLAKE_P)
[18:44:58] [PASSED] 0x462A (ALDERLAKE_P)
[18:44:58] [PASSED] 0x4626 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x4628 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46B0 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46B1 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46B2 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46B3 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46C0 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46C1 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46C2 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46C3 (ALDERLAKE_P)
[18:44:58] [PASSED] 0x46D0 (ALDERLAKE_N)
[18:44:58] [PASSED] 0x46D1 (ALDERLAKE_N)
[18:44:58] [PASSED] 0x46D2 (ALDERLAKE_N)
[18:44:58] [PASSED] 0x46D3 (ALDERLAKE_N)
[18:44:58] [PASSED] 0x46D4 (ALDERLAKE_N)
[18:44:58] [PASSED] 0xA721 (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7A1 (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7A9 (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7AC (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7AD (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA720 (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7A0 (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7A8 (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7AA (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA7AB (ALDERLAKE_P)
[18:44:59] [PASSED] 0xA780 (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA781 (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA782 (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA783 (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA788 (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA789 (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA78A (ALDERLAKE_S)
[18:44:59] [PASSED] 0xA78B (ALDERLAKE_S)
[18:44:59] [PASSED] 0x4905 (DG1)
[18:44:59] [PASSED] 0x4906 (DG1)
[18:44:59] [PASSED] 0x4907 (DG1)
[18:44:59] [PASSED] 0x4908 (DG1)
[18:44:59] [PASSED] 0x4909 (DG1)
[18:44:59] [PASSED] 0x56C0 (DG2)
[18:44:59] [PASSED] 0x56C2 (DG2)
[18:44:59] [PASSED] 0x56C1 (DG2)
[18:44:59] [PASSED] 0x7D51 (METEORLAKE)
[18:44:59] [PASSED] 0x7DD1 (METEORLAKE)
[18:44:59] [PASSED] 0x7D41 (METEORLAKE)
[18:44:59] [PASSED] 0x7D67 (METEORLAKE)
[18:44:59] [PASSED] 0xB640 (METEORLAKE)
[18:44:59] [PASSED] 0x56A0 (DG2)
[18:44:59] [PASSED] 0x56A1 (DG2)
[18:44:59] [PASSED] 0x56A2 (DG2)
[18:44:59] [PASSED] 0x56BE (DG2)
[18:44:59] [PASSED] 0x56BF (DG2)
[18:44:59] [PASSED] 0x5690 (DG2)
[18:44:59] [PASSED] 0x5691 (DG2)
[18:44:59] [PASSED] 0x5692 (DG2)
[18:44:59] [PASSED] 0x56A5 (DG2)
[18:44:59] [PASSED] 0x56A6 (DG2)
[18:44:59] [PASSED] 0x56B0 (DG2)
[18:44:59] [PASSED] 0x56B1 (DG2)
[18:44:59] [PASSED] 0x56BA (DG2)
[18:44:59] [PASSED] 0x56BB (DG2)
[18:44:59] [PASSED] 0x56BC (DG2)
[18:44:59] [PASSED] 0x56BD (DG2)
[18:44:59] [PASSED] 0x5693 (DG2)
[18:44:59] [PASSED] 0x5694 (DG2)
[18:44:59] [PASSED] 0x5695 (DG2)
[18:44:59] [PASSED] 0x56A3 (DG2)
[18:44:59] [PASSED] 0x56A4 (DG2)
[18:44:59] [PASSED] 0x56B2 (DG2)
[18:44:59] [PASSED] 0x56B3 (DG2)
[18:44:59] [PASSED] 0x5696 (DG2)
[18:44:59] [PASSED] 0x5697 (DG2)
[18:44:59] [PASSED] 0xB69 (PVC)
[18:44:59] [PASSED] 0xB6E (PVC)
[18:44:59] [PASSED] 0xBD4 (PVC)
[18:44:59] [PASSED] 0xBD5 (PVC)
[18:44:59] [PASSED] 0xBD6 (PVC)
[18:44:59] [PASSED] 0xBD7 (PVC)
[18:44:59] [PASSED] 0xBD8 (PVC)
[18:44:59] [PASSED] 0xBD9 (PVC)
[18:44:59] [PASSED] 0xBDA (PVC)
[18:44:59] [PASSED] 0xBDB (PVC)
[18:44:59] [PASSED] 0xBE0 (PVC)
[18:44:59] [PASSED] 0xBE1 (PVC)
[18:44:59] [PASSED] 0xBE5 (PVC)
[18:44:59] [PASSED] 0x7D40 (METEORLAKE)
[18:44:59] [PASSED] 0x7D45 (METEORLAKE)
[18:44:59] [PASSED] 0x7D55 (METEORLAKE)
[18:44:59] [PASSED] 0x7D60 (METEORLAKE)
[18:44:59] [PASSED] 0x7DD5 (METEORLAKE)
[18:44:59] [PASSED] 0x6420 (LUNARLAKE)
[18:44:59] [PASSED] 0x64A0 (LUNARLAKE)
[18:44:59] [PASSED] 0x64B0 (LUNARLAKE)
[18:44:59] [PASSED] 0xE202 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE209 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE20B (BATTLEMAGE)
[18:44:59] [PASSED] 0xE20C (BATTLEMAGE)
[18:44:59] [PASSED] 0xE20D (BATTLEMAGE)
[18:44:59] [PASSED] 0xE210 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE211 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE212 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE216 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE220 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE221 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE222 (BATTLEMAGE)
[18:44:59] [PASSED] 0xE223 (BATTLEMAGE)
[18:44:59] [PASSED] 0xB080 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB081 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB082 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB083 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB084 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB085 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB086 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB087 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB08F (PANTHERLAKE)
[18:44:59] [PASSED] 0xB090 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB0A0 (PANTHERLAKE)
[18:44:59] [PASSED] 0xB0B0 (PANTHERLAKE)
[18:44:59] [PASSED] 0xFD80 (PANTHERLAKE)
[18:44:59] [PASSED] 0xFD81 (PANTHERLAKE)
[18:44:59] [PASSED] 0xD740 (NOVALAKE_S)
[18:44:59] [PASSED] 0xD741 (NOVALAKE_S)
[18:44:59] [PASSED] 0xD742 (NOVALAKE_S)
[18:44:59] [PASSED] 0xD743 (NOVALAKE_S)
[18:44:59] [PASSED] 0xD745 (NOVALAKE_S)
[18:44:59] [PASSED] 0xD74A (NOVALAKE_S)
[18:44:59] [PASSED] 0xD74B (NOVALAKE_S)
[18:44:59] [PASSED] 0x674C (CRESCENTISLAND)
[18:44:59] [PASSED] 0x674D (CRESCENTISLAND)
[18:44:59] [PASSED] 0x674E (CRESCENTISLAND)
[18:44:59] [PASSED] 0x674F (CRESCENTISLAND)
[18:44:59] [PASSED] 0x6750 (CRESCENTISLAND)
[18:44:59] [PASSED] 0xD750 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD751 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD752 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD753 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD754 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD755 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD756 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD757 (NOVALAKE_P)
[18:44:59] [PASSED] 0xD75F (NOVALAKE_P)
[18:44:59] =============== [PASSED] check_platform_desc ===============
[18:44:59] ===================== [PASSED] xe_pci ======================
[18:44:59] ============= xe_rtp_tables_test (5 subtests) ==============
[18:44:59] ================== xe_rtp_table_gt_test ===================
[18:44:59] [PASSED] gt_was/14011060649
[18:44:59] [PASSED] gt_was/14011059788
[18:44:59] [PASSED] gt_was/14015795083
[18:44:59] [PASSED] gt_was/16021867713
[18:44:59] [PASSED] gt_was/14019449301
[18:44:59] [PASSED] gt_was/16028005424
[18:44:59] [PASSED] gt_was/14026578760
[18:44:59] [PASSED] gt_was/1409420604
[18:44:59] [PASSED] gt_was/1408615072
[18:44:59] [PASSED] gt_was/22010523718
[18:44:59] [PASSED] gt_was/14011006942
[18:44:59] [PASSED] gt_was/14014830051
[18:44:59] [PASSED] gt_was/18018781329
[18:44:59] [PASSED] gt_was/1509235366
[18:44:59] [PASSED] gt_was/18018781329
[18:44:59] [PASSED] gt_was/16016694945
[18:44:59] [PASSED] gt_was/14018575942
[18:44:59] [PASSED] gt_was/22016670082
[18:44:59] [PASSED] gt_was/22016670082
[18:44:59] [PASSED] gt_was/14017421178
[18:44:59] [PASSED] gt_was/16025250150
[18:44:59] [PASSED] gt_was/14021871409
[18:44:59] [PASSED] gt_was/16021865536
[18:44:59] [PASSED] gt_was/14021486841
[18:44:59] [PASSED] gt_was/14025160223
[18:44:59] [PASSED] gt_was/14026144927, 16029437861, 14026127056
[18:44:59] [PASSED] gt_was/14025635424
[18:44:59] [PASSED] gt_was/16028005424
[18:44:59] ============== [PASSED] xe_rtp_table_gt_test ===============
[18:44:59] ================== xe_rtp_table_gt_test ===================
[18:44:59] [PASSED] gt_tunings/Tuning: Blend Fill Caching Optimization Disable
[18:44:59] [PASSED] gt_tunings/Tuning: 32B Access Enable
[18:44:59] [PASSED] gt_tunings/Tuning: L3 cache
[18:44:59] [PASSED] gt_tunings/Tuning: L3 cache - media
[18:44:59] [PASSED] gt_tunings/Tuning: Compression Overfetch
[18:44:59] [PASSED] gt_tunings/Tuning: Compression Overfetch - media
[18:44:59] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3
[18:44:59] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3 - media
[18:44:59] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only
[18:44:59] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only - media
[18:44:59] [PASSED] gt_tunings/Tuning: Stateless compression control
[18:44:59] [PASSED] gt_tunings/Tuning: Stateless compression control - media
[18:44:59] [PASSED] gt_tunings/Tuning: L3 RW flush all Cache
[18:44:59] [PASSED] gt_tunings/Tuning: L3 RW flush all cache - media
[18:44:59] [PASSED] gt_tunings/Tuning: Set STLB Bank Hash Mode to 4KB
[18:44:59] ============== [PASSED] xe_rtp_table_gt_test ===============
[18:44:59] ================== xe_rtp_table_oob_test ==================
[18:44:59] [PASSED] oob_was/1607983814
[18:44:59] [PASSED] oob_was/16010904313
[18:44:59] [PASSED] oob_was/18022495364
[18:44:59] [PASSED] oob_was/22012773006
[18:44:59] [PASSED] oob_was/14014475959
[18:44:59] [PASSED] oob_was/22011391025
[18:44:59] [PASSED] oob_was/22012727170
[18:44:59] [PASSED] oob_was/22012727685
[18:44:59] [PASSED] oob_was/22016596838
[18:44:59] [PASSED] oob_was/18020744125
[18:44:59] [PASSED] oob_was/1409600907
[18:44:59] [PASSED] oob_was/22014953428
[18:44:59] [PASSED] oob_was/16017236439
[18:44:59] [PASSED] oob_was/14019821291
[18:44:59] [PASSED] oob_was/14015076503
[18:44:59] [PASSED] oob_was/14018913170
[18:44:59] [PASSED] oob_was/14018094691
[18:44:59] [PASSED] oob_was/18024947630
[18:44:59] [PASSED] oob_was/16022287689
[18:44:59] [PASSED] oob_was/13011645652
[18:44:59] [PASSED] oob_was/14022293748
[18:44:59] [PASSED] oob_was/22019794406
[18:44:59] [PASSED] oob_was/22019338487
[18:44:59] [PASSED] oob_was/16023588340
[18:44:59] [PASSED] oob_was/14019789679
[18:44:59] [PASSED] oob_was/14022866841
[18:44:59] [PASSED] oob_was/16021333562
[18:44:59] [PASSED] oob_was/14016712196
[18:44:59] [PASSED] oob_was/14015568240
[18:44:59] [PASSED] oob_was/18013179988
[18:44:59] [PASSED] oob_was/1508761755
[18:44:59] [PASSED] oob_was/16023105232
[18:44:59] [PASSED] oob_was/16026508708
[18:44:59] [PASSED] oob_was/14020001231
[18:44:59] [PASSED] oob_was/16023683509
[18:44:59] [PASSED] oob_was/14025515070
[18:44:59] [PASSED] oob_was/15015404425_disable
[18:44:59] [PASSED] oob_was/16026007364
[18:44:59] [PASSED] oob_was/14020316580
[18:44:59] [PASSED] oob_was/14025883347
[18:44:59] [PASSED] oob_was/16029380221
[18:44:59] [PASSED] oob_was/22022079272
[18:44:59] [PASSED] oob_was/16029897822
[18:44:59] [PASSED] oob_was/14027054324
[18:44:59] ============== [PASSED] xe_rtp_table_oob_test ==============
[18:44:59] ================ xe_rtp_table_dev_oob_test ================
[18:44:59] [PASSED] device_oob_was/22010954014
[18:44:59] [PASSED] device_oob_was/15015404425
[18:44:59] [PASSED] device_oob_was/22019338487_display
[18:44:59] [PASSED] device_oob_was/14022085890
[18:44:59] [PASSED] device_oob_was/14026539277
[18:44:59] [PASSED] device_oob_was/14026633728
[18:44:59] [PASSED] device_oob_was/14026746987
[18:44:59] [PASSED] device_oob_was/14026779378
[18:44:59] ============ [PASSED] xe_rtp_table_dev_oob_test ============
[18:44:59] ========== xe_rtp_table_missing_upper_bound_test ==========
[18:44:59] [PASSED] register_whitelist/WaAllowPMDepthAndInvocationCountAccessFromUMD, 1408556865
[18:44:59] [PASSED] register_whitelist/1508744258, 14012131227, 1808121037
[18:44:59] [PASSED] register_whitelist/1806527549
[18:44:59] [PASSED] register_whitelist/allow_read_ctx_timestamp
[18:44:59] [PASSED] register_whitelist/allow_read_queue_timestamp
[18:44:59] [PASSED] register_whitelist/16014440446
[18:44:59] [PASSED] register_whitelist/16017236439
[18:44:59] [PASSED] register_whitelist/16020183090
[18:44:59] [PASSED] register_whitelist/14024997852
[18:44:59] [PASSED] register_whitelist/14024997852
[18:44:59] ====== [PASSED] xe_rtp_table_missing_upper_bound_test ======
[18:44:59] =============== [PASSED] xe_rtp_tables_test ================
[18:44:59] =================== xe_rtp (3 subtests) ====================
[18:44:59] =================== xe_rtp_rules_tests ====================
[18:44:59] [PASSED] no
[18:44:59] [PASSED] yes
[18:44:59] [PASSED] no-and-no
[18:44:59] [PASSED] no-and-yes
[18:44:59] [PASSED] yes-and-no
[18:44:59] [PASSED] yes-and-yes
[18:44:59] [PASSED] no-or-no
[18:44:59] [PASSED] no-or-yes
[18:44:59] [PASSED] yes-or-no
[18:44:59] [PASSED] yes-or-yes
[18:44:59] [PASSED] no-yes-or-yes-no
[18:44:59] [PASSED] no-yes-or-yes-yes
[18:44:59] [PASSED] yes-yes-or-no-yes
[18:44:59] [PASSED] yes-yes-or-yes-yes
[18:44:59] [PASSED] no-no-or-yes-or-no
[18:44:59] [PASSED] or
[18:44:59] [PASSED] or-yes
[18:44:59] [PASSED] or-no
[18:44:59] [PASSED] yes-or
[18:44:59] [PASSED] no-or
[18:44:59] [PASSED] no-or-or-yes
[18:44:59] [PASSED] yes-or-or-no
[18:44:59] [PASSED] no-or-or-no
[18:44:59] [PASSED] missing-context-engine-class
[18:44:59] [PASSED] missing-context-engine-class-or-yes
[18:44:59] [PASSED] missing-context-engine-class-or-or-yes
[18:44:59] =============== [PASSED] xe_rtp_rules_tests ================
[18:44:59] =============== xe_rtp_process_to_sr_tests ================
[18:44:59] [PASSED] coalesce-same-reg
[18:44:59] [PASSED] coalesce-same-reg-literal-and-func
[18:44:59] [PASSED] no-match-no-add
[18:44:59] [PASSED] two-regs-two-entries
[18:44:59] [PASSED] clr-one-set-other
[18:44:59] [PASSED] set-field
[18:44:59] [PASSED] conflict-duplicate
[18:44:59] [PASSED] conflict-not-disjoint
[18:44:59] [PASSED] conflict-not-disjoint-literal-and-func
[18:44:59] [PASSED] conflict-reg-type
[18:44:59] [PASSED] bad-mcr-reg-forced-to-regular
[18:44:59] [PASSED] bad-regular-reg-forced-to-mcr
[18:44:59] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[18:44:59] ================== xe_rtp_process_tests ===================
[18:44:59] [PASSED] active1
[18:44:59] [PASSED] active2
[18:44:59] [PASSED] active-inactive
[18:44:59] [PASSED] inactive-active
[18:44:59] [PASSED] inactive-active-inactive
[18:44:59] [PASSED] inactive-inactive-inactive
[18:44:59] ============== [PASSED] xe_rtp_process_tests ===============
[18:44:59] ===================== [PASSED] xe_rtp ======================
[18:44:59] ==================== xe_wa (1 subtest) =====================
[18:44:59] ======================== xe_wa_gt =========================
[18:44:59] [PASSED] TIGERLAKE B0
[18:44:59] [PASSED] DG1 A0
[18:44:59] [PASSED] DG1 B0
[18:44:59] [PASSED] ALDERLAKE_S A0
[18:44:59] [PASSED] ALDERLAKE_S B0
[18:44:59] [PASSED] ALDERLAKE_S C0
[18:44:59] [PASSED] ALDERLAKE_S D0
[18:44:59] [PASSED] ALDERLAKE_P A0
[18:44:59] [PASSED] ALDERLAKE_P B0
[18:44:59] [PASSED] ALDERLAKE_P C0
[18:44:59] [PASSED] ALDERLAKE_S RPLS D0
[18:44:59] [PASSED] ALDERLAKE_P RPLU E0
[18:44:59] [PASSED] DG2 G10 C0
[18:44:59] [PASSED] DG2 G11 B1
[18:44:59] [PASSED] DG2 G12 A1
[18:44:59] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[18:44:59] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[18:44:59] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[18:44:59] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[18:44:59] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[18:44:59] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[18:44:59] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[18:44:59] ==================== [PASSED] xe_wa_gt =====================
[18:44:59] ====================== [PASSED] xe_wa ======================
[18:44:59] ============================================================
[18:44:59] Testing complete. Ran 789 tests: passed: 761, skipped: 28
[18:44:59] Elapsed time: 36.914s total, 4.440s configuring, 31.758s building, 0.673s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[18:44:59] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[18:45:01] 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
[18:45:25] Starting KUnit Kernel (1/1)...
[18:45:25] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[18:45:25] ============ drm_test_pick_cmdline (2 subtests) ============
[18:45:25] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[18:45:25] =============== drm_test_pick_cmdline_named ===============
[18:45:25] [PASSED] NTSC
[18:45:25] [PASSED] NTSC-J
[18:45:25] [PASSED] PAL
[18:45:25] [PASSED] PAL-M
[18:45:25] =========== [PASSED] drm_test_pick_cmdline_named ===========
[18:45:25] ============== [PASSED] drm_test_pick_cmdline ==============
[18:45:25] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[18:45:25] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[18:45:25] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[18:45:25] =========== drm_validate_clone_mode (2 subtests) ===========
[18:45:25] ============== drm_test_check_in_clone_mode ===============
[18:45:25] [PASSED] in_clone_mode
[18:45:25] [PASSED] not_in_clone_mode
[18:45:25] ========== [PASSED] drm_test_check_in_clone_mode ===========
[18:45:25] =============== drm_test_check_valid_clones ===============
[18:45:25] [PASSED] not_in_clone_mode
[18:45:25] [PASSED] valid_clone
[18:45:25] [PASSED] invalid_clone
[18:45:25] =========== [PASSED] drm_test_check_valid_clones ===========
[18:45:25] ============= [PASSED] drm_validate_clone_mode =============
[18:45:25] ============= drm_validate_modeset (1 subtest) =============
[18:45:25] [PASSED] drm_test_check_connector_changed_modeset
[18:45:25] ============== [PASSED] drm_validate_modeset ===============
[18:45:25] ====== drm_test_bridge_get_current_state (1 subtest) =======
[18:45:25] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[18:45:25] ======== [PASSED] drm_test_bridge_get_current_state ========
[18:45:25] ====== drm_test_bridge_helper_reset_crtc (3 subtests) ======
[18:45:25] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[18:45:25] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[18:45:25] [PASSED] drm_test_drm_bridge_helper_hdmi_output_bus_fmts
[18:45:25] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[18:45:25] ============== drm_bridge_alloc (2 subtests) ===============
[18:45:25] [PASSED] drm_test_drm_bridge_alloc_basic
[18:45:25] [PASSED] drm_test_drm_bridge_alloc_get_put
[18:45:25] ================ [PASSED] drm_bridge_alloc =================
[18:45:25] ============= drm_bridge_bus_fmt (5 subtests) ==============
[18:45:25] [PASSED] drm_test_bridge_rgb_yuv_rgb
[18:45:25] [PASSED] drm_test_bridge_must_convert_to_yuv444
[18:45:25] [PASSED] drm_test_bridge_hdmi_auto_rgb
[18:45:25] [PASSED] drm_test_bridge_auto_first
[18:45:25] [PASSED] drm_test_bridge_rgb_yuv_no_path
[18:45:25] =============== [PASSED] drm_bridge_bus_fmt ================
[18:45:25] ============= drm_cmdline_parser (40 subtests) =============
[18:45:25] [PASSED] drm_test_cmdline_force_d_only
[18:45:25] [PASSED] drm_test_cmdline_force_D_only_dvi
[18:45:25] [PASSED] drm_test_cmdline_force_D_only_hdmi
[18:45:25] [PASSED] drm_test_cmdline_force_D_only_not_digital
[18:45:25] [PASSED] drm_test_cmdline_force_e_only
[18:45:25] [PASSED] drm_test_cmdline_res
[18:45:25] [PASSED] drm_test_cmdline_res_vesa
[18:45:25] [PASSED] drm_test_cmdline_res_vesa_rblank
[18:45:25] [PASSED] drm_test_cmdline_res_rblank
[18:45:25] [PASSED] drm_test_cmdline_res_bpp
[18:45:25] [PASSED] drm_test_cmdline_res_refresh
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[18:45:25] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[18:45:25] [PASSED] drm_test_cmdline_res_margins_force_on
[18:45:25] [PASSED] drm_test_cmdline_res_vesa_margins
[18:45:25] [PASSED] drm_test_cmdline_name
[18:45:25] [PASSED] drm_test_cmdline_name_bpp
[18:45:25] [PASSED] drm_test_cmdline_name_option
[18:45:25] [PASSED] drm_test_cmdline_name_bpp_option
[18:45:25] [PASSED] drm_test_cmdline_rotate_0
[18:45:25] [PASSED] drm_test_cmdline_rotate_90
[18:45:25] [PASSED] drm_test_cmdline_rotate_180
[18:45:25] [PASSED] drm_test_cmdline_rotate_270
[18:45:25] [PASSED] drm_test_cmdline_hmirror
[18:45:25] [PASSED] drm_test_cmdline_vmirror
[18:45:25] [PASSED] drm_test_cmdline_margin_options
[18:45:25] [PASSED] drm_test_cmdline_multiple_options
[18:45:25] [PASSED] drm_test_cmdline_bpp_extra_and_option
[18:45:25] [PASSED] drm_test_cmdline_extra_and_option
[18:45:25] [PASSED] drm_test_cmdline_freestanding_options
[18:45:25] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[18:45:25] [PASSED] drm_test_cmdline_panel_orientation
[18:45:25] ================ drm_test_cmdline_invalid =================
[18:45:25] [PASSED] margin_only
[18:45:25] [PASSED] interlace_only
[18:45:25] [PASSED] res_missing_x
[18:45:25] [PASSED] res_missing_y
[18:45:25] [PASSED] res_bad_y
[18:45:25] [PASSED] res_missing_y_bpp
[18:45:25] [PASSED] res_bad_bpp
[18:45:25] [PASSED] res_bad_refresh
[18:45:25] [PASSED] res_bpp_refresh_force_on_off
[18:45:25] [PASSED] res_invalid_mode
[18:45:25] [PASSED] res_bpp_wrong_place_mode
[18:45:25] [PASSED] name_bpp_refresh
[18:45:25] [PASSED] name_refresh
[18:45:25] [PASSED] name_refresh_wrong_mode
[18:45:25] [PASSED] name_refresh_invalid_mode
[18:45:25] [PASSED] rotate_multiple
[18:45:25] [PASSED] rotate_invalid_val
[18:45:25] [PASSED] rotate_truncated
[18:45:25] [PASSED] invalid_option
[18:45:25] [PASSED] invalid_tv_option
[18:45:25] [PASSED] truncated_tv_option
[18:45:25] ============ [PASSED] drm_test_cmdline_invalid =============
[18:45:25] =============== drm_test_cmdline_tv_options ===============
[18:45:25] [PASSED] NTSC
[18:45:25] [PASSED] NTSC_443
[18:45:25] [PASSED] NTSC_J
[18:45:25] [PASSED] PAL
[18:45:25] [PASSED] PAL_M
[18:45:25] [PASSED] PAL_N
[18:45:25] [PASSED] SECAM
[18:45:25] [PASSED] MONO_525
[18:45:25] [PASSED] MONO_625
[18:45:25] =========== [PASSED] drm_test_cmdline_tv_options ===========
[18:45:25] =============== [PASSED] drm_cmdline_parser ================
[18:45:25] ========== drmm_connector_hdmi_init (20 subtests) ==========
[18:45:25] [PASSED] drm_test_connector_hdmi_init_valid
[18:45:25] [PASSED] drm_test_connector_hdmi_init_bpc_8
[18:45:25] [PASSED] drm_test_connector_hdmi_init_bpc_10
[18:45:25] [PASSED] drm_test_connector_hdmi_init_bpc_12
[18:45:25] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[18:45:25] [PASSED] drm_test_connector_hdmi_init_bpc_null
[18:45:25] [PASSED] drm_test_connector_hdmi_init_formats_empty
[18:45:25] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[18:45:25] === drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[18:45:25] [PASSED] supported_formats=0x9 yuv420_allowed=1
[18:45:25] [PASSED] supported_formats=0x9 yuv420_allowed=0
[18:45:25] [PASSED] supported_formats=0x5 yuv420_allowed=1
[18:45:25] [PASSED] supported_formats=0x5 yuv420_allowed=0
[18:45:25] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[18:45:25] [PASSED] drm_test_connector_hdmi_init_null_ddc
[18:45:25] [PASSED] drm_test_connector_hdmi_init_null_product
[18:45:25] [PASSED] drm_test_connector_hdmi_init_null_vendor
[18:45:25] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[18:45:25] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[18:45:25] [PASSED] drm_test_connector_hdmi_init_product_valid
[18:45:25] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[18:45:25] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[18:45:25] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[18:45:25] ========= drm_test_connector_hdmi_init_type_valid =========
[18:45:25] [PASSED] HDMI-A
[18:45:25] [PASSED] HDMI-B
[18:45:25] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[18:45:25] ======== drm_test_connector_hdmi_init_type_invalid ========
[18:45:25] [PASSED] Unknown
[18:45:25] [PASSED] VGA
[18:45:25] [PASSED] DVI-I
[18:45:25] [PASSED] DVI-D
[18:45:25] [PASSED] DVI-A
[18:45:25] [PASSED] Composite
[18:45:25] [PASSED] SVIDEO
[18:45:25] [PASSED] LVDS
[18:45:25] [PASSED] Component
[18:45:25] [PASSED] DIN
[18:45:25] [PASSED] DP
[18:45:25] [PASSED] TV
[18:45:25] [PASSED] eDP
[18:45:25] [PASSED] Virtual
[18:45:25] [PASSED] DSI
[18:45:25] [PASSED] DPI
[18:45:25] [PASSED] Writeback
[18:45:25] [PASSED] SPI
[18:45:25] [PASSED] USB
[18:45:25] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[18:45:25] ============ [PASSED] drmm_connector_hdmi_init =============
[18:45:25] ============= drmm_connector_init (3 subtests) =============
[18:45:25] [PASSED] drm_test_drmm_connector_init
[18:45:25] [PASSED] drm_test_drmm_connector_init_null_ddc
[18:45:25] ========= drm_test_drmm_connector_init_type_valid =========
[18:45:25] [PASSED] Unknown
[18:45:25] [PASSED] VGA
[18:45:25] [PASSED] DVI-I
[18:45:25] [PASSED] DVI-D
[18:45:25] [PASSED] DVI-A
[18:45:25] [PASSED] Composite
[18:45:25] [PASSED] SVIDEO
[18:45:25] [PASSED] LVDS
[18:45:25] [PASSED] Component
[18:45:25] [PASSED] DIN
[18:45:25] [PASSED] DP
[18:45:25] [PASSED] HDMI-A
[18:45:25] [PASSED] HDMI-B
[18:45:25] [PASSED] TV
[18:45:25] [PASSED] eDP
[18:45:25] [PASSED] Virtual
[18:45:25] [PASSED] DSI
[18:45:25] [PASSED] DPI
[18:45:25] [PASSED] Writeback
[18:45:25] [PASSED] SPI
[18:45:25] [PASSED] USB
[18:45:25] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[18:45:25] =============== [PASSED] drmm_connector_init ===============
[18:45:25] ========= drm_connector_dynamic_init (6 subtests) ==========
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_init
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_init_properties
[18:45:25] ===== drm_test_drm_connector_dynamic_init_type_valid ======
[18:45:25] [PASSED] Unknown
[18:45:25] [PASSED] VGA
[18:45:25] [PASSED] DVI-I
[18:45:25] [PASSED] DVI-D
[18:45:25] [PASSED] DVI-A
[18:45:25] [PASSED] Composite
[18:45:25] [PASSED] SVIDEO
[18:45:25] [PASSED] LVDS
[18:45:25] [PASSED] Component
[18:45:25] [PASSED] DIN
[18:45:25] [PASSED] DP
[18:45:25] [PASSED] HDMI-A
[18:45:25] [PASSED] HDMI-B
[18:45:25] [PASSED] TV
[18:45:25] [PASSED] eDP
[18:45:25] [PASSED] Virtual
[18:45:25] [PASSED] DSI
[18:45:25] [PASSED] DPI
[18:45:25] [PASSED] Writeback
[18:45:25] [PASSED] SPI
[18:45:25] [PASSED] USB
[18:45:25] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[18:45:25] ======== drm_test_drm_connector_dynamic_init_name =========
[18:45:25] [PASSED] Unknown
[18:45:25] [PASSED] VGA
[18:45:25] [PASSED] DVI-I
[18:45:25] [PASSED] DVI-D
[18:45:25] [PASSED] DVI-A
[18:45:25] [PASSED] Composite
[18:45:25] [PASSED] SVIDEO
[18:45:25] [PASSED] LVDS
[18:45:25] [PASSED] Component
[18:45:25] [PASSED] DIN
[18:45:25] [PASSED] DP
[18:45:25] [PASSED] HDMI-A
[18:45:25] [PASSED] HDMI-B
[18:45:25] [PASSED] TV
[18:45:25] [PASSED] eDP
[18:45:25] [PASSED] Virtual
[18:45:25] [PASSED] DSI
[18:45:25] [PASSED] DPI
[18:45:25] [PASSED] Writeback
[18:45:25] [PASSED] SPI
[18:45:25] [PASSED] USB
[18:45:25] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[18:45:25] =========== [PASSED] drm_connector_dynamic_init ============
[18:45:25] ==== drm_connector_dynamic_register_early (4 subtests) =====
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[18:45:25] ====== [PASSED] drm_connector_dynamic_register_early =======
[18:45:25] ======= drm_connector_dynamic_register (7 subtests) ========
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[18:45:25] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[18:45:25] ========= [PASSED] drm_connector_dynamic_register ==========
[18:45:25] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[18:45:25] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[18:45:25] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[18:45:25] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[18:45:25] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[18:45:25] ========== drm_test_get_tv_mode_from_name_valid ===========
[18:45:25] [PASSED] NTSC
[18:45:25] [PASSED] NTSC-443
[18:45:25] [PASSED] NTSC-J
[18:45:25] [PASSED] PAL
[18:45:25] [PASSED] PAL-M
[18:45:25] [PASSED] PAL-N
[18:45:25] [PASSED] SECAM
[18:45:25] [PASSED] Mono
[18:45:25] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[18:45:25] [PASSED] drm_test_get_tv_mode_from_name_truncated
[18:45:25] ============ [PASSED] drm_get_tv_mode_from_name ============
[18:45:25] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[18:45:25] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[18:45:25] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[18:45:25] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[18:45:25] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[18:45:25] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[18:45:25] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[18:45:25] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid =
[18:45:25] [PASSED] VIC 96
[18:45:25] [PASSED] VIC 97
[18:45:25] [PASSED] VIC 101
[18:45:25] [PASSED] VIC 102
[18:45:25] [PASSED] VIC 106
[18:45:25] [PASSED] VIC 107
[18:45:25] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[18:45:25] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[18:45:25] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[18:45:25] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[18:45:25] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[18:45:25] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[18:45:25] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[18:45:25] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[18:45:25] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name ====
[18:45:25] [PASSED] Automatic
[18:45:25] [PASSED] Full
[18:45:25] [PASSED] Limited 16:235
[18:45:25] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[18:45:25] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[18:45:25] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[18:45:25] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[18:45:25] === drm_test_drm_hdmi_connector_get_output_format_name ====
[18:45:25] [PASSED] RGB
[18:45:25] [PASSED] YUV 4:2:0
[18:45:25] [PASSED] YUV 4:2:2
[18:45:25] [PASSED] YUV 4:4:4
[18:45:25] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[18:45:25] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[18:45:25] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[18:45:25] ============= drm_damage_helper (21 subtests) ==============
[18:45:25] [PASSED] drm_test_damage_iter_no_damage
[18:45:25] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[18:45:25] [PASSED] drm_test_damage_iter_no_damage_src_moved
[18:45:25] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[18:45:25] [PASSED] drm_test_damage_iter_no_damage_not_visible
[18:45:25] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[18:45:25] [PASSED] drm_test_damage_iter_no_damage_no_fb
[18:45:25] [PASSED] drm_test_damage_iter_simple_damage
[18:45:25] [PASSED] drm_test_damage_iter_single_damage
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_outside_src
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_src_moved
[18:45:25] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[18:45:25] [PASSED] drm_test_damage_iter_damage
[18:45:25] [PASSED] drm_test_damage_iter_damage_one_intersect
[18:45:25] [PASSED] drm_test_damage_iter_damage_one_outside
[18:45:25] [PASSED] drm_test_damage_iter_damage_src_moved
[18:45:25] [PASSED] drm_test_damage_iter_damage_not_visible
[18:45:25] ================ [PASSED] drm_damage_helper ================
[18:45:25] ============== drm_dp_mst_helper (3 subtests) ==============
[18:45:25] ============== drm_test_dp_mst_calc_pbn_mode ==============
[18:45:25] [PASSED] Clock 154000 BPP 30 DSC disabled
[18:45:25] [PASSED] Clock 234000 BPP 30 DSC disabled
[18:45:25] [PASSED] Clock 297000 BPP 24 DSC disabled
[18:45:25] [PASSED] Clock 332880 BPP 24 DSC enabled
[18:45:25] [PASSED] Clock 324540 BPP 24 DSC enabled
[18:45:25] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[18:45:25] ============== drm_test_dp_mst_calc_pbn_div ===============
[18:45:25] [PASSED] Link rate 2000000 lane count 4
[18:45:25] [PASSED] Link rate 2000000 lane count 2
[18:45:25] [PASSED] Link rate 2000000 lane count 1
[18:45:25] [PASSED] Link rate 1350000 lane count 4
[18:45:25] [PASSED] Link rate 1350000 lane count 2
[18:45:25] [PASSED] Link rate 1350000 lane count 1
[18:45:25] [PASSED] Link rate 1000000 lane count 4
[18:45:25] [PASSED] Link rate 1000000 lane count 2
[18:45:25] [PASSED] Link rate 1000000 lane count 1
[18:45:25] [PASSED] Link rate 810000 lane count 4
[18:45:25] [PASSED] Link rate 810000 lane count 2
[18:45:26] [PASSED] Link rate 810000 lane count 1
[18:45:26] [PASSED] Link rate 540000 lane count 4
[18:45:26] [PASSED] Link rate 540000 lane count 2
[18:45:26] [PASSED] Link rate 540000 lane count 1
[18:45:26] [PASSED] Link rate 270000 lane count 4
[18:45:26] [PASSED] Link rate 270000 lane count 2
[18:45:26] [PASSED] Link rate 270000 lane count 1
[18:45:26] [PASSED] Link rate 162000 lane count 4
[18:45:26] [PASSED] Link rate 162000 lane count 2
[18:45:26] [PASSED] Link rate 162000 lane count 1
[18:45:26] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[18:45:26] ========= drm_test_dp_mst_sideband_msg_req_decode =========
[18:45:26] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[18:45:26] [PASSED] DP_POWER_UP_PHY with port number
[18:45:26] [PASSED] DP_POWER_DOWN_PHY with port number
[18:45:26] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[18:45:26] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[18:45:26] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[18:45:26] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[18:45:26] [PASSED] DP_QUERY_PAYLOAD with port number
[18:45:26] [PASSED] DP_QUERY_PAYLOAD with VCPI
[18:45:26] [PASSED] DP_REMOTE_DPCD_READ with port number
[18:45:26] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[18:45:26] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[18:45:26] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[18:45:26] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[18:45:26] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[18:45:26] [PASSED] DP_REMOTE_I2C_READ with port number
[18:45:26] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[18:45:26] [PASSED] DP_REMOTE_I2C_READ with transactions array
[18:45:26] [PASSED] DP_REMOTE_I2C_WRITE with port number
[18:45:26] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[18:45:26] [PASSED] DP_REMOTE_I2C_WRITE with data array
[18:45:26] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[18:45:26] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[18:45:26] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[18:45:26] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[18:45:26] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[18:45:26] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[18:45:26] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[18:45:26] ================ [PASSED] drm_dp_mst_helper ================
[18:45:26] ================== drm_exec (7 subtests) ===================
[18:45:26] [PASSED] sanitycheck
[18:45:26] [PASSED] test_lock
[18:45:26] [PASSED] test_lock_unlock
[18:45:26] [PASSED] test_duplicates
[18:45:26] [PASSED] test_prepare
[18:45:26] [PASSED] test_prepare_array
[18:45:26] [PASSED] test_multiple_loops
[18:45:26] ==================== [PASSED] drm_exec =====================
[18:45:26] =========== drm_format_helper_test (17 subtests) ===========
[18:45:26] ============== drm_test_fb_xrgb8888_to_gray8 ==============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[18:45:26] ============= drm_test_fb_xrgb8888_to_rgb332 ==============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[18:45:26] ============= drm_test_fb_xrgb8888_to_rgb565 ==============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[18:45:26] ============ drm_test_fb_xrgb8888_to_xrgb1555 =============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[18:45:26] ============ drm_test_fb_xrgb8888_to_argb1555 =============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[18:45:26] ============ drm_test_fb_xrgb8888_to_rgba5551 =============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[18:45:26] ============= drm_test_fb_xrgb8888_to_rgb888 ==============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[18:45:26] ============= drm_test_fb_xrgb8888_to_bgr888 ==============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[18:45:26] ============ drm_test_fb_xrgb8888_to_argb8888 =============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[18:45:26] =========== drm_test_fb_xrgb8888_to_xrgb2101010 ===========
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[18:45:26] =========== drm_test_fb_xrgb8888_to_argb2101010 ===========
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[18:45:26] ============== drm_test_fb_xrgb8888_to_mono ===============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[18:45:26] ==================== drm_test_fb_swab =====================
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ================ [PASSED] drm_test_fb_swab =================
[18:45:26] ============ drm_test_fb_xrgb8888_to_xbgr8888 =============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[18:45:26] ============ drm_test_fb_xrgb8888_to_abgr8888 =============
[18:45:26] [PASSED] single_pixel_source_buffer
[18:45:26] [PASSED] single_pixel_clip_rectangle
[18:45:26] [PASSED] well_known_colors
[18:45:26] [PASSED] destination_pitch
[18:45:26] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[18:45:26] ================= drm_test_fb_clip_offset =================
[18:45:26] [PASSED] pass through
[18:45:26] [PASSED] horizontal offset
[18:45:26] [PASSED] vertical offset
[18:45:26] [PASSED] horizontal and vertical offset
[18:45:26] [PASSED] horizontal offset (custom pitch)
[18:45:26] [PASSED] vertical offset (custom pitch)
[18:45:26] [PASSED] horizontal and vertical offset (custom pitch)
[18:45:26] ============= [PASSED] drm_test_fb_clip_offset =============
[18:45:26] =================== drm_test_fb_memcpy ====================
[18:45:26] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[18:45:26] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[18:45:26] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[18:45:26] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[18:45:26] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[18:45:26] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[18:45:26] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[18:45:26] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[18:45:26] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[18:45:26] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[18:45:26] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[18:45:26] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[18:45:26] =============== [PASSED] drm_test_fb_memcpy ================
[18:45:26] ============= [PASSED] drm_format_helper_test ==============
[18:45:26] ================= drm_format (18 subtests) =================
[18:45:26] [PASSED] drm_test_format_block_width_invalid
[18:45:26] [PASSED] drm_test_format_block_width_one_plane
[18:45:26] [PASSED] drm_test_format_block_width_two_plane
[18:45:26] [PASSED] drm_test_format_block_width_three_plane
[18:45:26] [PASSED] drm_test_format_block_width_tiled
[18:45:26] [PASSED] drm_test_format_block_height_invalid
[18:45:26] [PASSED] drm_test_format_block_height_one_plane
[18:45:26] [PASSED] drm_test_format_block_height_two_plane
[18:45:26] [PASSED] drm_test_format_block_height_three_plane
[18:45:26] [PASSED] drm_test_format_block_height_tiled
[18:45:26] [PASSED] drm_test_format_min_pitch_invalid
[18:45:26] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[18:45:26] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[18:45:26] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[18:45:26] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[18:45:26] [PASSED] drm_test_format_min_pitch_two_plane
[18:45:26] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[18:45:26] [PASSED] drm_test_format_min_pitch_tiled
[18:45:26] =================== [PASSED] drm_format ====================
[18:45:26] ============== drm_framebuffer (10 subtests) ===============
[18:45:26] ========== drm_test_framebuffer_check_src_coords ==========
[18:45:26] [PASSED] Success: source fits into fb
[18:45:26] [PASSED] Fail: overflowing fb with x-axis coordinate
[18:45:26] [PASSED] Fail: overflowing fb with y-axis coordinate
[18:45:26] [PASSED] Fail: overflowing fb with source width
[18:45:26] [PASSED] Fail: overflowing fb with source height
[18:45:26] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[18:45:26] [PASSED] drm_test_framebuffer_cleanup
[18:45:26] =============== drm_test_framebuffer_create ===============
[18:45:26] [PASSED] ABGR8888 normal sizes
[18:45:26] [PASSED] ABGR8888 max sizes
[18:45:26] [PASSED] ABGR8888 pitch greater than min required
[18:45:26] [PASSED] ABGR8888 pitch less than min required
[18:45:26] [PASSED] ABGR8888 Invalid width
[18:45:26] [PASSED] ABGR8888 Invalid buffer handle
[18:45:26] [PASSED] No pixel format
[18:45:26] [PASSED] ABGR8888 Width 0
[18:45:26] [PASSED] ABGR8888 Height 0
[18:45:26] [PASSED] ABGR8888 Out of bound height * pitch combination
[18:45:26] [PASSED] ABGR8888 Large buffer offset
[18:45:26] [PASSED] ABGR8888 Buffer offset for inexistent plane
[18:45:26] [PASSED] ABGR8888 Invalid flag
[18:45:26] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[18:45:26] [PASSED] ABGR8888 Valid buffer modifier
[18:45:26] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[18:45:26] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] NV12 Normal sizes
[18:45:26] [PASSED] NV12 Max sizes
[18:45:26] [PASSED] NV12 Invalid pitch
[18:45:26] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[18:45:26] [PASSED] NV12 different modifier per-plane
[18:45:26] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[18:45:26] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] NV12 Modifier for inexistent plane
[18:45:26] [PASSED] NV12 Handle for inexistent plane
[18:45:26] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[18:45:26] [PASSED] YVU420 Normal sizes
[18:45:26] [PASSED] YVU420 Max sizes
[18:45:26] [PASSED] YVU420 Invalid pitch
[18:45:26] [PASSED] YVU420 Different pitches
[18:45:26] [PASSED] YVU420 Different buffer offsets/pitches
[18:45:26] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[18:45:26] [PASSED] YVU420 Valid modifier
[18:45:26] [PASSED] YVU420 Different modifiers per plane
[18:45:26] [PASSED] YVU420 Modifier for inexistent plane
[18:45:26] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[18:45:26] [PASSED] X0L2 Normal sizes
[18:45:26] [PASSED] X0L2 Max sizes
[18:45:26] [PASSED] X0L2 Invalid pitch
[18:45:26] [PASSED] X0L2 Pitch greater than minimum required
[18:45:26] [PASSED] X0L2 Handle for inexistent plane
[18:45:26] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[18:45:26] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[18:45:26] [PASSED] X0L2 Valid modifier
[18:45:26] [PASSED] X0L2 Modifier for inexistent plane
[18:45:26] =========== [PASSED] drm_test_framebuffer_create ===========
[18:45:26] [PASSED] drm_test_framebuffer_free
[18:45:26] [PASSED] drm_test_framebuffer_init
[18:45:26] [PASSED] drm_test_framebuffer_init_bad_format
[18:45:26] [PASSED] drm_test_framebuffer_init_dev_mismatch
[18:45:26] [PASSED] drm_test_framebuffer_lookup
[18:45:26] [PASSED] drm_test_framebuffer_lookup_inexistent
[18:45:26] [PASSED] drm_test_framebuffer_modifiers_not_supported
[18:45:26] ================= [PASSED] drm_framebuffer =================
[18:45:26] ================ drm_gem_shmem (8 subtests) ================
[18:45:26] [PASSED] drm_gem_shmem_test_obj_create
[18:45:26] [PASSED] drm_gem_shmem_test_obj_create_private
[18:45:26] [PASSED] drm_gem_shmem_test_pin_pages
[18:45:26] [PASSED] drm_gem_shmem_test_vmap
[18:45:26] [PASSED] drm_gem_shmem_test_get_sg_table
[18:45:26] [PASSED] drm_gem_shmem_test_get_pages_sgt
[18:45:26] [PASSED] drm_gem_shmem_test_madvise
[18:45:26] [PASSED] drm_gem_shmem_test_purge
[18:45:26] ================== [PASSED] drm_gem_shmem ==================
[18:45:26] === drm_atomic_helper_connector_hdmi_check (29 subtests) ===
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[18:45:26] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420 =======
[18:45:26] [PASSED] Automatic
[18:45:26] [PASSED] Full
[18:45:26] [PASSED] Limited 16:235
[18:45:26] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[18:45:26] [PASSED] drm_test_check_disable_connector
[18:45:26] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[18:45:26] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[18:45:26] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[18:45:26] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[18:45:26] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[18:45:26] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[18:45:26] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[18:45:26] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[18:45:26] [PASSED] drm_test_check_output_bpc_dvi
[18:45:26] [PASSED] drm_test_check_output_bpc_format_vic_1
[18:45:26] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[18:45:26] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[18:45:26] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[18:45:26] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[18:45:26] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[18:45:26] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[18:45:26] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[18:45:26] ============ drm_test_check_hdmi_color_format =============
[18:45:26] [PASSED] AUTO -> RGB
[18:45:26] [PASSED] YCBCR422 -> YUV422
[18:45:26] [PASSED] YCBCR420 -> YUV420
[18:45:26] [PASSED] YCBCR444 -> YUV444
[18:45:26] [PASSED] RGB -> RGB
[18:45:26] ======== [PASSED] drm_test_check_hdmi_color_format =========
[18:45:26] ======== drm_test_check_hdmi_color_format_420_only ========
[18:45:26] [PASSED] RGB should fail
[18:45:26] [PASSED] YUV444 should fail
[18:45:26] [PASSED] YUV422 should fail
[18:45:26] [PASSED] YUV420 should work
[18:45:26] ==== [PASSED] drm_test_check_hdmi_color_format_420_only ====
[18:45:26] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[18:45:26] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[18:45:26] [PASSED] drm_test_check_broadcast_rgb_value
[18:45:26] [PASSED] drm_test_check_bpc_8_value
[18:45:26] [PASSED] drm_test_check_bpc_10_value
[18:45:26] [PASSED] drm_test_check_bpc_12_value
[18:45:26] [PASSED] drm_test_check_format_value
[18:45:26] [PASSED] drm_test_check_tmds_char_value
[18:45:26] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[18:45:26] = drm_atomic_helper_connector_hdmi_mode_valid (7 subtests) =
[18:45:26] [PASSED] drm_test_check_mode_valid
[18:45:26] [PASSED] drm_test_check_mode_valid_reject
[18:45:26] [PASSED] drm_test_check_mode_valid_reject_rate
[18:45:26] [PASSED] drm_test_check_mode_valid_reject_max_clock
[18:45:26] [PASSED] drm_test_check_mode_valid_yuv420_only_max_clock
[18:45:26] [PASSED] drm_test_check_mode_valid_reject_yuv420_only_connector
[18:45:26] [PASSED] drm_test_check_mode_valid_accept_yuv420_also_connector_rgb
[18:45:26] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[18:45:26] = drm_atomic_helper_connector_hdmi_infoframes (5 subtests) =
[18:45:26] [PASSED] drm_test_check_infoframes
[18:45:26] [PASSED] drm_test_check_reject_avi_infoframe
[18:45:26] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_8
[18:45:26] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_10
[18:45:26] [PASSED] drm_test_check_reject_audio_infoframe
[18:45:26] === [PASSED] drm_atomic_helper_connector_hdmi_infoframes ===
[18:45:26] ================= drm_managed (2 subtests) =================
[18:45:26] [PASSED] drm_test_managed_release_action
[18:45:26] [PASSED] drm_test_managed_run_action
[18:45:26] =================== [PASSED] drm_managed ===================
[18:45:26] =================== drm_mm (6 subtests) ====================
[18:45:26] [PASSED] drm_test_mm_init
[18:45:26] [PASSED] drm_test_mm_debug
[18:45:26] [PASSED] drm_test_mm_align32
[18:45:26] [PASSED] drm_test_mm_align64
[18:45:26] [PASSED] drm_test_mm_lowest
[18:45:26] [PASSED] drm_test_mm_highest
[18:45:26] ===================== [PASSED] drm_mm ======================
[18:45:26] ============= drm_modes_analog_tv (5 subtests) =============
[18:45:26] [PASSED] drm_test_modes_analog_tv_mono_576i
[18:45:26] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[18:45:26] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[18:45:26] [PASSED] drm_test_modes_analog_tv_pal_576i
[18:45:26] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[18:45:26] =============== [PASSED] drm_modes_analog_tv ===============
[18:45:26] ============== drm_plane_helper (2 subtests) ===============
[18:45:26] =============== drm_test_check_plane_state ================
[18:45:26] [PASSED] clipping_simple
[18:45:26] [PASSED] clipping_rotate_reflect
[18:45:26] [PASSED] positioning_simple
[18:45:26] [PASSED] upscaling
[18:45:26] [PASSED] downscaling
[18:45:26] [PASSED] rounding1
[18:45:26] [PASSED] rounding2
[18:45:26] [PASSED] rounding3
[18:45:26] [PASSED] rounding4
[18:45:26] =========== [PASSED] drm_test_check_plane_state ============
[18:45:26] =========== drm_test_check_invalid_plane_state ============
[18:45:26] [PASSED] positioning_invalid
[18:45:26] [PASSED] upscaling_invalid
[18:45:26] [PASSED] downscaling_invalid
[18:45:26] ======= [PASSED] drm_test_check_invalid_plane_state ========
[18:45:26] ================ [PASSED] drm_plane_helper =================
[18:45:26] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[18:45:26] ====== drm_test_connector_helper_tv_get_modes_check =======
[18:45:26] [PASSED] None
[18:45:26] [PASSED] PAL
[18:45:26] [PASSED] NTSC
[18:45:26] [PASSED] Both, NTSC Default
[18:45:26] [PASSED] Both, PAL Default
[18:45:26] [PASSED] Both, NTSC Default, with PAL on command-line
[18:45:26] [PASSED] Both, PAL Default, with NTSC on command-line
[18:45:26] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[18:45:26] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[18:45:26] ================== drm_rect (9 subtests) ===================
[18:45:26] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[18:45:26] [PASSED] drm_test_rect_clip_scaled_not_clipped
[18:45:26] [PASSED] drm_test_rect_clip_scaled_clipped
[18:45:26] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[18:45:26] ================= drm_test_rect_intersect =================
[18:45:26] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[18:45:26] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[18:45:26] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[18:45:26] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[18:45:26] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[18:45:26] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[18:45:26] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[18:45:26] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[18:45:26] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[18:45:26] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[18:45:26] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[18:45:26] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[18:45:26] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[18:45:26] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[18:45:26] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[18:45:26] ============= [PASSED] drm_test_rect_intersect =============
[18:45:26] ================ drm_test_rect_calc_hscale ================
[18:45:26] [PASSED] normal use
[18:45:26] [PASSED] out of max range
[18:45:26] [PASSED] out of min range
[18:45:26] [PASSED] zero dst
[18:45:26] [PASSED] negative src
[18:45:26] [PASSED] negative dst
[18:45:26] ============ [PASSED] drm_test_rect_calc_hscale ============
[18:45:26] ================ drm_test_rect_calc_vscale ================
[18:45:26] [PASSED] normal use
[18:45:26] [PASSED] out of max range
[18:45:26] [PASSED] out of min range
[18:45:26] [PASSED] zero dst
[18:45:26] [PASSED] negative src
[18:45:26] [PASSED] negative dst
[18:45:26] ============ [PASSED] drm_test_rect_calc_vscale ============
[18:45:26] ================== drm_test_rect_rotate ===================
[18:45:26] [PASSED] reflect-x
[18:45:26] [PASSED] reflect-y
[18:45:26] [PASSED] rotate-0
[18:45:26] [PASSED] rotate-90
[18:45:26] [PASSED] rotate-180
[18:45:26] [PASSED] rotate-270
[18:45:26] ============== [PASSED] drm_test_rect_rotate ===============
[18:45:26] ================ drm_test_rect_rotate_inv =================
[18:45:26] [PASSED] reflect-x
[18:45:26] [PASSED] reflect-y
[18:45:26] [PASSED] rotate-0
[18:45:26] [PASSED] rotate-90
[18:45:26] [PASSED] rotate-180
[18:45:26] [PASSED] rotate-270
[18:45:26] ============ [PASSED] drm_test_rect_rotate_inv =============
[18:45:26] ==================== [PASSED] drm_rect =====================
[18:45:26] ============ drm_sysfb_modeset_test (1 subtest) ============
[18:45:26] ============ drm_test_sysfb_build_fourcc_list =============
[18:45:26] [PASSED] no native formats
[18:45:26] [PASSED] XRGB8888 as native format
[18:45:26] [PASSED] remove duplicates
[18:45:26] [PASSED] convert alpha formats
[18:45:26] [PASSED] random formats
[18:45:26] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[18:45:26] ============= [PASSED] drm_sysfb_modeset_test ==============
[18:45:26] ================== drm_fixp (2 subtests) ===================
[18:45:26] [PASSED] drm_test_int2fixp
[18:45:26] [PASSED] drm_test_sm2fixp
[18:45:26] ==================== [PASSED] drm_fixp =====================
[18:45:26] ============================================================
[18:45:26] Testing complete. Ran 637 tests: passed: 637
[18:45:26] Elapsed time: 26.818s total, 1.804s configuring, 24.748s building, 0.239s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[18:45:26] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[18:45:28] 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
[18:45:37] Starting KUnit Kernel (1/1)...
[18:45:37] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[18:45:37] ================= ttm_device (5 subtests) ==================
[18:45:37] [PASSED] ttm_device_init_basic
[18:45:37] [PASSED] ttm_device_init_multiple
[18:45:37] [PASSED] ttm_device_fini_basic
[18:45:37] [PASSED] ttm_device_init_no_vma_man
[18:45:37] ================== ttm_device_init_pools ==================
[18:45:37] [PASSED] No DMA allocations, no DMA32 required
[18:45:37] [PASSED] DMA allocations, DMA32 required
[18:45:37] [PASSED] No DMA allocations, DMA32 required
[18:45:37] [PASSED] DMA allocations, no DMA32 required
[18:45:37] ============== [PASSED] ttm_device_init_pools ==============
[18:45:37] =================== [PASSED] ttm_device ====================
[18:45:37] ================== ttm_pool (8 subtests) ===================
[18:45:37] ================== ttm_pool_alloc_basic ===================
[18:45:37] [PASSED] One page
[18:45:37] [PASSED] More than one page
[18:45:37] [PASSED] Above the allocation limit
[18:45:37] [PASSED] One page, with coherent DMA mappings enabled
[18:45:37] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[18:45:37] ============== [PASSED] ttm_pool_alloc_basic ===============
[18:45:37] ============== ttm_pool_alloc_basic_dma_addr ==============
[18:45:37] [PASSED] One page
[18:45:37] [PASSED] More than one page
[18:45:37] [PASSED] Above the allocation limit
[18:45:37] [PASSED] One page, with coherent DMA mappings enabled
[18:45:37] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[18:45:37] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[18:45:37] [PASSED] ttm_pool_alloc_order_caching_match
[18:45:37] [PASSED] ttm_pool_alloc_caching_mismatch
[18:45:37] [PASSED] ttm_pool_alloc_order_mismatch
[18:45:37] [PASSED] ttm_pool_free_dma_alloc
[18:45:37] [PASSED] ttm_pool_free_no_dma_alloc
[18:45:37] [PASSED] ttm_pool_fini_basic
[18:45:37] ==================== [PASSED] ttm_pool =====================
[18:45:37] ================ ttm_resource (8 subtests) =================
[18:45:37] ================= ttm_resource_init_basic =================
[18:45:37] [PASSED] Init resource in TTM_PL_SYSTEM
[18:45:37] [PASSED] Init resource in TTM_PL_VRAM
[18:45:37] [PASSED] Init resource in a private placement
[18:45:37] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[18:45:37] ============= [PASSED] ttm_resource_init_basic =============
[18:45:37] [PASSED] ttm_resource_init_pinned
[18:45:37] [PASSED] ttm_resource_fini_basic
[18:45:37] [PASSED] ttm_resource_manager_init_basic
[18:45:37] [PASSED] ttm_resource_manager_usage_basic
[18:45:37] [PASSED] ttm_resource_manager_set_used_basic
[18:45:37] [PASSED] ttm_sys_man_alloc_basic
[18:45:37] [PASSED] ttm_sys_man_free_basic
[18:45:37] ================== [PASSED] ttm_resource ===================
[18:45:37] =================== ttm_tt (15 subtests) ===================
[18:45:37] ==================== ttm_tt_init_basic ====================
[18:45:37] [PASSED] Page-aligned size
[18:45:37] [PASSED] Extra pages requested
[18:45:37] ================ [PASSED] ttm_tt_init_basic ================
[18:45:37] [PASSED] ttm_tt_init_misaligned
[18:45:37] [PASSED] ttm_tt_fini_basic
[18:45:37] [PASSED] ttm_tt_fini_sg
[18:45:37] [PASSED] ttm_tt_fini_shmem
[18:45:37] [PASSED] ttm_tt_create_basic
[18:45:37] [PASSED] ttm_tt_create_invalid_bo_type
[18:45:37] [PASSED] ttm_tt_create_ttm_exists
[18:45:37] [PASSED] ttm_tt_create_failed
[18:45:37] [PASSED] ttm_tt_destroy_basic
[18:45:37] [PASSED] ttm_tt_populate_null_ttm
[18:45:37] [PASSED] ttm_tt_populate_populated_ttm
[18:45:37] [PASSED] ttm_tt_unpopulate_basic
[18:45:37] [PASSED] ttm_tt_unpopulate_empty_ttm
[18:45:37] [PASSED] ttm_tt_swapin_basic
[18:45:37] ===================== [PASSED] ttm_tt ======================
[18:45:37] =================== ttm_bo (14 subtests) ===================
[18:45:37] =========== ttm_bo_reserve_optimistic_no_ticket ===========
[18:45:37] [PASSED] Cannot be interrupted and sleeps
[18:45:37] [PASSED] Cannot be interrupted, locks straight away
[18:45:37] [PASSED] Can be interrupted, sleeps
[18:45:37] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[18:45:37] [PASSED] ttm_bo_reserve_locked_no_sleep
[18:45:37] [PASSED] ttm_bo_reserve_no_wait_ticket
[18:45:37] [PASSED] ttm_bo_reserve_double_resv
[18:45:37] [PASSED] ttm_bo_reserve_interrupted
[18:45:37] [PASSED] ttm_bo_reserve_deadlock
[18:45:37] [PASSED] ttm_bo_unreserve_basic
[18:45:37] [PASSED] ttm_bo_unreserve_pinned
[18:45:37] [PASSED] ttm_bo_unreserve_bulk
[18:45:37] [PASSED] ttm_bo_fini_basic
[18:45:37] [PASSED] ttm_bo_fini_shared_resv
[18:45:37] [PASSED] ttm_bo_pin_basic
[18:45:37] [PASSED] ttm_bo_pin_unpin_resource
[18:45:37] [PASSED] ttm_bo_multiple_pin_one_unpin
[18:45:37] ===================== [PASSED] ttm_bo ======================
[18:45:37] ============== ttm_bo_validate (22 subtests) ===============
[18:45:37] ============== ttm_bo_init_reserved_sys_man ===============
[18:45:37] [PASSED] Buffer object for userspace
[18:45:37] [PASSED] Kernel buffer object
[18:45:37] [PASSED] Shared buffer object
[18:45:37] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[18:45:37] ============== ttm_bo_init_reserved_mock_man ==============
[18:45:37] [PASSED] Buffer object for userspace
[18:45:37] [PASSED] Kernel buffer object
[18:45:37] [PASSED] Shared buffer object
[18:45:37] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[18:45:37] [PASSED] ttm_bo_init_reserved_resv
[18:45:37] ================== ttm_bo_validate_basic ==================
[18:45:37] [PASSED] Buffer object for userspace
[18:45:37] [PASSED] Kernel buffer object
[18:45:37] [PASSED] Shared buffer object
[18:45:37] ============== [PASSED] ttm_bo_validate_basic ==============
[18:45:37] [PASSED] ttm_bo_validate_invalid_placement
[18:45:37] ============= ttm_bo_validate_same_placement ==============
[18:45:37] [PASSED] System manager
[18:45:37] [PASSED] VRAM manager
[18:45:37] ========= [PASSED] ttm_bo_validate_same_placement ==========
[18:45:37] [PASSED] ttm_bo_validate_failed_alloc
[18:45:37] [PASSED] ttm_bo_validate_pinned
[18:45:37] [PASSED] ttm_bo_validate_busy_placement
[18:45:37] ================ ttm_bo_validate_multihop =================
[18:45:37] [PASSED] Buffer object for userspace
[18:45:37] [PASSED] Kernel buffer object
[18:45:37] [PASSED] Shared buffer object
[18:45:37] ============ [PASSED] ttm_bo_validate_multihop =============
[18:45:37] ========== ttm_bo_validate_no_placement_signaled ==========
[18:45:37] [PASSED] Buffer object in system domain, no page vector
[18:45:37] [PASSED] Buffer object in system domain with an existing page vector
[18:45:37] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[18:45:37] ======== ttm_bo_validate_no_placement_not_signaled ========
[18:45:37] [PASSED] Buffer object for userspace
[18:45:37] [PASSED] Kernel buffer object
[18:45:37] [PASSED] Shared buffer object
[18:45:37] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[18:45:37] [PASSED] ttm_bo_validate_move_fence_signaled
[18:45:37] ========= ttm_bo_validate_move_fence_not_signaled =========
[18:45:37] [PASSED] Waits for GPU
[18:45:37] [PASSED] Tries to lock straight away
[18:45:37] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[18:45:37] [PASSED] ttm_bo_validate_swapout
[18:45:37] [PASSED] ttm_bo_validate_happy_evict
[18:45:37] [PASSED] ttm_bo_validate_all_pinned_evict
[18:45:37] [PASSED] ttm_bo_validate_allowed_only_evict
[18:45:37] [PASSED] ttm_bo_validate_deleted_evict
[18:45:37] [PASSED] ttm_bo_validate_busy_domain_evict
[18:45:37] [PASSED] ttm_bo_validate_evict_gutting
[18:45:37] [PASSED] ttm_bo_validate_recrusive_evict
[18:45:37] ================= [PASSED] ttm_bo_validate =================
[18:45:37] ============================================================
[18:45:37] Testing complete. Ran 102 tests: passed: 102
[18:45:38] Elapsed time: 11.805s total, 1.808s configuring, 9.781s building, 0.177s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/dma-buf/.kunitconfig
[18:45:38] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[18:45:39] 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
[18:45:48] Starting KUnit Kernel (1/1)...
[18:45:48] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[18:45:48] =============== dma-buf-fence (12 subtests) ================
[18:45:48] [PASSED] test_sanitycheck
[18:45:48] [PASSED] test_signaling
[18:45:48] [PASSED] test_add_callback
[18:45:48] [PASSED] test_late_add_callback
[18:45:48] [PASSED] test_rm_callback
[18:45:48] [PASSED] test_late_rm_callback
[18:45:48] [PASSED] test_status
[18:45:48] [PASSED] test_error
[18:45:48] [PASSED] test_wait
[18:45:48] [PASSED] test_wait_timeout
[18:45:48] [PASSED] test_stub
[18:45:48] [SKIPPED] test_race_signal_callback (requires at least 2 CPUs)
[18:45:48] ================== [PASSED] dma-buf-fence ==================
[18:45:48] ============ dma-buf-fence-chain (11 subtests) =============
[18:45:48] [PASSED] test_sanitycheck
[18:45:48] [PASSED] test_find_seqno
[18:45:48] [PASSED] test_find_signaled
[18:45:48] [PASSED] test_find_out_of_order
[18:45:53] [PASSED] test_find_gap
[18:45:53] [PASSED] test_find_race
[18:45:53] [PASSED] test_signal_forward
[18:45:53] [PASSED] test_signal_backward
[18:45:53] [PASSED] test_wait_forward
[18:45:53] [PASSED] test_wait_backward
[18:45:53] [PASSED] test_wait_random
[18:45:53] =============== [PASSED] dma-buf-fence-chain ===============
[18:45:53] ============ dma-buf-fence-unwrap (10 subtests) ============
[18:45:53] [PASSED] test_sanitycheck
[18:45:53] [PASSED] test_unwrap_array
[18:45:53] [PASSED] test_unwrap_chain
[18:45:53] [PASSED] test_unwrap_chain_array
[18:45:53] [PASSED] test_unwrap_merge
[18:45:53] [PASSED] test_unwrap_merge_duplicate
[18:45:53] [PASSED] test_unwrap_merge_seqno
[18:45:53] [PASSED] test_unwrap_merge_order
[18:45:53] [PASSED] test_unwrap_merge_complex
[18:45:53] [PASSED] test_unwrap_merge_complex_seqno
[18:45:53] ============== [PASSED] dma-buf-fence-unwrap ===============
[18:45:53] ================ dma-buf-resv (5 subtests) =================
[18:45:53] [PASSED] test_sanitycheck
[18:45:53] ===================== test_signaling ======================
[18:45:53] [PASSED] kernel
[18:45:53] [PASSED] write
[18:45:53] [PASSED] read
[18:45:53] [PASSED] bookkeep
[18:45:53] ================= [PASSED] test_signaling ==================
[18:45:53] ====================== test_for_each ======================
[18:45:53] [PASSED] kernel
[18:45:53] [PASSED] write
[18:45:53] [PASSED] read
[18:45:53] [PASSED] bookkeep
[18:45:53] ================== [PASSED] test_for_each ==================
[18:45:53] ================= test_for_each_unlocked ==================
[18:45:53] [PASSED] kernel
[18:45:53] [PASSED] write
[18:45:53] [PASSED] read
[18:45:53] [PASSED] bookkeep
[18:45:53] ============= [PASSED] test_for_each_unlocked ==============
[18:45:53] ===================== test_get_fences =====================
[18:45:53] [PASSED] kernel
[18:45:53] [PASSED] write
[18:45:53] [PASSED] read
[18:45:53] [PASSED] bookkeep
[18:45:53] ================= [PASSED] test_get_fences =================
[18:45:53] ================== [PASSED] dma-buf-resv ===================
[18:45:53] ============================================================
[18:45:53] Testing complete. Ran 50 tests: passed: 49, skipped: 1
[18:45:53] Elapsed time: 15.517s total, 1.748s configuring, 8.448s building, 5.278s running
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
2026-08-18 18:45 ` ✓ CI.KUnit: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
@ 2026-08-18 18:51 ` sashiko-bot
2026-08-18 20:54 ` Niranjana Vishwanathapura
2026-08-18 19:32 ` ✓ Xe.CI.BAT: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
` (3 subsequent siblings)
5 siblings, 1 reply; 11+ messages in thread
From: sashiko-bot @ 2026-08-18 18:51 UTC (permalink / raw)
To: Jagmeet Randhawa; +Cc: intel-xe
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
Pre-existing issues:
- [High] TDR (Timeout Delay Recovery) races with asynchronous queue suspend, leading to misattributed SCHED_DONE messages, an assertion failure, and a TDR hang.
--
--- Patch [1]: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access ---
commit 256a06cec5d42e79b9711d9f1861b3717e23b08f
Author: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
drm/xe/guc: Fix race around q->guc->suspend_pending access
This commit wraps accesses to q->guc->suspend_pending with xe_sched_msg_lock
in the suspend, resume, kill, and ban paths. This ensures that checking the
flag and calling suspend_fence_signal() is atomic with respect to the
asynchronous clears in the kill and ban paths.
> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> index 9036f89dff7d9..c565c1d32d3a4 100644
> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
[ ... ]
> @@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
> smp_wmb();
> wake_up_all(&guc->ct.wq);
> } else {
> + bool was_pending;
> +
> xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
> xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
>
> - if (q->guc->suspend_pending) {
> + xe_sched_msg_lock(&q->guc->sched);
> + was_pending = q->guc->suspend_pending;
> + if (was_pending) {
> clear_exec_queue_pending_disable(q);
> suspend_fence_signal(q);
> - } else {
> + }
> + xe_sched_msg_unlock(&q->guc->sched);
[Severity: High]
This is a pre-existing issue, but does this logic misattribute a SCHED_DONE
message if TDR races with an asynchronous queue suspend?
If a queue is being suspended (setting q->guc->suspend_pending = true) and
concurrently a job times out, the TDR handler stops the DRM scheduler,
trapping the SUSPEND message before it can execute. TDR then calls
disable_scheduling(), which directly sends a DISABLE command to the GuC.
When the GuC responds to the DISABLE command with a SCHED_DONE message,
won't handle_sched_done() incorrectly attribute it to the trapped SUSPEND
message because was_pending is still true?
It looks like this would call suspend_fence_signal(q) and trigger an
assertion failure, because the queue was never actually suspended (the
message was trapped) or killed (TDR only bans it). Additionally, skipping
the wake_up_all() call in the other branch might cause the TDR thread to
wait and hang for 5 seconds until it times out.
> +
> + if (!was_pending) {
> if (exec_queue_banned(q)) {
> smp_wmb();
> wake_up_all(&guc->ct.wq);
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260818183838.273486-2-jagmeet.randhawa@intel.com?part=1
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Xe.CI.BAT: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2)
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
2026-08-18 18:45 ` ✓ CI.KUnit: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
2026-08-18 18:51 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access sashiko-bot
@ 2026-08-18 19:32 ` Patchwork
2026-08-18 20:53 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Niranjana Vishwanathapura
` (2 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2026-08-18 19:32 UTC (permalink / raw)
To: Jagmeet Randhawa; +Cc: intel-xe
[-- Attachment #1: Type: text/plain, Size: 4979 bytes --]
== Series Details ==
Series: drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2)
URL : https://patchwork.freedesktop.org/series/172176/
State : success
== Summary ==
CI Bug Log - changes from xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef_BAT -> xe-pw-172176v2_BAT
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (12 -> 12)
------------------------------
Additional (1): bat-bmg-2
Missing (1): bat-nvls-2
Known issues
------------
Here are the changes found in xe-pw-172176v2_BAT that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@fbdev@write:
- bat-bmg-2: NOTRUN -> [SKIP][1] ([Intel XE#2134]) +4 other tests skip
[1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@fbdev@write.html
* igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
- bat-bmg-2: NOTRUN -> [SKIP][2] ([Intel XE#2233])
[2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
- bat-bmg-2: NOTRUN -> [SKIP][3] ([Intel XE#2489] / [Intel XE#3419]) +13 other tests skip
[3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
* igt@kms_flip@basic-flip-vs-modeset:
- bat-bmg-2: NOTRUN -> [SKIP][4] ([Intel XE#2482]) +3 other tests skip
[4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@kms_flip@basic-flip-vs-modeset.html
* igt@kms_frontbuffer_tracking@basic:
- bat-bmg-2: NOTRUN -> [SKIP][5] ([Intel XE#2434] / [Intel XE#2548] / [Intel XE#6314])
[5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@kms_frontbuffer_tracking@basic.html
* igt@kms_psr@psr-sprite-plane-onoff:
- bat-bmg-2: NOTRUN -> [SKIP][6] ([Intel XE#2234] / [Intel XE#2850]) +2 other tests skip
[6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@kms_psr@psr-sprite-plane-onoff.html
* igt@xe_exec_multi_queue@priority:
- bat-bmg-2: NOTRUN -> [SKIP][7] ([Intel XE#8364]) +13 other tests skip
[7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@xe_exec_multi_queue@priority.html
* igt@xe_live_ktest@xe_bo@xe_ccs_migrate_kunit:
- bat-bmg-2: NOTRUN -> [SKIP][8] ([Intel XE#2229])
[8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@xe_live_ktest@xe_bo@xe_ccs_migrate_kunit.html
* igt@xe_pat@pat-index-xehpc:
- bat-bmg-2: NOTRUN -> [SKIP][9] ([Intel XE#1420] / [Intel XE#7590])
[9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@xe_pat@pat-index-xehpc.html
* igt@xe_pat@pat-index-xelp:
- bat-bmg-2: NOTRUN -> [SKIP][10] ([Intel XE#2245] / [Intel XE#7590])
[10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@xe_pat@pat-index-xelp.html
* igt@xe_pat@pat-index-xelpg:
- bat-bmg-2: NOTRUN -> [SKIP][11] ([Intel XE#2236] / [Intel XE#7590])
[11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/bat-bmg-2/igt@xe_pat@pat-index-xelpg.html
[Intel XE#1420]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1420
[Intel XE#2134]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2134
[Intel XE#2229]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2229
[Intel XE#2233]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2233
[Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
[Intel XE#2236]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2236
[Intel XE#2245]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2245
[Intel XE#2434]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2434
[Intel XE#2482]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2482
[Intel XE#2489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2489
[Intel XE#2548]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2548
[Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
[Intel XE#3419]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3419
[Intel XE#6314]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6314
[Intel XE#7590]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7590
[Intel XE#8364]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8364
Build changes
-------------
* Linux: xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef -> xe-pw-172176v2
IGT_9059: f5a4ff79434df36db1d6b13deb37c90a6602565e @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef: 275df33dcf4c5d018717867a0b29ed1d3b62c1ef
xe-pw-172176v2: 172176v2
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/index.html
[-- Attachment #2: Type: text/html, Size: 5903 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
` (2 preceding siblings ...)
2026-08-18 19:32 ` ✓ Xe.CI.BAT: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
@ 2026-08-18 20:53 ` Niranjana Vishwanathapura
2026-08-18 21:06 ` Matthew Brost
2026-08-18 21:05 ` Matthew Brost
2026-08-18 22:08 ` ✓ Xe.CI.FULL: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
5 siblings, 1 reply; 11+ messages in thread
From: Niranjana Vishwanathapura @ 2026-08-18 20:53 UTC (permalink / raw)
To: Jagmeet Randhawa; +Cc: intel-xe
On Wed, Aug 19, 2026 at 02:38:39AM +0800, Jagmeet Randhawa wrote:
>q->guc->suspend_pending is accessed without any common lock.
>__suspend_fence_signal(), called from guc_exec_queue_kill() and the
>suspend-timeout ban path, clears the flag asynchronously. Meanwhile
>handle_sched_done(), guc_exec_queue_stop() and
>__guc_exec_queue_process_msg_suspend() check the flag and then call
>suspend_fence_signal(), which asserts that it is still set.
>
>As the check and suspend_fence_signal() are not atomic, the clear can
>land in between and trip the xe_gt_assert(q->guc->suspend_pending).
>
>The flag is already set and read under the per-queue msg_lock
>(xe_sched_msg_lock()) on the suspend and resume paths. Extend that same
>lock to the clear paths (kill and ban) and to the three check-then-act
>sites so the check and the signal are atomic with respect to the clear.
>In __guc_exec_queue_process_msg_suspend() only the non-sleeping branch is
>wrapped, since the other branch waits. In handle_sched_done() the flag is
>snapshotted under the lock and deregister_exec_queue() is kept outside it.
>
>v2: Document that sched->msg_lock also protects
> guc->suspend_pending, which indicates a suspend message is in
> flight, in addition to the sched->msgs list (Niranjana)
>
>Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
LGTM
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
>---
> drivers/gpu/drm/xe/xe_gpu_scheduler_types.h | 5 +++-
> drivers/gpu/drm/xe/xe_guc_exec_queue_types.h | 5 +++-
> drivers/gpu/drm/xe/xe_guc_submit.c | 29 ++++++++++++++++----
> 3 files changed, 32 insertions(+), 7 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
>index 63d9bf92583c..78ef2e8ded4f 100644
>--- a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
>+++ b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
>@@ -47,7 +47,10 @@ struct xe_gpu_scheduler {
> const struct xe_sched_backend_ops *ops;
> /** @msgs: list of messages to be processed in @work_process_msg */
> struct list_head msgs;
>- /** @msg_lock: Message lock */
>+ /**
>+ * @msg_lock: Protects @msgs and guc->suspend_pending (indicates a
>+ * suspend message is in flight) of exec queues on this scheduler.
>+ */
> spinlock_t msg_lock;
> /** @work_process_msg: processes messages */
> struct work_struct work_process_msg;
>diff --git a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
>index d27826b36649..74b711abe257 100644
>--- a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
>+++ b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
>@@ -52,7 +52,10 @@ struct xe_guc_exec_queue {
> u16 id;
> /** @suspend_wait: wait queue used to wait on pending suspends */
> wait_queue_head_t suspend_wait;
>- /** @suspend_pending: a suspend of the exec_queue is pending */
>+ /**
>+ * @suspend_pending: a suspend of the exec_queue is pending.
>+ * Protected by @sched.msg_lock.
>+ */
> bool suspend_pending;
> /**
> * @suspend_count: Reference count of active suspend requests. The
>diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
>index 9036f89dff7d..c565c1d32d3a 100644
>--- a/drivers/gpu/drm/xe/xe_guc_submit.c
>+++ b/drivers/gpu/drm/xe/xe_guc_submit.c
>@@ -1928,9 +1928,13 @@ static void __guc_exec_queue_process_msg_suspend(struct xe_sched_msg *msg)
> set_exec_queue_suspended(q);
> disable_scheduling(q, false);
> }
>- } else if (q->guc->suspend_pending) {
>- set_exec_queue_suspended(q);
>- suspend_fence_signal(q);
>+ } else {
>+ xe_sched_msg_lock(&q->guc->sched);
>+ if (q->guc->suspend_pending) {
>+ set_exec_queue_suspended(q);
>+ suspend_fence_signal(q);
>+ }
>+ xe_sched_msg_unlock(&q->guc->sched);
> }
> }
>
>@@ -2130,7 +2134,9 @@ static void guc_exec_queue_kill(struct xe_exec_queue *q)
> {
> trace_xe_exec_queue_kill(q);
> set_exec_queue_killed(q);
>+ xe_sched_msg_lock(&q->guc->sched);
> __suspend_fence_signal(q);
>+ xe_sched_msg_unlock(&q->guc->sched);
> xe_guc_exec_queue_trigger_cleanup(q);
> }
>
>@@ -2392,11 +2398,15 @@ static void guc_exec_queue_suspend_timeout_ban(struct xe_exec_queue *q)
> */
> if (xe_exec_queue_is_multi_queue(q)) {
> set_exec_queue_group_banned(q);
>+ xe_sched_msg_lock(&q->guc->sched);
> __suspend_fence_signal(q);
>+ xe_sched_msg_unlock(&q->guc->sched);
> xe_guc_exec_queue_group_trigger_cleanup(q);
> } else {
> set_exec_queue_banned(q);
>+ xe_sched_msg_lock(&q->guc->sched);
> __suspend_fence_signal(q);
>+ xe_sched_msg_unlock(&q->guc->sched);
> xe_guc_exec_queue_trigger_cleanup(q);
> }
> }
>@@ -2614,10 +2624,12 @@ static void guc_exec_queue_stop(struct xe_guc *guc, struct xe_exec_queue *q)
> if (exec_queue_destroyed(q))
> do_destroy = true;
> }
>+ xe_sched_msg_lock(sched);
> if (q->guc->suspend_pending) {
> set_exec_queue_suspended(q);
> suspend_fence_signal(q);
> }
>+ xe_sched_msg_unlock(sched);
> atomic_and(EXEC_QUEUE_STATE_WEDGED | EXEC_QUEUE_STATE_BANNED |
> EXEC_QUEUE_STATE_KILLED | EXEC_QUEUE_STATE_DESTROYED |
> EXEC_QUEUE_STATE_SUSPENDED,
>@@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
> smp_wmb();
> wake_up_all(&guc->ct.wq);
> } else {
>+ bool was_pending;
>+
> xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
> xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
>
>- if (q->guc->suspend_pending) {
>+ xe_sched_msg_lock(&q->guc->sched);
>+ was_pending = q->guc->suspend_pending;
>+ if (was_pending) {
> clear_exec_queue_pending_disable(q);
> suspend_fence_signal(q);
>- } else {
>+ }
>+ xe_sched_msg_unlock(&q->guc->sched);
>+
>+ if (!was_pending) {
> if (exec_queue_banned(q)) {
> smp_wmb();
> wake_up_all(&guc->ct.wq);
>--
>2.53.0
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 18:51 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access sashiko-bot
@ 2026-08-18 20:54 ` Niranjana Vishwanathapura
0 siblings, 0 replies; 11+ messages in thread
From: Niranjana Vishwanathapura @ 2026-08-18 20:54 UTC (permalink / raw)
To: sashiko-reviews; +Cc: Jagmeet Randhawa, intel-xe
On Tue, Aug 18, 2026 at 06:51:42PM +0000, sashiko-bot@kernel.org wrote:
>Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
>
>Pre-existing issues:
>- [High] TDR (Timeout Delay Recovery) races with asynchronous queue suspend, leading to misattributed SCHED_DONE messages, an assertion failure, and a TDR hang.
>--
>
>--- Patch [1]: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access ---
>commit 256a06cec5d42e79b9711d9f1861b3717e23b08f
>Author: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
>
>drm/xe/guc: Fix race around q->guc->suspend_pending access
>
>This commit wraps accesses to q->guc->suspend_pending with xe_sched_msg_lock
>in the suspend, resume, kill, and ban paths. This ensures that checking the
>flag and calling suspend_fence_signal() is atomic with respect to the
>asynchronous clears in the kill and ban paths.
>
>> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
>> index 9036f89dff7d9..c565c1d32d3a4 100644
>> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
>> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
>[ ... ]
>> @@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
>> smp_wmb();
>> wake_up_all(&guc->ct.wq);
>> } else {
>> + bool was_pending;
>> +
>> xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
>> xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
>>
>> - if (q->guc->suspend_pending) {
>> + xe_sched_msg_lock(&q->guc->sched);
>> + was_pending = q->guc->suspend_pending;
>> + if (was_pending) {
>> clear_exec_queue_pending_disable(q);
>> suspend_fence_signal(q);
>> - } else {
>> + }
>> + xe_sched_msg_unlock(&q->guc->sched);
>
>[Severity: High]
>This is a pre-existing issue, but does this logic misattribute a SCHED_DONE
>message if TDR races with an asynchronous queue suspend?
>
>If a queue is being suspended (setting q->guc->suspend_pending = true) and
>concurrently a job times out, the TDR handler stops the DRM scheduler,
>trapping the SUSPEND message before it can execute. TDR then calls
>disable_scheduling(), which directly sends a DISABLE command to the GuC.
>
>When the GuC responds to the DISABLE command with a SCHED_DONE message,
>won't handle_sched_done() incorrectly attribute it to the trapped SUSPEND
>message because was_pending is still true?
>
>It looks like this would call suspend_fence_signal(q) and trigger an
>assertion failure, because the queue was never actually suspended (the
>message was trapped) or killed (TDR only bans it). Additionally, skipping
>the wake_up_all() call in the other branch might cause the TDR thread to
>wait and hang for 5 seconds until it times out.
>
As the report says, it it unrelated, pre-existing issue which should
be addressed separately.
Niranjana
>> +
>> + if (!was_pending) {
>> if (exec_queue_banned(q)) {
>> smp_wmb();
>> wake_up_all(&guc->ct.wq);
>
>--
>Sashiko AI review · https://sashiko.dev/#/patchset/20260818183838.273486-2-jagmeet.randhawa@intel.com?part=1
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
` (3 preceding siblings ...)
2026-08-18 20:53 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Niranjana Vishwanathapura
@ 2026-08-18 21:05 ` Matthew Brost
2026-08-18 23:27 ` Niranjana Vishwanathapura
2026-08-18 22:08 ` ✓ Xe.CI.FULL: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
5 siblings, 1 reply; 11+ messages in thread
From: Matthew Brost @ 2026-08-18 21:05 UTC (permalink / raw)
To: Jagmeet Randhawa; +Cc: intel-xe, niranjana.vishwanathapura
On Wed, Aug 19, 2026 at 02:38:39AM +0800, Jagmeet Randhawa wrote:
This is designed to be lockless.
> q->guc->suspend_pending is accessed without any common lock.
> __suspend_fence_signal(), called from guc_exec_queue_kill() and the
This is actually the problem. __suspend_fence_signal shouldn't be called
from guc_exec_queue_kill(). This can prematurely signal a suspend fence
while the hardware is still executing.
__suspend_fence_signal should be called in two possible places:
- Naturally in G2H handler (handle_sched_done)
- Or in global event that takes down the GuC firmware
(guc_exec_queue_stop)
With that, a lock isn't need because the state machine / firmware
interaction ensures everything is race free.
So I think the solution is ensure __suspend_fence_signal is called in
the correct places rather than adding protection via a lock.
Matt
> suspend-timeout ban path, clears the flag asynchronously. Meanwhile
> handle_sched_done(), guc_exec_queue_stop() and
> __guc_exec_queue_process_msg_suspend() check the flag and then call
> suspend_fence_signal(), which asserts that it is still set.
>
> As the check and suspend_fence_signal() are not atomic, the clear can
> land in between and trip the xe_gt_assert(q->guc->suspend_pending).
>
> The flag is already set and read under the per-queue msg_lock
> (xe_sched_msg_lock()) on the suspend and resume paths. Extend that same
> lock to the clear paths (kill and ban) and to the three check-then-act
> sites so the check and the signal are atomic with respect to the clear.
> In __guc_exec_queue_process_msg_suspend() only the non-sleeping branch is
> wrapped, since the other branch waits. In handle_sched_done() the flag is
> snapshotted under the lock and deregister_exec_queue() is kept outside it.
>
> v2: Document that sched->msg_lock also protects
> guc->suspend_pending, which indicates a suspend message is in
> flight, in addition to the sched->msgs list (Niranjana)
>
> Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
> ---
> drivers/gpu/drm/xe/xe_gpu_scheduler_types.h | 5 +++-
> drivers/gpu/drm/xe/xe_guc_exec_queue_types.h | 5 +++-
> drivers/gpu/drm/xe/xe_guc_submit.c | 29 ++++++++++++++++----
> 3 files changed, 32 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> index 63d9bf92583c..78ef2e8ded4f 100644
> --- a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> +++ b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> @@ -47,7 +47,10 @@ struct xe_gpu_scheduler {
> const struct xe_sched_backend_ops *ops;
> /** @msgs: list of messages to be processed in @work_process_msg */
> struct list_head msgs;
> - /** @msg_lock: Message lock */
> + /**
> + * @msg_lock: Protects @msgs and guc->suspend_pending (indicates a
> + * suspend message is in flight) of exec queues on this scheduler.
> + */
> spinlock_t msg_lock;
> /** @work_process_msg: processes messages */
> struct work_struct work_process_msg;
> diff --git a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> index d27826b36649..74b711abe257 100644
> --- a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> +++ b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> @@ -52,7 +52,10 @@ struct xe_guc_exec_queue {
> u16 id;
> /** @suspend_wait: wait queue used to wait on pending suspends */
> wait_queue_head_t suspend_wait;
> - /** @suspend_pending: a suspend of the exec_queue is pending */
> + /**
> + * @suspend_pending: a suspend of the exec_queue is pending.
> + * Protected by @sched.msg_lock.
> + */
> bool suspend_pending;
> /**
> * @suspend_count: Reference count of active suspend requests. The
> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> index 9036f89dff7d..c565c1d32d3a 100644
> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> @@ -1928,9 +1928,13 @@ static void __guc_exec_queue_process_msg_suspend(struct xe_sched_msg *msg)
> set_exec_queue_suspended(q);
> disable_scheduling(q, false);
> }
> - } else if (q->guc->suspend_pending) {
> - set_exec_queue_suspended(q);
> - suspend_fence_signal(q);
> + } else {
> + xe_sched_msg_lock(&q->guc->sched);
> + if (q->guc->suspend_pending) {
> + set_exec_queue_suspended(q);
> + suspend_fence_signal(q);
> + }
> + xe_sched_msg_unlock(&q->guc->sched);
> }
> }
>
> @@ -2130,7 +2134,9 @@ static void guc_exec_queue_kill(struct xe_exec_queue *q)
> {
> trace_xe_exec_queue_kill(q);
> set_exec_queue_killed(q);
> + xe_sched_msg_lock(&q->guc->sched);
> __suspend_fence_signal(q);
> + xe_sched_msg_unlock(&q->guc->sched);
> xe_guc_exec_queue_trigger_cleanup(q);
> }
>
> @@ -2392,11 +2398,15 @@ static void guc_exec_queue_suspend_timeout_ban(struct xe_exec_queue *q)
> */
> if (xe_exec_queue_is_multi_queue(q)) {
> set_exec_queue_group_banned(q);
> + xe_sched_msg_lock(&q->guc->sched);
> __suspend_fence_signal(q);
> + xe_sched_msg_unlock(&q->guc->sched);
> xe_guc_exec_queue_group_trigger_cleanup(q);
> } else {
> set_exec_queue_banned(q);
> + xe_sched_msg_lock(&q->guc->sched);
> __suspend_fence_signal(q);
> + xe_sched_msg_unlock(&q->guc->sched);
> xe_guc_exec_queue_trigger_cleanup(q);
> }
> }
> @@ -2614,10 +2624,12 @@ static void guc_exec_queue_stop(struct xe_guc *guc, struct xe_exec_queue *q)
> if (exec_queue_destroyed(q))
> do_destroy = true;
> }
> + xe_sched_msg_lock(sched);
> if (q->guc->suspend_pending) {
> set_exec_queue_suspended(q);
> suspend_fence_signal(q);
> }
> + xe_sched_msg_unlock(sched);
> atomic_and(EXEC_QUEUE_STATE_WEDGED | EXEC_QUEUE_STATE_BANNED |
> EXEC_QUEUE_STATE_KILLED | EXEC_QUEUE_STATE_DESTROYED |
> EXEC_QUEUE_STATE_SUSPENDED,
> @@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
> smp_wmb();
> wake_up_all(&guc->ct.wq);
> } else {
> + bool was_pending;
> +
> xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
> xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
>
> - if (q->guc->suspend_pending) {
> + xe_sched_msg_lock(&q->guc->sched);
> + was_pending = q->guc->suspend_pending;
> + if (was_pending) {
> clear_exec_queue_pending_disable(q);
> suspend_fence_signal(q);
> - } else {
> + }
> + xe_sched_msg_unlock(&q->guc->sched);
> +
> + if (!was_pending) {
> if (exec_queue_banned(q)) {
> smp_wmb();
> wake_up_all(&guc->ct.wq);
> --
> 2.53.0
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 20:53 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Niranjana Vishwanathapura
@ 2026-08-18 21:06 ` Matthew Brost
0 siblings, 0 replies; 11+ messages in thread
From: Matthew Brost @ 2026-08-18 21:06 UTC (permalink / raw)
To: Niranjana Vishwanathapura; +Cc: Jagmeet Randhawa, intel-xe
On Tue, Aug 18, 2026 at 01:53:15PM -0700, Niranjana Vishwanathapura wrote:
> On Wed, Aug 19, 2026 at 02:38:39AM +0800, Jagmeet Randhawa wrote:
> > q->guc->suspend_pending is accessed without any common lock.
> > __suspend_fence_signal(), called from guc_exec_queue_kill() and the
> > suspend-timeout ban path, clears the flag asynchronously. Meanwhile
> > handle_sched_done(), guc_exec_queue_stop() and
> > __guc_exec_queue_process_msg_suspend() check the flag and then call
> > suspend_fence_signal(), which asserts that it is still set.
> >
> > As the check and suspend_fence_signal() are not atomic, the clear can
> > land in between and trip the xe_gt_assert(q->guc->suspend_pending).
> >
> > The flag is already set and read under the per-queue msg_lock
> > (xe_sched_msg_lock()) on the suspend and resume paths. Extend that same
> > lock to the clear paths (kill and ban) and to the three check-then-act
> > sites so the check and the signal are atomic with respect to the clear.
> > In __guc_exec_queue_process_msg_suspend() only the non-sleeping branch is
> > wrapped, since the other branch waits. In handle_sched_done() the flag is
> > snapshotted under the lock and deregister_exec_queue() is kept outside it.
> >
> > v2: Document that sched->msg_lock also protects
> > guc->suspend_pending, which indicates a suspend message is in
> > flight, in addition to the sched->msgs list (Niranjana)
> >
> > Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
>
> LGTM
> Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
>
So my latest reply, I don't think patch is needed in its current form.
Matt
> > ---
> > drivers/gpu/drm/xe/xe_gpu_scheduler_types.h | 5 +++-
> > drivers/gpu/drm/xe/xe_guc_exec_queue_types.h | 5 +++-
> > drivers/gpu/drm/xe/xe_guc_submit.c | 29 ++++++++++++++++----
> > 3 files changed, 32 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> > index 63d9bf92583c..78ef2e8ded4f 100644
> > --- a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> > +++ b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> > @@ -47,7 +47,10 @@ struct xe_gpu_scheduler {
> > const struct xe_sched_backend_ops *ops;
> > /** @msgs: list of messages to be processed in @work_process_msg */
> > struct list_head msgs;
> > - /** @msg_lock: Message lock */
> > + /**
> > + * @msg_lock: Protects @msgs and guc->suspend_pending (indicates a
> > + * suspend message is in flight) of exec queues on this scheduler.
> > + */
> > spinlock_t msg_lock;
> > /** @work_process_msg: processes messages */
> > struct work_struct work_process_msg;
> > diff --git a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> > index d27826b36649..74b711abe257 100644
> > --- a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> > +++ b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> > @@ -52,7 +52,10 @@ struct xe_guc_exec_queue {
> > u16 id;
> > /** @suspend_wait: wait queue used to wait on pending suspends */
> > wait_queue_head_t suspend_wait;
> > - /** @suspend_pending: a suspend of the exec_queue is pending */
> > + /**
> > + * @suspend_pending: a suspend of the exec_queue is pending.
> > + * Protected by @sched.msg_lock.
> > + */
> > bool suspend_pending;
> > /**
> > * @suspend_count: Reference count of active suspend requests. The
> > diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> > index 9036f89dff7d..c565c1d32d3a 100644
> > --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> > +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> > @@ -1928,9 +1928,13 @@ static void __guc_exec_queue_process_msg_suspend(struct xe_sched_msg *msg)
> > set_exec_queue_suspended(q);
> > disable_scheduling(q, false);
> > }
> > - } else if (q->guc->suspend_pending) {
> > - set_exec_queue_suspended(q);
> > - suspend_fence_signal(q);
> > + } else {
> > + xe_sched_msg_lock(&q->guc->sched);
> > + if (q->guc->suspend_pending) {
> > + set_exec_queue_suspended(q);
> > + suspend_fence_signal(q);
> > + }
> > + xe_sched_msg_unlock(&q->guc->sched);
> > }
> > }
> >
> > @@ -2130,7 +2134,9 @@ static void guc_exec_queue_kill(struct xe_exec_queue *q)
> > {
> > trace_xe_exec_queue_kill(q);
> > set_exec_queue_killed(q);
> > + xe_sched_msg_lock(&q->guc->sched);
> > __suspend_fence_signal(q);
> > + xe_sched_msg_unlock(&q->guc->sched);
> > xe_guc_exec_queue_trigger_cleanup(q);
> > }
> >
> > @@ -2392,11 +2398,15 @@ static void guc_exec_queue_suspend_timeout_ban(struct xe_exec_queue *q)
> > */
> > if (xe_exec_queue_is_multi_queue(q)) {
> > set_exec_queue_group_banned(q);
> > + xe_sched_msg_lock(&q->guc->sched);
> > __suspend_fence_signal(q);
> > + xe_sched_msg_unlock(&q->guc->sched);
> > xe_guc_exec_queue_group_trigger_cleanup(q);
> > } else {
> > set_exec_queue_banned(q);
> > + xe_sched_msg_lock(&q->guc->sched);
> > __suspend_fence_signal(q);
> > + xe_sched_msg_unlock(&q->guc->sched);
> > xe_guc_exec_queue_trigger_cleanup(q);
> > }
> > }
> > @@ -2614,10 +2624,12 @@ static void guc_exec_queue_stop(struct xe_guc *guc, struct xe_exec_queue *q)
> > if (exec_queue_destroyed(q))
> > do_destroy = true;
> > }
> > + xe_sched_msg_lock(sched);
> > if (q->guc->suspend_pending) {
> > set_exec_queue_suspended(q);
> > suspend_fence_signal(q);
> > }
> > + xe_sched_msg_unlock(sched);
> > atomic_and(EXEC_QUEUE_STATE_WEDGED | EXEC_QUEUE_STATE_BANNED |
> > EXEC_QUEUE_STATE_KILLED | EXEC_QUEUE_STATE_DESTROYED |
> > EXEC_QUEUE_STATE_SUSPENDED,
> > @@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
> > smp_wmb();
> > wake_up_all(&guc->ct.wq);
> > } else {
> > + bool was_pending;
> > +
> > xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
> > xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
> >
> > - if (q->guc->suspend_pending) {
> > + xe_sched_msg_lock(&q->guc->sched);
> > + was_pending = q->guc->suspend_pending;
> > + if (was_pending) {
> > clear_exec_queue_pending_disable(q);
> > suspend_fence_signal(q);
> > - } else {
> > + }
> > + xe_sched_msg_unlock(&q->guc->sched);
> > +
> > + if (!was_pending) {
> > if (exec_queue_banned(q)) {
> > smp_wmb();
> > wake_up_all(&guc->ct.wq);
> > --
> > 2.53.0
> >
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Xe.CI.FULL: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2)
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
` (4 preceding siblings ...)
2026-08-18 21:05 ` Matthew Brost
@ 2026-08-18 22:08 ` Patchwork
5 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2026-08-18 22:08 UTC (permalink / raw)
To: Jagmeet Randhawa; +Cc: intel-xe
[-- Attachment #1: Type: text/plain, Size: 17485 bytes --]
== Series Details ==
Series: drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2)
URL : https://patchwork.freedesktop.org/series/172176/
State : success
== Summary ==
CI Bug Log - changes from xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef_FULL -> xe-pw-172176v2_FULL
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (2 -> 2)
------------------------------
No changes in participating hosts
Known issues
------------
Here are the changes found in xe-pw-172176v2_FULL that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_big_fb@4-tiled-32bpp-rotate-90:
- shard-bmg: NOTRUN -> [SKIP][1] ([Intel XE#2327])
[1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_big_fb@4-tiled-32bpp-rotate-90.html
* igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip:
- shard-bmg: NOTRUN -> [SKIP][2] ([Intel XE#1124]) +2 other tests skip
[2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip.html
* igt@kms_bw@connected-linear-tiling-4-displays-target-1920x1080p:
- shard-bmg: NOTRUN -> [SKIP][3] ([Intel XE#7679])
[3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_bw@connected-linear-tiling-4-displays-target-1920x1080p.html
* igt@kms_ccs@ccs-on-another-bo-y-tiled-gen12-rc-ccs:
- shard-bmg: NOTRUN -> [SKIP][4] ([Intel XE#2887])
[4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_ccs@ccs-on-another-bo-y-tiled-gen12-rc-ccs.html
* igt@kms_chamelium_frames@dp-crc-multiple:
- shard-bmg: NOTRUN -> [SKIP][5] ([Intel XE#2252]) +1 other test skip
[5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@kms_chamelium_frames@dp-crc-multiple.html
* igt@kms_cursor_crc@cursor-sliding-32x10:
- shard-bmg: NOTRUN -> [SKIP][6] ([Intel XE#2320])
[6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_cursor_crc@cursor-sliding-32x10.html
* igt@kms_dsc@dsc-fractional-bpp-with-bpc-bigjoiner:
- shard-bmg: NOTRUN -> [SKIP][7] ([Intel XE#8265])
[7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@kms_dsc@dsc-fractional-bpp-with-bpc-bigjoiner.html
* igt@kms_feature_discovery@display-4x:
- shard-bmg: NOTRUN -> [SKIP][8] ([Intel XE#1138] / [Intel XE#7344])
[8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-2/igt@kms_feature_discovery@display-4x.html
* igt@kms_frontbuffer_tracking@drrs-1p-primscrn-spr-indfb-fullscreen:
- shard-bmg: NOTRUN -> [SKIP][9] ([Intel XE#2311]) +6 other tests skip
[9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_frontbuffer_tracking@drrs-1p-primscrn-spr-indfb-fullscreen.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-move:
- shard-bmg: NOTRUN -> [SKIP][10] ([Intel XE#4141]) +1 other test skip
[10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-move.html
* igt@kms_frontbuffer_tracking@fbc-abgr161616f-draw-mmap-wc:
- shard-bmg: NOTRUN -> [SKIP][11] ([Intel XE#7061] / [Intel XE#7356])
[11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@kms_frontbuffer_tracking@fbc-abgr161616f-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psrhdr-modesetfrombusy:
- shard-bmg: NOTRUN -> [SKIP][12] ([Intel XE#2313]) +7 other tests skip
[12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@kms_frontbuffer_tracking@psrhdr-modesetfrombusy.html
* igt@kms_joiner@basic-ultra-joiner:
- shard-bmg: NOTRUN -> [SKIP][13] ([Intel XE#6911] / [Intel XE#7378])
[13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-2/igt@kms_joiner@basic-ultra-joiner.html
* igt@kms_mst@mst-suspend-read-crc:
- shard-bmg: NOTRUN -> [SKIP][14] ([Intel XE#8348])
[14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_mst@mst-suspend-read-crc.html
* igt@kms_pm_backlight@brightness-with-dpms:
- shard-bmg: NOTRUN -> [SKIP][15] ([Intel XE#2938] / [Intel XE#7376] / [Intel XE#7760])
[15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@kms_pm_backlight@brightness-with-dpms.html
* igt@kms_psr2_sf@fbc-psr2-overlay-plane-move-continuous-exceed-fully-sf:
- shard-bmg: NOTRUN -> [SKIP][16] ([Intel XE#1489])
[16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_psr2_sf@fbc-psr2-overlay-plane-move-continuous-exceed-fully-sf.html
* igt@kms_psr@fbc-pr-no-drrs:
- shard-bmg: NOTRUN -> [SKIP][17] ([Intel XE#2234] / [Intel XE#2850]) +2 other tests skip
[17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_psr@fbc-pr-no-drrs.html
* igt@kms_sharpness_filter@invalid-filter-with-plane:
- shard-bmg: NOTRUN -> [SKIP][18] ([Intel XE#6503])
[18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@kms_sharpness_filter@invalid-filter-with-plane.html
* igt@sriov_basic@pf-unbind-with-vf-probed:
- shard-bmg: NOTRUN -> [ABORT][19] ([Intel XE#8868])
[19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@sriov_basic@pf-unbind-with-vf-probed.html
* igt@xe_evict@evict-mixed-many-threads-small:
- shard-bmg: [PASS][20] -> [INCOMPLETE][21] ([Intel XE#6321] / [Intel XE#8355])
[20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-8/igt@xe_evict@evict-mixed-many-threads-small.html
[21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-4/igt@xe_evict@evict-mixed-many-threads-small.html
* igt@xe_evict@evict-small-multi-queue-priority:
- shard-bmg: NOTRUN -> [SKIP][22] ([Intel XE#8370])
[22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-2/igt@xe_evict@evict-small-multi-queue-priority.html
* igt@xe_exec_fault_mode@many-execqueues-multi-queue-userptr-invalidate-imm:
- shard-bmg: NOTRUN -> [SKIP][23] ([Intel XE#8374]) +1 other test skip
[23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-2/igt@xe_exec_fault_mode@many-execqueues-multi-queue-userptr-invalidate-imm.html
* igt@xe_exec_multi_queue@few-execs-preempt-mode-dyn-priority:
- shard-bmg: NOTRUN -> [SKIP][24] ([Intel XE#8364]) +4 other tests skip
[24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@xe_exec_multi_queue@few-execs-preempt-mode-dyn-priority.html
* igt@xe_exec_reset@multi-queue-long-spin-reuse-many-queue-switch:
- shard-bmg: NOTRUN -> [SKIP][25] ([Intel XE#8369])
[25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@xe_exec_reset@multi-queue-long-spin-reuse-many-queue-switch.html
* igt@xe_exec_system_allocator@many-64k-malloc-fork-read:
- shard-bmg: NOTRUN -> [INCOMPLETE][26] ([Intel XE#8159])
[26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@xe_exec_system_allocator@many-64k-malloc-fork-read.html
* igt@xe_exec_system_allocator@many-stride-new-prefetch:
- shard-bmg: NOTRUN -> [INCOMPLETE][27] ([Intel XE#7098] / [Intel XE#8159])
[27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@xe_exec_system_allocator@many-stride-new-prefetch.html
* igt@xe_exec_threads@threads-multi-queue-cm-fd-rebind:
- shard-bmg: NOTRUN -> [SKIP][28] ([Intel XE#8378]) +3 other tests skip
[28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@xe_exec_threads@threads-multi-queue-cm-fd-rebind.html
* igt@xe_fault_injection@vm-bind-fail-xe_vma_ops_alloc:
- shard-bmg: [PASS][29] -> [ABORT][30] ([Intel XE#8007])
[29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-8/igt@xe_fault_injection@vm-bind-fail-xe_vma_ops_alloc.html
[30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-4/igt@xe_fault_injection@vm-bind-fail-xe_vma_ops_alloc.html
* igt@xe_pxp@pxp-stale-bo-bind-post-rpm:
- shard-bmg: NOTRUN -> [SKIP][31] ([Intel XE#4733] / [Intel XE#7417])
[31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-6/igt@xe_pxp@pxp-stale-bo-bind-post-rpm.html
#### Possible fixes ####
* igt@kms_atomic_transition@plane-all-modeset-transition-fencing:
- shard-bmg: [INCOMPLETE][32] ([Intel XE#6819] / [Intel XE#8174]) -> [PASS][33] +1 other test pass
[32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-3/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html
[33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-2/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html
* igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs:
- shard-bmg: [INCOMPLETE][34] ([Intel XE#7084] / [Intel XE#8150]) -> [PASS][35] +1 other test pass
[34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-10/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs.html
[35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-7/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-lnl: [FAIL][36] ([Intel XE#301] / [Intel XE#3149]) -> [PASS][37] +1 other test pass
[36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-lnl-4/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-lnl-2/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_setmode@basic:
- shard-bmg: [FAIL][38] ([Intel XE#8618]) -> [PASS][39] +1 other test pass
[38]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-5/igt@kms_setmode@basic.html
[39]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-9/igt@kms_setmode@basic.html
* igt@kms_setmode@basic@pipe-c-hdmi-a-3:
- shard-bmg: [FAIL][40] -> [PASS][41]
[40]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-5/igt@kms_setmode@basic@pipe-c-hdmi-a-3.html
[41]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-9/igt@kms_setmode@basic@pipe-c-hdmi-a-3.html
* igt@xe_pmu@engine-activity-accuracy-50@engine-drm_xe_engine_class_compute0:
- shard-bmg: [FAIL][42] ([Intel XE#8555]) -> [PASS][43] +2 other tests pass
[42]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-2/igt@xe_pmu@engine-activity-accuracy-50@engine-drm_xe_engine_class_compute0.html
[43]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-8/igt@xe_pmu@engine-activity-accuracy-50@engine-drm_xe_engine_class_compute0.html
#### Warnings ####
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions:
- shard-lnl: [SKIP][44] ([Intel XE#309] / [Intel XE#7343] / [Intel XE#7935]) -> [SKIP][45] ([Intel XE#309] / [Intel XE#7343])
[44]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-lnl-8/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html
[45]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-lnl-5/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html
* igt@kms_tiled_display@basic-test-pattern:
- shard-bmg: [SKIP][46] ([Intel XE#2426] / [Intel XE#5848]) -> [FAIL][47] ([Intel XE#1729] / [Intel XE#7424])
[46]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-6/igt@kms_tiled_display@basic-test-pattern.html
[47]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-5/igt@kms_tiled_display@basic-test-pattern.html
* igt@kms_tiled_display@basic-test-pattern-with-chamelium:
- shard-bmg: [SKIP][48] ([Intel XE#2509] / [Intel XE#7437]) -> [SKIP][49] ([Intel XE#2426] / [Intel XE#5848])
[48]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef/shard-bmg-4/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
[49]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/shard-bmg-2/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
[Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
[Intel XE#1138]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1138
[Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
[Intel XE#1729]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1729
[Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
[Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
[Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
[Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
[Intel XE#2320]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2320
[Intel XE#2327]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2327
[Intel XE#2426]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2426
[Intel XE#2509]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2509
[Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
[Intel XE#2887]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2887
[Intel XE#2938]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2938
[Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
[Intel XE#309]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/309
[Intel XE#3149]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3149
[Intel XE#4141]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4141
[Intel XE#4733]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4733
[Intel XE#5848]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5848
[Intel XE#6321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6321
[Intel XE#6503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6503
[Intel XE#6819]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6819
[Intel XE#6911]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6911
[Intel XE#7061]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7061
[Intel XE#7084]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7084
[Intel XE#7098]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7098
[Intel XE#7343]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7343
[Intel XE#7344]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7344
[Intel XE#7356]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7356
[Intel XE#7376]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7376
[Intel XE#7378]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7378
[Intel XE#7417]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7417
[Intel XE#7424]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7424
[Intel XE#7437]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7437
[Intel XE#7679]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7679
[Intel XE#7760]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7760
[Intel XE#7935]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7935
[Intel XE#8007]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8007
[Intel XE#8150]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8150
[Intel XE#8159]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8159
[Intel XE#8174]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8174
[Intel XE#8265]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8265
[Intel XE#8348]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8348
[Intel XE#8355]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8355
[Intel XE#8364]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8364
[Intel XE#8369]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8369
[Intel XE#8370]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8370
[Intel XE#8374]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8374
[Intel XE#8378]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8378
[Intel XE#8555]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8555
[Intel XE#8618]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8618
[Intel XE#8868]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8868
Build changes
-------------
* Linux: xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef -> xe-pw-172176v2
IGT_9059: f5a4ff79434df36db1d6b13deb37c90a6602565e @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
xe-5610-275df33dcf4c5d018717867a0b29ed1d3b62c1ef: 275df33dcf4c5d018717867a0b29ed1d3b62c1ef
xe-pw-172176v2: 172176v2
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-172176v2/index.html
[-- Attachment #2: Type: text/html, Size: 19129 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 21:05 ` Matthew Brost
@ 2026-08-18 23:27 ` Niranjana Vishwanathapura
2026-08-19 1:01 ` Matthew Brost
0 siblings, 1 reply; 11+ messages in thread
From: Niranjana Vishwanathapura @ 2026-08-18 23:27 UTC (permalink / raw)
To: Matthew Brost; +Cc: Jagmeet Randhawa, intel-xe
On Tue, Aug 18, 2026 at 02:05:10PM -0700, Matthew Brost wrote:
>On Wed, Aug 19, 2026 at 02:38:39AM +0800, Jagmeet Randhawa wrote:
>
>This is designed to be lockless.
>
>> q->guc->suspend_pending is accessed without any common lock.
>> __suspend_fence_signal(), called from guc_exec_queue_kill() and the
>
>This is actually the problem. __suspend_fence_signal shouldn't be called
>from guc_exec_queue_kill(). This can prematurely signal a suspend fence
>while the hardware is still executing.
>
>__suspend_fence_signal should be called in two possible places:
>
>- Naturally in G2H handler (handle_sched_done)
>- Or in global event that takes down the GuC firmware
> (guc_exec_queue_stop)
>
>With that, a lock isn't need because the state machine / firmware
>interaction ensures everything is race free.
>
>So I think the solution is ensure __suspend_fence_signal is called in
>the correct places rather than adding protection via a lock.
>
Matt,
I think it is probably not as trivial as dropping __suspend_fence_signal()
from guc_exec_queue_kill() for following reasons.
1. __suspend_fence_signal() is also called from
guc_exec_queue_suspend_timeout_ban() if suspend_wait times out.
2. Calling __suspend_fence_signal() wakes up any suspend_wait(),
which otherwise will have to wait 5 seconds before timing out.
If in guc_exec_queue_kill(), if we just try to wake up suspend_wait,
without clearing suspend_pending, then a resume() might run before
TDR kicks in and hits the !suspend_pending assert.
3. Even if we drop __suspend_pending_signal() from guc_exec_queue_kill()
and guc_exec_queue_suspend_timeout_ban(), we still have
handle_sched_done() and guc_exec_queue_stop() which can race against
each other in accessing suspend_pending and clearing it.
So, probably this locking extention patch here might be simpler and
effective.
Niranjana
>Matt
>
>> suspend-timeout ban path, clears the flag asynchronously. Meanwhile
>> handle_sched_done(), guc_exec_queue_stop() and
>> __guc_exec_queue_process_msg_suspend() check the flag and then call
>> suspend_fence_signal(), which asserts that it is still set.
>>
>> As the check and suspend_fence_signal() are not atomic, the clear can
>> land in between and trip the xe_gt_assert(q->guc->suspend_pending).
>>
>> The flag is already set and read under the per-queue msg_lock
>> (xe_sched_msg_lock()) on the suspend and resume paths. Extend that same
>> lock to the clear paths (kill and ban) and to the three check-then-act
>> sites so the check and the signal are atomic with respect to the clear.
>> In __guc_exec_queue_process_msg_suspend() only the non-sleeping branch is
>> wrapped, since the other branch waits. In handle_sched_done() the flag is
>> snapshotted under the lock and deregister_exec_queue() is kept outside it.
>>
>> v2: Document that sched->msg_lock also protects
>> guc->suspend_pending, which indicates a suspend message is in
>> flight, in addition to the sched->msgs list (Niranjana)
>>
>> Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
>> ---
>> drivers/gpu/drm/xe/xe_gpu_scheduler_types.h | 5 +++-
>> drivers/gpu/drm/xe/xe_guc_exec_queue_types.h | 5 +++-
>> drivers/gpu/drm/xe/xe_guc_submit.c | 29 ++++++++++++++++----
>> 3 files changed, 32 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
>> index 63d9bf92583c..78ef2e8ded4f 100644
>> --- a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
>> +++ b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
>> @@ -47,7 +47,10 @@ struct xe_gpu_scheduler {
>> const struct xe_sched_backend_ops *ops;
>> /** @msgs: list of messages to be processed in @work_process_msg */
>> struct list_head msgs;
>> - /** @msg_lock: Message lock */
>> + /**
>> + * @msg_lock: Protects @msgs and guc->suspend_pending (indicates a
>> + * suspend message is in flight) of exec queues on this scheduler.
>> + */
>> spinlock_t msg_lock;
>> /** @work_process_msg: processes messages */
>> struct work_struct work_process_msg;
>> diff --git a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
>> index d27826b36649..74b711abe257 100644
>> --- a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
>> +++ b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
>> @@ -52,7 +52,10 @@ struct xe_guc_exec_queue {
>> u16 id;
>> /** @suspend_wait: wait queue used to wait on pending suspends */
>> wait_queue_head_t suspend_wait;
>> - /** @suspend_pending: a suspend of the exec_queue is pending */
>> + /**
>> + * @suspend_pending: a suspend of the exec_queue is pending.
>> + * Protected by @sched.msg_lock.
>> + */
>> bool suspend_pending;
>> /**
>> * @suspend_count: Reference count of active suspend requests. The
>> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
>> index 9036f89dff7d..c565c1d32d3a 100644
>> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
>> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
>> @@ -1928,9 +1928,13 @@ static void __guc_exec_queue_process_msg_suspend(struct xe_sched_msg *msg)
>> set_exec_queue_suspended(q);
>> disable_scheduling(q, false);
>> }
>> - } else if (q->guc->suspend_pending) {
>> - set_exec_queue_suspended(q);
>> - suspend_fence_signal(q);
>> + } else {
>> + xe_sched_msg_lock(&q->guc->sched);
>> + if (q->guc->suspend_pending) {
>> + set_exec_queue_suspended(q);
>> + suspend_fence_signal(q);
>> + }
>> + xe_sched_msg_unlock(&q->guc->sched);
>> }
>> }
>>
>> @@ -2130,7 +2134,9 @@ static void guc_exec_queue_kill(struct xe_exec_queue *q)
>> {
>> trace_xe_exec_queue_kill(q);
>> set_exec_queue_killed(q);
>> + xe_sched_msg_lock(&q->guc->sched);
>> __suspend_fence_signal(q);
>> + xe_sched_msg_unlock(&q->guc->sched);
>> xe_guc_exec_queue_trigger_cleanup(q);
>> }
>>
>> @@ -2392,11 +2398,15 @@ static void guc_exec_queue_suspend_timeout_ban(struct xe_exec_queue *q)
>> */
>> if (xe_exec_queue_is_multi_queue(q)) {
>> set_exec_queue_group_banned(q);
>> + xe_sched_msg_lock(&q->guc->sched);
>> __suspend_fence_signal(q);
>> + xe_sched_msg_unlock(&q->guc->sched);
>> xe_guc_exec_queue_group_trigger_cleanup(q);
>> } else {
>> set_exec_queue_banned(q);
>> + xe_sched_msg_lock(&q->guc->sched);
>> __suspend_fence_signal(q);
>> + xe_sched_msg_unlock(&q->guc->sched);
>> xe_guc_exec_queue_trigger_cleanup(q);
>> }
>> }
>> @@ -2614,10 +2624,12 @@ static void guc_exec_queue_stop(struct xe_guc *guc, struct xe_exec_queue *q)
>> if (exec_queue_destroyed(q))
>> do_destroy = true;
>> }
>> + xe_sched_msg_lock(sched);
>> if (q->guc->suspend_pending) {
>> set_exec_queue_suspended(q);
>> suspend_fence_signal(q);
>> }
>> + xe_sched_msg_unlock(sched);
>> atomic_and(EXEC_QUEUE_STATE_WEDGED | EXEC_QUEUE_STATE_BANNED |
>> EXEC_QUEUE_STATE_KILLED | EXEC_QUEUE_STATE_DESTROYED |
>> EXEC_QUEUE_STATE_SUSPENDED,
>> @@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
>> smp_wmb();
>> wake_up_all(&guc->ct.wq);
>> } else {
>> + bool was_pending;
>> +
>> xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
>> xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
>>
>> - if (q->guc->suspend_pending) {
>> + xe_sched_msg_lock(&q->guc->sched);
>> + was_pending = q->guc->suspend_pending;
>> + if (was_pending) {
>> clear_exec_queue_pending_disable(q);
>> suspend_fence_signal(q);
>> - } else {
>> + }
>> + xe_sched_msg_unlock(&q->guc->sched);
>> +
>> + if (!was_pending) {
>> if (exec_queue_banned(q)) {
>> smp_wmb();
>> wake_up_all(&guc->ct.wq);
>> --
>> 2.53.0
>>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access
2026-08-18 23:27 ` Niranjana Vishwanathapura
@ 2026-08-19 1:01 ` Matthew Brost
0 siblings, 0 replies; 11+ messages in thread
From: Matthew Brost @ 2026-08-19 1:01 UTC (permalink / raw)
To: Niranjana Vishwanathapura; +Cc: Jagmeet Randhawa, intel-xe
On Tue, Aug 18, 2026 at 04:27:14PM -0700, Niranjana Vishwanathapura wrote:
> On Tue, Aug 18, 2026 at 02:05:10PM -0700, Matthew Brost wrote:
> > On Wed, Aug 19, 2026 at 02:38:39AM +0800, Jagmeet Randhawa wrote:
> >
> > This is designed to be lockless.
> >
> > > q->guc->suspend_pending is accessed without any common lock.
> > > __suspend_fence_signal(), called from guc_exec_queue_kill() and the
> >
> > This is actually the problem. __suspend_fence_signal shouldn't be called
> > from guc_exec_queue_kill(). This can prematurely signal a suspend fence
> > while the hardware is still executing.
> >
> > __suspend_fence_signal should be called in two possible places:
> >
> > - Naturally in G2H handler (handle_sched_done)
> > - Or in global event that takes down the GuC firmware
> > (guc_exec_queue_stop)
> >
> > With that, a lock isn't need because the state machine / firmware
> > interaction ensures everything is race free.
> >
> > So I think the solution is ensure __suspend_fence_signal is called in
> > the correct places rather than adding protection via a lock.
> >
>
> Matt,
>
> I think it is probably not as trivial as dropping __suspend_fence_signal()
> from guc_exec_queue_kill() for following reasons.
Let's take a step back, has existing code been linked to any bugs?
>
> 1. __suspend_fence_signal() is also called from
> guc_exec_queue_suspend_timeout_ban() if suspend_wait times out.
>
Yes, this is another example of where this should not be called.
> 2. Calling __suspend_fence_signal() wakes up any suspend_wait(),
> which otherwise will have to wait 5 seconds before timing out.
> If in guc_exec_queue_kill(), if we just try to wake up suspend_wait,
> without clearing suspend_pending, then a resume() might run before
> TDR kicks in and hits the !suspend_pending assert.
>
Don't do a wake here.
> 3. Even if we drop __suspend_pending_signal() from guc_exec_queue_kill()
> and guc_exec_queue_suspend_timeout_ban(), we still have
> handle_sched_done() and guc_exec_queue_stop() which can race against
> each other in accessing suspend_pending and clearing it.
I don't think this part can race.
- __guc_exec_queue_process_msg_suspend, this is only there for GT
resets racing (I think). This should be executed before or after
guc_exec_queue_stop() but not in parallel.
- guc_exec_queue_wait_suspend_done(), this on wait queue and we don't
have lock upon reading, at least in this patch. So if justification
is all readers need a lock, then this is missing in this patch.
- guc_exec_queue_stop() touch this but this code is only reachable
when GuC exec queue is stopped and CTs are down. I guess a new
suspend could come in and race, so maybe in a lock is needed here.
>
> So, probably this locking extention patch here might be simpler and
> effective.
I'm thinking this need a bit more rework and would like to get this
right in single patch. I'm fine with a lock, but let's at least make
guc_exec_queue_wait_suspend_done() consistent in using a lock and remove
the two places we should not be calling __suspend_pending_signal().
Matt
>
> Niranjana
>
> > Matt
> >
> > > suspend-timeout ban path, clears the flag asynchronously. Meanwhile
> > > handle_sched_done(), guc_exec_queue_stop() and
> > > __guc_exec_queue_process_msg_suspend() check the flag and then call
> > > suspend_fence_signal(), which asserts that it is still set.
> > >
> > > As the check and suspend_fence_signal() are not atomic, the clear can
> > > land in between and trip the xe_gt_assert(q->guc->suspend_pending).
> > >
> > > The flag is already set and read under the per-queue msg_lock
> > > (xe_sched_msg_lock()) on the suspend and resume paths. Extend that same
> > > lock to the clear paths (kill and ban) and to the three check-then-act
> > > sites so the check and the signal are atomic with respect to the clear.
> > > In __guc_exec_queue_process_msg_suspend() only the non-sleeping branch is
> > > wrapped, since the other branch waits. In handle_sched_done() the flag is
> > > snapshotted under the lock and deregister_exec_queue() is kept outside it.
> > >
> > > v2: Document that sched->msg_lock also protects
> > > guc->suspend_pending, which indicates a suspend message is in
> > > flight, in addition to the sched->msgs list (Niranjana)
> > >
> > > Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa@intel.com>
> > > ---
> > > drivers/gpu/drm/xe/xe_gpu_scheduler_types.h | 5 +++-
> > > drivers/gpu/drm/xe/xe_guc_exec_queue_types.h | 5 +++-
> > > drivers/gpu/drm/xe/xe_guc_submit.c | 29 ++++++++++++++++----
> > > 3 files changed, 32 insertions(+), 7 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> > > index 63d9bf92583c..78ef2e8ded4f 100644
> > > --- a/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> > > +++ b/drivers/gpu/drm/xe/xe_gpu_scheduler_types.h
> > > @@ -47,7 +47,10 @@ struct xe_gpu_scheduler {
> > > const struct xe_sched_backend_ops *ops;
> > > /** @msgs: list of messages to be processed in @work_process_msg */
> > > struct list_head msgs;
> > > - /** @msg_lock: Message lock */
> > > + /**
> > > + * @msg_lock: Protects @msgs and guc->suspend_pending (indicates a
> > > + * suspend message is in flight) of exec queues on this scheduler.
> > > + */
> > > spinlock_t msg_lock;
> > > /** @work_process_msg: processes messages */
> > > struct work_struct work_process_msg;
> > > diff --git a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> > > index d27826b36649..74b711abe257 100644
> > > --- a/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> > > +++ b/drivers/gpu/drm/xe/xe_guc_exec_queue_types.h
> > > @@ -52,7 +52,10 @@ struct xe_guc_exec_queue {
> > > u16 id;
> > > /** @suspend_wait: wait queue used to wait on pending suspends */
> > > wait_queue_head_t suspend_wait;
> > > - /** @suspend_pending: a suspend of the exec_queue is pending */
> > > + /**
> > > + * @suspend_pending: a suspend of the exec_queue is pending.
> > > + * Protected by @sched.msg_lock.
> > > + */
> > > bool suspend_pending;
> > > /**
> > > * @suspend_count: Reference count of active suspend requests. The
> > > diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> > > index 9036f89dff7d..c565c1d32d3a 100644
> > > --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> > > +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> > > @@ -1928,9 +1928,13 @@ static void __guc_exec_queue_process_msg_suspend(struct xe_sched_msg *msg)
> > > set_exec_queue_suspended(q);
> > > disable_scheduling(q, false);
> > > }
> > > - } else if (q->guc->suspend_pending) {
> > > - set_exec_queue_suspended(q);
> > > - suspend_fence_signal(q);
> > > + } else {
> > > + xe_sched_msg_lock(&q->guc->sched);
> > > + if (q->guc->suspend_pending) {
> > > + set_exec_queue_suspended(q);
> > > + suspend_fence_signal(q);
> > > + }
> > > + xe_sched_msg_unlock(&q->guc->sched);
> > > }
> > > }
> > >
> > > @@ -2130,7 +2134,9 @@ static void guc_exec_queue_kill(struct xe_exec_queue *q)
> > > {
> > > trace_xe_exec_queue_kill(q);
> > > set_exec_queue_killed(q);
> > > + xe_sched_msg_lock(&q->guc->sched);
> > > __suspend_fence_signal(q);
> > > + xe_sched_msg_unlock(&q->guc->sched);
> > > xe_guc_exec_queue_trigger_cleanup(q);
> > > }
> > >
> > > @@ -2392,11 +2398,15 @@ static void guc_exec_queue_suspend_timeout_ban(struct xe_exec_queue *q)
> > > */
> > > if (xe_exec_queue_is_multi_queue(q)) {
> > > set_exec_queue_group_banned(q);
> > > + xe_sched_msg_lock(&q->guc->sched);
> > > __suspend_fence_signal(q);
> > > + xe_sched_msg_unlock(&q->guc->sched);
> > > xe_guc_exec_queue_group_trigger_cleanup(q);
> > > } else {
> > > set_exec_queue_banned(q);
> > > + xe_sched_msg_lock(&q->guc->sched);
> > > __suspend_fence_signal(q);
> > > + xe_sched_msg_unlock(&q->guc->sched);
> > > xe_guc_exec_queue_trigger_cleanup(q);
> > > }
> > > }
> > > @@ -2614,10 +2624,12 @@ static void guc_exec_queue_stop(struct xe_guc *guc, struct xe_exec_queue *q)
> > > if (exec_queue_destroyed(q))
> > > do_destroy = true;
> > > }
> > > + xe_sched_msg_lock(sched);
> > > if (q->guc->suspend_pending) {
> > > set_exec_queue_suspended(q);
> > > suspend_fence_signal(q);
> > > }
> > > + xe_sched_msg_unlock(sched);
> > > atomic_and(EXEC_QUEUE_STATE_WEDGED | EXEC_QUEUE_STATE_BANNED |
> > > EXEC_QUEUE_STATE_KILLED | EXEC_QUEUE_STATE_DESTROYED |
> > > EXEC_QUEUE_STATE_SUSPENDED,
> > > @@ -3222,13 +3234,20 @@ static void handle_sched_done(struct xe_guc *guc, struct xe_exec_queue *q,
> > > smp_wmb();
> > > wake_up_all(&guc->ct.wq);
> > > } else {
> > > + bool was_pending;
> > > +
> > > xe_gt_assert(guc_to_gt(guc), runnable_state == 0);
> > > xe_gt_assert(guc_to_gt(guc), exec_queue_pending_disable(q));
> > >
> > > - if (q->guc->suspend_pending) {
> > > + xe_sched_msg_lock(&q->guc->sched);
> > > + was_pending = q->guc->suspend_pending;
> > > + if (was_pending) {
> > > clear_exec_queue_pending_disable(q);
> > > suspend_fence_signal(q);
> > > - } else {
> > > + }
> > > + xe_sched_msg_unlock(&q->guc->sched);
> > > +
> > > + if (!was_pending) {
> > > if (exec_queue_banned(q)) {
> > > smp_wmb();
> > > wake_up_all(&guc->ct.wq);
> > > --
> > > 2.53.0
> > >
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2026-08-19 1:01 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-18 18:38 [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Jagmeet Randhawa
2026-08-18 18:45 ` ✓ CI.KUnit: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
2026-08-18 18:51 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access sashiko-bot
2026-08-18 20:54 ` Niranjana Vishwanathapura
2026-08-18 19:32 ` ✓ Xe.CI.BAT: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
2026-08-18 20:53 ` [PATCH v2] drm/xe/guc: Fix race around q->guc->suspend_pending access Niranjana Vishwanathapura
2026-08-18 21:06 ` Matthew Brost
2026-08-18 21:05 ` Matthew Brost
2026-08-18 23:27 ` Niranjana Vishwanathapura
2026-08-19 1:01 ` Matthew Brost
2026-08-18 22:08 ` ✓ Xe.CI.FULL: success for drm/xe/guc: Fix race around q->guc->suspend_pending access (rev2) Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.