[Intel-gfx] [PATCH] drm/i915: Only run idle processing from i915_gem_retire_requests_worker

[Daniel Vetter]

On Wed, Jan 30, 2013 at 03:32:33PM -0200, Rodrigo Vivi wrote:
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi at gmail.com>
> 
> 
> On Mon, Jan 28, 2013 at 9:02 AM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> > On Tue, Jan 08, 2013 at 11:02:57AM +0000, Chris Wilson wrote:
> >> When adding the fb idle detection to mark-inactive, it was forgotten
> >> that userspace can drive the processing of retire-requests. We assumed
> >> that it would be principally driven by the retire requests worker,
> >> running once every second whilst active and so we would get the deferred
> >> timer for free. Instead we spend too many CPU cycles reclocking the LVDS
> >> preventing real work from being done.
> >>
> >> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> >> Reported-and-tested-by: Alexander Lam <lambchop468 at gmail.com>
> >> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=58843
> >> Cc: stable at vger.kernel.org
> >
> > Poke.

Queued for -next, thanks for the patch.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch