[PATCH] drm/i915/guc: Change GEM_WARN_ON to guc_err to prevent taints in CI

[Cavitt, Jonathan]

-----Original Message-----
From: Intel-gfx <intel-gfx-bounces at lists.freedesktop.org> On Behalf Of Jesus Narvaez
Sent: Thursday, August 8, 2024 1:50 PM
To: intel-gfx at lists.freedesktop.org
Cc: Narvaez, Jesus <jesus.narvaez at intel.com>
Subject: [PATCH] drm/i915/guc: Change GEM_WARN_ON to guc_err to prevent taints in CI
> 
> This warning was supposed to catch a harmless issue, but changing to
> guc_error should prevent kernel taints in CI runs.

I'll defer to your judgement on this, but IMO if we just want to log when this
happens and guarantee we don't throw kernel taints in CI, we might want to
consider using guc_info instead.  Not blocking on it, though.
Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
-Jonathan Cavitt

> 
> Signed-off-by: Jesus Narvaez <jesus.narvaez at intel.com>
> ---
>  drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> index 9400d0eb682b..c3a5d9e1288e 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
> @@ -2014,11 +2014,12 @@ void intel_guc_submission_reset_finish(struct intel_guc *guc)
>  
>  	/*
>  	 * Technically possible for either of these values to be non-zero here,
> -	 * but very unlikely + harmless. Regardless let's add a warn so we can
> +	 * but very unlikely + harmless. Regardless let's add an error so we can
>  	 * see in CI if this happens frequently / a precursor to taking down the
>  	 * machine.
>  	 */
> -	GEM_WARN_ON(atomic_read(&guc->outstanding_submission_g2h));
> +	if (atomic_read(&guc->outstanding_submission_g2h))
> +		guc_err(guc, "Unexpected outstanding GuC to Host in reset finish\n");
>  	atomic_set(&guc->outstanding_submission_g2h, 0);
>  
>  	intel_guc_global_policies_update(guc);
> -- 
> 2.34.1
> 
>