Re: [PATCH v2] tracing: Show the tracer options in boot-time created instance

From: Google

Date: Wed Nov 19 2025 - 03:31:42 EST


On Wed, 19 Nov 2025 13:39:42 +0900
Masami Hiramatsu (Google) <mhiramat@xxxxxxxxxx> wrote:
> > diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c
> > index 8ae95800592d..4a6784057855 100644
> > --- a/kernel/trace/trace.c
> > +++ b/kernel/trace/trace.c
> > @@ -10231,11 +10231,16 @@ static __init int __update_tracer_options(struct trace_array *tr)
> > return ret;
> > }
> >
> > -static __init void update_tracer_options(struct trace_array *tr)
> > +static __init void update_tracer_options(void)
> > {
> > + struct trace_array *tr;
> > +
> > guard(mutex)(&trace_types_lock);
> > +
> > + list_for_each_entry(tr, &ftrace_trace_arrays, list)
> > + __update_tracer_options(tr);
> > +
> > tracer_options_updated = true;

Oops, I misunderstood this flag. This should be set *before* calling
__update_tracer_options(). Not updated.

Thanks,

--
Masami Hiramatsu (Google) <mhiramat@xxxxxxxxxx>