From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: Intel i915 freeze on latest git Date: Tue, 1 Feb 2011 08:31:12 -0800 Message-ID: <20110201083112.3cfefc3a@jbarnes-desktop> References: <20110201112200.GA2349@fujiko> <1bdc18$jeuv05@fmsmga002.fm.intel.com> <20110201122943.GA2011@fujiko> <20110201140754.GA2229@fujiko> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from cpoproxy3-pub.bluehost.com (cpoproxy3-pub.bluehost.com [67.222.54.6]) by gabe.freedesktop.org (Postfix) with SMTP id 6CE5B9E76B for ; Tue, 1 Feb 2011 08:31:23 -0800 (PST) In-Reply-To: <20110201140754.GA2229@fujiko> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Francesco Allertsen Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Tue, 1 Feb 2011 15:07:55 +0100 Francesco Allertsen wrote: > On Tue, Feb 01, 2011 at 01:16:01PM +0000, Chris Wilson wrote: > > Useful for me. Just confirms that you have an equivalent machine to > > this Lenovo x201s, on which those patches were tested after receiving > > from Jesse. :| > > Yes, I have a Lenovo x201s :-P. > > > The dmesg would have been useful to know whether the i915.ko module is > > loaded during X startup, and so reveal some information about the sequence > > of function calls (i.e. whether the modified routines were running at the > > time of the hang, or whether they had completed much earlier during boot). > > The drm.debug dmesg is usually a gold mine of information. > > Attacched there is the full dmesg log. > > > Another useful test is whether i915.powersave=0 also prevents the hang? > > No, it doesn't have any effect. Yeah I don't think we use that flag for rc6, though we probably should. The bisect is interesting, I'd have expected a failure when we re-enabled rc6 on ILK or when we fixed up the ring buffer init. The cleanup patch should be just that, but at least reverting it shouldn't cause any trouble. -- Jesse Barnes, Intel Open Source Technology Center