From: Frederic Weisbecker <fweisbec@gmail.com>
To: Jean Pihet <jean.pihet@newoldbits.com>
Cc: Thomas Renninger <trenn@suse.de>,
tardyp@gmail.com, acme@ghostprotocols.net,
linux-perf-users@vger.kernel.org, mathieu.desnoyers@efficios.com,
linux-trace-users@vger.kernel.org
Subject: Re: Perf ABI versioning
Date: Mon, 24 Jan 2011 21:05:37 +0100 [thread overview]
Message-ID: <20110124200534.GB2318@nowhere> (raw)
In-Reply-To: <AANLkTinbOkn15TM3CC9pMmceb1V_gxWm8EBMKLxnM50J@mail.gmail.com>
On Mon, Jan 24, 2011 at 06:28:54PM +0100, Jean Pihet wrote:
> Hi Thomas,
>
> On Mon, Jan 24, 2011 at 5:04 PM, Thomas Renninger <trenn@suse.de> wrote:
> > 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
>
> If needed the ABI versioning should at least contain a major and minor
> number, i.e. 'x.y'.
>
> The big question IMO is if it is needed or not. Is this a real problem
> in the userland? Are the apps always fitting the kernel ABI in a
> packaged distro?
This may help solving some tracepoint ABI issues that we have.
But let me add more tracing people and LKML in Cc in an answer to the explanation
of the idea, to discuss that more widely.
next prev parent reply other threads:[~2011-01-24 20:07 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-24 16:04 Perf ABI versioning Thomas Renninger
2011-01-24 16:18 ` Thomas Renninger
2011-01-24 17:26 ` Pierre Tardy
2011-01-24 17:28 ` Jean Pihet
2011-01-24 20:05 ` Frederic Weisbecker [this message]
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=20110124200534.GB2318@nowhere \
--to=fweisbec@gmail.com \
--cc=acme@ghostprotocols.net \
--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 \
--cc=trenn@suse.de \
/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).