Re: [Intel-gfx] [PATCH 1/4] drm/i915: don't block resume on fb console resume v2

2012-11-03 Thread Keith Packard
Jesse Barnes writes: > v2: use console_trylock() to try to resume the console immediately > (Chris) This will cause other printks to stall if i915 grabs the lock first. Seems like a way to get random resume delays to me. -- keith.pack...@intel.com pgpas36b6iooM.pgp Description: PGP signature

[Intel-gfx] [PATCH 1/4] drm/i915: don't block resume on fb console resume v2

2012-11-02 Thread Jesse Barnes
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/

[Intel-gfx] [PATCH 1/4] drm/i915: don't block resume on fb console resume v2

2012-11-02 Thread Jesse Barnes
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/