Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 8 Jan 2023 16:31:28 -0500
From:   Steven Rostedt <rostedt@goodmis.org>
To:     "Masami Hiramatsu (Google)" <mhiramat@kernel.org>
Cc:     Shuah Khan <shuah@kernel.org>, linux-kernel@vger.kernel.org,
        linux-trace-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org
Subject: Re: [PATCH 0/3] selftests/tracing: Fix some testcases for recent
 change
Message-ID: <20230108163128.2860894d@rorschach.local.home>
In-Reply-To: <167309832823.640500.13244630381161014364.stgit@devnote3>
References: <167309832823.640500.13244630381161014364.stgit@devnote3>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Sat,  7 Jan 2023 22:32:08 +0900
"Masami Hiramatsu (Google)" <mhiramat@kernel.org> wrote:

> This includes some patches to fix 2 issues on ftrace selftests.
> 
> - eprobe filter and eprobe syntax test case were introduced but it
>   doesn't check whether the kernel supports eprobe filter. Thus the
>   new test case fails on the kernel which has eprobe but not support
>   eprobe filter. To solve this issue, add a filter description to
>   README file [1/3] and run the filter syntax error test only if the
>   description is found in the README file [2/3].
> 
> - Recently objtool adds prefix symbols for the function padding nops,
>   and the probepoint test case fails because this probepoint test case
>   tests whether the kprobe event can probe the target function and the
>   functions next to the target function. But the prefix symbols can not
>   be probed. Thus these prefix symbols must be skipped [3/3].
> 
> Thank you,
> 
> ---
> 
> Masami Hiramatsu (Google) (3):
>       tracing/eprobe: Fix to add filter on eprobe description in README file
>       selftests/ftrace: Fix eprobe syntax test case to check filter support
>       selftests/ftrace: Fix probepoint testcase to ignore __pfx_* symbols
> 
>

Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>

-- Steve