Hi, Oops, I forgot to Cc Shuah. I'll update a bit this series and send v5.
Thanks, On Thu, 28 Feb 2019 21:33:00 +0900 Masami Hiramatsu <mhira...@kernel.org> wrote: > Add a user-memory access syntax testcase which checks > new user-memory access syntax and ustring type. > > Signed-off-by: Masami Hiramatsu <mhira...@kernel.org> > --- > .../ftrace/test.d/kprobe/kprobe_args_user.tc | 31 > ++++++++++++++++++++ > 1 file changed, 31 insertions(+) > create mode 100644 > tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_user.tc > > diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_user.tc > b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_user.tc > new file mode 100644 > index 000000000000..996da9d1eec5 > --- /dev/null > +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_user.tc > @@ -0,0 +1,31 @@ > +#!/bin/sh > +# SPDX-License-Identifier: GPL-2.0 > +# description: Kprobe event user-memory access > + > +[ -f kprobe_events ] || exit_unsupported # this is configurable > + > +grep -A10 "fetcharg:" README | grep -q 'ustring' || exit_unsupported > +grep -A10 "fetcharg:" README | grep -q '\[u\]<offset>' || exit_unsupported > + > +:;: "user-memory access syntax and ustring working on user memory";: > +echo 'p:myevent do_sys_open path=+0($arg2):ustring path2=+u0($arg2):string' \ > + > kprobe_events > + > +grep myevent kprobe_events | \ > + grep -q 'path=+0($arg2):ustring path2=+u0($arg2):string' > +echo 1 > events/kprobes/myevent/enable > +echo > /dev/null > +echo 0 > events/kprobes/myevent/enable > + > +grep myevent trace | grep -q 'path="/dev/null" path2="/dev/null"' > + > +:;: "user-memory access syntax and ustring not working with kernel memory";: > +echo 'p:myevent vfs_symlink path=+0($arg3):ustring path2=+u0($arg3):string' \ > + > kprobe_events > +echo 1 > events/kprobes/myevent/enable > +ln -s foo $TMPDIR/bar > +echo 0 > events/kprobes/myevent/enable > + > +grep myevent trace | grep -q 'path=(fault) path2=(fault)' > + > +exit 0 > -- Masami Hiramatsu <mhira...@kernel.org>