perf pmu: Support alias descriptions
Add support to print alias descriptions in perf list, which are taken from the generated event files. The sorting code is changed to put the events with descriptions at the end. The descriptions are printed as possibly multiple word wrapped lines. Example output: % perf list ... arith.fpu_div [Divide operations executed] arith.fpu_div_active [Cycles when divider is busy executing divide operations] Committer notes: Further testing on a Broadwell machine (ThinkPad t450s), using these files: $ find tools/perf/pmu-events/arch/x86/ tools/perf/pmu-events/arch/x86/ tools/perf/pmu-events/arch/x86/Broadwell tools/perf/pmu-events/arch/x86/Broadwell/Cache.json tools/perf/pmu-events/arch/x86/Broadwell/Other.json tools/perf/pmu-events/arch/x86/Broadwell/Frontend.json tools/perf/pmu-events/arch/x86/Broadwell/Virtual-Memory.json tools/perf/pmu-events/arch/x86/Broadwell/Pipeline.json tools/perf/pmu-events/arch/x86/Broadwell/Floating-point.json tools/perf/pmu-events/arch/x86/Broadwell/Memory.json tools/perf/pmu-events/arch/x86/mapfile.csv $ Taken from: https://github.com/sukadev/linux/tree/json-code+data-v21/tools/perf/pmu-events/arch/x86/ to get this machinery to actually parse JSON files, generate $(OUTPUT)pmu-events/pmu-events.c, compile it and link it with perf, that will then use the table it contains, these files will be submitted right after this patchkit. [acme@jouet linux]$ perf list page_walker List of pre-defined events (to be used in -e): page_walker_loads.dtlb_l1 [Number of DTLB page walker hits in the L1+FB] page_walker_loads.dtlb_l2 [Number of DTLB page walker hits in the L2] page_walker_loads.dtlb_l3 [Number of DTLB page walker hits in the L3 + XSNP] page_walker_loads.dtlb_memory [Number of DTLB page walker hits in Memory] page_walker_loads.itlb_l1 [Number of ITLB page walker hits in the L1+FB] page_walker_loads.itlb_l2 [Number of ITLB page walker hits in the L2] page_walker_loads.itlb_l3 [Number of ITLB page walker hits in the L3 + XSNP] [acme@jouet linux]$ Signed-off-by: Andi Kleen <ak@linux.intel.com> Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> Acked-by: Ingo Molnar <mingo@kernel.org> Acked-by: Jiri Olsa <jolsa@redhat.com> Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: Madhavan Srinivasan <maddy@linux.vnet.ibm.com> Cc: Peter Zijlstra <peterz@infradead.org> Cc: linuxppc-dev@lists.ozlabs.org Link: http://lkml.kernel.org/r/1473978296-20712-7-git-send-email-sukadev@linux.vnet.ibm.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
parent
dc720ffc98
commit
08e60ed15d
|
@ -222,7 +222,7 @@ static int perf_pmu__parse_snapshot(struct perf_pmu_alias *alias,
|
|||
}
|
||||
|
||||
static int __perf_pmu__new_alias(struct list_head *list, char *dir, char *name,
|
||||
char *desc __maybe_unused, char *val)
|
||||
char *desc, char *val)
|
||||
{
|
||||
struct perf_pmu_alias *alias;
|
||||
int ret;
|
||||
|
@ -255,6 +255,8 @@ static int __perf_pmu__new_alias(struct list_head *list, char *dir, char *name,
|
|||
perf_pmu__parse_snapshot(alias, dir, name);
|
||||
}
|
||||
|
||||
alias->desc = desc ? strdup(desc) : NULL;
|
||||
|
||||
list_add_tail(&alias->list, list);
|
||||
|
||||
return 0;
|
||||
|
@ -1043,11 +1045,42 @@ static char *format_alias_or(char *buf, int len, struct perf_pmu *pmu,
|
|||
return buf;
|
||||
}
|
||||
|
||||
static int cmp_string(const void *a, const void *b)
|
||||
struct pair {
|
||||
char *name;
|
||||
char *desc;
|
||||
};
|
||||
|
||||
static int cmp_pair(const void *a, const void *b)
|
||||
{
|
||||
const char * const *as = a;
|
||||
const char * const *bs = b;
|
||||
return strcmp(*as, *bs);
|
||||
const struct pair *as = a;
|
||||
const struct pair *bs = b;
|
||||
|
||||
/* Put extra events last */
|
||||
if (!!as->desc != !!bs->desc)
|
||||
return !!as->desc - !!bs->desc;
|
||||
return strcmp(as->name, bs->name);
|
||||
}
|
||||
|
||||
static void wordwrap(char *s, int start, int max, int corr)
|
||||
{
|
||||
int column = start;
|
||||
int n;
|
||||
|
||||
while (*s) {
|
||||
int wlen = strcspn(s, " \t");
|
||||
|
||||
if (column + wlen >= max && column > start) {
|
||||
printf("\n%*s", start, "");
|
||||
column = start + corr;
|
||||
}
|
||||
n = printf("%s%.*s", column > start ? " " : "", wlen, s);
|
||||
if (n <= 0)
|
||||
break;
|
||||
s += wlen;
|
||||
column += n;
|
||||
while (isspace(*s))
|
||||
s++;
|
||||
}
|
||||
}
|
||||
|
||||
void print_pmu_events(const char *event_glob, bool name_only)
|
||||
|
@ -1057,7 +1090,9 @@ void print_pmu_events(const char *event_glob, bool name_only)
|
|||
char buf[1024];
|
||||
int printed = 0;
|
||||
int len, j;
|
||||
char **aliases;
|
||||
struct pair *aliases;
|
||||
int numdesc = 0;
|
||||
int columns = 78;
|
||||
|
||||
pmu = NULL;
|
||||
len = 0;
|
||||
|
@ -1067,14 +1102,15 @@ void print_pmu_events(const char *event_glob, bool name_only)
|
|||
if (pmu->selectable)
|
||||
len++;
|
||||
}
|
||||
aliases = zalloc(sizeof(char *) * len);
|
||||
aliases = zalloc(sizeof(struct pair) * len);
|
||||
if (!aliases)
|
||||
goto out_enomem;
|
||||
pmu = NULL;
|
||||
j = 0;
|
||||
while ((pmu = perf_pmu__scan(pmu)) != NULL) {
|
||||
list_for_each_entry(alias, &pmu->aliases, list) {
|
||||
char *name = format_alias(buf, sizeof(buf), pmu, alias);
|
||||
char *name = alias->desc ? alias->name :
|
||||
format_alias(buf, sizeof(buf), pmu, alias);
|
||||
bool is_cpu = !strcmp(pmu->name, "cpu");
|
||||
|
||||
if (event_glob != NULL &&
|
||||
|
@ -1083,12 +1119,19 @@ void print_pmu_events(const char *event_glob, bool name_only)
|
|||
event_glob))))
|
||||
continue;
|
||||
|
||||
if (is_cpu && !name_only)
|
||||
if (is_cpu && !name_only && !alias->desc)
|
||||
name = format_alias_or(buf, sizeof(buf), pmu, alias);
|
||||
|
||||
aliases[j] = strdup(name);
|
||||
if (aliases[j] == NULL)
|
||||
aliases[j].name = name;
|
||||
if (is_cpu && !name_only && !alias->desc)
|
||||
aliases[j].name = format_alias_or(buf,
|
||||
sizeof(buf),
|
||||
pmu, alias);
|
||||
aliases[j].name = strdup(aliases[j].name);
|
||||
if (!aliases[j].name)
|
||||
goto out_enomem;
|
||||
|
||||
aliases[j].desc = alias->desc;
|
||||
j++;
|
||||
}
|
||||
if (pmu->selectable &&
|
||||
|
@ -1096,25 +1139,33 @@ void print_pmu_events(const char *event_glob, bool name_only)
|
|||
char *s;
|
||||
if (asprintf(&s, "%s//", pmu->name) < 0)
|
||||
goto out_enomem;
|
||||
aliases[j] = s;
|
||||
aliases[j].name = s;
|
||||
j++;
|
||||
}
|
||||
}
|
||||
len = j;
|
||||
qsort(aliases, len, sizeof(char *), cmp_string);
|
||||
qsort(aliases, len, sizeof(struct pair), cmp_pair);
|
||||
for (j = 0; j < len; j++) {
|
||||
if (name_only) {
|
||||
printf("%s ", aliases[j]);
|
||||
printf("%s ", aliases[j].name);
|
||||
continue;
|
||||
}
|
||||
printf(" %-50s [Kernel PMU event]\n", aliases[j]);
|
||||
if (aliases[j].desc) {
|
||||
if (numdesc++ == 0)
|
||||
printf("\n");
|
||||
printf(" %-50s\n", aliases[j].name);
|
||||
printf("%*s", 8, "[");
|
||||
wordwrap(aliases[j].desc, 8, columns, 0);
|
||||
printf("]\n");
|
||||
} else
|
||||
printf(" %-50s [Kernel PMU event]\n", aliases[j].name);
|
||||
printed++;
|
||||
}
|
||||
if (printed && pager_in_use())
|
||||
printf("\n");
|
||||
out_free:
|
||||
for (j = 0; j < len; j++)
|
||||
zfree(&aliases[j]);
|
||||
zfree(&aliases[j].name);
|
||||
zfree(&aliases);
|
||||
return;
|
||||
|
||||
|
|
|
@ -40,6 +40,7 @@ struct perf_pmu_info {
|
|||
|
||||
struct perf_pmu_alias {
|
||||
char *name;
|
||||
char *desc;
|
||||
struct list_head terms; /* HEAD struct parse_events_term -> list */
|
||||
struct list_head list; /* ELEM */
|
||||
char unit[UNIT_MAX_LEN+1];
|
||||
|
|
Loading…
Reference in New Issue