From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 3/6] drm/i915: Downgrade pipe state mismatches to DRM_DEBUG_KMS Date: Thu, 5 Dec 2013 12:49:05 +0100 Message-ID: <20131205114905.GG27344@phenom.ffwll.local> References: <1385990771-3877-1-git-send-email-rodrigo.vivi@gmail.com> <1385990771-3877-4-git-send-email-rodrigo.vivi@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f45.google.com (mail-wg0-f45.google.com [74.125.82.45]) by gabe.freedesktop.org (Postfix) with ESMTP id 25E15FA6CC for ; Thu, 5 Dec 2013 03:48:20 -0800 (PST) Received: by mail-wg0-f45.google.com with SMTP id y10so14882175wgg.24 for ; Thu, 05 Dec 2013 03:48:18 -0800 (PST) Content-Disposition: inline In-Reply-To: <1385990771-3877-4-git-send-email-rodrigo.vivi@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Rodrigo Vivi Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, Dec 02, 2013 at 11:26:08AM -0200, Rodrigo Vivi wrote: > From: Adam Jackson > > This is, by far, the most common kernel retrace in i915 I'm seeing: > > https://retrace.fedoraproject.org/faf/problems/1346879/ > > There's not enough information in the backtrace to know if something > actually went wrong or if this is just an assertion failure, so it's > pretty useless. Downgrade this to DRM_DEBUG_KMS so we can still see it > if we want it. > > Signed-off-by: Adam Jackson > Signed-off-by: Rodrigo Vivi Nacked. I want these reports, and if the default dmesg spam doesn't contain enough information to make those actionable then we need to fix that. Atm our approach is to just ask for a drm.debug=0xe log reproducing the issue, which contains all the relevant information. If distros don't want to bother, then they can carry this locally. -Daniel > --- > drivers/gpu/drm/i915/intel_display.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 035588a..a403611 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -9463,7 +9463,7 @@ check_crtc_state(struct drm_device *dev) > > if (active && > !intel_pipe_config_compare(dev, &crtc->config, &pipe_config)) { > - WARN(1, "pipe state doesn't match!\n"); > + DRM_DEBUG_KMS("pipe state doesn't match!\n"); > intel_dump_pipe_config(crtc, &pipe_config, > "[hw state]"); > intel_dump_pipe_config(crtc, &crtc->config, > -- > 1.8.3.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch