ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Steven Rostedt <rostedt@goodmis.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: ksummit <ksummit-discuss@lists.linuxfoundation.org>,
	Peter Zijlstra <peterz@infradead.org>,
	Julien Desfossez <jdesfossez@efficios.com>,
	daolivei <daolivei@redhat.com>, bristot <bristot@redhat.com>,
	Ingo Molnar <mingo@kernel.org>
Subject: Re: [Ksummit-discuss] [TECH TOPIC] Pulling away from the tracing ABI quicksands
Date: Thu, 29 Jun 2017 20:52:18 -0400	[thread overview]
Message-ID: <20170629205218.5b9a7923@gandalf.local.home> (raw)
In-Reply-To: <20170629203224.6bf7f29a@gandalf.local.home>

On Thu, 29 Jun 2017 20:32:24 -0400
Steven Rostedt <rostedt@goodmis.org> wrote:


> The problem we have is how to implement it?
> 
> We could make one tracepoint hook location have several different
> "tracepoints" in the tracefs directory letting the user choose how much
> information they want to trace. Have different tracepoints that can be
> enabled for a single location, where it may show extended fields.
> 
> I know people would like to have a way to cut down some fields, as
> real-estate in the ring buffer is of high value, and the smaller the
> events are, the more data one can collect. People who use tracing
> really do care about any wasted space (which is why we like to avoid
> writing zeros in fields no longer valued, it makes it harder to get the
> data you are after).
> 
> In summary, this is not another beat the dead horse how to do stable
> tracepoints. The focus is, how to make tracepoints more user
> customizable for their use cases.

Just to explain what Mathieu was talking about with echo and such, is
that Peter Zijlstra has been against multiple tracepoints for that one
sched switch location. He would prefer a dynamic case, instead of
having to enable all tracepoints, to get full functionality, as he
tends to use echo / cat to interact with ftrace than by using the
tools. He doesn't want the hassle of enabling more than one tracepoint
for sched_switch. One could call him lazy ;) but he's also the
maintainer of that tracepoint!

What would be really nice is a way to have sched_switch be a dynamic
tracepoint where it can trace differently depending on what is being
scheduled in and out. I'm sure other tracepoints locations may want a
similar feature.

But the devil is in the details of implementation, and we need to do
all this without breaking the existing tools that use the tracing
system. (See I have been listening to you!)

-- Steve

  parent reply	other threads:[~2017-06-30  0:52 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-29 21:20 Mathieu Desnoyers
2017-06-29 23:55 ` Steven Rostedt
2017-06-30  0:03   ` Linus Torvalds
2017-06-30  0:32     ` Steven Rostedt
2017-06-30  0:41       ` Linus Torvalds
2017-06-30  0:59         ` Steven Rostedt
2017-06-30  0:52       ` Steven Rostedt [this message]
2017-06-30  1:00         ` Linus Torvalds
2017-06-30  1:16           ` Steven Rostedt
2017-06-30  1:27             ` Steven Rostedt
2017-06-30  1:51               ` Linus Torvalds
2017-06-30  2:12                 ` Steven Rostedt
2017-06-30  2:34                   ` Linus Torvalds
2017-06-30  2:48                     ` Steven Rostedt
2017-06-30  2:58                     ` Alexei Starovoitov
2017-06-30  3:02                       ` Steven Rostedt
2017-06-30  3:20                         ` Steven Rostedt
2017-07-27 14:35                           ` Mathieu Desnoyers
2017-07-27 15:57                             ` Steven Rostedt
2017-06-30 18:24                         ` Josef Bacik
2017-06-30 18:29                           ` Steven Rostedt
2017-06-30 18:30                             ` Steven Rostedt
2017-06-30 18:37                               ` Josef Bacik
2017-07-06 19:10                                 ` Steven Rostedt
2017-07-21 21:45                                   ` Mathieu Desnoyers
2017-07-21 23:15                                     ` James Bottomley
2017-07-22  2:18                                     ` Steven Rostedt
2017-07-23 16:24                                       ` Josef Bacik
2017-07-23 21:25                                         ` Steven Rostedt
2017-07-04 14:51                   ` Peter Zijlstra
2017-06-30  1:38             ` Linus Torvalds
2017-06-30  1:45               ` Steven Rostedt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170629205218.5b9a7923@gandalf.local.home \
    --to=rostedt@goodmis.org \
    --cc=bristot@redhat.com \
    --cc=daolivei@redhat.com \
    --cc=jdesfossez@efficios.com \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=mingo@kernel.org \
    --cc=peterz@infradead.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox