[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Mon, 4 Mar 2024 00:48:23 +0900
From: Masami Hiramatsu (Google) <mhiramat@...nel.org>
To: Steven Rostedt <rostedt@...dmis.org>
Cc: Jiri Olsa <jolsa@...nel.org>, linux-kernel@...r.kernel.org,
linux-trace-kernel@...r.kernel.org, linux-doc@...r.kernel.org,
linux-kselftest@...r.kernel.org
Subject: Re: [PATCH v2 3/7] tracing/probes: cleanup: Set
trace_probe::nr_args at trace_probe_init
On Thu, 29 Feb 2024 22:37:53 -0500
Steven Rostedt <rostedt@...dmis.org> wrote:
> On Mon, 26 Feb 2024 12:58:23 +0900
> "Masami Hiramatsu (Google)" <mhiramat@...nel.org> wrote:
>
> > From: Masami Hiramatsu (Google) <mhiramat@...nel.org>
> >
> > Instead of incrementing the trace_probe::nr_args, init it at trace_probe_init().
> > This is a cleanup, so the behavior is not changed.
>
> The change is fine, but the change log does not say why this was done.
This change is to get the number of trace_probe arguments while parsing
the argument strings.
Thank you,
>
> Reviewed-by: Steven Rostedt (Google) <rostedt@...dmis.org>
>
> -- Steve
>
>
> >
> > Signed-off-by: Masami Hiramatsu (Google) <mhiramat@...nel.org>
> > ---
--
Masami Hiramatsu (Google) <mhiramat@...nel.org>
Powered by blists - more mailing lists