[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20100120084910.a869aca8.randy.dunlap@oracle.com>
Date: Wed, 20 Jan 2010 08:49:10 -0800
From: Randy Dunlap <randy.dunlap@...cle.com>
To: "Michael S. Tsirkin" <mst@...hat.com>
Cc: Rusty Russell <rusty@...tcorp.com.au>,
"David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org,
Stephen Rothwell <sfr@...b.auug.org.au>
Subject: Re: [PATCH] vhost: fix TUN=m VHOST_NET=y
On Wed, 20 Jan 2010 13:30:38 +0200 Michael S. Tsirkin wrote:
> drivers/built-in.o: In function `get_tun_socket':
> net.c:(.text+0x15436e): undefined reference to `tun_get_socket'
>
> If tun is a module, vhost must be a module, too.
> If tun is built-in or disabled, vhost can be built-in.
>
> Note: TUN || !TUN might look a bit strange until you realize
> that boolean logic rules do not apply for tristate variables.
>
> Reported-by: Randy Dunlap <randy.dunlap@...cle.com>
> Signed-off-by: Michael S. Tsirkin <mst@...hat.com>
Acked-by: Randy Dunlap <randy.dunlap@...cle.com>
Thanks.
> ---
>
> The above was already applied by Rusty, but got lost.
> The patch is on top of net-next.
>
> drivers/vhost/Kconfig | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/vhost/Kconfig b/drivers/vhost/Kconfig
> index 9f409f4..9e93553 100644
> --- a/drivers/vhost/Kconfig
> +++ b/drivers/vhost/Kconfig
> @@ -1,6 +1,6 @@
> config VHOST_NET
> tristate "Host kernel accelerator for virtio net (EXPERIMENTAL)"
> - depends on NET && EVENTFD && EXPERIMENTAL
> + depends on NET && EVENTFD && (TUN || !TUN) && EXPERIMENTAL
> ---help---
> This kernel module can be loaded in host kernel to accelerate
> guest networking with virtio_net. Not to be confused with virtio_net
> --
> 1.6.6.144.g5c3af
---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists