[Intel-gfx] [PATCH 1/4] drm/i915: don't block resume on fb console resume
Chris Wilson
chris at chris-wilson.co.uk
Fri Nov 2 14:35:58 CET 2012
On Wed, 31 Oct 2012 15:41:02 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> The console lock can be contended, so rather than prevent other drivers
> after us from being held up, queue the console suspend into the global
> work queue that can happen anytime. I've measured this to take around
> 200ms on my T420. Combined with the ring freq/turbo change, we should
> save almost 1/2 a second on resume.
Hmm, I'd rather not postpone the work unless actually contended,
otherwise we may end up just increasing the resume time whilst avoiding
the blame. Perhaps:
if (console_trylock() {
intel_fbdev_set_suspend(dev, 0);
console_unlock();
} else
schedule_work(&dev_priv->console_resume_work);
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list