diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-11-05 13:32:36 -0500 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-11-05 13:32:36 -0500 |
| commit | 316c7136f8bad924609163b9b115f68d59a68c82 (patch) | |
| tree | ba664f98d92563544aa63dfb63ae4b49c27929c6 /tools | |
| parent | 714647bdc516330e4405b39677d7f763e016c685 (diff) | |
perf tools: Finish the removal of 'self' arguments
They convey no information, perhaps I was bitten by some snake at some
point, complete the detox by naming the last of those arguments more
sensibly.
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-u1r0dnjoro08dgztiy2g3t2q@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/perf/scripts/python/Perf-Trace-Util/Context.c | 6 | ||||
| -rw-r--r-- | tools/perf/ui/browser.h | 32 | ||||
| -rw-r--r-- | tools/perf/ui/browsers/hists.c | 2 | ||||
| -rw-r--r-- | tools/perf/ui/browsers/map.c | 40 | ||||
| -rw-r--r-- | tools/perf/ui/browsers/map.h | 2 | ||||
| -rw-r--r-- | tools/perf/ui/browsers/scripts.c | 8 | ||||
| -rw-r--r-- | tools/perf/ui/stdio/hist.c | 14 | ||||
| -rw-r--r-- | tools/perf/util/build-id.h | 3 | ||||
| -rw-r--r-- | tools/perf/util/event.c | 6 | ||||
| -rw-r--r-- | tools/perf/util/event.h | 3 | ||||
| -rw-r--r-- | tools/perf/util/evlist.h | 2 | ||||
| -rw-r--r-- | tools/perf/util/hist.h | 33 | ||||
| -rw-r--r-- | tools/perf/util/probe-finder.c | 113 | ||||
| -rw-r--r-- | tools/perf/util/probe-finder.h | 10 | ||||
| -rw-r--r-- | tools/perf/util/pstack.h | 10 | ||||
| -rw-r--r-- | tools/perf/util/session.c | 121 | ||||
| -rw-r--r-- | tools/perf/util/session.h | 27 | ||||
| -rw-r--r-- | tools/perf/util/sort.h | 2 | ||||
| -rw-r--r-- | tools/perf/util/strfilter.c | 32 | ||||
| -rw-r--r-- | tools/perf/util/strfilter.h | 12 | ||||
| -rw-r--r-- | tools/perf/util/thread.h | 10 |
21 files changed, 242 insertions, 246 deletions
diff --git a/tools/perf/scripts/python/Perf-Trace-Util/Context.c b/tools/perf/scripts/python/Perf-Trace-Util/Context.c index 315067b8f552..fcd1dd667906 100644 --- a/tools/perf/scripts/python/Perf-Trace-Util/Context.c +++ b/tools/perf/scripts/python/Perf-Trace-Util/Context.c | |||
| @@ -25,7 +25,7 @@ | |||
| 25 | 25 | ||
| 26 | PyMODINIT_FUNC initperf_trace_context(void); | 26 | PyMODINIT_FUNC initperf_trace_context(void); |
| 27 | 27 | ||
| 28 | static PyObject *perf_trace_context_common_pc(PyObject *self, PyObject *args) | 28 | static PyObject *perf_trace_context_common_pc(PyObject *obj, PyObject *args) |
| 29 | { | 29 | { |
| 30 | static struct scripting_context *scripting_context; | 30 | static struct scripting_context *scripting_context; |
| 31 | PyObject *context; | 31 | PyObject *context; |
| @@ -40,7 +40,7 @@ static PyObject *perf_trace_context_common_pc(PyObject *self, PyObject *args) | |||
| 40 | return Py_BuildValue("i", retval); | 40 | return Py_BuildValue("i", retval); |
| 41 | } | 41 | } |
| 42 | 42 | ||
| 43 | static PyObject *perf_trace_context_common_flags(PyObject *self, | 43 | static PyObject *perf_trace_context_common_flags(PyObject *obj, |
| 44 | PyObject *args) | 44 | PyObject *args) |
| 45 | { | 45 | { |
| 46 | static struct scripting_context *scripting_context; | 46 | static struct scripting_context *scripting_context; |
| @@ -56,7 +56,7 @@ static PyObject *perf_trace_context_common_flags(PyObject *self, | |||
| 56 | return Py_BuildValue("i", retval); | 56 | return Py_BuildValue("i", retval); |
| 57 | } | 57 | } |
| 58 | 58 | ||
| 59 | static PyObject *perf_trace_context_common_lock_depth(PyObject *self, | 59 | static PyObject *perf_trace_context_common_lock_depth(PyObject *obj, |
| 60 | PyObject *args) | 60 | PyObject *args) |
| 61 | { | 61 | { |
| 62 | static struct scripting_context *scripting_context; | 62 | static struct scripting_context *scripting_context; |
diff --git a/tools/perf/ui/browser.h b/tools/perf/ui/browser.h index 404ff66a3e36..7d45d2f53601 100644 --- a/tools/perf/ui/browser.h +++ b/tools/perf/ui/browser.h | |||
| @@ -21,32 +21,32 @@ struct ui_browser { | |||
| 21 | void *priv; | 21 | void *priv; |
| 22 | const char *title; | 22 | const char *title; |
| 23 | char *helpline; | 23 | char *helpline; |
| 24 | unsigned int (*refresh)(struct ui_browser *self); | 24 | unsigned int (*refresh)(struct ui_browser *browser); |
| 25 | void (*write)(struct ui_browser *self, void *entry, int row); | 25 | void (*write)(struct ui_browser *browser, void *entry, int row); |
| 26 | void (*seek)(struct ui_browser *self, off_t offset, int whence); | 26 | void (*seek)(struct ui_browser *browser, off_t offset, int whence); |
| 27 | bool (*filter)(struct ui_browser *self, void *entry); | 27 | bool (*filter)(struct ui_browser *browser, void *entry); |
| 28 | u32 nr_entries; | 28 | u32 nr_entries; |
| 29 | bool navkeypressed; | 29 | bool navkeypressed; |
| 30 | bool use_navkeypressed; | 30 | bool use_navkeypressed; |
| 31 | }; | 31 | }; |
| 32 | 32 | ||
| 33 | int ui_browser__set_color(struct ui_browser *browser, int color); | 33 | int ui_browser__set_color(struct ui_browser *browser, int color); |
| 34 | void ui_browser__set_percent_color(struct ui_browser *self, | 34 | void ui_browser__set_percent_color(struct ui_browser *browser, |
| 35 | double percent, bool current); | 35 | double percent, bool current); |
| 36 | bool ui_browser__is_current_entry(struct ui_browser *self, unsigned row); | 36 | bool ui_browser__is_current_entry(struct ui_browser *browser, unsigned row); |
| 37 | void ui_browser__refresh_dimensions(struct ui_browser *self); | 37 | void ui_browser__refresh_dimensions(struct ui_browser *browser); |
| 38 | void ui_browser__reset_index(struct ui_browser *self); | 38 | void ui_browser__reset_index(struct ui_browser *browser); |
| 39 | 39 | ||
| 40 | void ui_browser__gotorc(struct ui_browser *self, int y, int x); | 40 | void ui_browser__gotorc(struct ui_browser *browser, int y, int x); |
| 41 | void ui_browser__write_graph(struct ui_browser *browser, int graph); | 41 | void ui_browser__write_graph(struct ui_browser *browser, int graph); |
| 42 | void __ui_browser__line_arrow(struct ui_browser *browser, unsigned int column, | 42 | void __ui_browser__line_arrow(struct ui_browser *browser, unsigned int column, |
| 43 | u64 start, u64 end); | 43 | u64 start, u64 end); |
| 44 | void __ui_browser__show_title(struct ui_browser *browser, const char *title); | 44 | void __ui_browser__show_title(struct ui_browser *browser, const char *title); |
| 45 | void ui_browser__show_title(struct ui_browser *browser, const char *title); | 45 | void ui_browser__show_title(struct ui_browser *browser, const char *title); |
| 46 | int ui_browser__show(struct ui_browser *self, const char *title, | 46 | int ui_browser__show(struct ui_browser *browser, const char *title, |
| 47 | const char *helpline, ...); | 47 | const char *helpline, ...); |
| 48 | void ui_browser__hide(struct ui_browser *self); | 48 | void ui_browser__hide(struct ui_browser *browser); |
| 49 | int ui_browser__refresh(struct ui_browser *self); | 49 | int ui_browser__refresh(struct ui_browser *browser); |
| 50 | int ui_browser__run(struct ui_browser *browser, int delay_secs); | 50 | int ui_browser__run(struct ui_browser *browser, int delay_secs); |
| 51 | void ui_browser__update_nr_entries(struct ui_browser *browser, u32 nr_entries); | 51 | void ui_browser__update_nr_entries(struct ui_browser *browser, u32 nr_entries); |
| 52 | void ui_browser__handle_resize(struct ui_browser *browser); | 52 | void ui_browser__handle_resize(struct ui_browser *browser); |
| @@ -63,11 +63,11 @@ int ui_browser__input_window(const char *title, const char *text, char *input, | |||
| 63 | void ui_browser__argv_seek(struct ui_browser *browser, off_t offset, int whence); | 63 | void ui_browser__argv_seek(struct ui_browser *browser, off_t offset, int whence); |
| 64 | unsigned int ui_browser__argv_refresh(struct ui_browser *browser); | 64 | unsigned int ui_browser__argv_refresh(struct ui_browser *browser); |
| 65 | 65 | ||
| 66 | void ui_browser__rb_tree_seek(struct ui_browser *self, off_t offset, int whence); | 66 | void ui_browser__rb_tree_seek(struct ui_browser *browser, off_t offset, int whence); |
| 67 | unsigned int ui_browser__rb_tree_refresh(struct ui_browser *self); | 67 | unsigned int ui_browser__rb_tree_refresh(struct ui_browser *browser); |
| 68 | 68 | ||
| 69 | void ui_browser__list_head_seek(struct ui_browser *self, off_t offset, int whence); | 69 | void ui_browser__list_head_seek(struct ui_browser *browser, off_t offset, int whence); |
| 70 | unsigned int ui_browser__list_head_refresh(struct ui_browser *self); | 70 | unsigned int ui_browser__list_head_refresh(struct ui_browser *browser); |
| 71 | 71 | ||
| 72 | void ui_browser__init(void); | 72 | void ui_browser__init(void); |
| 73 | void annotate_browser__init(void); | 73 | void annotate_browser__init(void); |
diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c index a91b6b219412..16848bb4c418 100644 --- a/tools/perf/ui/browsers/hists.c +++ b/tools/perf/ui/browsers/hists.c | |||
| @@ -1889,7 +1889,7 @@ out: | |||
| 1889 | return key; | 1889 | return key; |
| 1890 | } | 1890 | } |
| 1891 | 1891 | ||
| 1892 | static bool filter_group_entries(struct ui_browser *self __maybe_unused, | 1892 | static bool filter_group_entries(struct ui_browser *browser __maybe_unused, |
| 1893 | void *entry) | 1893 | void *entry) |
| 1894 | { | 1894 | { |
| 1895 | struct perf_evsel *evsel = list_entry(entry, struct perf_evsel, node); | 1895 | struct perf_evsel *evsel = list_entry(entry, struct perf_evsel, node); |
diff --git a/tools/perf/ui/browsers/map.c b/tools/perf/ui/browsers/map.c index 95c7cfb8f2c6..b11639f33682 100644 --- a/tools/perf/ui/browsers/map.c +++ b/tools/perf/ui/browsers/map.c | |||
| @@ -18,30 +18,30 @@ struct map_browser { | |||
| 18 | u8 addrlen; | 18 | u8 addrlen; |
| 19 | }; | 19 | }; |
| 20 | 20 | ||
| 21 | static void map_browser__write(struct ui_browser *self, void *nd, int row) | 21 | static void map_browser__write(struct ui_browser *browser, void *nd, int row) |
| 22 | { | 22 | { |
| 23 | struct symbol *sym = rb_entry(nd, struct symbol, rb_node); | 23 | struct symbol *sym = rb_entry(nd, struct symbol, rb_node); |
| 24 | struct map_browser *mb = container_of(self, struct map_browser, b); | 24 | struct map_browser *mb = container_of(browser, struct map_browser, b); |
| 25 | bool current_entry = ui_browser__is_current_entry(self, row); | 25 | bool current_entry = ui_browser__is_current_entry(browser, row); |
| 26 | int width; | 26 | int width; |
| 27 | 27 | ||
| 28 | ui_browser__set_percent_color(self, 0, current_entry); | 28 | ui_browser__set_percent_color(browser, 0, current_entry); |
| 29 | slsmg_printf("%*" PRIx64 " %*" PRIx64 " %c ", | 29 | slsmg_printf("%*" PRIx64 " %*" PRIx64 " %c ", |
| 30 | mb->addrlen, sym->start, mb->addrlen, sym->end, | 30 | mb->addrlen, sym->start, mb->addrlen, sym->end, |
| 31 | sym->binding == STB_GLOBAL ? 'g' : | 31 | sym->binding == STB_GLOBAL ? 'g' : |
| 32 | sym->binding == STB_LO | ||
