Commit e74cfed5 authored by Chris Wilson's avatar Chris Wilson

Revert "drm/i915/ringbuffer: Ignore failure to setup the ring on Sandybridge"

This reverts commit 629e8941.
parent cae5852d
...@@ -180,30 +180,14 @@ static int init_ring_common(struct intel_ring_buffer *ring) ...@@ -180,30 +180,14 @@ static int init_ring_common(struct intel_ring_buffer *ring)
if ((I915_READ_CTL(ring) & RING_VALID) == 0 || if ((I915_READ_CTL(ring) & RING_VALID) == 0 ||
I915_READ_START(ring) != obj_priv->gtt_offset || I915_READ_START(ring) != obj_priv->gtt_offset ||
(I915_READ_HEAD(ring) & HEAD_ADDR) != 0) { (I915_READ_HEAD(ring) & HEAD_ADDR) != 0) {
if (IS_GEN6(ring->dev) && ring->dev->pdev->revision <= 8) { DRM_ERROR("%s initialization failed "
/* Early revisions of Sandybridge do not like "ctl %08x head %08x tail %08x start %08x\n",
* revealing the contents of the ring buffer ring->name,
* registers whilst idle. Fortunately, the I915_READ_CTL(ring),
* auto-reporting mechanism prevents most hangs, I915_READ_HEAD(ring),
* but this will bite us eventually... I915_READ_TAIL(ring),
*/ I915_READ_START(ring));
DRM_DEBUG("%s initialization failed " return -EIO;
"ctl %08x head %08x tail %08x start %08x. Ignoring, hope for the best!\n",
ring->name,
I915_READ_CTL(ring),
I915_READ_HEAD(ring),
I915_READ_TAIL(ring),
I915_READ_START(ring));
} else {
DRM_ERROR("%s initialization failed "
"ctl %08x head %08x tail %08x start %08x\n",
ring->name,
I915_READ_CTL(ring),
I915_READ_HEAD(ring),
I915_READ_TAIL(ring),
I915_READ_START(ring));
return -EIO;
}
} }
if (!drm_core_check_feature(ring->dev, DRIVER_MODESET)) if (!drm_core_check_feature(ring->dev, DRIVER_MODESET))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment