On Sun, 16 Dec 2012 22:46:00 +0000, Chris Wilson <chris at chris-wilson.co.uk> wrote: > @@ -1087,6 +1087,9 @@ i915_error_first_batchbuffer(struct drm_i915_private *dev_priv, > if (!ring->get_seqno) > return NULL; > > + if (HAS_BROKEN_CS_TLB(dev_priv->dev)) > + return i915_error_object_create(dev_priv, ring->private); Hmm, this is complicated by userspace opting out of the CS w/a, and imposes quite a burden upon our simple seq interface. -Chris -- Chris Wilson, Intel Open Source Technology Centre