On Oct 9, 2024, at 6:36 AM, Greg KH gregkh@linuxfoundation.org wrote:
On Tue, Oct 08, 2024 at 03:29:44PM -0700, Sherry Yang wrote:
5.10.y backported the commit 09bcf9254838 ("selftests/ftrace: Add new test case which checks non unique symbol") which added a new test case to check non-unique symbol. However, 5.10.y didn't backport the kernel commit b022f0c7e404 ("tracing/kprobes: Return EADDRNOTAVAIL when func matches several symbols")to support the functionality from kernel side. Backport it in this patch series.
The first two patches are presiquisites. The 4th commit is a fix commit for the 3rd one.
Should we just revert the selftest test instead? That seems simpler instead of adding a new feature to this old and obsolete kernel tree, right?
Sorry about the confusion. If kprobe attaches a function which is not the user wants to attach to, I would say it’s a bug. The test case uncovers the bug, so it’s a fix.
Sherry