> If NUMA information can't be derived from a vHost User device, only print
> an error if the VHOST_NUMA option is enabled in DPDK. Otherwise 'fail'
> silently.
>
> Fixes: 0a0f39df1d5a ("netdev-dpdk: Add support for DPDK 16.07")
> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com>
> Reported-by: Ian Stokes <ian.sto...@intel.com>
> ---
> acinclude.m4 | 3 ++-
> lib/netdev-dpdk.c | 2 ++
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/acinclude.m4 b/acinclude.m4 index 5a6dca7..1b05f8d 100644
> --- a/acinclude.m4
> +++ b/acinclude.m4
> @@ -207,7 +207,8 @@ AC_DEFUN([OVS_CHECK_DPDK], [
> ], [])
> ], [],
> [AC_SEARCH_LIBS([get_mempolicy],[numa],[],[AC_MSG_ERROR([unable to
> find libnuma, install the dependency package])])
> - DPDK_EXTRA_LIB="-lnuma"])
> + DPDK_EXTRA_LIB="-lnuma"
> + AC_DEFINE([VHOST_NUMA], [1], [NUMA Aware vHost support detected
> + in DPDK.])])
>
> AC_COMPILE_IFELSE([
> AC_LANG_PROGRAM(
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index e5f2cdd..f7e844d
> 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -2357,8 +2357,10 @@ new_device(int vid)
> /* Get NUMA information */
> newnode = rte_vhost_get_numa_node(vid);
> if (newnode == -1) {
> +#ifdef VHOST_NUMA
> VLOG_INFO("Error getting NUMA info for vHost Device
> '%s'",
> ifname);
> +#endif
> newnode = dev->socket_id;
> }
>
> --
Thanks for the patch Ciara.
Tested-by: Ian Stokes <ian.sto...@intel.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
