On Thu, 3 May 2012 17:07:54 +0200, Daniel Vetter <daniel at ffwll.ch> wrote: > On Thu, Apr 26, 2012 at 11:28:42AM +0100, Chris Wilson wrote: > > This patch does have a side-effect. It removes the mark-busy for just > > moving the cursor - we don't want to increase the render clock just for > > the sprite, though we may want to bump the display frequency. I'd argue > > that we do not, and certainly don't want to take the struct_mutex here > > due to the large latencies that introduces. > > - > > - if (visible) > > - intel_mark_busy(dev, to_intel_framebuffer(crtc->fb)->obj); > > Hm, what's the reason to drop this one here? See above. -Chris -- Chris Wilson, Intel Open Source Technology Centre
