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.
