On Wed, 19 Nov 2025 13:39:42 +0900 Masami Hiramatsu (Google) <[email protected]> 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) <[email protected]>
