[Intel-gfx] [PATCH] drm/i915/contexts: Always use arbitration reset workaround

Ben Widawsky ben at bwidawsk.net
Wed Aug 22 21:08:59 CEST 2012


VLV needs it, HSW needs it, and it shouldn't hurt on SNB.

Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
---
 drivers/gpu/drm/i915/i915_gem_context.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c
index 5c2d354..f12ef29 100644
--- a/drivers/gpu/drm/i915/i915_gem_context.c
+++ b/drivers/gpu/drm/i915/i915_gem_context.c
@@ -337,10 +337,8 @@ mi_set_context(struct intel_ring_buffer *ring,
 	if (ret)
 		return ret;
 
-	if (IS_GEN7(ring->dev))
-		intel_ring_emit(ring, MI_ARB_ON_OFF | MI_ARB_DISABLE);
-	else
-		intel_ring_emit(ring, MI_NOOP);
+	/* w/a */
+	intel_ring_emit(ring, MI_ARB_ON_OFF | MI_ARB_DISABLE);
 
 	intel_ring_emit(ring, MI_NOOP);
 	intel_ring_emit(ring, MI_SET_CONTEXT);
@@ -352,10 +350,7 @@ mi_set_context(struct intel_ring_buffer *ring,
 	/* w/a: MI_SET_CONTEXT must always be followed by MI_NOOP */
 	intel_ring_emit(ring, MI_NOOP);
 
-	if (IS_GEN7(ring->dev))
-		intel_ring_emit(ring, MI_ARB_ON_OFF | MI_ARB_ENABLE);
-	else
-		intel_ring_emit(ring, MI_NOOP);
+	intel_ring_emit(ring, MI_ARB_ON_OFF | MI_ARB_ENABLE);
 
 	intel_ring_advance(ring);
 
-- 
1.7.11.5




More information about the Intel-gfx mailing list