[linux-3.16-rc5/arch/mips/net/bpf_jit.c:1337]: (style) Checking if unsigned variable 'off' is less than zero. Source code is off = pkt_type_offset(); if (off < 0) return -1; but unsigned int i, off, load_order, condt; and static int pkt_type_offset(void) Suggest put the return type from pkt_type_offset into a signed variable, test that < 0, then use a cast to get the value into an unsigned variable. Something like signedVar = pkt_type_offset(); if (signedVar < 0) return -1; off = (unsigned int) signedVar;
Thanks, patch submitted: https://patchwork.kernel.org/patch/4575021/
Resubmitted: https://patchwork.kernel.org/patch/4575881/
Applied ages ago as 98fb24af5e6885ae5ae6b1598a875922e0d83a36 (arch/mips/net/bpf_jit.c: fix failure check)