On Mon, Mar 17, 2025 at 11:59 AM Joe Lawrence <joe.lawrence@xxxxxxxxxx> wrote: > > On 3/17/25 12:51, Song Liu wrote: > > CONFIG_KPROBES_ON_FTRACE is required for test-kprobe. Skip test-kprobe > > when CONFIG_KPROBES_ON_FTRACE is not set. > > > > Signed-off-by: Song Liu <song@xxxxxxxxxx> > > --- > > tools/testing/selftests/livepatch/test-kprobe.sh | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/tools/testing/selftests/livepatch/test-kprobe.sh b/tools/testing/selftests/livepatch/test-kprobe.sh > > index 115065156016..fd823dd5dd7f 100755 > > --- a/tools/testing/selftests/livepatch/test-kprobe.sh > > +++ b/tools/testing/selftests/livepatch/test-kprobe.sh > > @@ -5,6 +5,8 @@ > > > > . $(dirname $0)/functions.sh > > > > +zgrep KPROBES_ON_FTRACE /proc/config.gz || skip "test-kprobe requires CONFIG_KPROBES_ON_FTRACE" > > + > > Hi Song, > > This in turn depends on CONFIG_IKCONFIG_PROC for /proc/config.gz (not > set for RHEL distro kernels). I was actually worrying about this when testing it. > Is there a dynamic way to figure out CONFIG_KPROBES_ON_FTRACE support? How about we grep kprobe_ftrace_ops from /proc/kallsyms? Thanks, Song > Without looking into it very long, maybe test_klp_kprobe.c's call to > register_kprobe() could fail with -ENOTSUPP and the test script could > gracefully skip the test? > > Regards, > > -- > Joe >
