On Thu, May 03, 2012 at 04:13:32PM +0100, Chris Wilson wrote: > 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. Oops, pardon me for being blind. Patch queued for -next, thanks. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
