p71924506
/
linux_old1
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

perf stat: Move 'big_num' data to 'struct perf_stat_config' 

Move the static variable 'big_num' to 'struct perf_stat_config', so that
it can be passed around and used outside 'perf stat' command.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/20180830063252.23729-39-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
Jiri Olsa 2018-08-30 08:32:47 +02:00 committed by Arnaldo Carvalho de Melo
parent a138af6635
commit 34ff0866d4
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tools/perf/builtin-stat.c
View File

@ -151,7 +151,6 @@ static bool transaction_run;
static bool topdown_run = false; static bool topdown_run = false;
static bool smi_cost = false; static bool smi_cost = false;
static bool smi_reset = false; static bool smi_reset = false;
static bool big_num = true;
static int big_num_opt = -1; static int big_num_opt = -1;
static bool group = false; static bool group = false;
static const char *pre_cmd = NULL; static const char *pre_cmd = NULL;
@ -192,6 +191,7 @@ static struct perf_stat_config stat_config = {
.run_count = 1, .run_count = 1,
.metric_only_len = METRIC_ONLY_LEN, .metric_only_len = METRIC_ONLY_LEN,
.walltime_nsecs_stats = &walltime_nsecs_stats, .walltime_nsecs_stats = &walltime_nsecs_stats,
.big_num = true,
}; };
static bool is_duration_time(struct perf_evsel *evsel) static bool is_duration_time(struct perf_evsel *evsel)
@ -945,7 +945,7 @@ static void abs_printout(struct perf_stat_config *config,
if (config->csv_output) { if (config->csv_output) {
fmt = floor(sc) != sc ? "%.2f%s" : "%.0f%s"; fmt = floor(sc) != sc ? "%.2f%s" : "%.0f%s";
} else { } else {
if (big_num) if (config->big_num)
fmt = floor(sc) != sc ? "%'18.2f%s" : "%'18.0f%s"; fmt = floor(sc) != sc ? "%'18.2f%s" : "%'18.0f%s";
else else
fmt = floor(sc) != sc ? "%18.2f%s" : "%18.0f%s"; fmt = floor(sc) != sc ? "%18.2f%s" : "%18.0f%s";
@ -2847,9 +2847,9 @@ int cmd_stat(int argc, const char **argv)
parse_options_usage(NULL, stat_options, "x", 1); parse_options_usage(NULL, stat_options, "x", 1);
goto out; goto out;
} else /* Nope, so disable big number formatting */ } else /* Nope, so disable big number formatting */
big_num = false; stat_config.big_num = false;
} else if (big_num_opt == 0) /* User passed --no-big-num */ } else if (big_num_opt == 0) /* User passed --no-big-num */
big_num = false; stat_config.big_num = false;
setup_system_wide(argc); setup_system_wide(argc);

1
tools/perf/util/stat.h
View File

@ -103,6 +103,7 @@ struct perf_stat_config {
bool metric_only; bool metric_only;
bool null_run; bool null_run;
bool ru_display; bool ru_display;
bool big_num;
FILE *output; FILE *output;
unsigned int interval; unsigned int interval;
unsigned int timeout; unsigned int timeout;