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

From: Tom Zanussi

Date: Wed Dec 03 2025 - 17:58:29 EST


On Wed, 2025-12-03 at 17:46 -0500, Steven Rostedt wrote:
> 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

No problem, my fault for being so slow to get to these. :-/

Tom