[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250411143132.56096f76@gandalf.local.home>
Date: Fri, 11 Apr 2025 14:31:32 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Mark Brown <broonie@...nel.org>
Cc: linux-kernel@...r.kernel.org, linux-trace-kernel@...r.kernel.org,
bpf@...r.kernel.org, Masami Hiramatsu <mhiramat@...nel.org>, Mark Rutland
<mark.rutland@....com>, Mathieu Desnoyers <mathieu.desnoyers@...icios.com>,
Andrew Morton <akpm@...ux-foundation.org>, Sven Schnelle
<svens@...ux.ibm.com>, Paul Walmsley <paul.walmsley@...ive.com>, Palmer
Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>, Guo Ren
<guoren@...nel.org>, Donglin Peng <dolinux.peng@...il.com>, Zheng Yejian
<zhengyejian@...weicloud.com>, Aishwarya.TCV@....com
Subject: Re: [PATCH v4 2/4] ftrace: Add support for function argument to
graph tracer
On Fri, 11 Apr 2025 14:24:27 -0400
Steven Rostedt <rostedt@...dmis.org> wrote:
> > Yeah, if I bodge ftracetest to be a bash script then the test runs fine
> > so it'll be a bashism. We're running the tests in a Debian rootfs so
> > /bin/sh will be dash.
>
> Interesting, as one of the ftracetests checks for bashisms:
>
> test.d/selftest/bashisms.tc
>
> Did it not catch something?
Hmm, I just tested this, and it fails on my box too (I test on a debian VM).
It fails with and without setting it to bash. I'll take a look too.
-- Steve
Powered by blists - more mailing lists