[v1] security.bpf xattr name prefix

[PATCH bpf-next 0/2] security.bpf xattr name prefix

Song Liu posted 2 patches 1 month, 3 weeks ago

Diff against v2
Download series mbox

There is a newer version of this series

fs/bpf_fs_kfuncs.c                            | 19 ++++++++-
include/uapi/linux/xattr.h                    |  4 ++
.../selftests/bpf/prog_tests/fs_kfuncs.c      | 40 ++++++++++++++-----
.../selftests/bpf/progs/test_get_xattr.c      | 30 ++++++++++++--
4 files changed, 78 insertions(+), 15 deletions(-)

Expand all Fold all

[PATCH bpf-next 0/2] security.bpf xattr name prefix

Posted by Song Liu 1 month, 3 weeks ago

Follow up discussion in LPC 2024 [1], that we need security.bpf xattr
prefix. This set adds "security.bpf" xattr name prefix, and allows
bpf kfuncs bpf_get_[file|dentry]_xattr() to read these xattrs.


[1] https://lpc.events/event/18/contributions/1940/

Song Liu (2):
  fs/xattr: bpf: Introduce security.bpf xattr name prefix
  selftests/bpf: Extend test fs_kfuncs to cover security.bpf xattr names

 fs/bpf_fs_kfuncs.c                            | 19 ++++++++-
 include/uapi/linux/xattr.h                    |  4 ++
 .../selftests/bpf/prog_tests/fs_kfuncs.c      | 40 ++++++++++++++-----
 .../selftests/bpf/progs/test_get_xattr.c      | 30 ++++++++++++--
 4 files changed, 78 insertions(+), 15 deletions(-)

--
2.43.5

Re: [PATCH bpf-next 0/2] security.bpf xattr name prefix

Posted by Song Liu 1 month, 2 weeks ago

Hi Christian, Al, and Jan,

Could you please review and share your comments on this set?

Thanks,
Song

On Wed, Oct 2, 2024 at 2:47 PM Song Liu <song@kernel.org> wrote:
>
> Follow up discussion in LPC 2024 [1], that we need security.bpf xattr
> prefix. This set adds "security.bpf" xattr name prefix, and allows
> bpf kfuncs bpf_get_[file|dentry]_xattr() to read these xattrs.
>
>
> [1] https://lpc.events/event/18/contributions/1940/
>
> Song Liu (2):
>   fs/xattr: bpf: Introduce security.bpf xattr name prefix
>   selftests/bpf: Extend test fs_kfuncs to cover security.bpf xattr names
>
>  fs/bpf_fs_kfuncs.c                            | 19 ++++++++-
>  include/uapi/linux/xattr.h                    |  4 ++
>  .../selftests/bpf/prog_tests/fs_kfuncs.c      | 40 ++++++++++++++-----
>  .../selftests/bpf/progs/test_get_xattr.c      | 30 ++++++++++++--
>  4 files changed, 78 insertions(+), 15 deletions(-)
>
> --
> 2.43.5

Re: [PATCH bpf-next 0/2] security.bpf xattr name prefix

Posted by Christian Brauner 1 month, 2 weeks ago

On Fri, Oct 11, 2024 at 10:40:02AM -0700, Song Liu wrote:
> Hi Christian, Al, and Jan,
> 
> Could you please review and share your comments on this set?

Yes, as I said in-person, I think adding security.bpf is fine.