[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