drm/i915: Warn about missing context state workarounds only once
authorMichel Thierry <michel.thierry@intel.com>
Wed, 26 Nov 2014 14:21:02 +0000 (14:21 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 16 Dec 2014 09:39:12 +0000 (10:39 +0100)
Otherwise, new platforms without workarounds will hit this warning for
every new context created.

Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Signed-off-by: Michel Thierry <michel.thierry@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_lrc.c
drivers/gpu/drm/i915/intel_ringbuffer.c

index 4dc6d4263f3d339cc55c2f4f753f8d43f61fbfe1..7670a0f0f62043b70dd530541ba79e67f88cde3b 100644 (file)
@@ -1096,7 +1096,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;
index f1ce16997a7959708a96be9e9af5cbd6f476029f..3cad32a80108f3bc8059dbe4d5f9c8f9513a8ff0 100644 (file)
@@ -679,7 +679,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;