From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752520AbcHIGLg (ORCPT ); Tue, 9 Aug 2016 02:11:36 -0400 Received: from mail-wm0-f65.google.com ([74.125.82.65]:35912 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752249AbcHIGLb (ORCPT ); Tue, 9 Aug 2016 02:11:31 -0400 Date: Tue, 9 Aug 2016 08:11:26 +0200 From: Daniel Vetter To: Alex Deucher Cc: Lyude , Tom St Denis , Jammy Zhou , xorg-driver-ati@lists.freedesktop.org, LKML , amd-gfx list , Maling list - DRI developers , Alex Deucher , Ken Wang , Christian =?iso-8859-1?Q?K=F6nig?= Subject: Re: [PATCH 0/7] Minor DP aux transaction fixes Message-ID: <20160809061126.GF6232@phenom.ffwll.local> Mail-Followup-To: Alex Deucher , Lyude , Tom St Denis , Jammy Zhou , xorg-driver-ati@lists.freedesktop.org, LKML , amd-gfx list , Maling list - DRI developers , Alex Deucher , Ken Wang , Christian =?iso-8859-1?Q?K=F6nig?= References: <1470443443-27252-1-git-send-email-cpaul@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Operating-System: Linux phenom 4.6.0-1-amd64 User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 08, 2016 at 01:30:37PM -0400, Alex Deucher wrote: > On Fri, Aug 5, 2016 at 8:30 PM, Lyude wrote: > > While I was investigating an unrelated bug on the radeon driver, I noticed that > > it's become rather difficult to actually read through dmesg with drm.debug > > turned on, on account of the huge number of messages we end up printing from > > failed DP aux transactions that happen every time we reprobe each connector. > > > > Timed out transactions are relatively normal, and as well there's a lot of > > places in radeon/amdgpu where we're printing redundant debugging information > > dozens of times each time we attempt a DP aux transactions. > > > > Additionally, I've removed some of the retry loops in amdgpu/radeon. These were > > definitely useful at one point, but since we now retry any failed aux > > transaction unconditionally in DRM's dp helpers they don't serve much purpose > > other then to make failing aux transactions take a lot more time then they need > > to. > > I've applied the amdgpu and radeon patches. For the drm patches, I > can either take them through my tree or via drm-misc. I applied the 2 core patches to drm-misc, thanks. -Daniel > > Alex > > > > > Lyude (7): > > drm/dp_helper: Print first error received on failure in > > drm_dp_dpcd_access() > > drm/radeon: Don't print error on aux transaction timeouts > > drm/radeon: Don't retry 7 times in radeon_dp_dpcd() > > drm/amdgpu: Don't print error on aux transaction timeouts > > drm/amdgpu: Don't retry 7 times in amdgpu_atombios_dp_get_dpcd() > > drm: Add ratelimited versions of the DRM_DEBUG* macros > > drm/dp_helper: Rate limit timeout errors from drm_dp_i2c_do_msg() > > > > drivers/gpu/drm/amd/amdgpu/atombios_dp.c | 22 ++++++++++------------ > > drivers/gpu/drm/drm_dp_helper.c | 14 ++++++++++++-- > > drivers/gpu/drm/radeon/atombios_dp.c | 21 ++++++++++----------- > > drivers/gpu/drm/radeon/radeon_dp_auxch.c | 1 - > > include/drm/drmP.h | 30 ++++++++++++++++++++++++++++++ > > 5 files changed, 62 insertions(+), 26 deletions(-) > > > > -- > > 2.7.4 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch