On Sun, Nov 06, 2011 at 09:51:38AM -0800, Kenneth Graunke wrote: > This looks like it was part of the BLT ring workaround which the prior > patch removed. However, this is a separate patch because it changes > behavior on shipping platforms.
Nope, this should actually be part of your previous patch. ring->private for the blt ring is only set for this gen6 preproduction-hw w/a, so this entire if block should go. You're missing a similar if block in the ring cleanup. I've posted that patch a few months back, but due to the Giant Whitespace Patch haven't yet bothered to forward port it ... Cheers, Daniel -- Daniel Vetter Mail: dan...@ffwll.ch Mobile: +41 (0)79 365 57 48 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx