On Mon, 16 Apr 2012 15:29:41 +0100, Chris Wilson <chris at chris-wilson.co.uk> wrote: > + ring = obj->ring; > + if (ring == NULL || ring == &dev_priv->ring[RCS]) should have been: ring == &dev_priv->ring[VCS] This does require testing on IVB, since the BCS was choosen for a reason, I'm sure. -Chris -- Chris Wilson, Intel Open Source Technology Centre