On Wed, Jan 7, 2015 at 4:58 PM, Rusty Russell <rusty@xxxxxxxxxxxxxxx> wrote: > --- a/kernel/bpf/core.c > +++ b/kernel/bpf/core.c > void bpf_jit_binary_free(struct bpf_binary_header *hdr) > { > - module_free(NULL, hdr); > + module_memfree(hdr); > } ... > -void __weak module_free(struct module *mod, void *module_region) > +void __weak module_memfree(void *module_region) > { > vfree(module_region); > } Looks obviously correct. Acked-by: Alexei Starovoitov <ast@xxxxxxxxxx>