<#part sign=pgpmime>
On Wed, 8 Feb 2012 15:54:25 +0100, Daniel Vetter <dan...@ffwll.ch> wrote:

> I'm on the fence whether we should include this in -fixes. On one hand it
> fixes a severe issue on snb, but introduces a perf regression without the
> second patch. Otoh we've never shipped snb without it broken like this.
> But if it turns out that this is broken on ilk and earlier, too, I think
> we definitely need these two patches in -fixes.

The first one is a simple bug fix -- not attempting to use the status
page value. That seems reasonable for -fixes.

The second one has a bunch of global changes which seems less suitable
for -fixes, especially as it now requires that the driver reliably track
ring positions, which seems a bit perilous to me.

-- 
keith.pack...@intel.com
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to