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

From: Markus Elfring

Date: Thu Jul 02 2026 - 06:19:45 EST



> +++ 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?

Regards,
Markus