[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <849307$b3orev@azsmga001.ch.intel.com>
Date: Tue, 11 Jan 2011 20:05:23 +0000
From: Chris Wilson <chris@...is-wilson.co.uk>
To: Jesse Barnes <jbarnes@...tuousgeek.org>,
Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Dave Airlie <airlied@...il.com>, Dave Airlie <airlied@...ux.ie>,
Eric Anholt <eric@...olt.net>,
Daniel Vetter <daniel.vetter@...ll.ch>,
linux-kernel@...r.kernel.org,
DRI mailing list <dri-devel@...ts.freedesktop.org>
Subject: Re: [git pull] drm for rc1
On Tue, 11 Jan 2011 11:45:05 -0800, Jesse Barnes <jbarnes@...tuousgeek.org> wrote:
> On Tue, 11 Jan 2011 11:25:39 -0800
> Linus Torvalds <torvalds@...ux-foundation.org> wrote:
>
> > On Tue, Jan 11, 2011 at 11:12 AM, Jesse Barnes <jbarnes@...tuousgeek.org> wrote:
> > >
> > > Have you tried reproducing it using xset dpms force off or similar?
> >
> > That doesn't seem to do anything bad.
> >
> > In fact, I think the second time it happened the screen never went
> > black - just the random photo thing was on. But no, forcing the screen
> > saver on doesn't do it either (ie pressing the "lock screen" icon and
> > waiting for a few pictures to cycle).
> >
> > Maybe the screen just has to be inactive for a longer time: do you do
> > some dynamic "let's power things down if nothing is changing"?
>
> There are some timeouts, the FBC engine will recompress about once
> every 15s; the self-refresh timers are much smaller though so it should
> be active anytime the CPU enters a deep sleep state. The clock
> frequency changes in millisecond time too, you can check the status of
> that in /sys/kernel/debug/dri/0/i915_drpc_status and i915_cur_delayinfo.
>
> I wonder if re-enabling rc6 may have caused your issues? That would be:
>
> commit 29a15061ff5df5bf9bf49c05c17f41eb2807a55a
> Author: Jesse Barnes <jbarnes@...tuousgeek.org>
> Date: Wed Jan 5 12:01:24 2011 -0800
>
> drm/i915: re-enable rc6 support for Ironlake+
We haven't actually got as far as that patch yet, that's waiting for a
suitable juncture to send the request. Now that the trees have converged I
can pick which patches need to go for rc1 and which can wait for -next.
Looking at the set of outstanding patch, my suspect would be the ring irq
refcounting which needed a fix and seems implicated by the error message.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists