Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1031098AbbD1WX6 (ORCPT ); Tue, 28 Apr 2015 18:23:58 -0400 Received: from mail-wg0-f48.google.com ([74.125.82.48]:33232 "EHLO mail-wg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030984AbbD1WX4 (ORCPT ); Tue, 28 Apr 2015 18:23:56 -0400 From: Thomas Gummerer To: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org, David Airlie , Jani Nikula , Daniel Vetter Subject: Re: [PATCH/RFC] i915: fix screen flickering In-Reply-To: <1430244832-7987-1-git-send-email-t.gummerer@gmail.com> References: <1430244832-7987-1-git-send-email-t.gummerer@gmail.com> User-Agent: Notmuch/0.19~rc1+3~g0d8251d (http://notmuchmail.org) Emacs/24.5.1 (x86_64-unknown-linux-gnu) Date: Wed, 29 Apr 2015 00:23:43 +0200 Message-ID: <87zj5rx574.fsf@hank.tgummerer.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2856 Lines: 78 Thomas Gummerer writes: > Commit c9f038a1a592 ("drm/i915: Don't assume primary & cursor are > always on for wm calculation (v4)") fixes a null pointer dereference. > Setting the primary and cursor panes to false in > ilk_compute_wm_parameters to false does however give the following > errors in the kernel log and causes the screen to flicker. > > [ 101.133716] [drm:intel_set_cpu_fifo_underrun_reporting [i915]] > *ERROR* uncleared fifo underrun on pipe A > [ 101.133725] [drm:intel_cpu_fifo_underrun_irq_handler [i915]] > *ERROR* CPU pipe A FIFO underrun > > Always setting the panes to enabled fixes this error. > > Signed-off-by: Thomas Gummerer > --- > Hi, > > I've noticed the regression in v4.1-rc1. I'm not sure this is the best > solution, but it's the best I could come up with in the time I had > available. I forgot to mention, the bug only occurs after suspending the system and waking it up again. In that case, moving the mouse pointer around on the build in laptop screen triggers the flickering, while the external monitor works just fine. Might be unrelated, but I only managed to trigger the bug when the system is suspended after the synergy server has started. > drivers/gpu/drm/i915/intel_pm.c | 22 ++++++++++------------ > 1 file changed, 10 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index fa4ccb3..377df60 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -2045,22 +2045,20 @@ static void ilk_compute_wm_parameters(struct drm_crtc *crtc, > p->pipe_htotal = intel_crtc->config->base.adjusted_mode.crtc_htotal; > p->pixel_rate = ilk_pipe_pixel_rate(dev, crtc); > > - if (crtc->primary->state->fb) { > - p->pri.enabled = true; > + if (crtc->primary->state->fb) > p->pri.bytes_per_pixel = > crtc->primary->state->fb->bits_per_pixel / 8; > - } else { > - p->pri.enabled = false; > + else > p->pri.bytes_per_pixel = 0; > - } > > - if (crtc->cursor->state->fb) { > - p->cur.enabled = true; > - p->cur.bytes_per_pixel = 4; > - } else { > - p->cur.enabled = false; > - p->cur.bytes_per_pixel = 0; > - } > + p->cur.bytes_per_pixel = 4; > + /* > + * TODO: for now, assume primary and cursor planes are always enabled. > + * Setting them to false makes the screen flicker. > + */ > + p->pri.enabled = true; > + p->cur.enabled = true; > + > p->pri.horiz_pixels = intel_crtc->config->pipe_src_w; > p->cur.horiz_pixels = intel_crtc->base.cursor->state->crtc_w; > > -- > 2.1.0.264.g0463184.dirty -- 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/