[Intel-gfx] [PATCH 4/4] drm/i915: don't rewrite the GTT on resume v4

Jesse Barnes jbarnes at virtuousgeek.org
Sat Nov 3 00:28:31 CET 2012


On Fri, 2 Nov 2012 23:55:04 +0100
Daniel Vetter <daniel at ffwll.ch> wrote:

> On Fri, Nov 02, 2012 at 11:14:02AM -0700, Jesse Barnes wrote:
> > The BIOS shouldn't be touching this memory across suspend/resume, so
> > just leave it alone.  This saves us ~6ms on resume on my T420 (retested
> > with write combined PTEs).
> > 
> > v2: change gtt restore default on pre-gen4 (Chris)
> >     move needs_gtt_restore flag into dev_priv
> > v3: make sure we restore GTT on resume from hibernate (Daniel)
> >     use opregion support as the cutoff for restore from resume (Chris)
> > v4: use a better check for opregion (Chris)
> > 
> > Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> 
> Merged, with some bikeshed applied, and the less-than-careful locking
> switch in patch 3 fixed. Please check whether I haven't botched it up.

Hm I even changed the name of the rps struct to catch all the cases at
compile time.  Maybe I sent out an old version or forgot a git add...
thanks for fixing it up.

-- 
Jesse Barnes, Intel Open Source Technology Center



More information about the Intel-gfx mailing list