Commit a4e0ea0e authored by Steven Rostedt's avatar Steven Rostedt Committed by Steven Rostedt (Google)
Browse files

tracing: Remove redundant call to event_trigger_reset_filter() in event_hist_trigger_parse() 

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/

Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Cc: Miaoqian Lin <linmq006@gmail.com>
Link: https://patch.msgid.link/20260108174429.2d9ca51f@gandalf.local.home


Acked-by: default avatarMasami Hiramatsu (Google) <mhiramat@kernel.org>
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
parent 0550069c

kernel/trace/trace_events_hist.c

+0 −2
Original line number Diff line number Diff line
@@ -6907,8 +6907,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);