diff options
-rw-r--r-- | tools/perf/builtin-diff.c | 4 | ||||
-rw-r--r-- | tools/perf/builtin-report.c | 20 | ||||
-rw-r--r-- | tools/perf/builtin-top.c | 4 | ||||
-rw-r--r-- | tools/perf/util/sort.c | 45 | ||||
-rw-r--r-- | tools/perf/util/sort.h | 3 |
5 files changed, 47 insertions, 29 deletions
diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c index 2d0462d89a97..cabbea5f0bc2 100644 --- a/tools/perf/builtin-diff.c +++ b/tools/perf/builtin-diff.c | |||
@@ -611,9 +611,7 @@ int cmd_diff(int argc, const char **argv, const char *prefix __maybe_unused) | |||
611 | 611 | ||
612 | setup_pager(); | 612 | setup_pager(); |
613 | 613 | ||
614 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "dso", NULL); | 614 | sort__setup_elide(NULL); |
615 | sort_entry__setup_elide(&sort_comm, symbol_conf.comm_list, "comm", NULL); | ||
616 | sort_entry__setup_elide(&sort_sym, symbol_conf.sym_list, "symbol", NULL); | ||
617 | 615 | ||
618 | return __cmd_diff(); | 616 | return __cmd_diff(); |
619 | } | 617 | } |
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 669405c9b8a2..d45bf9b0361d 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
@@ -937,25 +937,7 @@ repeat: | |||
937 | report.symbol_filter_str = argv[0]; | 937 | report.symbol_filter_str = argv[0]; |
938 | } | 938 | } |
939 | 939 | ||
940 | sort_entry__setup_elide(&sort_comm, symbol_conf.comm_list, "comm", stdout); | 940 | sort__setup_elide(stdout); |
941 | |||
942 | if (sort__mode == SORT_MODE__BRANCH) { | ||
943 | sort_entry__setup_elide(&sort_dso_from, symbol_conf.dso_from_list, "dso_from", stdout); | ||
944 | sort_entry__setup_elide(&sort_dso_to, symbol_conf.dso_to_list, "dso_to", stdout); | ||
945 | sort_entry__setup_elide(&sort_sym_from, symbol_conf.sym_from_list, "sym_from", stdout); | ||
946 | sort_entry__setup_elide(&sort_sym_to, symbol_conf.sym_to_list, "sym_to", stdout); | ||
947 | } else { | ||
948 | if (report.mem_mode) { | ||
949 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "symbol_daddr", stdout); | ||
950 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "dso_daddr", stdout); | ||
951 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "mem", stdout); | ||
952 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "local_weight", stdout); | ||
953 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "tlb", stdout); | ||
954 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "snoop", stdout); | ||
955 | } | ||
956 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "dso", stdout); | ||
957 | sort_entry__setup_elide(&sort_sym, symbol_conf.sym_list, "symbol", stdout); | ||
958 | } | ||
959 | 941 | ||
960 | ret = __cmd_report(&report); | 942 | ret = __cmd_report(&report); |
961 | if (ret == K_SWITCH_INPUT_DATA) { | 943 | if (ret == K_SWITCH_INPUT_DATA) { |
diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c index 67bdb9f14ad6..2eb272d8753c 100644 --- a/tools/perf/builtin-top.c +++ b/tools/perf/builtin-top.c | |||
@@ -1200,9 +1200,7 @@ int cmd_top(int argc, const char **argv, const char *prefix __maybe_unused) | |||
1200 | if (symbol__init() < 0) | 1200 | if (symbol__init() < 0) |
1201 | return -1; | 1201 | return -1; |
1202 | 1202 | ||
1203 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "dso", stdout); | 1203 | sort__setup_elide(stdout); |
1204 | sort_entry__setup_elide(&sort_comm, symbol_conf.comm_list, "comm", stdout); | ||
1205 | sort_entry__setup_elide(&sort_sym, symbol_conf.sym_list, "symbol", stdout); | ||
1206 | 1204 | ||
1207 | /* | 1205 | /* |
1208 | * Avoid annotation data structures overhead when symbols aren't on the | 1206 | * Avoid annotation data structures overhead when symbols aren't on the |
diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index 1dbf16949250..701ab1d84894 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c | |||
@@ -1,5 +1,6 @@ | |||
1 | #include "sort.h" | 1 | #include "sort.h" |
2 | #include "hist.h" | 2 | #include "hist.h" |
3 | #include "symbol.h" | ||
3 | 4 | ||
4 | regex_t parent_regex; | 5 | regex_t parent_regex; |
5 | const char default_parent_pattern[] = "^sys_|^do_page_fault"; | 6 | const char default_parent_pattern[] = "^sys_|^do_page_fault"; |
@@ -1009,8 +1010,9 @@ int setup_sorting(void) | |||
1009 | return ret; | 1010 | return ret; |
1010 | } | 1011 | } |
1011 | 1012 | ||
1012 | void sort_entry__setup_elide(struct sort_entry *self, struct strlist *list, | 1013 | static void sort_entry__setup_elide(struct sort_entry *self, |
1013 | const char *list_name, FILE *fp) | 1014 | struct strlist *list, |
1015 | const char *list_name, FILE *fp) | ||
1014 | { | 1016 | { |
1015 | if (list && strlist__nr_entries(list) == 1) { | 1017 | if (list && strlist__nr_entries(list) == 1) { |
1016 | if (fp != NULL) | 1018 | if (fp != NULL) |
@@ -1019,3 +1021,42 @@ void sort_entry__setup_elide(struct sort_entry *self, struct strlist *list, | |||
1019 | self->elide = true; | 1021 | self->elide = true; |
1020 | } | 1022 | } |
1021 | } | 1023 | } |
1024 | |||
1025 | void sort__setup_elide(FILE *output) | ||
1026 | { | ||
1027 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1028 | "dso", output); | ||
1029 | sort_entry__setup_elide(&sort_comm, symbol_conf.comm_list, | ||
1030 | "comm", output); | ||
1031 | sort_entry__setup_elide(&sort_sym, symbol_conf.sym_list, | ||
1032 | "symbol", output); | ||
1033 | |||
1034 | if (sort__mode == SORT_MODE__BRANCH) { | ||
1035 | sort_entry__setup_elide(&sort_dso_from, | ||
1036 | symbol_conf.dso_from_list, | ||
1037 | "dso_from", output); | ||
1038 | sort_entry__setup_elide(&sort_dso_to, | ||
1039 | symbol_conf.dso_to_list, | ||
1040 | "dso_to", output); | ||
1041 | sort_entry__setup_elide(&sort_sym_from, | ||
1042 | symbol_conf.sym_from_list, | ||
1043 | "sym_from", output); | ||
1044 | sort_entry__setup_elide(&sort_sym_to, | ||
1045 | symbol_conf.sym_to_list, | ||
1046 | "sym_to", output); | ||
1047 | } else if (sort__mode == SORT_MODE__MEMORY) { | ||
1048 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1049 | "symbol_daddr", output); | ||
1050 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1051 | "dso_daddr", output); | ||
1052 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1053 | "mem", output); | ||
1054 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1055 | "local_weight", output); | ||
1056 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1057 | "tlb", output); | ||
1058 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, | ||
1059 | "snoop", output); | ||
1060 | } | ||
1061 | |||
1062 | } | ||
diff --git a/tools/perf/util/sort.h b/tools/perf/util/sort.h index 0232d476da87..51f1b5a854e7 100644 --- a/tools/perf/util/sort.h +++ b/tools/perf/util/sort.h | |||
@@ -181,7 +181,6 @@ extern struct list_head hist_entry__sort_list; | |||
181 | 181 | ||
182 | int setup_sorting(void); | 182 | int setup_sorting(void); |
183 | extern int sort_dimension__add(const char *); | 183 | extern int sort_dimension__add(const char *); |
184 | void sort_entry__setup_elide(struct sort_entry *self, struct strlist *list, | 184 | void sort__setup_elide(FILE *fp); |
185 | const char *list_name, FILE *fp); | ||
186 | 185 | ||
187 | #endif /* __PERF_SORT_H */ | 186 | #endif /* __PERF_SORT_H */ |