Shuah,
Can you pull this into your urgent branch? Thanks, -- Steve On Tue, 7 Apr 2026 20:37:27 -0400 Steven Rostedt <[email protected]> wrote: > Shuah, > > Care to take this through your tree. Probably could even add: > > Cc: [email protected] > Fixes: 8b55572e51805 ("tracing/selftests: Add tracefs mount options test") > > As well as: > > Reviewed-by: Steven Rostedt (Google) <[email protected]> > > -- Steve > > > On Tue, 7 Apr 2026 18:26:13 +0800 > CaoRuichuang <[email protected]> wrote: > > > From: Cao Ruichuang <[email protected]> > > > > test_ownership.tc is sourced by ftracetest under /bin/sh. > > > > The script currently declares mount_point with local at file scope, > > which makes /bin/sh abort with "local: not in a function" before the > > test can reach the eventfs ownership checks. > > > > Replace the top-level local declaration with a normal shell variable so > > kernels that support the gid= tracefs mount option can run the test at > > all. > > > > Signed-off-by: Cao Ruichuang <[email protected]> > > --- > > tools/testing/selftests/ftrace/test.d/00basic/test_ownership.tc | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git > > a/tools/testing/selftests/ftrace/test.d/00basic/test_ownership.tc > > b/tools/testing/selftests/ftrace/test.d/00basic/test_ownership.tc > > index e71cc3ad0..6d00d3c0f 100644 > > --- a/tools/testing/selftests/ftrace/test.d/00basic/test_ownership.tc > > +++ b/tools/testing/selftests/ftrace/test.d/00basic/test_ownership.tc > > @@ -6,7 +6,7 @@ > > original_group=`stat -c "%g" .` > > original_owner=`stat -c "%u" .` > > > > -local mount_point=$(get_mount_point) > > +mount_point=$(get_mount_point) > > > > mount_options=$(get_mnt_options "$mount_point") > > >
