Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752121AbbBQRtf (ORCPT ); Tue, 17 Feb 2015 12:49:35 -0500 Received: from mga01.intel.com ([192.55.52.88]:47582 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751739AbbBQRtd (ORCPT ); Tue, 17 Feb 2015 12:49:33 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,595,1418112000"; d="scan'208";a="686922784" Message-ID: <1424195346.12687.6.camel@theros.lm.intel.com> Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume From: Ross Zwisler To: "Deucher, Alexander" Cc: Michel =?ISO-8859-1?Q?D=E4nzer?= , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Dave Airlie , Lauri Kasanen , "Koenig, Christian" Date: Tue, 17 Feb 2015 10:49:06 -0700 In-Reply-To: References: <1423773026-5941-1-git-send-email-ross.zwisler@linux.intel.com> <54DD6469.9060809@daenzer.net> <1423886115.5037.12.camel@theros.lm.intel.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-4.fc20.rez) Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2456 Lines: 55 On Sat, 2015-02-14 at 06:25 +0000, Deucher, Alexander wrote: > > -----Original Message----- > > From: Ross Zwisler [mailto:ross.zwisler@linux.intel.com] > > Sent: Friday, February 13, 2015 10:55 PM > > To: Michel Dänzer > > Cc: linux-kernel@vger.kernel.org; dri-devel@lists.freedesktop.org; Deucher, > > Alexander; Dave Airlie; Lauri Kasanen; Koenig, Christian > > Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume > > > > On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: > > > On 13.02.2015 05:30, Ross Zwisler wrote: > > > > This patch reverts the changes made in this commit: > > > > > > > > deadcb36f49b ("drm/radeon: Use two-ended allocation by size, v2") > > > > > > > > That patch caused a regression on my system where the bottom of the > > > > screen flickers after my laptop goes thorough a suspend and resume. > > > > > > What kind of flicker is it? E.g. does it only affect X or also console, > > > does it flicker all the time or only when there is activity, what does > > > it look like, ... > > > > It's kind of hard to describe it precisely, so I made a video. :) > > > > http://youtu.be/ESm9SMnr0do > > > > It only affects X, not the console, and it seems to go away if you log > > out back to the login manager (I'm using GDM on Fedora 20) and back into > > your window manager. > > Does a VT switch or forcing a dpms cycle (sleep 5; xset dpms force off) > also fix it? It doesn't look related to the patch in question at all. > Is the flickering 100% reproducible or does it only happen > periodically? >From kernels 3.14 or so (when the deadcb36f49b patch was introduced) till 3.18 it happened 100% of the time. With 3.19 it only seems to happen maybe 50% of the time, but is still very easily reproducible. It's entirely possible that the patch isn't the root cause, but it just brought out a bug somewhere else. All I know is that I did a bisect, and with the commit before this the issue never happens, and after this commit it happens 100% of the time. :) Also, reverting that commit with 3.19 makes the issue go away. Nope, "xset dpms force off" doesn't fix it. After the screen goes black and comes back, the flicker is still there. - Ross -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/