From mboxrd@z Thu Jan 1 00:00:00 1970 From: Masami Hiramatsu Subject: Re: [PATCH v9 tip 2/9] tracing: add kprobe flag Date: Sat, 21 Mar 2015 21:16:03 +0900 Message-ID: <550D6103.9090001@hitachi.com> References: <1426894210-27441-1-git-send-email-ast@plumgrid.com> <1426894210-27441-3-git-send-email-ast@plumgrid.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-2022-JP Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1426894210-27441-3-git-send-email-ast-uqk4Ao+rVK5Wk0Htik3J/w@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Alexei Starovoitov Cc: Ingo Molnar , Steven Rostedt , Namhyung Kim , Arnaldo Carvalho de Melo , Jiri Olsa , "David S. Miller" , Daniel Borkmann , Peter Zijlstra , linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-api@vger.kernel.org (2015/03/21 8:30), Alexei Starovoitov wrote: > add TRACE_EVENT_FL_KPROBE flag to differentiate kprobe type of tracepoints, > since bpf programs can only be attached to kprobe type of > PERF_TYPE_TRACEPOINT perf events. > Looks ok to me. Reviewed-by: Thank you! > Signed-off-by: Alexei Starovoitov > Reviewed-by: Steven Rostedt > --- > include/linux/ftrace_event.h | 3 +++ > kernel/trace/trace_kprobe.c | 2 +- > 2 files changed, 4 insertions(+), 1 deletion(-) > > diff --git a/include/linux/ftrace_event.h b/include/linux/ftrace_event.h > index c674ee8f7fca..77325e1a1816 100644 > --- a/include/linux/ftrace_event.h > +++ b/include/linux/ftrace_event.h > @@ -252,6 +252,7 @@ enum { > TRACE_EVENT_FL_WAS_ENABLED_BIT, > TRACE_EVENT_FL_USE_CALL_FILTER_BIT, > TRACE_EVENT_FL_TRACEPOINT_BIT, > + TRACE_EVENT_FL_KPROBE_BIT, > }; > > /* > @@ -265,6 +266,7 @@ enum { > * it is best to clear the buffers that used it). > * USE_CALL_FILTER - For ftrace internal events, don't use file filter > * TRACEPOINT - Event is a tracepoint > + * KPROBE - Event is a kprobe > */ > enum { > TRACE_EVENT_FL_FILTERED = (1 << TRACE_EVENT_FL_FILTERED_BIT), > @@ -274,6 +276,7 @@ enum { > TRACE_EVENT_FL_WAS_ENABLED = (1 << TRACE_EVENT_FL_WAS_ENABLED_BIT), > TRACE_EVENT_FL_USE_CALL_FILTER = (1 << TRACE_EVENT_FL_USE_CALL_FILTER_BIT), > TRACE_EVENT_FL_TRACEPOINT = (1 << TRACE_EVENT_FL_TRACEPOINT_BIT), > + TRACE_EVENT_FL_KPROBE = (1 << TRACE_EVENT_FL_KPROBE_BIT), > }; > > struct ftrace_event_call { > diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c > index d73f565b4e06..8fa549f6f528 100644 > --- a/kernel/trace/trace_kprobe.c > +++ b/kernel/trace/trace_kprobe.c > @@ -1286,7 +1286,7 @@ static int register_kprobe_event(struct trace_kprobe *tk) > kfree(call->print_fmt); > return -ENODEV; > } > - call->flags = 0; > + call->flags = TRACE_EVENT_FL_KPROBE; > call->class->reg = kprobe_register; > call->data = tk; > ret = trace_add_event_call(call); > -- Masami HIRAMATSU Software Platform Research Dept. Linux Technology Research Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt-FCd8Q96Dh0JBDgjK7y7TUQ@public.gmane.org