On Wed, 8 Jan 2025 10:11:11 -0500
Steven Rostedt [off-list ref] wrote:

On Wed,  8 Jan 2025 11:11:18 +0900
"Masami Hiramatsu (Google)" [off-list ref] wrote:

quoted

-static int __trace_kprobe_create(int argc, const char *argv[])
+static int ___trace_kprobe_create(int argc, const char *argv[],
+				  struct traceprobe_parse_context *ctx)
 {

I'm not sure I care about all the underscores. It just adds to confusion.

Maybe just:

static int __create_trace_kprobe(..)

Ok. Or __trace_kprobe_create_internal()?

Then it will be

trace_kprobe_create()
 -> __trace_kprobe_create()
   -> __trace_kprobe_create_internal()

Thanks,

?

-- Steve

-- 
Masami Hiramatsu (Google) [off-list ref]