[Intel-gfx] BISECTED: Suspend regression on drm-intel-next

[Peter Clifton]

On Sat, 2010-12-18 at 21:42 +0000, Chris Wilson wrote:
> On Sat, 18 Dec 2010 20:02:21 +0000, Peter Clifton <pcjc2 at cam.ac.uk> wrote:
> > Hi Chris,
> > 
> > I bisected the suspend regression I mentioned, and came to this commit:
> > 
> > commit 0cdab21f9a1fca50dd27e488839f5a6578e333b2
> > Author: Chris Wilson <chris at chris-wilson.co.uk>
> > Date:   Sun Dec 5 17:27:06 2010 +0000
> > 
> >     drm/i915: Uncouple render/power ctx before suspending
> >     
> >     Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > 
> > 
> > Disabling power context tear down on suspend with this diff:
> 
> And you verified the revert against drm-intel-next HEAD? I'll revert just
> as soon as I see your confirmation. :)

Confirmed.. drm-intel-next is still broken.

I'm vaguely curious why it makes it lock up, but I'll settle for it not
doing so ;)  Only since suspend has been broken did I realise just how
much I rely upon it!

> [The patch was only trying to see if it made Ironlake stable on suspending
> with renderctx, to no avail so there should be no repercussions in
> reverting.]

Good to hear.

Best wishes,

-- 
Peter Clifton

Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA

Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328 - (Shared lab phone, ask for me)