Message ID | 515657BF.7010101@asianux.com |
---|---|
State | Not Applicable, archived |
Delegated to: | David Miller |
Headers | show |
From: Chen Gang <gang.chen@asianux.com> Date: Sat, 30 Mar 2013 11:10:55 +0800 > On 2013年03月30日 03:34, David Miller wrote: >> From: Chen Gang <gang.chen@asianux.com> >> Date: Fri, 29 Mar 2013 09:24:53 +0800 >> >>> > >>> > for make V=1 EXTRA_CFLAGS=-W ARCH=arm allmodconfig >>> > printk is need when CONFIG_BPF_JIT is defined >>> > or it will report pr_err and print_hex_dump are implicit declaration >>> > >>> > Signed-off-by: Chen Gang <gang.chen@asianux.com> >> There are no references to pr_err nor print_hex_dump in this >> filter.h header file. > > maybe we see the different tree. > I find it in next-20130328. When you submit patches, always mention explicitly what tree you want the patch applied to. In this case, this appropriate tree to be "net-next" since that is the tree where the problem actually exists. Patch applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 2013年03月30日 11:32, David Miller wrote: > From: Chen Gang <gang.chen@asianux.com> > Date: Sat, 30 Mar 2013 11:10:55 +0800 > >> > On 2013$BG/(B03$B7n(B30$BF|(B 03:34, David Miller wrote: >>> >> From: Chen Gang <gang.chen@asianux.com> >>> >> Date: Fri, 29 Mar 2013 09:24:53 +0800 >>> >> >>>>> >>> > >>>>> >>> > for make V=1 EXTRA_CFLAGS=-W ARCH=arm allmodconfig >>>>> >>> > printk is need when CONFIG_BPF_JIT is defined >>>>> >>> > or it will report pr_err and print_hex_dump are implicit declaration >>>>> >>> > >>>>> >>> > Signed-off-by: Chen Gang <gang.chen@asianux.com> >>> >> There are no references to pr_err nor print_hex_dump in this >>> >> filter.h header file. >> > >> > maybe we see the different tree. >> > I find it in next-20130328. > When you submit patches, always mention explicitly what tree you > want the patch applied to. > > In this case, this appropriate tree to be "net-next" since that > is the tree where the problem actually exists. I should notice, next time. > > Patch applied, thanks. > > thanks, too.
diff --git a/include/linux/filter.h b/include/linux/filter.h index d2059cb..d7d2508 100644 --- a/include/linux/filter.h +++ b/include/linux/filter.h @@ -50,6 +50,16 @@ extern int sk_get_filter(struct sock *sk, struct sock_filter __user *filter, uns #ifdef CONFIG_BPF_JIT extern void bpf_jit_compile(struct sk_filter *fp); extern void bpf_jit_free(struct sk_filter *fp); + +static inline void bpf_jit_dump(unsigned int flen, unsigned int proglen, + u32 pass, void *image) +{ + pr_err("flen=%u proglen=%u pass=%u image=%p\n", + flen, proglen, pass, image); + if (image) + print_hex_dump(KERN_ERR, "JIT code: ", DUMP_PREFIX_ADDRESS, + 16, 1, image, proglen, false); +} #define SK_RUN_FILTER(FILTER, SKB) (*FILTER->bpf_func)(SKB, FILTER->insns) #else static inline void bpf_jit_compile(struct sk_filter *fp)