[Intel-gfx] [PATCH 6/6] drm/i915: Perform intel_enable_fbc() from a delayed task

[Jesse Barnes]

On Thu,  7 Jul 2011 12:48:11 +0100
Chris Wilson <chris at chris-wilson.co.uk> wrote:

> In order to accommodate the requirements of re-enabling FBC after
> page-flipping, but to avoid doing so and incurring the cost of a wait
> for vblank in the middle of a page-flip sequence, we defer the actual
> enablement by 50ms. If any request to disable FBC arrive within that
> interval, the enablement is cancelled and we are saved from blocking on
> the wait.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---

Assuming we can't get away with what Keith suggested, this looks like a
good change.

Did you test module unload/reload as well to make sure the work
teardown is working correctly?

Reviewed-by: Jesse Barnes <jbarnes at virtuousgeek.org>

-- 
Jesse Barnes, Intel Open Source Technology Center