* [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined
[not found] <5153DC24.4010104@asianux.com>
@ 2013-03-29 1:24 ` Chen Gang
2013-03-29 19:34 ` David Miller
0 siblings, 1 reply; 6+ messages in thread
From: Chen Gang @ 2013-03-29 1:24 UTC (permalink / raw)
To: edumazet, dborkman, wad, eparis; +Cc: David Miller, netdev
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>
---
include/linux/filter.h | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/include/linux/filter.h b/include/linux/filter.h
index d7d2508..d1248f4 100644
--- a/include/linux/filter.h
+++ b/include/linux/filter.h
@@ -48,6 +48,9 @@ extern int sk_chk_filter(struct sock_filter *filter, unsigned int flen);
extern int sk_get_filter(struct sock *sk, struct sock_filter __user *filter, unsigned len);
#ifdef CONFIG_BPF_JIT
+#include <linux/linkage.h>
+#include <linux/printk.h>
+
extern void bpf_jit_compile(struct sk_filter *fp);
extern void bpf_jit_free(struct sk_filter *fp);
--
1.7.7.6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined
2013-03-29 1:24 ` [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined Chen Gang
@ 2013-03-29 19:34 ` David Miller
2013-03-30 3:10 ` Chen Gang
2013-03-30 3:17 ` Chen Gang
0 siblings, 2 replies; 6+ messages in thread
From: David Miller @ 2013-03-29 19:34 UTC (permalink / raw)
To: gang.chen; +Cc: edumazet, dborkman, wad, eparis, netdev
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.
I think the issue is more like that the ARM bpf jit code uses
those routines, and therefore the ARM bpf jit code is what needs
to add these includes.
I'm not applying this patch, it isn't correct.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined
2013-03-29 19:34 ` David Miller
@ 2013-03-30 3:10 ` Chen Gang
2013-03-30 3:32 ` David Miller
2013-03-30 3:17 ` Chen Gang
1 sibling, 1 reply; 6+ messages in thread
From: Chen Gang @ 2013-03-30 3:10 UTC (permalink / raw)
To: David Miller; +Cc: edumazet, dborkman, wad, eparis, netdev
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.
please help check, thanks.
commit 79617801ea0c0e6664cb497d4c1892c2ff407364
Author: Daniel Borkmann <dborkman@redhat.com>
Date: Thu Mar 21 22:22:03 2013 +0100
filter: bpf_jit_comp: refactor and unify BPF JIT image dump output
If bpf_jit_enable > 1, then we dump the emitted JIT compiled image
after creation. Currently, only SPARC and PowerPC has similar output
as in the reference implementation on x86_64. Make a small helper
function in order to reduce duplicated code and make the dump output
uniform across architectures x86_64, SPARC, PPC, ARM (e.g. on ARM
flen, pass and proglen are currently not shown, but would be
interesting to know as well), also for future BPF JIT implementations
on other archs.
Cc: Mircea Gherzan <mgherzan@gmail.com>
Cc: Matt Evans <matt@ozlabs.org>
Cc: Eric Dumazet <eric.dumazet@google.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
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)
--
Chen Gang
Asianux Corporation
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined
2013-03-29 19:34 ` David Miller
2013-03-30 3:10 ` Chen Gang
@ 2013-03-30 3:17 ` Chen Gang
1 sibling, 0 replies; 6+ messages in thread
From: Chen Gang @ 2013-03-30 3:17 UTC (permalink / raw)
To: David Miller; +Cc: edumazet, dborkman, wad, eparis, netdev
On 2013年03月30日 03:34, David Miller wrote:
> I think the issue is more like that the ARM bpf jit code uses
> those routines, and therefore the ARM bpf jit code is what needs
> to add these includes.
for arm, I should continue to try to find new valuable issues.
:-)
--
Chen Gang
Asianux Corporation
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined
2013-03-30 3:10 ` Chen Gang
@ 2013-03-30 3:32 ` David Miller
2013-03-30 3:41 ` Chen Gang
0 siblings, 1 reply; 6+ messages in thread
From: David Miller @ 2013-03-30 3:32 UTC (permalink / raw)
To: gang.chen; +Cc: edumazet, dborkman, wad, eparis, netdev
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined
2013-03-30 3:32 ` David Miller
@ 2013-03-30 3:41 ` Chen Gang
0 siblings, 0 replies; 6+ messages in thread
From: Chen Gang @ 2013-03-30 3:41 UTC (permalink / raw)
To: David Miller; +Cc: edumazet, dborkman, wad, eparis, netdev
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.
--
Chen Gang
Asianux Corporation
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-03-30 3:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <5153DC24.4010104@asianux.com>
2013-03-29 1:24 ` [PATCH v2] include/linux: printk is needed in filter.h when CONFIG_BPF_JIT is defined Chen Gang
2013-03-29 19:34 ` David Miller
2013-03-30 3:10 ` Chen Gang
2013-03-30 3:32 ` David Miller
2013-03-30 3:41 ` Chen Gang
2013-03-30 3:17 ` Chen Gang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).