From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 14/16] drm/i915: Add watermark tracepoints Date: Tue, 15 Oct 2013 12:59:42 +0200 Message-ID: <20131015105942.GE4830@phenom.ffwll.local> References: <1381335490-4906-1-git-send-email-ville.syrjala@linux.intel.com> <1381335490-4906-15-git-send-email-ville.syrjala@linux.intel.com> <20131015084331.GB4830@phenom.ffwll.local> <20131015101149.GN13047@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-we0-f176.google.com (mail-we0-f176.google.com [74.125.82.176]) by gabe.freedesktop.org (Postfix) with ESMTP id CDBD644917 for ; Tue, 15 Oct 2013 03:59:22 -0700 (PDT) Received: by mail-we0-f176.google.com with SMTP id w62so8197600wes.21 for ; Tue, 15 Oct 2013 03:59:21 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20131015101149.GN13047@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Tue, Oct 15, 2013 at 01:11:49PM +0300, Ville Syrj=E4l=E4 wrote: > On Tue, Oct 15, 2013 at 10:43:31AM +0200, Daniel Vetter wrote: > > On Fri, Oct 11, 2013 at 04:40:24PM -0300, Paulo Zanoni wrote: > > > 2013/10/9 : > > > > From: Ville Syrj=E4l=E4 > > > > > > > > We may want to know what kind of watermarks got computed and progra= mmed > > > > into the hardware. Using tracepoints is much leaner than debug prin= ts. > > > > > > > > Also add trace call for the watermark state we read out of the > > > > hardware during init, though I;m not sure there's any way to see th= at > > > > trace as the events aren't available until the module is loaded. > > > > > > > > Signed-off-by: Ville Syrj=E4l=E4 > > > = > > > I never worked with these things before, but on a quick look it all s= ounds sane. > > > = > > > Acked-by: Paulo Zanoni > > = > > I'm not sold on tracepoints being the right tool here. DRM_DEBUG_KMS > > probably isn't it, since that would needlessly spam dmesg since it's way > > too coarse. But the kernel has this neat dynamic debug subsystem, which > > has the upshot that it's all nicely inline with the other modeset debug > > noise in dmesg. > = > I need to trace the watermark updates in relation to plane updates and > vblanks. Tracepoints seem like a good tool to me, though it does make > it a bit less useful for bug reports and such. I'm just worried that > regular printks add too much overhead to be all that useful for such > timing sensitive work with potentially quite a bit of trace data. Hm, I've thought we'd only dump the wm state once it's computed before we bash it into the hw. That part doesn't really feel timing critical, at least as long as we have the printks disabled by default. Otherwise syslogd will bring the system down ;-) Now I haven't really played around with it yet, but the dynamic printk stuff seemed to fit that bill. -Daniel -- = Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch