* [PATCH -next] drivers: net: WIREGUARD depends on IPV6
@ 2020-02-25 6:39 Chen Zhou
2020-02-25 6:52 ` Jason A. Donenfeld
0 siblings, 1 reply; 3+ messages in thread
From: Chen Zhou @ 2020-02-25 6:39 UTC (permalink / raw)
To: davem, jiri, krzk, gregkh, Jason; +Cc: netdev, linux-kernel, chenzhou10
If CONFIG_IPV6 is n, build fails:
drivers/net/wireguard/device.o: In function `wg_xmit':
device.c:(.text+0xb2d): undefined reference to `icmpv6_ndo_send'
make: *** [vmlinux] Error 1
Set WIREGUARD depending on IPV6 to fix this.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Chen Zhou <chenzhou10@huawei.com>
---
drivers/net/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
index 25a8f93..824292e 100644
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -74,7 +74,7 @@ config DUMMY
config WIREGUARD
tristate "WireGuard secure network tunnel"
depends on NET && INET
- depends on IPV6 || !IPV6
+ depends on IPV6
select NET_UDP_TUNNEL
select DST_CACHE
select CRYPTO
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH -next] drivers: net: WIREGUARD depends on IPV6
2020-02-25 6:39 [PATCH -next] drivers: net: WIREGUARD depends on IPV6 Chen Zhou
@ 2020-02-25 6:52 ` Jason A. Donenfeld
2020-02-25 10:08 ` Jason A. Donenfeld
0 siblings, 1 reply; 3+ messages in thread
From: Jason A. Donenfeld @ 2020-02-25 6:52 UTC (permalink / raw)
To: Chen Zhou; +Cc: davem, jiri, krzk, gregkh, netdev, linux-kernel
On 2/25/20, Chen Zhou <chenzhou10@huawei.com> wrote:
> If CONFIG_IPV6 is n, build fails:
>
> drivers/net/wireguard/device.o: In function `wg_xmit':
> device.c:(.text+0xb2d): undefined reference to `icmpv6_ndo_send'
> make: *** [vmlinux] Error 1
>
> Set WIREGUARD depending on IPV6 to fix this.
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Chen Zhou <chenzhou10@huawei.com>
> ---
> drivers/net/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index 25a8f93..824292e 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -74,7 +74,7 @@ config DUMMY
> config WIREGUARD
> tristate "WireGuard secure network tunnel"
> depends on NET && INET
> - depends on IPV6 || !IPV6
> + depends on IPV6
Thanks for reporting the breakage. However, this is not the correct
fix, as wireguard should work without IPv6. Rather, the recent icmp
fixes changed something. I'll investigate why when I'm off the road in
several hours.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next] drivers: net: WIREGUARD depends on IPV6
2020-02-25 6:52 ` Jason A. Donenfeld
@ 2020-02-25 10:08 ` Jason A. Donenfeld
0 siblings, 0 replies; 3+ messages in thread
From: Jason A. Donenfeld @ 2020-02-25 10:08 UTC (permalink / raw)
To: Chen Zhou
Cc: David Miller, jiri, Krzysztof Kozlowski, Greg Kroah-Hartman,
Netdev, LKML
On Tue, Feb 25, 2020 at 2:52 PM Jason A. Donenfeld <Jason@zx2c4.com> wrote:
>
> On 2/25/20, Chen Zhou <chenzhou10@huawei.com> wrote:
> > If CONFIG_IPV6 is n, build fails:
> >
> > drivers/net/wireguard/device.o: In function `wg_xmit':
> > device.c:(.text+0xb2d): undefined reference to `icmpv6_ndo_send'
> > make: *** [vmlinux] Error 1
> >
> > Set WIREGUARD depending on IPV6 to fix this.
> >
> > Reported-by: Hulk Robot <hulkci@huawei.com>
> > Signed-off-by: Chen Zhou <chenzhou10@huawei.com>
> > ---
> > drivers/net/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> > index 25a8f93..824292e 100644
> > --- a/drivers/net/Kconfig
> > +++ b/drivers/net/Kconfig
> > @@ -74,7 +74,7 @@ config DUMMY
> > config WIREGUARD
> > tristate "WireGuard secure network tunnel"
> > depends on NET && INET
> > - depends on IPV6 || !IPV6
> > + depends on IPV6
>
> Thanks for reporting the breakage. However, this is not the correct
> fix, as wireguard should work without IPv6. Rather, the recent icmp
> fixes changed something. I'll investigate why when I'm off the road in
> several hours.
Off the road. Fix posted to mailing list:
https://lore.kernel.org/netdev/20200225100535.45146-1-Jason@zx2c4.com/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-02-25 10:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-02-25 6:39 [PATCH -next] drivers: net: WIREGUARD depends on IPV6 Chen Zhou
2020-02-25 6:52 ` Jason A. Donenfeld
2020-02-25 10:08 ` Jason A. Donenfeld
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).