Messages in this thread Patch in this message | | | From | Wang Nan <> | Subject | [PATCH] perf tools: Allow BPF program attach to modules | Date | Tue, 27 Oct 2015 03:07:28 +0000 |
| |
By extending the syntax of BPF object section names, this patch allows user to attach BPF programs to symbol in modules. For example:
SEC("module=i915\n" "parse_cmds=i915_parse_cmds") int parse_cmds(void *ctx) { return 1; }
Implementation is very simple: like what 'perf probe' does, for module, fill 'uprobe' field in 'struct perf_probe_event'. Other parts would be done automatically.
Signed-off-by: Wang Nan <wangnan0@huawei.com> Cc: Alexei Starovoitov <ast@plumgrid.com> Cc: Brendan Gregg <brendan.d.gregg@gmail.com> Cc: Daniel Borkmann <daniel@iogearbox.net> Cc: David Ahern <dsahern@gmail.com> Cc: He Kuang <hekuang@huawei.com> Cc: Jiri Olsa <jolsa@kernel.org> Cc: Kaixu Xia <xiakaixu@huawei.com> Cc: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Zefan Li <lizefan@huawei.com> Cc: pi3orama@163.com Cc: Arnaldo Carvalho de Melo <acme@redhat.com> Link: http://lkml.kernel.org/n/ebpf-x1mxko3tby054o573zd575qp@git.kernel.org ---
This patch is based on commit ad14ba45236a496a7ce25f4ea947d245b1406e40 in my git tree[1], and will be appear in the next pull request of this perf eBPF support patchset.
[1] https://git.kernel.org/cgit/linux/kernel/git/pi3orama/linux.git
--- tools/perf/util/bpf-loader.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/tools/perf/util/bpf-loader.c b/tools/perf/util/bpf-loader.c index b14aff4..a85caae 100644 --- a/tools/perf/util/bpf-loader.c +++ b/tools/perf/util/bpf-loader.c @@ -120,10 +120,17 @@ do_config(const char *key, const char *value, pev->target = strdup(value); return 0; } + if (strcmp(key, "module") == 0) { + pev->uprobes = false; + pev->target = strdup(value); + return 0; + } pr_warning("BPF: WARNING: invalid config option in object: %s=%s\n", key, value); - pr_warning("\tHint: Currently only valid option is 'exec=<full patch of file>'\n"); + pr_warning("\tHint: Currently valid options are:\n"); + pr_warning("\t 'exec=<full patch of file>'\n"); + pr_warning("\t 'module=<module name>'\n"); return 0; } -- 1.8.3.4
| |