Re: [PATCH v2 3/3] tracing: Use strim() in trigger_process_regex() instead of skip_spaces()

From: Steven Rostedt

Date: Wed Dec 03 2025 - 17:45:05 EST


On Wed, 03 Dec 2025 16:40:04 -0600
Tom Zanussi <zanussi@xxxxxxxxxx> wrote:

> > Remove the strim() from event_trigger_regex_write() and have
> > trigger_process_regex() use strim() instead of skip_spaces(). The buff
> > variable is not passed in as const, so it can be modified.
> >
> > Acked-by: Masami Hiramatsu (Google) <mhiramat@xxxxxxxxxx>
> > Signed-off-by: Steven Rostedt (Google) <rostedt@xxxxxxxxxxx>
>
> Looks good to me.
>
> Reviewed-by: Tom Zanussi <zanussi@xxxxxxxxxx>

Thanks for all the reviews, but I already did the pull request ;-)

Although it hasn't been accepted yet and there's always a chance Linus may
have an issue with one of the commits (I had a lot of commits this time,
affecting various aspects of the code, but couldn't use topic branches due
to too many conflicts). If it is rejected, I'll add your reviewed-by tags.

-- Steve