Re: [RFC PATCH v3 35/37] tools: Add FUSE, update bpf includes

From: Daniel Rosenberg
Date: Thu Apr 27 2023 - 20:48:28 EST


On Wed, Apr 26, 2023 at 9:24 PM Andrii Nakryiko
<andrii.nakryiko@xxxxxxxxx> wrote:
>
> On Mon, Apr 17, 2023 at 6:42 PM Daniel Rosenberg <drosen@xxxxxxxxxx> wrote:
> >
> > +/* Return Codes for Fuse BPF struct_op programs */
> > +#define BPF_FUSE_CONTINUE 0
> > +#define BPF_FUSE_USER 1
> > +#define BPF_FUSE_USER_PREFILTER 2
> > +#define BPF_FUSE_POSTFILTER 3
> > +#define BPF_FUSE_USER_POSTFILTER 4
>
> nit: can this be an enum instead? It would be more self-documenting,
> IMO. At given it's FUSE BPF-specific, why is it not in
> uapi/linux/fuse.h?
>

An enum would be nicer. And I'm sure there are plenty of things that
are probably in the wrong place right now. I'll be moving most of the
changes in bpf specific areas over to fuse specific areas when
struct_ops can handle modules. This particular one can move now
though.