diff mbox

net, filter: make new test_bpf depend on networking

Message ID alpine.DEB.2.02.1405131255380.3392@chino.kir.corp.google.com
State Not Applicable, archived
Delegated to: David Miller
Headers show

Commit Message

David Rientjes May 13, 2014, 7:57 p.m. UTC
CONFIG_TEST_BPF depends on CONFIG_NET, otherwise the build fails:

ERROR: "sk_unattached_filter_destroy" [lib/test_bpf.ko] undefined!
ERROR: "kfree_skb" [lib/test_bpf.ko] undefined!
ERROR: "sk_unattached_filter_create" [lib/test_bpf.ko] undefined!
ERROR: "sk_run_filter_int_skb" [lib/test_bpf.ko] undefined!
ERROR: "__alloc_skb" [lib/test_bpf.ko] undefined!

Add the dependency.

Cc: Alexei Starovoitov <ast@plumgrid.com>
Signed-off-by: David Rientjes <rientjes@google.com>
---
 lib/Kconfig.debug | 1 +
 1 file changed, 1 insertion(+)

--
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

Comments

Alexei Starovoitov May 13, 2014, 8:07 p.m. UTC | #1
On Tue, May 13, 2014 at 12:57 PM, David Rientjes <rientjes@google.com> wrote:
> CONFIG_TEST_BPF depends on CONFIG_NET, otherwise the build fails:
>
> ERROR: "sk_unattached_filter_destroy" [lib/test_bpf.ko] undefined!
> ERROR: "kfree_skb" [lib/test_bpf.ko] undefined!
> ERROR: "sk_unattached_filter_create" [lib/test_bpf.ko] undefined!
> ERROR: "sk_run_filter_int_skb" [lib/test_bpf.ko] undefined!
> ERROR: "__alloc_skb" [lib/test_bpf.ko] undefined!
>
> Add the dependency.
>
> Cc: Alexei Starovoitov <ast@plumgrid.com>
> Signed-off-by: David Rientjes <rientjes@google.com>

commit 98920ba6911c7 ("net: fix test_bpf build to depend on NET")
is already in net-next.
Sorry about the breakage.

> ---
>  lib/Kconfig.debug | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -1639,6 +1639,7 @@ config TEST_USER_COPY
>  config TEST_BPF
>         tristate "Test BPF filter functionality"
>         default n
> +       depends on NET
>         depends on m
>         help
>           This builds the "test_bpf" module that runs various test vectors
--
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
David Miller May 13, 2014, 9:13 p.m. UTC | #2
From: David Rientjes <rientjes@google.com>
Date: Tue, 13 May 2014 12:57:56 -0700 (PDT)

> CONFIG_TEST_BPF depends on CONFIG_NET, otherwise the build fails:
> 
> ERROR: "sk_unattached_filter_destroy" [lib/test_bpf.ko] undefined!
> ERROR: "kfree_skb" [lib/test_bpf.ko] undefined!
> ERROR: "sk_unattached_filter_create" [lib/test_bpf.ko] undefined!
> ERROR: "sk_run_filter_int_skb" [lib/test_bpf.ko] undefined!
> ERROR: "__alloc_skb" [lib/test_bpf.ko] undefined!
> 
> Add the dependency.
> 
> Cc: Alexei Starovoitov <ast@plumgrid.com>
> Signed-off-by: David Rientjes <rientjes@google.com>

I just applied a patch from Randy Dunlap that does the same thing,
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
diff mbox

Patch

diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -1639,6 +1639,7 @@  config TEST_USER_COPY
 config TEST_BPF
 	tristate "Test BPF filter functionality"
 	default n
+	depends on NET
 	depends on m
 	help
 	  This builds the "test_bpf" module that runs various test vectors