Re: [PATCH] selftests/tracing: Add basic test for trace_marker_raw file

[Steven Rostedt]

Masami,

  Ping!

-- Steve


On Tue, 14 Oct 2025 16:14:03 -0400
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> Shuah,
> 
> After Masami gives an ack, could you take this through your tree.
> 
> I don't think it's urgent, but I want to make sure it gets upstream.
> 
> -- Steve
> 
> 
> On Tue, 14 Oct 2025 14:51:49 -0400
> Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> 
> > From: Steven Rostedt <rostedt@xxxxxxxxxxx>
> > 
> > Commit 64cf7d058a00 ("tracing: Have trace_marker use per-cpu data to read
> > user space") made an update that fixed both trace_marker and
> > trace_marker_raw. But the small difference made to trace_marker_raw had a
> > blatant bug in it that any basic testing would have uncovered.
> > Unfortunately, the self tests have tests for trace_marker but nothing for
> > trace_marker_raw which allowed the bug to get upstream.
> > 
> > Add basic selftests to test trace_marker_raw so that this doesn't happen
> > again.
> > 
> > Signed-off-by: Steven Rostedt (Google) <rostedt@xxxxxxxxxxx>
> >