[ovs-dev] [PATCH v3] netdev-dpdk: add dpdk vhost-user ports

Flavio Leitner fbl at sysclose.org
Wed May 13 13:09:49 UTC 2015


On Wed, May 13, 2015 at 11:28:50AM +0300, Panu Matilainen wrote:
> On 05/11/2015 01:56 PM, Ciara Loftus wrote:
> >This patch adds support for a new port type to the userspace
> >datapath called dpdkvhostuser.
> >
> >A new dpdkvhostuser port will create a unix domain socket which
> >when provided to QEMU is used to facilitate communication between
> >the virtio-net device on the VM and the OVS port on the host.
> >
> >vhost-cuse ('dpdkvhost') ports are still available, and will be
> >enabled if vhost-cuse support is detected in the DPDK build
> >specified during compilation of the switch. Otherwise, vhost-user
> >ports are enabled.
> >
> >Signed-off-by: Ciara Loftus <ciara.loftus at intel.com>
> >---
> [...]
> >diff --git a/acinclude.m4 b/acinclude.m4
> >index e9d0ed9..2873480 100644
> >--- a/acinclude.m4
> >+++ b/acinclude.m4
> >@@ -218,6 +218,9 @@ AC_DEFUN([OVS_CHECK_DPDK], [
> >      DPDK_vswitchd_LDFLAGS=-Wl,--whole-archive,$DPDK_LIB,--no-whole-archive
> >      AC_SUBST([DPDK_vswitchd_LDFLAGS])
> >      AC_DEFINE([DPDK_NETDEV], [1], [System uses the DPDK module.])
> >+
> >+    OVS_GREP_IFELSE([$RTE_SDK/include/rte_config.h], [define RTE_LIBRTE_VHOST_USER 1],
> >+                    [], [AC_DEFINE([VHOST_CUSE], [1], [DPDK vhost-cuse support enabled, vhost-user disabled.])])
> >    else
> >      RTE_SDK=
> >    fi
> 
> This isn't really needed, you could just include rte_config.h from
> netdev-dpdk.c ... but maybe this is better afterall as it leaves a trace in
> the build log as to which version was chosen.

Leave it there please.  It's useful to check right at the beginning
what is going to be compiled and doesn't cost anything.

fbl




More information about the dev mailing list