On Wed, Dec 6, 2023 at 9:21 AM Andrii Nakryiko <andrii.nakryiko@xxxxxxxxx> wrote: > > On Tue, Dec 5, 2023 at 11:36 PM Jie Jiang <jiejiang@xxxxxxxxxxxx> wrote: > > > > Parse uid and gid in bpf_parse_param() so that they can be passed in as > > the `data` parameter when mount() bpffs. This will be useful when we > > want to control which user/group has the control to the mounted bpffs, > > otherwise a separate chown() call will be needed. > > > > Signed-off-by: Jie Jiang <jiejiang@xxxxxxxxxxxx> > > --- > > v1 -> v2: Add additional validation in bpf_parse_param() for if the > > requested uid/gid is representable in the fs's idmapping. > > > > kernel/bpf/inode.c | 52 ++++++++++++++++++++++++++++++++++++++++++++-- > > 1 file changed, 50 insertions(+), 2 deletions(-) > > > > LGTM, but I want to point out that this will conflict with the BPF > token series ([0]), so depending which one goes in first, the other > will have to be rebased. The token series are much bigger, so I applied them first. Please rebase, resend, and keep acks. > Acked-by: Andrii Nakryiko <andrii@xxxxxxxxxx>
