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

Reply via email to