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

[Matthew Brost <matthew.brost@intel.com>]

On Tue, Apr 30, 2024 at 06:42:29PM +0530, Tejas Upadhyay wrote:
> 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.
> 
> V2:
>   - Just skip error capture - MattB
> 
> Signed-off-by: Tejas Upadhyay <tejas.upadhyay@intel.com>

Finally send to right verison...

Reviewed-by: Matthew Brost <matthew.brost@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_guc_submit.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> index d274a139010b..2c0aa3443cd9 100644
> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> @@ -980,8 +980,10 @@ guc_exec_queue_timedout_job(struct drm_sched_job *drm_job)
>  	xe_gt_WARN(q->gt, q->flags & EXEC_QUEUE_FLAG_VM && !exec_queue_killed(q),
>  		   "VM job timed out on non-killed execqueue\n");
>  
> -	simple_error_capture(q);
> -	xe_devcoredump(job);
> +	if (!exec_queue_killed(q)) {
> +		simple_error_capture(q);
> +		xe_devcoredump(job);
> +	}
>  
>  	trace_xe_sched_job_timedout(job);
>  
> -- 
> 2.25.1
>