From: Robert Richter <robert.richter@amd.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <peterz@infradead.org>,
Arnaldo Carvalho de Melo <acme@redhat.com>,
Stephane Eranian <eranian@google.com>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 4/4] perf script: Add script to collect and display IBS samples
Date: Fri, 23 Dec 2011 17:50:30 +0100 [thread overview]
Message-ID: <20111223165030.GG16765@erda.amd.com> (raw)
In-Reply-To: <20111223163930.GA5983@elte.hu>
On 23.12.11 17:39:30, Ingo Molnar wrote:
> No, i think you misunderstood me, i definitely think getting all
> the data out is perfectly fine and useful, if the hardware
> provides it. No need to make 'every' conceivable piece of IBS
> data useful straight away.
>
> All i want is at least *one* common usecase to work fine. This
> allows us to see things in practice and it also provides the
> seed for further improvements.
>
> Right now this does not seem to be the case: the 'report'
> portion only reports a raw hexa dump, not very useful to users.
>
> Once something like 'perf report' works for some useful looking
> sub-case you can also tack on all the extra IBS goodies that
> specialized tools might need as well, i have no problem with
> providing that.
Isn't that outlook I gave about pebs like implementation of the
precise bit with IBS "useful enough"? This would be on top patches
anyway, so no need to hold back current patches.
-Robert
--
Advanced Micro Devices, Inc.
Operating System Research Center
next prev parent reply other threads:[~2011-12-23 16:50 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-15 17:23 [PATCH 0/4] perf tools: Add support for IBS Robert Richter
2011-12-15 17:23 ` [PATCH 1/4] perf tool: Parse general/raw events from sysfs Robert Richter
2011-12-21 16:52 ` Robert Richter
2012-01-02 11:07 ` Stephane Eranian
2011-12-15 17:23 ` [PATCH 2/4] perf tools: Add pmu mappings to header information Robert Richter
2011-12-15 17:23 ` [PATCH 3/4] perf script: Add generic perl handler to process events Robert Richter
2011-12-29 20:56 ` [tip:perf/core] " tip-bot for Robert Richter
2011-12-15 17:23 ` [PATCH 4/4] perf script: Add script to collect and display IBS samples Robert Richter
2011-12-15 19:19 ` David Ahern
2011-12-15 23:47 ` Robert Richter
2011-12-23 10:33 ` Ingo Molnar
2011-12-23 11:19 ` Robert Richter
2011-12-23 13:53 ` Ingo Molnar
2011-12-23 14:14 ` Peter Zijlstra
2011-12-23 14:40 ` Ingo Molnar
2011-12-23 16:17 ` Robert Richter
2011-12-23 16:39 ` Ingo Molnar
2011-12-23 16:50 ` Robert Richter [this message]
2011-12-30 9:55 ` Ingo Molnar
2012-02-02 11:21 ` Robert Richter
2012-03-08 12:19 ` Ingo Molnar
2012-03-09 11:41 ` Robert Richter
2012-03-21 18:13 ` Robert Richter
2012-03-22 7:51 ` Ingo Molnar
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=20111223165030.GG16765@erda.amd.com \
--to=robert.richter@amd.com \
--cc=acme@redhat.com \
--cc=eranian@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=peterz@infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.