[PATCH] drm/xe: skip error capture when exec queue is killed

From: Tejas Upadhyay <tejas.upadhyay@intel.com>
To: intel-xe@lists.freedesktop.org
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>,
	Matthew Brost <matthew.brost@intel.com>,
	Tejas Upadhyay <tejas.upadhyay@intel.com>
Subject: [PATCH] drm/xe: skip error capture when exec queue is killed
Date: Thu, 25 Apr 2024 17:59:31 +0530	[thread overview]
Message-ID: <20240425122931.1851837-1-tejas.upadhyay@intel.com> (raw)

When user closes exec queue soon after job submission,
we are generating error coredump. Instead check if
exec queue is killed during job timeout then skip
error coredump capture, just free the job and return
proper scheduler state.

Signed-off-by: Tejas Upadhyay <tejas.upadhyay@intel.com>
---
 drivers/gpu/drm/xe/xe_guc_submit.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
index 93e1ee183e4a..376a2c04e899 100644
--- a/drivers/gpu/drm/xe/xe_guc_submit.c
+++ b/drivers/gpu/drm/xe/xe_guc_submit.c
@@ -971,7 +971,8 @@ guc_exec_queue_timedout_job(struct drm_sched_job *drm_job)
 	 * TDR has fired before free job worker. Common if exec queue
 	 * immediately closed after last fence signaled.
 	 */
-	if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &job->fence->flags)) {
+	if (exec_queue_killed(q) || 
+	    test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &job->fence->flags)) {
 		guc_exec_queue_free_job(drm_job);
 
 		return DRM_GPU_SCHED_STAT_NOMINAL;
-- 
2.25.1

