On Thu, 8 Jan 2026 17:44:29 -0500 Steven Rostedt <[email protected]> wrote:
> From: Steven Rostedt <[email protected]> > > With the change to replace kfree() with trigger_data_free(), which starts > out doing the exact same thing as event_trigger_reset_filter(), there's no > reason to call event_trigger_reset_filter() before calling > trigger_data_free(). Remove the call to it. > > Link: > https://lore.kernel.org/linux-trace-kernel/20251211204520.0f3ba6d1@fedora/ > Looks good to me. Acked-by: Masami Hiramatsu (Google) <[email protected]> Thanks, > Signed-off-by: Steven Rostedt (Google) <[email protected]> > --- > kernel/trace/trace_events_hist.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/kernel/trace/trace_events_hist.c > b/kernel/trace/trace_events_hist.c > index f9886fff7123..4a7f945e368d 100644 > --- a/kernel/trace/trace_events_hist.c > +++ b/kernel/trace/trace_events_hist.c > @@ -6898,8 +6898,6 @@ static int event_hist_trigger_parse(struct > event_command *cmd_ops, > out_unreg: > event_trigger_unregister(cmd_ops, file, glob+1, trigger_data); > out_free: > - event_trigger_reset_filter(cmd_ops, trigger_data); > - > remove_hist_vars(hist_data); > > trigger_data_free(trigger_data); > -- > 2.51.0 > -- Masami Hiramatsu (Google) <[email protected]>
