Re: [PATCH] selftests/ftrace: Disable multiple_kprobes test on powerpc

From: Masami Hiramatsu
Date: Tue Jan 05 2021 - 05:02:46 EST


On Tue, 5 Jan 2021 12:27:30 +0530
"Naveen N. Rao" <naveen.n.rao@xxxxxxxxxxxxxxxxxx> wrote:

> Not all symbols are blacklisted on powerpc. Disable multiple_kprobes
> test until that is sorted, so that rest of ftrace and kprobe selftests
> can be run.

This looks good to me, but could you try to find the functions
which should be blocked from kprobes?
(Usually, the function which are involved in the sw-breakpoint
handling, including locks etc.)

Thank you,

>
> Signed-off-by: Naveen N. Rao <naveen.n.rao@xxxxxxxxxxxxxxxxxx>
> ---
> .../testing/selftests/ftrace/test.d/kprobe/multiple_kprobes.tc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/multiple_kprobes.tc b/tools/testing/selftests/ftrace/test.d/kprobe/multiple_kprobes.tc
> index 312d237800969e..41503c32f53eed 100644
> --- a/tools/testing/selftests/ftrace/test.d/kprobe/multiple_kprobes.tc
> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/multiple_kprobes.tc
> @@ -7,7 +7,7 @@
> # Currently HAVE_KPROBES_ON_FTRACE defined on x86 and powerpc64le
> case `uname -m` in
> x86_64|i[3456]86) OFFS=5;;
> - ppc64le) OFFS=8;;
> + ppc*) exit_unsupported;;
> *) OFFS=0;;
> esac
>
>
> base-commit: 36bbbd0e234d817938bdc52121a0f5473b3e58f5
> --
> 2.25.4
>


--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>