Re: [PATCH] perf tools: Fix reading of perf.data file header

Brice Goglin
2009-08-07T08:18:29Z

Ingo Molnar wrote:
> I completely forgot about it. And since we now have --scale enabled
> by default (and it doesnt make much sense to disable it i guess), it
> would not be a big issue to rename that -c/--scale to free up that
> flag for -S/--stat?
>
> Ingo
>

Here you are. But how do you switch "scale" off now that it's disabled
by default?
I tried --scale=0, --no-scale and so on without apparently succeeding.

Brice

[PATCH] perf stat: change -S for --scale into -c

Signed-off-by: Brice Goglin <Brice.Goglin@inria.fr>

Index: linux-rc/tools/perf/Documentation/perf-stat.txt
===================================================================
--- linux-rc.orig/tools/perf/Documentation/perf-stat.txt 2009-08-07 10:09:40.000000000 +0200
+++ linux-rc/tools/perf/Documentation/perf-stat.txt 2009-08-07 10:09:46.000000000 +0200
@@ -40,7 +40,7 @@
-a::
system-wide collection

--S::
+-c::
scale counter values

EXAMPLES
Index: linux-rc/tools/perf/builtin-stat.c
===================================================================
--- linux-rc.orig/tools/perf/builtin-stat.c 2009-08-07 09:46:49.000000000 +0200
+++ linux-rc/tools/perf/builtin-stat.c 2009-08-07 10:07:44.000000000 +0200
@@ -496,7 +496,7 @@
"stat events on existing pid"),
OPT_BOOLEAN('a', "all-cpus", &system_wide,
"system-wide collection from all CPUs"),
- OPT_BOOLEAN('S', "scale", &scale,
+ OPT_BOOLEAN('c', "scale", &scale,
"scale/normalize counters"),
OPT_BOOLEAN('v', "verbose", &verbose,
"be more verbose (show counter open errors, etc)"),