[Intel-xe] [PATCH] fixup! drm/xe: Introduce a new DRM driver for Intel GPUs

[Matthew Brost]

On Thu, Aug 03, 2023 at 03:00:29PM -0700, Daniele Ceraolo Spurio wrote:
> Resets can be caused by userspace (and we do so in our testing),
> so we can't print at warning level when they occur.
> 
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Cc: Matthew Brost <matthew.brost at intel.com>

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

> ---
>  drivers/gpu/drm/xe/xe_guc_submit.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> index 193362518a62..60c311079fcc 100644
> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> @@ -843,8 +843,8 @@ guc_exec_queue_timedout_job(struct drm_sched_job *drm_job)
>  		XE_WARN_ON(q->flags & EXEC_QUEUE_FLAG_KERNEL);
>  		XE_WARN_ON(q->flags & EXEC_QUEUE_FLAG_VM && !exec_queue_killed(q));
>  
> -		drm_warn(&xe->drm, "Timedout job: seqno=%u, guc_id=%d, flags=0x%lx",
> -			 xe_sched_job_seqno(job), q->guc->id, q->flags);
> +		drm_notice(&xe->drm, "Timedout job: seqno=%u, guc_id=%d, flags=0x%lx",
> +			   xe_sched_job_seqno(job), q->guc->id, q->flags);
>  		simple_error_capture(q);
>  		xe_devcoredump(q);
>  	} else {
> @@ -1597,7 +1597,7 @@ int xe_guc_exec_queue_reset_handler(struct xe_guc *guc, u32 *msg, u32 len)
>  	if (unlikely(!q))
>  		return -EPROTO;
>  
> -	drm_warn(&xe->drm, "Engine reset: guc_id=%d", guc_id);
> +	drm_info(&xe->drm, "Engine reset: guc_id=%d", guc_id);
>  
>  	/* FIXME: Do error capture, most likely async */
>  
> -- 
> 2.41.0
>