* [PATCH bpf v1] selftests/bpf: Rename do_filp_open() to do_file_open()
@ 2026-02-10 23:58 Ihor Solodrai
2026-02-11 2:20 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 2+ messages in thread
From: Ihor Solodrai @ 2026-02-10 23:58 UTC (permalink / raw)
To: Alexei Starovoitov, Andrii Nakryiko, Eduard Zingerman, Daniel Borkmann
Cc: Shung-Hsi Yu, Al Viro, bpf, linux-kernel
do_filp_open() was renamed (fixing typo) in commit
541003b576c3 ("rename do_filp_open() to do_file_open()")
This broke test_profiler, because it uses a kretprobe on that
function. Fix it by renaming accordingly.
Fixes: 541003b576c3 ("rename do_filp_open() to do_file_open()")
Reported-by: Shung-Hsi Yu <shung-hsi.yu@suse.com>
Closes: https://lore.kernel.org/bpf/djwjf2vfb7gro3rfag666bojod6ytcectahnb5z6hx2hawimtj@sx47ghzjg4lw/
Signed-off-by: Ihor Solodrai <ihor.solodrai@linux.dev>
---
tools/testing/selftests/bpf/progs/profiler.h | 2 +-
tools/testing/selftests/bpf/progs/profiler.inc.h | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/testing/selftests/bpf/progs/profiler.h b/tools/testing/selftests/bpf/progs/profiler.h
index 3bac4fdd4bdf..637fbf2c2652 100644
--- a/tools/testing/selftests/bpf/progs/profiler.h
+++ b/tools/testing/selftests/bpf/progs/profiler.h
@@ -169,7 +169,7 @@ enum bpf_function_id {
profiler_bpf_sched_process_exec,
profiler_bpf_sched_process_exit,
profiler_bpf_sys_enter_kill,
- profiler_bpf_do_filp_open_ret,
+ profiler_bpf_do_file_open_ret,
profiler_bpf_sched_process_fork,
profiler_bpf_vfs_link,
profiler_bpf_vfs_symlink,
diff --git a/tools/testing/selftests/bpf/progs/profiler.inc.h b/tools/testing/selftests/bpf/progs/profiler.inc.h
index 813143b4985d..9044dd8aff11 100644
--- a/tools/testing/selftests/bpf/progs/profiler.inc.h
+++ b/tools/testing/selftests/bpf/progs/profiler.inc.h
@@ -751,11 +751,11 @@ int raw_tracepoint__sched_process_exec(struct bpf_raw_tracepoint_args* ctx)
return 0;
}
-SEC("kretprobe/do_filp_open")
-int kprobe_ret__do_filp_open(struct pt_regs* ctx)
+SEC("kretprobe/do_file_open")
+int kprobe_ret__do_file_open(struct pt_regs *ctx)
{
struct bpf_func_stats_ctx stats_ctx;
- bpf_stats_enter(&stats_ctx, profiler_bpf_do_filp_open_ret);
+ bpf_stats_enter(&stats_ctx, profiler_bpf_do_file_open_ret);
struct file* filp = (struct file*)PT_REGS_RC_CORE(ctx);
--
2.53.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH bpf v1] selftests/bpf: Rename do_filp_open() to do_file_open()
2026-02-10 23:58 [PATCH bpf v1] selftests/bpf: Rename do_filp_open() to do_file_open() Ihor Solodrai
@ 2026-02-11 2:20 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2026-02-11 2:20 UTC (permalink / raw)
To: Ihor Solodrai
Cc: ast, andrii, eddyz87, daniel, shung-hsi.yu, viro, bpf, linux-kernel
Hello:
This patch was applied to bpf/bpf.git (master)
by Alexei Starovoitov <ast@kernel.org>:
On Tue, 10 Feb 2026 15:58:55 -0800 you wrote:
> do_filp_open() was renamed (fixing typo) in commit
> 541003b576c3 ("rename do_filp_open() to do_file_open()")
>
> This broke test_profiler, because it uses a kretprobe on that
> function. Fix it by renaming accordingly.
>
> Fixes: 541003b576c3 ("rename do_filp_open() to do_file_open()")
> Reported-by: Shung-Hsi Yu <shung-hsi.yu@suse.com>
> Closes: https://lore.kernel.org/bpf/djwjf2vfb7gro3rfag666bojod6ytcectahnb5z6hx2hawimtj@sx47ghzjg4lw/
> Signed-off-by: Ihor Solodrai <ihor.solodrai@linux.dev>
>
> [...]
Here is the summary with links:
- [bpf,v1] selftests/bpf: Rename do_filp_open() to do_file_open()
https://git.kernel.org/bpf/bpf/c/1a7eb7a3d740
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-02-11 2:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-02-10 23:58 [PATCH bpf v1] selftests/bpf: Rename do_filp_open() to do_file_open() Ihor Solodrai
2026-02-11 2:20 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®