From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: 3.15-rc5: Regression in i915 driver? Date: Mon, 19 May 2014 09:07:05 +0200 Message-ID: <20140519070705.GA8790@phenom.ffwll.local> References: <1399796911.2442.7.camel@localhost.localdomain> <20140512063355.GA3813@nuc-i3427.alporthouse.com> <1400440120.3400.5.camel@localhost.localdomain> <20140518191338.GA11754@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f41.google.com (mail-ee0-f41.google.com [74.125.83.41]) by gabe.freedesktop.org (Postfix) with ESMTP id D08F86E15B for ; Mon, 19 May 2014 00:07:09 -0700 (PDT) Received: by mail-ee0-f41.google.com with SMTP id t10so3254700eei.0 for ; Mon, 19 May 2014 00:07:09 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140518191338.GA11754@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , Thomas Meyer , Daniel Vetter , Dave Airlie , "intel-gfx@lists.freedesktop.org" , Linux Kernel Mailing List List-Id: intel-gfx@lists.freedesktop.org On Sun, May 18, 2014 at 08:13:38PM +0100, Chris Wilson wrote: > On Sun, May 18, 2014 at 09:08:40PM +0200, Thomas Meyer wrote: > > Am Montag, den 12.05.2014, 07:33 +0100 schrieb Chris Wilson: > > > On Sun, May 11, 2014 at 07:40:57PM +0200, Daniel Vetter wrote: > > > > On Sun, May 11, 2014 at 11:02 AM, Dave Airlie wrote: > > > > > On 11 May 2014 18:28, Thomas Meyer wrote: > > > > >> Hi, > > > > >> > > > > >> 3.14.3 works as expected. > > > > >> 3.15-rc5 shows a strange behaviour: When resuming from ram the X server > > > > >> seems to be disfunctional. > > > > >> > > > > >> I see this WARNING in the kernel log before suspend to ram in the early > > > > >> boot process: > > > > > > > > Doesn't ring a bell really. > > > > > > Same symptoms as > > > https://bugs.freedesktop.org/show_bug.cgi?id=76554 > > > -Chris > > > > > > > bisected this to: > > # first bad commit: [78f2975eec9faff353a6194e854d3d39907bab68] drm/i915: Move all ring resets before setting the HWS page > > > > commit 78f2975eec9faff353a6194e854d3d39907bab68 > > Author: Chris Wilson > > Date: Wed Apr 2 16:36:07 2014 +0100 > > > > drm/i915: Move all ring resets before setting the HWS page > > > > In commit a51435a3137ad8ae75c288c39bd2d8b2696bae8f > > Author: Naresh Kumar Kachhi > > Date: Wed Mar 12 16:39:40 2014 +0530 > > > > drm/i915: disable rings before HW status page setup > > > > we reordered stopping the rings to do so before we set the HWS register. > > However, there is an extra workaround for g45 to reset the rings twice, > > and for consistency we should apply that workaround before setting the > > HWS to be sure that the rings are truly stopped. > > > > Reference: http://lkml.kernel.org/r/20140423202248.GA3621@amd.pavel.ucw.cz > > Tested-by: Pavel Machek > > Cc: Naresh Kumar Kachhi > > Signed-off-by: Chris Wilson > > Reviewed-by: Jesse Barnes > > Signed-off-by: Daniel Vetter > > Signed-off-by: Jani Nikula > > > > Reverting above commit from current linus' master tree makes the display server behave correctly after resume from ram. > > Boot time regression trumps resume regression? Doubly-so as we have an > earlier resume regression with identical symptoms? I'll check on my gm45 - that has has become much more crashy ime in 3.15 somehow, with ring init failures. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch