perf stat: Move noise/running printing into printout
Move the running/noise printing into printout to avoid duplicated code in the callers. v2: Merged with other patches. Remove unnecessary hunk. Readd hunk that ended in earlier patch. v3: Fix noise/running output in CSV mode v4: Merge with later patch that also moves not supported printing. Signed-off-by: Andi Kleen <ak@linux.intel.com> Acked-by: Jiri Olsa <jolsa@kernel.org> Cc: Stephane Eranian <eranian@google.com> Link: http://lkml.kernel.org/r/1454173616-17710-4-git-send-email-andi@firstfloor.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
parent
f948339290
commit
cb110f4710
|
@ -848,7 +848,7 @@ static void abs_printout(int id, int nr, struct perf_evsel *evsel, double avg)
|
||||||
}
|
}
|
||||||
|
|
||||||
static void printout(int id, int nr, struct perf_evsel *counter, double uval,
|
static void printout(int id, int nr, struct perf_evsel *counter, double uval,
|
||||||
char *prefix)
|
char *prefix, u64 run, u64 ena, double noise)
|
||||||
{
|
{
|
||||||
struct perf_stat_output_ctx out;
|
struct perf_stat_output_ctx out;
|
||||||
struct outstate os = {
|
struct outstate os = {
|
||||||
|
@ -860,6 +860,30 @@ static void printout(int id, int nr, struct perf_evsel *counter, double uval,
|
||||||
|
|
||||||
nl = new_line_std;
|
nl = new_line_std;
|
||||||
|
|
||||||
|
if (run == 0 || ena == 0) {
|
||||||
|
aggr_printout(counter, id, nr);
|
||||||
|
|
||||||
|
fprintf(stat_config.output, "%*s%s",
|
||||||
|
csv_output ? 0 : 18,
|
||||||
|
counter->supported ? CNTR_NOT_COUNTED : CNTR_NOT_SUPPORTED,
|
||||||
|
csv_sep);
|
||||||
|
|
||||||
|
fprintf(stat_config.output, "%-*s%s",
|
||||||
|
csv_output ? 0 : unit_width,
|
||||||
|
counter->unit, csv_sep);
|
||||||
|
|
||||||
|
fprintf(stat_config.output, "%*s",
|
||||||
|
csv_output ? 0 : -25,
|
||||||
|
perf_evsel__name(counter));
|
||||||
|
|
||||||
|
if (counter->cgrp)
|
||||||
|
fprintf(stat_config.output, "%s%s",
|
||||||
|
csv_sep, counter->cgrp->name);
|
||||||
|
|
||||||
|
print_running(run, ena);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
if (nsec_counter(counter))
|
if (nsec_counter(counter))
|
||||||
nsec_printout(id, nr, counter, uval);
|
nsec_printout(id, nr, counter, uval);
|
||||||
else
|
else
|
||||||
|
@ -874,6 +898,9 @@ static void printout(int id, int nr, struct perf_evsel *counter, double uval,
|
||||||
stat_config.aggr_mode == AGGR_GLOBAL ? 0 :
|
stat_config.aggr_mode == AGGR_GLOBAL ? 0 :
|
||||||
cpu_map__id_to_cpu(id),
|
cpu_map__id_to_cpu(id),
|
||||||
&out);
|
&out);
|
||||||
|
|
||||||
|
print_noise(counter, noise);
|
||||||
|
print_running(run, ena);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void print_aggr(char *prefix)
|
static void print_aggr(char *prefix)
|
||||||
|
@ -904,36 +931,8 @@ static void print_aggr(char *prefix)
|
||||||
if (prefix)
|
if (prefix)
|
||||||
fprintf(output, "%s", prefix);
|
fprintf(output, "%s", prefix);
|
||||||
|
|
||||||
if (run == 0 || ena == 0) {
|
|
||||||
aggr_printout(counter, id, nr);
|
|
||||||
|
|
||||||
fprintf(output, "%*s%s",
|
|
||||||
csv_output ? 0 : 18,
|
|
||||||
counter->supported ? CNTR_NOT_COUNTED : CNTR_NOT_SUPPORTED,
|
|
||||||
csv_sep);
|
|
||||||
|
|
||||||
fprintf(output, "%-*s%s",
|
|
||||||
csv_output ? 0 : unit_width,
|
|
||||||
counter->unit, csv_sep);
|
|
||||||
|
|
||||||
fprintf(output, "%*s",
|
|
||||||
csv_output ? 0 : -25,
|
|
||||||
perf_evsel__name(counter));
|
|
||||||
|
|
||||||
if (counter->cgrp)
|
|
||||||
fprintf(output, "%s%s",
|
|
||||||
csv_sep, counter->cgrp->name);
|
|
||||||
|
|
||||||
print_running(run, ena);
|
|
||||||
fputc('\n', output);
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
uval = val * counter->scale;
|
uval = val * counter->scale;
|
||||||
printout(id, nr, counter, uval, prefix);
|
printout(id, nr, counter, uval, prefix, run, ena, 1.0);
|
||||||
if (!csv_output)
|
|
||||||
print_noise(counter, 1.0);
|
|
||||||
|
|
||||||
print_running(run, ena);
|
|
||||||
fputc('\n', output);
|
fputc('\n', output);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -960,12 +959,7 @@ static void print_aggr_thread(struct perf_evsel *counter, char *prefix)
|
||||||
fprintf(output, "%s", prefix);
|
fprintf(output, "%s", prefix);
|
||||||
|
|
||||||
uval = val * counter->scale;
|
uval = val * counter->scale;
|
||||||
printout(thread, 0, counter, uval, prefix);
|
printout(thread, 0, counter, uval, prefix, run, ena, 1.0);
|
||||||
|
|
||||||
if (!csv_output)
|
|
||||||
print_noise(counter, 1.0);
|
|
||||||
|
|
||||||
print_running(run, ena);
|
|
||||||
fputc('\n', output);
|
fputc('\n', output);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -979,7 +973,6 @@ static void print_counter_aggr(struct perf_evsel *counter, char *prefix)
|
||||||
FILE *output = stat_config.output;
|
FILE *output = stat_config.output;
|
||||||
struct perf_stat_evsel *ps = counter->priv;
|
struct perf_stat_evsel *ps = counter->priv;
|
||||||
double avg = avg_stats(&ps->res_stats[0]);
|
double avg = avg_stats(&ps->res_stats[0]);
|
||||||
int scaled = counter->counts->scaled;
|
|
||||||
double uval;
|
double uval;
|
||||||
double avg_enabled, avg_running;
|
double avg_enabled, avg_running;
|
||||||
|
|
||||||
|
@ -989,32 +982,8 @@ static void print_counter_aggr(struct perf_evsel *counter, char *prefix)
|
||||||
if (prefix)
|
if (prefix)
|
||||||
fprintf(output, "%s", prefix);
|
fprintf(output, "%s", prefix);
|
||||||
|
|
||||||
if (scaled == -1 || !counter->supported) {
|
|
||||||
fprintf(output, "%*s%s",
|
|
||||||
csv_output ? 0 : 18,
|
|
||||||
counter->supported ? CNTR_NOT_COUNTED : CNTR_NOT_SUPPORTED,
|
|
||||||
csv_sep);
|
|
||||||
fprintf(output, "%-*s%s",
|
|
||||||
csv_output ? 0 : unit_width,
|
|
||||||
counter->unit, csv_sep);
|
|
||||||
fprintf(output, "%*s",
|
|
||||||
csv_output ? 0 : -25,
|
|
||||||
perf_evsel__name(counter));
|
|
||||||
|
|
||||||
if (counter->cgrp)
|
|
||||||
fprintf(output, "%s%s", csv_sep, counter->cgrp->name);
|
|
||||||
|
|
||||||
print_running(avg_running, avg_enabled);
|
|
||||||
fputc('\n', output);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
uval = avg * counter->scale;
|
uval = avg * counter->scale;
|
||||||
printout(-1, 0, counter, uval, prefix);
|
printout(-1, 0, counter, uval, prefix, avg_running, avg_enabled, avg);
|
||||||
|
|
||||||
print_noise(counter, avg);
|
|
||||||
|
|
||||||
print_running(avg_running, avg_enabled);
|
|
||||||
fprintf(output, "\n");
|
fprintf(output, "\n");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1037,36 +1006,8 @@ static void print_counter(struct perf_evsel *counter, char *prefix)
|
||||||
if (prefix)
|
if (prefix)
|
||||||
fprintf(output, "%s", prefix);
|
fprintf(output, "%s", prefix);
|
||||||
|
|
||||||
if (run == 0 || ena == 0) {
|
|
||||||
fprintf(output, "CPU%*d%s%*s%s",
|
|
||||||
csv_output ? 0 : -4,
|
|
||||||
perf_evsel__cpus(counter)->map[cpu], csv_sep,
|
|
||||||
csv_output ? 0 : 18,
|
|
||||||
counter->supported ? CNTR_NOT_COUNTED : CNTR_NOT_SUPPORTED,
|
|
||||||
csv_sep);
|
|
||||||
|
|
||||||
fprintf(output, "%-*s%s",
|
|
||||||
csv_output ? 0 : unit_width,
|
|
||||||
counter->unit, csv_sep);
|
|
||||||
|
|
||||||
fprintf(output, "%*s",
|
|
||||||
csv_output ? 0 : -25,
|
|
||||||
perf_evsel__name(counter));
|
|
||||||
|
|
||||||
if (counter->cgrp)
|
|
||||||
fprintf(output, "%s%s",
|
|
||||||
csv_sep, counter->cgrp->name);
|
|
||||||
|
|
||||||
print_running(run, ena);
|
|
||||||
fputc('\n', output);
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
|
|
||||||
uval = val * counter->scale;
|
uval = val * counter->scale;
|
||||||
printout(cpu, 0, counter, uval, prefix);
|
printout(cpu, 0, counter, uval, prefix, run, ena, 1.0);
|
||||||
if (!csv_output)
|
|
||||||
print_noise(counter, 1.0);
|
|
||||||
print_running(run, ena);
|
|
||||||
|
|
||||||
fputc('\n', output);
|
fputc('\n', output);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue