Message ID | 20241114084345.1564165-5-song@kernel.org (mailing list archive) |
---|---|
State | RFC |
Delegated to: | BPF |
Headers | show |
Series | Fanotify fastpath handler | expand |
Context | Check | Description |
---|---|---|
netdev/tree_selection | success | Clearly marked for bpf-next, async |
netdev/apply | fail | Patch does not apply to bpf-next-0 |
bpf/vmtest-bpf-net-VM_Test-5 | success | Logs for aarch64-gcc / build-release |
bpf/vmtest-bpf-net-VM_Test-1 | success | Logs for ShellCheck |
bpf/vmtest-bpf-net-VM_Test-0 | success | Logs for Lint |
bpf/vmtest-bpf-net-VM_Test-3 | success | Logs for Validate matrix.py |
bpf/vmtest-bpf-net-VM_Test-2 | success | Logs for Unittests |
bpf/vmtest-bpf-net-VM_Test-13 | fail | Logs for x86_64-gcc / build / build for x86_64 with gcc |
bpf/vmtest-bpf-net-PR | fail | PR summary |
bpf/vmtest-bpf-net-VM_Test-7 | success | Logs for aarch64-gcc / veristat |
bpf/vmtest-bpf-net-VM_Test-11 | success | Logs for s390x-gcc / veristat |
bpf/vmtest-bpf-net-VM_Test-4 | fail | Logs for aarch64-gcc / build / build for aarch64 with gcc |
bpf/vmtest-bpf-net-VM_Test-12 | success | Logs for set-matrix |
bpf/vmtest-bpf-net-VM_Test-10 | success | Logs for s390x-gcc / test |
bpf/vmtest-bpf-net-VM_Test-9 | success | Logs for s390x-gcc / build-release |
bpf/vmtest-bpf-net-VM_Test-6 | success | Logs for aarch64-gcc / test |
bpf/vmtest-bpf-net-VM_Test-8 | fail | Logs for s390x-gcc / build / build for s390x with gcc |
bpf/vmtest-bpf-net-VM_Test-14 | success | Logs for x86_64-gcc / build-release |
bpf/vmtest-bpf-net-VM_Test-15 | success | Logs for x86_64-gcc / test |
bpf/vmtest-bpf-net-VM_Test-16 | success | Logs for x86_64-gcc / veristat |
bpf/vmtest-bpf-net-VM_Test-17 | fail | Logs for x86_64-llvm-17 / build / build for x86_64 with llvm-17 |
bpf/vmtest-bpf-net-VM_Test-18 | fail | Logs for x86_64-llvm-17 / build-release / build for x86_64 with llvm-17-O2 |
bpf/vmtest-bpf-net-VM_Test-19 | success | Logs for x86_64-llvm-17 / test |
bpf/vmtest-bpf-net-VM_Test-20 | success | Logs for x86_64-llvm-17 / veristat |
bpf/vmtest-bpf-net-VM_Test-21 | fail | Logs for x86_64-llvm-18 / build / build for x86_64 with llvm-18 |
bpf/vmtest-bpf-net-VM_Test-22 | fail | Logs for x86_64-llvm-18 / build-release / build for x86_64 with llvm-18-O2 |
bpf/vmtest-bpf-net-VM_Test-23 | success | Logs for x86_64-llvm-18 / test |
bpf/vmtest-bpf-net-VM_Test-24 | success | Logs for x86_64-llvm-18 / veristat |
diff --git a/fs/bpf_fs_kfuncs.c b/fs/bpf_fs_kfuncs.c index 3fe9f59ef867..03ad3a2faec8 100644 --- a/fs/bpf_fs_kfuncs.c +++ b/fs/bpf_fs_kfuncs.c @@ -152,6 +152,44 @@ __bpf_kfunc int bpf_get_file_xattr(struct file *file, const char *name__str, return bpf_get_dentry_xattr(dentry, name__str, value_p); } +/** + * bpf_iput - Drop a reference on the inode + * + * @inode: inode to drop reference. + * + * Drop a refcount on inode. + */ +__bpf_kfunc void bpf_iput(struct inode *inode) +{ + iput(inode); +} + +/** + * bpf_dput - Drop a reference on the dentry + * + * @dentry: dentry to drop reference. + * + * Drop a refcount on dentry. + */ +__bpf_kfunc void bpf_dput(struct dentry *dentry) +{ + dput(dentry); +} + +/** + * bpf_is_subdir - is new dentry a subdirectory of old_dentry + * @new_dentry: new dentry + * @old_dentry: old dentry + * + * Returns true if new_dentry is a subdirectory of the parent (at any depth). + * Returns false otherwise. + * Caller must ensure that "new_dentry" is pinned before calling is_subdir() + */ +__bpf_kfunc bool bpf_is_subdir(struct dentry *new_dentry, struct dentry *old_dentry) +{ + return is_subdir(new_dentry, old_dentry); +} + __bpf_kfunc_end_defs(); BTF_KFUNCS_START(bpf_fs_kfunc_set_ids) @@ -161,6 +199,9 @@ BTF_ID_FLAGS(func, bpf_put_file, KF_RELEASE) BTF_ID_FLAGS(func, bpf_path_d_path, KF_TRUSTED_ARGS) BTF_ID_FLAGS(func, bpf_get_dentry_xattr, KF_SLEEPABLE | KF_TRUSTED_ARGS) BTF_ID_FLAGS(func, bpf_get_file_xattr, KF_SLEEPABLE | KF_TRUSTED_ARGS) +BTF_ID_FLAGS(func, bpf_iput, KF_SLEEPABLE | KF_RELEASE) +BTF_ID_FLAGS(func, bpf_dput, KF_SLEEPABLE | KF_RELEASE) +BTF_ID_FLAGS(func, bpf_is_subdir, KF_RCU) BTF_KFUNCS_END(bpf_fs_kfunc_set_ids) static int bpf_fs_kfuncs_filter(const struct bpf_prog *prog, u32 kfunc_id) diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c index 9a7ed527e47e..65abb2d74ee5 100644 --- a/kernel/bpf/verifier.c +++ b/kernel/bpf/verifier.c @@ -5432,6 +5432,7 @@ BTF_ID(struct, bpf_cpumask) #endif BTF_ID(struct, task_struct) BTF_ID(struct, bpf_crypto_ctx) +BTF_ID(struct, dentry) BTF_SET_END(rcu_protected_types) static bool rcu_protected_object(const struct btf *btf, u32 btf_id)
Add the following kfuncs: - bpf_iput - bpf_dput - bpf_is_subdir These kfuncs can be used by bpf fanotify fastpath. Both bpf_iput and bpf_dput are marked as KF_SLEEPABLE | KF_RELEASE. They will be used to release reference on inode and dentry. bpf_is_subdir is marked as KF_RCU. It will be used to take rcu protected pointers, for example, kptr saved to a bpf map. Signed-off-by: Song Liu <song@kernel.org> --- fs/bpf_fs_kfuncs.c | 41 +++++++++++++++++++++++++++++++++++++++++ kernel/bpf/verifier.c | 1 + 2 files changed, 42 insertions(+)