perf tool: Rename perf_tool::bpf_event to bpf
authorArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 26 Aug 2019 22:28:13 +0000 (19:28 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 26 Aug 2019 22:39:11 +0000 (19:39 -0300)
No need for that _event suffix, do just like all the other meta event
handlers and suppress that suffix.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Song Liu <songliubraving@fb.com>
Link: https://lkml.kernel.org/n/tip-03spzxtqafbabbbmnm7y4xfx@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-script.c
tools/perf/util/bpf-event.c
tools/perf/util/bpf-event.h
tools/perf/util/event.c
tools/perf/util/event.h
tools/perf/util/machine.c
tools/perf/util/session.c
tools/perf/util/tool.h

index 6f389b33fbe5f9cd9da605fa13bdaf312ec7c663..51e7e6d0eee6f342701f32e6870131936ad1fe39 100644 (file)
@@ -2492,8 +2492,8 @@ static int __cmd_script(struct perf_script *script)
                script->tool.finished_round = process_finished_round_event;
        }
        if (script->show_bpf_events) {
-               script->tool.ksymbol   = process_bpf_events;
-               script->tool.bpf_event = process_bpf_events;
+               script->tool.ksymbol = process_bpf_events;
+               script->tool.bpf     = process_bpf_events;
        }
 
        if (perf_script__setup_per_event_dump(script)) {
index 28fa2b1ce66ee8da326d83d994fa546ab81a44b4..2d6d500c9af72bf8ca73db7ea4eb75a679156584 100644 (file)
@@ -64,12 +64,11 @@ static int machine__process_bpf_event_load(struct machine *machine,
        return 0;
 }
 
-int machine__process_bpf_event(struct machine *machine __maybe_unused,
-                              union perf_event *event,
-                              struct perf_sample *sample __maybe_unused)
+int machine__process_bpf(struct machine *machine, union perf_event *event,
+                        struct perf_sample *sample)
 {
        if (dump_trace)
-               perf_event__fprintf_bpf_event(event, stdout);
+               perf_event__fprintf_bpf(event, stdout);
 
        switch (event->bpf.type) {
        case PERF_BPF_EVENT_PROG_LOAD:
@@ -83,7 +82,7 @@ int machine__process_bpf_event(struct machine *machine __maybe_unused,
                 */
                break;
        default:
-               pr_debug("unexpected bpf_event type of %d\n", event->bpf.type);
+               pr_debug("unexpected bpf event type of %d\n", event->bpf.type);
                break;
        }
        return 0;
@@ -410,7 +409,7 @@ static int bpf_event__sb_cb(union perf_event *event, void *data)
                 */
                break;
        default:
-               pr_debug("unexpected bpf_event type of %d\n", event->bpf.type);
+               pr_debug("unexpected bpf event type of %d\n", event->bpf.type);
                break;
        }
 
index 26ab9239f98605934f316de38105505a153188d0..417b78835ea09858b34a4f34a900e6f5a0ea9623 100644 (file)
@@ -30,8 +30,8 @@ struct btf_node {
 };
 
 #ifdef HAVE_LIBBPF_SUPPORT
-int machine__process_bpf_event(struct machine *machine, union perf_event *event,
-                              struct perf_sample *sample);
+int machine__process_bpf(struct machine *machine, union perf_event *event,
+                        struct perf_sample *sample);
 
 int perf_event__synthesize_bpf_events(struct perf_session *session,
                                      perf_event__handler_t process,
@@ -43,9 +43,9 @@ void bpf_event__print_bpf_prog_info(struct bpf_prog_info *info,
                                    struct perf_env *env,
                                    FILE *fp);
 #else
-static inline int machine__process_bpf_event(struct machine *machine __maybe_unused,
-                                            union perf_event *event __maybe_unused,
-                                            struct perf_sample *sample __maybe_unused)
+static inline int machine__process_bpf(struct machine *machine __maybe_unused,
+                                      union perf_event *event __maybe_unused,
+                                      struct perf_sample *sample __maybe_unused)
 {
        return 0;
 }
index 17304df44fc2049e361ed618783bf2997cf7180a..33616ea62a4794d7096729279dc1a41c6d80e7f7 100644 (file)
@@ -1343,12 +1343,12 @@ int perf_event__process_ksymbol(struct perf_tool *tool __maybe_unused,
        return machine__process_ksymbol(machine, event, sample);
 }
 
-int perf_event__process_bpf_event(struct perf_tool *tool __maybe_unused,
-                                 union perf_event *event,
-                                 struct perf_sample *sample __maybe_unused,
-                                 struct machine *machine)
+int perf_event__process_bpf(struct perf_tool *tool __maybe_unused,
+                           union perf_event *event,
+                           struct perf_sample *sample,
+                           struct machine *machine)
 {
-       return machine__process_bpf_event(machine, event, sample);
+       return machine__process_bpf(machine, event, sample);
 }
 
 size_t perf_event__fprintf_mmap(union perf_event *event, FILE *fp)
@@ -1491,7 +1491,7 @@ size_t perf_event__fprintf_ksymbol(union perf_event *event, FILE *fp)
                       event->ksymbol.flags, event->ksymbol.name);
 }
 
-size_t perf_event__fprintf_bpf_event(union perf_event *event, FILE *fp)
+size_t perf_event__fprintf_bpf(union perf_event *event, FILE *fp)
 {
        return fprintf(fp, " type %u, flags %u, id %u\n",
                       event->bpf.type, event->bpf.flags, event->bpf.id);
@@ -1536,7 +1536,7 @@ size_t perf_event__fprintf(union perf_event *event, FILE *fp)
                ret += perf_event__fprintf_ksymbol(event, fp);
                break;
        case PERF_RECORD_BPF_EVENT:
-               ret += perf_event__fprintf_bpf_event(event, fp);
+               ret += perf_event__fprintf_bpf(event, fp);
                break;
        default:
                ret += fprintf(fp, "\n");
index 7251e2eee4419b33cc3876c475d4e580ebea2317..429a3fe52d6c7e63859aeefa7b818da91dd2caf3 100644 (file)
@@ -683,10 +683,10 @@ int perf_event__process_ksymbol(struct perf_tool *tool,
                                union perf_event *event,
                                struct perf_sample *sample,
                                struct machine *machine);
-int perf_event__process_bpf_event(struct perf_tool *tool,
-                                 union perf_event *event,
-                                 struct perf_sample *sample,
-                                 struct machine *machine);
+int perf_event__process_bpf(struct perf_tool *tool,
+                           union perf_event *event,
+                           struct perf_sample *sample,
+                           struct machine *machine);
 int perf_tool__process_synth_event(struct perf_tool *tool,
                                   union perf_event *event,
                                   struct machine *machine,
@@ -751,7 +751,7 @@ size_t perf_event__fprintf_thread_map(union perf_event *event, FILE *fp);
 size_t perf_event__fprintf_cpu_map(union perf_event *event, FILE *fp);
 size_t perf_event__fprintf_namespaces(union perf_event *event, FILE *fp);
 size_t perf_event__fprintf_ksymbol(union perf_event *event, FILE *fp);
-size_t perf_event__fprintf_bpf_event(union perf_event *event, FILE *fp);
+size_t perf_event__fprintf_bpf(union perf_event *event, FILE *fp);
 size_t perf_event__fprintf(union perf_event *event, FILE *fp);
 
 int kallsyms__get_function_start(const char *kallsyms_filename,
index 86b7fd24b1e12c1a1c9a40dbdf8f7d4be13b7080..93483f1764d3e7f8ac614af99183cc94c35655f0 100644 (file)
@@ -1922,7 +1922,7 @@ int machine__process_event(struct machine *machine, union perf_event *event,
        case PERF_RECORD_KSYMBOL:
                ret = machine__process_ksymbol(machine, event, sample); break;
        case PERF_RECORD_BPF_EVENT:
-               ret = machine__process_bpf_event(machine, event, sample); break;
+               ret = machine__process_bpf(machine, event, sample); break;
        default:
                ret = -1;
                break;
index 4bfec9db36d639a66c4c9f2b50823bb2a89a40c8..5786e9c807c540ddc7beeb7d225a1247f679fed3 100644 (file)
@@ -473,8 +473,8 @@ void perf_tool__fill_defaults(struct perf_tool *tool)
                tool->context_switch = perf_event__process_switch;
        if (tool->ksymbol == NULL)
                tool->ksymbol = perf_event__process_ksymbol;
-       if (tool->bpf_event == NULL)
-               tool->bpf_event = perf_event__process_bpf_event;
+       if (tool->bpf == NULL)
+               tool->bpf = perf_event__process_bpf;
        if (tool->read == NULL)
                tool->read = process_event_sample_stub;
        if (tool->throttle == NULL)
@@ -1452,7 +1452,7 @@ static int machines__deliver_event(struct machines *machines,
        case PERF_RECORD_KSYMBOL:
                return tool->ksymbol(tool, event, sample, machine);
        case PERF_RECORD_BPF_EVENT:
-               return tool->bpf_event(tool, event, sample, machine);
+               return tool->bpf(tool, event, sample, machine);
        default:
                ++evlist->stats.nr_unknown_events;
                return -1;
index 7f95dd1d6883abd4b91fff294c363457da2d8c1f..2abbf668b8dec1a2293cce9e7b6600a06823dd37 100644 (file)
@@ -56,7 +56,7 @@ struct perf_tool {
                        throttle,
                        unthrottle,
                        ksymbol,
-                       bpf_event;
+                       bpf;
 
        event_attr_op   attr;
        event_attr_op   event_update;