[Intel-gfx] [PATCH 3/3] drm/i915: properly cancel rps_work on module unload v2
Chris Wilson
chris at chris-wilson.co.uk
Thu Sep 8 22:51:30 CEST 2011
On Thu, 8 Sep 2011 14:00:22 +0200, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> The rps disabling code wasn't properly cancelling outstanding work
> items. Also add a comment that explains why we're not racing with
> the work item that could unmask interrupts - that piece of code
> confused me quite a bit.
>
> v2: Ben Widawsky pointed out that the first patch would deadlock
> (and a few lesser problems). All corrected.
>
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> Reviewed-by: Ben Widawsky <ben at bwidawsk.net>
And since presumably the chip is in a completely different thermal
envelope and thawed, waking up the task function is pointless after
resume. ;-)
With the original deadlocks gone,
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list