* [PATCH] selftests/uprobes: Add a basic uprobe testcase
@ 2024-05-24 0:49 Masami Hiramatsu (Google)
2024-08-07 20:36 ` Steven Rostedt
0 siblings, 1 reply; 3+ messages in thread
From: Masami Hiramatsu (Google) @ 2024-05-24 0:49 UTC (permalink / raw)
To: Steven Rostedt, Shuah Khan
Cc: Andrii Nakryiko, oleg, jolsa, linux-kernel, linux-trace-kernel,
mhiramat, linux-kselftest
From: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Add a basic uprobe testcase which checks whether add/remove/trace
operations works on /bin/sh.
Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
---
.../ftrace/test.d/dynevent/add_remove_uprobe.tc | 26 ++++++++++++++++++++
1 file changed, 26 insertions(+)
create mode 100644 tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc
diff --git a/tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc b/tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc
new file mode 100644
index 000000000000..a275decdc880
--- /dev/null
+++ b/tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc
@@ -0,0 +1,26 @@
+#!/bin/sh
+# SPDX-License-Identifier: GPL-2.0
+# description: Generic dynamic event - add/remove/test uprobe events
+# requires: uprobe_events
+
+echo 0 > events/enable
+echo > dynamic_events
+
+echo 'cat /proc/$$/maps' | /bin/sh | \
+ grep "r-xp .*/bin/.*sh$" | \
+ awk '{printf "p:myevent %s:0x%s\n", $6,$3 }' >> uprobe_events
+
+grep -q myevent uprobe_events
+test -d events/uprobes/myevent
+
+echo 1 > events/uprobes/myevent/enable
+echo 'ls' | /bin/sh > /dev/null
+echo 0 > events/uprobes/myevent/enable
+grep -q myevent trace
+
+echo "-:myevent" >> uprobe_events
+! grep -q myevent uprobe_events
+
+echo > uprobe_events
+
+clear_trace
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] selftests/uprobes: Add a basic uprobe testcase
2024-05-24 0:49 [PATCH] selftests/uprobes: Add a basic uprobe testcase Masami Hiramatsu (Google)
@ 2024-08-07 20:36 ` Steven Rostedt
2024-08-07 20:46 ` Shuah Khan
0 siblings, 1 reply; 3+ messages in thread
From: Steven Rostedt @ 2024-08-07 20:36 UTC (permalink / raw)
To: Masami Hiramatsu (Google)
Cc: Shuah Khan, Andrii Nakryiko, oleg, jolsa, linux-kernel,
linux-trace-kernel, linux-kselftest
On Fri, 24 May 2024 09:49:55 +0900
"Masami Hiramatsu (Google)" <mhiramat@kernel.org> wrote:
> From: Masami Hiramatsu (Google) <mhiramat@kernel.org>
>
> Add a basic uprobe testcase which checks whether add/remove/trace
> operations works on /bin/sh.
>
> Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Shuah, can you take this too.
-- Steve
> ---
> .../ftrace/test.d/dynevent/add_remove_uprobe.tc | 26 ++++++++++++++++++++
> 1 file changed, 26 insertions(+)
> create mode 100644 tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc
>
> diff --git a/tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc b/tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc
> new file mode 100644
> index 000000000000..a275decdc880
> --- /dev/null
> +++ b/tools/testing/selftests/ftrace/test.d/dynevent/add_remove_uprobe.tc
> @@ -0,0 +1,26 @@
> +#!/bin/sh
> +# SPDX-License-Identifier: GPL-2.0
> +# description: Generic dynamic event - add/remove/test uprobe events
> +# requires: uprobe_events
> +
> +echo 0 > events/enable
> +echo > dynamic_events
> +
> +echo 'cat /proc/$$/maps' | /bin/sh | \
> + grep "r-xp .*/bin/.*sh$" | \
> + awk '{printf "p:myevent %s:0x%s\n", $6,$3 }' >> uprobe_events
> +
> +grep -q myevent uprobe_events
> +test -d events/uprobes/myevent
> +
> +echo 1 > events/uprobes/myevent/enable
> +echo 'ls' | /bin/sh > /dev/null
> +echo 0 > events/uprobes/myevent/enable
> +grep -q myevent trace
> +
> +echo "-:myevent" >> uprobe_events
> +! grep -q myevent uprobe_events
> +
> +echo > uprobe_events
> +
> +clear_trace
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] selftests/uprobes: Add a basic uprobe testcase
2024-08-07 20:36 ` Steven Rostedt
@ 2024-08-07 20:46 ` Shuah Khan
0 siblings, 0 replies; 3+ messages in thread
From: Shuah Khan @ 2024-08-07 20:46 UTC (permalink / raw)
To: Steven Rostedt, Masami Hiramatsu (Google)
Cc: Shuah Khan, Andrii Nakryiko, oleg, jolsa, linux-kernel,
linux-trace-kernel, linux-kselftest, Shuah Khan
On 8/7/24 14:36, Steven Rostedt wrote:
> On Fri, 24 May 2024 09:49:55 +0900
> "Masami Hiramatsu (Google)" <mhiramat@kernel.org> wrote:
>
>> From: Masami Hiramatsu (Google) <mhiramat@kernel.org>
>>
>> Add a basic uprobe testcase which checks whether add/remove/trace
>> operations works on /bin/sh.
>>
>> Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
>
> Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>
>
> Shuah, can you take this too.
Yes.
>
> -- Steve
Applied to linux-kselftest next for Linux 6.12-rc1.
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-08-07 20:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-24 0:49 [PATCH] selftests/uprobes: Add a basic uprobe testcase Masami Hiramatsu (Google)
2024-08-07 20:36 ` Steven Rostedt
2024-08-07 20:46 ` Shuah Khan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox