Re: [PATCH] tracing: Cleanup event_enable_trigger_parse() by using __free()

From: Steven Rostedt

Date: Thu Jul 02 2026 - 16:54:18 EST


On Thu, 2 Jul 2026 11:58:02 +0200
Markus Elfring <Markus.Elfring@xxxxxx> wrote:

>
> > +++ b/kernel/trace/trace_events_trigger.c
> > @@ -1739,7 +1739,7 @@ int event_enable_trigger_parse(struct event_command *cmd_ops,
> > char *glob, char *cmd, char *param_and_filter)
> > {
> > struct trace_event_file *event_enable_file;
> > - struct enable_trigger_data *enable_data;
> > + struct enable_trigger_data *enable_data __free(kfree) = NULL;
>
>
> How do you think about to reduce the scope for this local variable?
>

No, it's fine as is.

-- Steve