| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sat, 11 Jul 2015 10:13:28 +0900 From: Masami Hiramatsu <masami.hiramatsu.pt@...achi.com> To: Steven Rostedt <rostedt@...dmis.org> CC: Vince Weaver <vincent.weaver@...ne.edu>, linux-api@...r.kernel.org, Heiko Carstens <heiko.carstens@...ibm.com>, Shuah Khan <shuahkh@....samsung.com>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Ingo Molnar <mingo@...hat.com>, Namhyung Kim <namhyung@...nel.org>, Ingo Molnar <mingo@...nel.org> Subject: Re: Re: [PATCH tip/master ] ftracetest: Update kprobe-tracer testcases because of renaming do_fork On 2015/07/09 21:03, Steven Rostedt wrote: > On Thu, 09 Jul 2015 19:10:12 +0900 > Masami Hiramatsu <masami.hiramatsu.pt@...achi.com> wrote: > >> echo 0 > events/enable >> echo > kprobe_events >> -echo p:myevent do_fork > kprobe_events >> +echo p:myevent _do_fork > kprobe_events >> grep myevent kprobe_events >> test -d events/kprobes/myevent >> echo > kprobe_events >> diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/busy_check.tc b/tools/testing/selftests/ftrace/test.d/kprobe/busy_check.tc >> index d8c7bb6..74507db 100644 >> --- a/tools/testing/selftests/ftrace/test.d/kprobe/busy_check.tc >> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/busy_check.tc >> @@ -5,7 +5,7 @@ >> >> echo 0 > events/enable >> echo > kprobe_events >> -echo p:myevent do_fork > kprobe_events >> +echo p:myevent _do_fork > kprobe_events >> test -d events/kprobes/myevent >> echo 1 > events/kprobes/myevent/enable >> echo > kprobe_events && exit 1 # this must fail >> diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args.tc >> index c45ee27..64949d4 100644 >> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args.tc >> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args.tc >> @@ -5,7 +5,7 @@ >> >> echo 0 > events/enable >> echo > kprobe_events >> -echo 'p:testprobe do_fork $stack $stack0 +0($stack)' > kprobe_events >> +echo 'p:testprobe _do_fork $stack $stack0 +0($stack)' > kprobe_events >> grep testprobe kprobe_events >> test -d events/kprobes/testprobe >> echo 1 > events/kprobes/testprobe/enable >> diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_ftrace.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_ftrace.tc >> index ab41d2b..d6f2f49 100644 >> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_ftrace.tc >> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_ftrace.tc >> @@ -6,31 +6,31 @@ grep function available_tracers || exit_unsupported # this is configurable >> >> # prepare >> echo nop > current_tracer >> -echo do_fork > set_ftrace_filter >> +echo _do_fork > set_ftrace_filter >> echo 0 > events/enable >> echo > kprobe_events >> -echo 'p:testprobe do_fork' > kprobe_events >> +echo 'p:testprobe _do_fork' > kprobe_events >> >> # kprobe on / ftrace off >> echo 1 > events/kprobes/testprobe/enable >> echo > trace >> ( echo "forked") >> grep testprobe trace >> -! grep 'do_fork <-' trace >> +! grep '_do_fork <-' trace >> >> # kprobe on / ftrace on >> echo function > current_tracer >> echo > trace >> ( echo "forked") >> grep testprobe trace >> -grep 'do_fork <-' trace >> +grep '_do_fork <-' trace >> >> # kprobe off / ftrace on >> echo 0 > events/kprobes/testprobe/enable >> echo > trace >> ( echo "forked") >> ! grep testprobe trace >> -grep 'do_fork <-' trace >> +grep '_do_fork <-' trace >> >> # kprobe on / ftrace on >> echo 1 > events/kprobes/testprobe/enable >> @@ -38,14 +38,14 @@ echo function > current_tracer >> echo > trace >> ( echo "forked") >> grep testprobe trace >> -grep 'do_fork <-' trace >> +grep '_do_fork <-' trace >> >> # kprobe on / ftrace off >> echo nop > current_tracer >> echo > trace >> ( echo "forked") >> grep testprobe trace >> -! grep 'do_fork <-' trace >> +! grep '_do_fork <-' trace >> >> # cleanup >> echo nop > current_tracer >> diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kretprobe_args.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kretprobe_args.tc >> index 3171798..0d09546 100644 >> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kretprobe_args.tc >> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kretprobe_args.tc >> @@ -5,7 +5,7 @@ >> >> echo 0 > events/enable >> echo > kprobe_events >> -echo 'r:testprobe2 do_fork $retval' > kprobe_events >> +echo 'r:testprobe2 _do_fork $retval' > kprobe_events >> grep testprobe2 kprobe_events >> test -d events/kprobes/testprobe2 >> echo 1 > events/kprobes/testprobe2/enable > > > Instead of hard coding the name, what about adding a global variable > for the function, in case we need to change it again, it would only > need to be changed in one place? OK, I'll introduce FORK_SYMBOL="_do_fork" :) Thank you, -- Masami HIRAMATSU Linux Technology Research Center, System Productivity Research Dept. Center for Technology Innovation - Systems Engineering Hitachi, Ltd., Research & Development Group E-mail: masami.hiramatsu.pt@...achi.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists