dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Matt Mackall <mpm@selenic.com>
To: Marek Otahal <markotahal@gmail.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: Regression with i915 and suspend on Thinkpad x220
Date: Sun, 22 May 2011 11:25:19 -0500	[thread overview]
Message-ID: <1306081519.4204.5.camel@calx> (raw)
In-Reply-To: <201105221813.59862.markotahal@gmail.com>

On Sun, 2011-05-22 at 18:13 +0200, Marek Otahal wrote:
> On Sunday 22 of May 2011 08:23:34 Keith Packard wrote:
> > On Sat, 21 May 2011 23:55:57 -0500, Matt Mackall <mpm@selenic.com> wrote:
> > > I've got a new Thinkpad x220 which won't wake up from suspend with
> > > 2.6.39, but works fine with 2.6.37.
> > > 
> > > I bisected it down to this cset:
> > > 
> > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit
> > > ;h=4efe070896e1f7373c98a13713e659d1f5dee52a
> > > 
> > > drm/i915: make the blitter report buffer modifications to the FBC unit
> > 
> > For now, you should be able to disable FBC:
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_drv.c
> > b/drivers/gpu/drm/i915/i915_drv.c index c34a8dd..2a82895 100644
> > --- a/drivers/gpu/drm/i915/i915_drv.c
> > +++ b/drivers/gpu/drm/i915/i915_drv.c
> > @@ -183,7 +183,7 @@ static const struct intel_device_info
> > intel_sandybridge_d_info = { static const struct intel_device_info
> > intel_sandybridge_m_info = { .gen = 6, .is_mobile = 1,
> >  	.need_gfx_hws = 1, .has_hotplug = 1,
> > -	.has_fbc = 1,
> > +	.has_fbc = 0,
> >  	.has_bsd_ring = 1,
> >  	.has_blt_ring = 1,
> >  };
> > 
> > Let us know if this works. There's a patch heading for 2.6.40 that
> > provides a module parameter to control FBC from the kernel command
> > line. We've got a couple X220s on order and I hope we'll be able to
> > reproduce (and fix) this issue -- FBC saves enough power that we'd like
> > to be able to leave it enabled on hardware that supports it.
> 
> Hello, 
> thanks for finding and working on this issue. 
> 
> I seem to be struck by the same bug, intel atom netbook, i915 graphics driver, 
> resume from suspend stopped working. 
> For details please see my bug reported at archlinux : 
> https://bugs.archlinux.org/24029
> 
> I tried suspending with the 2.6.37(.5) kernel as Matt suggested but it didn't 
> work for me either - Matt, could you tell me the last version you found 
> working? I'll try custom kernel with the patch provided when i have some time 
> for testing (is it working for you?). 

The last released kernel that worked for me was 2.6.37. Keith's patch
appears to work fine as well.

-- 
Mathematics is the supreme nostalgia of our time.

      reply	other threads:[~2011-05-22 16:25 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-22  4:55 Regression with i915 and suspend on Thinkpad x220 Matt Mackall
2011-05-22  6:23 ` Keith Packard
2011-05-22 15:28   ` Matt Mackall
2011-05-22 23:02     ` Keith Packard
2011-05-23  0:42       ` Matt Mackall
2011-05-22 16:13   ` Marek Otahal
2011-05-22 16:25     ` Matt Mackall [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1306081519.4204.5.camel@calx \
    --to=mpm@selenic.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=markotahal@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).