Re: [PATCH V2 6/8] perf tools: Enable configs required for test_uprobe_from_different_cu.sh

[Masami Hiramatsu (Google) <mhiramat@kernel.org>]

On Mon,  8 Apr 2024 11:52:28 +0530
Chaitanya S Prakash <ChaitanyaS.Prakash@arm.com> wrote:

> From: Chaitanya S Prakash <chaitanyas.prakash@arm.com>
> 
> Test "perf probe of function from different CU" fails due to certain
> configs not being enabled. Building the kernel with
> CONFIG_KPROBE_EVENTS=y and CONFIG_UPROBE_EVENTS=y fixes the issue. As
> CONFIG_KPROBE_EVENTS is dependent on CONFIG_KPROBES, enable it as well.
> Some platforms enable these configs as a part of their defconfig, so
> this change is only required for the ones that don't do so.
> 

Looks good to me.

Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>

Thank you,

> Signed-off-by: Chaitanya S Prakash <chaitanyas.prakash@arm.com>
> ---
>  tools/perf/tests/config-fragments/config | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tools/perf/tests/config-fragments/config b/tools/perf/tests/config-fragments/config
> index c340b3195fca..4fca12851016 100644
> --- a/tools/perf/tests/config-fragments/config
> +++ b/tools/perf/tests/config-fragments/config
> @@ -9,3 +9,6 @@ CONFIG_GENERIC_TRACER=y
>  CONFIG_FTRACE=y
>  CONFIG_FTRACE_SYSCALLS=y
>  CONFIG_BRANCH_PROFILE_NONE=y
> +CONFIG_KPROBES=y
> +CONFIG_KPROBE_EVENTS=y
> +CONFIG_UPROBE_EVENTS=y
> -- 
> 2.30.2
> 
> 


-- 
Masami Hiramatsu (Google) <mhiramat@kernel.org>