Message ID | 20200608124531.819838-1-jean-philippe@linaro.org |
---|---|
State | Accepted |
Delegated to: | BPF Maintainers |
Headers | show |
Series | tracing/probe: Fix bpf_task_fd_query() for kprobes and uprobes | expand |
On 6/8/20 5:45 AM, Jean-Philippe Brucker wrote: > Commit 60d53e2c3b75 ("tracing/probe: Split trace_event related data from > trace_probe") removed the trace_[ku]probe structure from the > trace_event_call->data pointer. As bpf_get_[ku]probe_info() were > forgotten in that change, fix them now. These functions are currently > only used by the bpf_task_fd_query() syscall handler to collect > information about a perf event. > > Fixes: 60d53e2c3b75 ("tracing/probe: Split trace_event related data from trace_probe") > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> Thanks for the fix. Acked-by: Yonghong Song <yhs@fb.com> > --- > Found while trying to run the task_fd_query BPF sample. I intend to try > and move that sample to kselftests since it seems like a useful > regression test. That will be great. Thanks!
On Mon, 8 Jun 2020 14:45:32 +0200 Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: > Commit 60d53e2c3b75 ("tracing/probe: Split trace_event related data from > trace_probe") removed the trace_[ku]probe structure from the > trace_event_call->data pointer. As bpf_get_[ku]probe_info() were > forgotten in that change, fix them now. These functions are currently > only used by the bpf_task_fd_query() syscall handler to collect > information about a perf event. > Oops, good catch! Acked-by: Masami Hiramatsu <mhiramat@kernel.org> > Fixes: 60d53e2c3b75 ("tracing/probe: Split trace_event related data from trace_probe") > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> Cc: stable@vger.kernel.org Thank you! > --- > Found while trying to run the task_fd_query BPF sample. I intend to try > and move that sample to kselftests since it seems like a useful > regression test. > --- > kernel/trace/trace_kprobe.c | 2 +- > kernel/trace/trace_uprobe.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c > index 35989383ae113..8eeb95e04bf52 100644 > --- a/kernel/trace/trace_kprobe.c > +++ b/kernel/trace/trace_kprobe.c > @@ -1629,7 +1629,7 @@ int bpf_get_kprobe_info(const struct perf_event *event, u32 *fd_type, > if (perf_type_tracepoint) > tk = find_trace_kprobe(pevent, group); > else > - tk = event->tp_event->data; > + tk = trace_kprobe_primary_from_call(event->tp_event); > if (!tk) > return -EINVAL; > > diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c > index 2a8e8e9c1c754..fdd47f99b18fd 100644 > --- a/kernel/trace/trace_uprobe.c > +++ b/kernel/trace/trace_uprobe.c > @@ -1412,7 +1412,7 @@ int bpf_get_uprobe_info(const struct perf_event *event, u32 *fd_type, > if (perf_type_tracepoint) > tu = find_probe_event(pevent, group); > else > - tu = event->tp_event->data; > + tu = trace_uprobe_primary_from_call(event->tp_event); > if (!tu) > return -EINVAL; > > -- > 2.27.0 >
On Tue, Jun 9, 2020 at 12:12 AM Masami Hiramatsu <mhiramat@kernel.org> wrote: > > On Mon, 8 Jun 2020 14:45:32 +0200 > Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: > > > Commit 60d53e2c3b75 ("tracing/probe: Split trace_event related data from > > trace_probe") removed the trace_[ku]probe structure from the > > trace_event_call->data pointer. As bpf_get_[ku]probe_info() were > > forgotten in that change, fix them now. These functions are currently > > only used by the bpf_task_fd_query() syscall handler to collect > > information about a perf event. > > > > Oops, good catch! > > Acked-by: Masami Hiramatsu <mhiramat@kernel.org> > > > > Fixes: 60d53e2c3b75 ("tracing/probe: Split trace_event related data from trace_probe") > > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> > > Cc: stable@vger.kernel.org Applied to bpf tree. Thanks
diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c index 35989383ae113..8eeb95e04bf52 100644 --- a/kernel/trace/trace_kprobe.c +++ b/kernel/trace/trace_kprobe.c @@ -1629,7 +1629,7 @@ int bpf_get_kprobe_info(const struct perf_event *event, u32 *fd_type, if (perf_type_tracepoint) tk = find_trace_kprobe(pevent, group); else - tk = event->tp_event->data; + tk = trace_kprobe_primary_from_call(event->tp_event); if (!tk) return -EINVAL; diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c index 2a8e8e9c1c754..fdd47f99b18fd 100644 --- a/kernel/trace/trace_uprobe.c +++ b/kernel/trace/trace_uprobe.c @@ -1412,7 +1412,7 @@ int bpf_get_uprobe_info(const struct perf_event *event, u32 *fd_type, if (perf_type_tracepoint) tu = find_probe_event(pevent, group); else - tu = event->tp_event->data; + tu = trace_uprobe_primary_from_call(event->tp_event); if (!tu) return -EINVAL;
Commit 60d53e2c3b75 ("tracing/probe: Split trace_event related data from trace_probe") removed the trace_[ku]probe structure from the trace_event_call->data pointer. As bpf_get_[ku]probe_info() were forgotten in that change, fix them now. These functions are currently only used by the bpf_task_fd_query() syscall handler to collect information about a perf event. Fixes: 60d53e2c3b75 ("tracing/probe: Split trace_event related data from trace_probe") Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> --- Found while trying to run the task_fd_query BPF sample. I intend to try and move that sample to kselftests since it seems like a useful regression test. --- kernel/trace/trace_kprobe.c | 2 +- kernel/trace/trace_uprobe.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)