From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Subject: Re: [PATCH 14/16] drm/i915: Add watermark tracepoints
Date: Tue, 15 Oct 2013 13:11:49 +0300
Message-ID: <20131015101149.GN13047@intel.com>
References: <1381335490-4906-1-git-send-email-ville.syrjala@linux.intel.com>
	<1381335490-4906-15-git-send-email-ville.syrjala@linux.intel.com>
	<CA+gsUGTVa8MiGUvoup4n11p4=9zZZREekOgAGOB0aqAZyzBgdw@mail.gmail.com>
	<20131015084331.GB4830@phenom.ffwll.local>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from mga14.intel.com (mga14.intel.com [143.182.124.37])
	by gabe.freedesktop.org (Postfix) with ESMTP id A9A7844753
	for <intel-gfx@lists.freedesktop.org>;
	Tue, 15 Oct 2013 03:12:02 -0700 (PDT)
Content-Disposition: inline
In-Reply-To: <20131015084331.GB4830@phenom.ffwll.local>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=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: Daniel Vetter <daniel@ffwll.ch>
Cc: Intel Graphics Development <intel-gfx@lists.freedesktop.org>
List-Id: intel-gfx@lists.freedesktop.org

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  <ville.syrjala@linux.intel.com>:
> > > From: Ville Syrj=E4l=E4 <ville.syrjala@linux.intel.com>
> > >
> > > We may want to know what kind of watermarks got computed and programm=
ed
> > > into the hardware. Using tracepoints is much leaner than debug prints.
> > >
> > > 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 that
> > > trace as the events aren't available until the module is loaded.
> > >
> > > Signed-off-by: Ville Syrj=E4l=E4 <ville.syrjala@linux.intel.com>
> > =

> > I never worked with these things before, but on a quick look it all sou=
nds sane.
> > =

> > Acked-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
> =

> 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.

-- =

Ville Syrj=E4l=E4
Intel OTC