On 9/13/12 8:56 PM, Xiao Guangrong wrote:
We'd better keep the record/report function, sometimes, we can only get
perf.data
from the customers whose machine can not be reached for us.
Especially, other tracepoints are also interesting for us when the customers
encounter
the performance issue,
On 09/13/2012 12:56 PM, David Ahern wrote:
>
> That suggests what is really needed is a 'live' mode - a continual updating
> of the output like perf top, not a record and analyze later mode. Which does
> come back to why I responded to this email -- the syntax is klunky and
> awkward.
>
> So,
Em Thu, Sep 13, 2012 at 08:14:37AM -0600, David Ahern escreveu:
> On 9/13/12 7:45 AM, Arnaldo Carvalho de Melo wrote:
> >Em Wed, Sep 12, 2012 at 10:56:44PM -0600, David Ahern escreveu:
> >>So, I spent a fair amount of time today implementing a live mode.
> >>And after a lot of swearing at the trac
On 9/13/12 7:45 AM, Arnaldo Carvalho de Melo wrote:
Em Wed, Sep 12, 2012 at 10:56:44PM -0600, David Ahern escreveu:
static const char * const kvm_usage[] = {
+"perf kvm [] {top|record|report|diff|buildid-list|stat}",
The usage for the report/record sub commands of stat is never shown. e
Em Wed, Sep 12, 2012 at 10:56:44PM -0600, David Ahern escreveu:
> >> static const char * const kvm_usage[] = {
> >>+"perf kvm [] {top|record|report|diff|buildid-list|stat}",
> >The usage for the report/record sub commands of stat is never shown. e.g.,
> >$ perf kvm stat
> >--> shows help for
static const char * const kvm_usage[] = {
-"perf kvm [] {top|record|report|diff|buildid-list}",
+"perf kvm [] {top|record|report|diff|buildid-list|stat}",
NULL
};
The usage for the report/record sub commands of stat is never shown. e.g.,
$ perf kvm stat
--> shows help for perf
On 9/3/12 2:48 AM, don wrote:
于 2012年08月31日 02:29, David Ahern 写道:
In addition to Andrew's comment about making the stats struct and
functions generic...
Yes. :-)
On 8/27/12 3:51 AM, Dong Hao wrote:
---8<---
+static void exit_event_decode_key(struct event_key *key, char
decode[20])
+{
+
于 2012年08月31日 02:29, David Ahern 写道:
In addition to Andrew's comment about making the stats struct and
functions generic...
Yes. :-)
On 8/27/12 3:51 AM, Dong Hao wrote:
---8<---
+static void exit_event_decode_key(struct event_key *key, char
decode[20])
+{
+ const char *exit_reason = g
于 2012年08月27日 23:53, Andrew Jones 写道:
On Mon, Aug 27, 2012 at 05:51:46PM +0800, Dong Hao wrote:
+struct event_stats {
+ u64 count;
+ u64 time;
+
+ /* used to calculate stddev. */
+ double mean;
+ double M2;
+};
How about moving the stats functions from builtin-s
In addition to Andrew's comment about making the stats struct and
functions generic...
On 8/27/12 3:51 AM, Dong Hao wrote:
---8<---
+static void exit_event_decode_key(struct event_key *key, char decode[20])
+{
+ const char *exit_reason = get_exit_reason(key->key);
+
+ snprintf(deco
On 8/28/12 12:35 AM, Andrew Jones wrote:
This is the second user. As I said, move the stats code from
builtin-stat.c. Or are you suggesting to duplicate the code once,
and then refactor the second (really the third) time it's needed?
gah, I misunderstood your comment. I see your point now.
Dav
On Mon, Aug 27, 2012 at 01:34:36PM -0600, David Ahern wrote:
> On 8/27/12 9:53 AM, Andrew Jones wrote:
> >On Mon, Aug 27, 2012 at 05:51:46PM +0800, Dong Hao wrote:
> >
> >
> >
> >>+struct event_stats {
> >>+ u64 count;
> >>+ u64 time;
> >>+
> >>+ /* used to calculate stddev. */
> >>+ double
On 8/27/12 9:53 AM, Andrew Jones wrote:
On Mon, Aug 27, 2012 at 05:51:46PM +0800, Dong Hao wrote:
+struct event_stats {
+ u64 count;
+ u64 time;
+
+ /* used to calculate stddev. */
+ double mean;
+ double M2;
+};
How about moving the stats functions from builti
On Mon, Aug 27, 2012 at 05:51:46PM +0800, Dong Hao wrote:
> +struct event_stats {
> + u64 count;
> + u64 time;
> +
> + /* used to calculate stddev. */
> + double mean;
> + double M2;
> +};
How about moving the stats functions from builtin-stat.c to e.g.
util/stats.c, and the
This one does not apply to Arnaldo's current perf/core branch. Please
rebase.
David
On 8/23/12 7:15 PM, Dong Hao wrote:
From: Xiao Guangrong
Add 'perf kvm stat' support to analyze kvm vmexit/mmio/ioport smartly
Usage:
- kvm stat
run a command and gather performance counter statistics, it
15 matches
Mail list logo
