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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----Original Message-----
From: Intel-gfx <intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Jesus Narvaez
Sent: Thursday, August 8, 2024 1:50 PM
To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
Cc: Narvaez, Jesus <jesus.narvaez@xxxxxxxxx>
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@xxxxxxxxx>
-Jonathan Cavitt

> 
> Signed-off-by: Jesus Narvaez <jesus.narvaez@xxxxxxxxx>
> ---
>  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
> 
> 




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux