Re: [PATCH v3 bpf-next 3/3] selftests/bpf: Add tests for bpf_get_dentry_xattr

From: Song Liu
Date: Tue Aug 06 2024 - 18:22:58 EST




> On Aug 6, 2024, at 1:33 PM, Song Liu <song@xxxxxxxxxx> wrote:
>
> Add test for bpf_get_dentry_xattr on hook security_inode_getxattr.
> Verify that the kfunc can read the xattr. Also test failing getxattr
> from user space by returning non-zero from the LSM bpf program.
>
> Acked-by: Christian Brauner <brauner@xxxxxxxxxx>
> Signed-off-by: Song Liu <song@xxxxxxxxxx>
> ---
> tools/testing/selftests/bpf/bpf_kfuncs.h | 8 ++++
> .../selftests/bpf/prog_tests/fs_kfuncs.c | 9 ++++-
> .../selftests/bpf/progs/test_get_xattr.c | 37 ++++++++++++++++---
> 3 files changed, 48 insertions(+), 6 deletions(-)
>
> diff --git a/tools/testing/selftests/bpf/bpf_kfuncs.h b/tools/testing/selftests/bpf/bpf_kfuncs.h
> index 3b6675ab4086..cf25421c36c9 100644
> --- a/tools/testing/selftests/bpf/bpf_kfuncs.h
> +++ b/tools/testing/selftests/bpf/bpf_kfuncs.h
> @@ -78,4 +78,12 @@ extern int bpf_verify_pkcs7_signature(struct bpf_dynptr *data_ptr,
>
> extern bool bpf_session_is_return(void) __ksym __weak;
> extern __u64 *bpf_session_cookie(void) __ksym __weak;
> +
> +/* Description
> + * Returns xattr of a dentry
> + * Returns__bpf_kfunc
> + * Error code
> + */
> +extern int bpf_get_dentry_xattr(struct dentry *dentry, const char *name,
> + struct bpf_dynptr *value_ptr) __ksym __weak;
> #endif

Oops.. forgot to declare struct dentry. Let me fix this. Sorry for the noise.

Thanks,
Song