linux-perf-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Renninger <trenn@suse.de>
To: tardyp@gmail.com
Cc: jean.pihet@newoldbits.com, acme@ghostprotocols.net,
	fweisbec@gmail.com, linux-perf-users@vger.kernel.org,
	mathieu.desnoyers@efficios.com,
	linux-trace-users@vger.kernel.org
Subject: Perf ABI versioning
Date: Mon, 24 Jan 2011 17:04:00 +0100	[thread overview]
Message-ID: <201101241704.01021.trenn@suse.de> (raw)

Hi,

as I recently cleaned up the power events with
some format/abi adjustings...

I realized that it was rather easy to adjust
the stuff in drivers/tools/perf, because there
are some nice facilities to query which events
are available.

I looked at pytimechart, there it's not that nice
and convenient would be some kind of version number.

pytimechart can parse traces which are simply copied
away from /sys/kernel/debug/trace and to check for
available events and their format, one would have to
do a separate, ugly to implement, iteration of parsing.

Would it make sense to introduce a trace
ABI version number?
It would be enough to increase it by one every time
any event (don't know about other perf facilities) is
added, removed or modified.
Ideally the modifications are tracked with a short
description in a file like Documentation/trace/format.changes

Untested, but the patch at the end should simply add
a version number on top of
/sys/kernel/debug/trace file

While this should be enough for the kernel, perf record
would need to include it into it's binary perf.data format
as well?

Then userspace apps could easily test whether supported
events would be included at all, whether only a subset of
supported events were made available and which format they have.

Does this make sense?

Thanks,

    Thomas

             reply	other threads:[~2011-01-24 16:04 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-24 16:04 Thomas Renninger [this message]
2011-01-24 16:18 ` Perf ABI versioning Thomas Renninger
2011-01-24 17:26 ` Pierre Tardy
2011-01-24 17:28 ` Jean Pihet
2011-01-24 20:05   ` Frederic Weisbecker
2011-01-24 20:12 ` Mathieu Desnoyers
2011-01-24 21:02   ` Thomas Renninger
2011-01-24 20:34 ` Frederic Weisbecker
2011-01-24 20:39   ` Peter Zijlstra
2011-01-24 20:40     ` Frederic Weisbecker
2011-01-24 21:28   ` Ingo Molnar
2011-01-24 21:49     ` Thomas Renninger
2011-01-24 21:57       ` Arnaldo Carvalho de Melo
2011-01-24 22:30         ` Thomas Renninger
2011-01-24 22:46           ` Frederic Weisbecker
2011-01-24 23:05             ` Arnaldo Carvalho de Melo
2011-01-24 21:59     ` Frederic Weisbecker

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=201101241704.01021.trenn@suse.de \
    --to=trenn@suse.de \
    --cc=acme@ghostprotocols.net \
    --cc=fweisbec@gmail.com \
    --cc=jean.pihet@newoldbits.com \
    --cc=linux-perf-users@vger.kernel.org \
    --cc=linux-trace-users@vger.kernel.org \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=tardyp@gmail.com \
    /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;
as well as URLs for NNTP newsgroup(s).