lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.18 059/879] selftests/bpf: Fix vfs_link kprobe definition
    Date
    From: Nikolay Borisov <nborisov@suse.com>

    [ Upstream commit e299bcd4d16ff86f46c48df1062c8aae0eca1ed8 ]

    Since commit 6521f8917082 ("namei: prepare for idmapped mounts")
    vfs_link's prototype was changed, the kprobe definition in
    profiler selftest in turn wasn't updated. The result is that all
    argument after the first are now stored in different registers. This
    means that self-test has been broken ever since. Fix it by updating the
    kprobe definition accordingly.

    Signed-off-by: Nikolay Borisov <nborisov@suse.com>
    Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
    Link: https://lore.kernel.org/bpf/20220331140949.1410056-1-nborisov@suse.com
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    tools/testing/selftests/bpf/progs/profiler.inc.h | 5 +++--
    1 file changed, 3 insertions(+), 2 deletions(-)

    diff --git a/tools/testing/selftests/bpf/progs/profiler.inc.h b/tools/testing/selftests/bpf/progs/profiler.inc.h
    index 4896fdf816f7..92331053dba3 100644
    --- a/tools/testing/selftests/bpf/progs/profiler.inc.h
    +++ b/tools/testing/selftests/bpf/progs/profiler.inc.h
    @@ -826,8 +826,9 @@ int kprobe_ret__do_filp_open(struct pt_regs* ctx)

    SEC("kprobe/vfs_link")
    int BPF_KPROBE(kprobe__vfs_link,
    - struct dentry* old_dentry, struct inode* dir,
    - struct dentry* new_dentry, struct inode** delegated_inode)
    + struct dentry* old_dentry, struct user_namespace *mnt_userns,
    + struct inode* dir, struct dentry* new_dentry,
    + struct inode** delegated_inode)
    {
    struct bpf_func_stats_ctx stats_ctx;
    bpf_stats_enter(&stats_ctx, profiler_bpf_vfs_link);
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-06-08 00:02    [W:2.256 / U:1.128 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site