On Tue, Jul 25, 2023, at 20:15, Alexei Starovoitov wrote: > On Mon, Jul 24, 2023 at 1:41 PM Arnd Bergmann <arnd@xxxxxxxx> wrote: >> >> Sure, that's fine. Between this and the two suggestions I had >> (__always_inline or passing the flags from inc_active as a >> return code), I don't have a strong preference, so pick whichever >> you like. > > I think: > static void dec_active(struct bpf_mem_cache *c, unsigned long *flags) > is cleaner. > Could you send a patch? Ok, done, Arnd