[Intel-gfx] [PATCH] drm/i915: Warn about missing context state workarounds only once

[Tvrtko Ursulin]

Backtraces make me jump and this one is very noisy, please merge. :)

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

On 11/26/2014 02:21 PM, Michel Thierry wrote:
> Otherwise, new platforms without workarounds will hit this warning for
> every new context created.
>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> Signed-off-by: Michel Thierry <michel.thierry at intel.com>
> ---
>   drivers/gpu/drm/i915/intel_lrc.c        | 2 +-
>   drivers/gpu/drm/i915/intel_ringbuffer.c | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> index 4d41f5a..4111cdd 100644
> --- a/drivers/gpu/drm/i915/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/intel_lrc.c
> @@ -1164,7 +1164,7 @@ static int intel_logical_ring_workarounds_emit(struct intel_engine_cs *ring,
>   	struct drm_i915_private *dev_priv = dev->dev_private;
>   	struct i915_workarounds *w = &dev_priv->workarounds;
>
> -	if (WARN_ON(w->count == 0))
> +	if (WARN_ON_ONCE(w->count == 0))
>   		return 0;
>
>   	ring->gpu_caches_dirty = true;
> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c
> index 1d01b51..104edf6 100644
> --- a/drivers/gpu/drm/i915/intel_ringbuffer.c
> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c
> @@ -669,7 +669,7 @@ static int intel_ring_workarounds_emit(struct intel_engine_cs *ring,
>   	struct drm_i915_private *dev_priv = dev->dev_private;
>   	struct i915_workarounds *w = &dev_priv->workarounds;
>
> -	if (WARN_ON(w->count == 0))
> +	if (WARN_ON_ONCE(w->count == 0))
>   		return 0;
>
>   	ring->gpu_caches_dirty = true;
>