From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrien Mazarguil Subject: [PATCH v1 3/3] net/hyperv: add "force" parameter Date: Mon, 18 Dec 2017 17:46:25 +0100 Message-ID: <20171218162443.12971-4-adrien.mazarguil@6wind.com> References: <20171124172132.GW4062@6wind.com> <20171218162443.12971-1-adrien.mazarguil@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, Stephen Hemminger To: Ferruh Yigit Return-path: Received: from mail-wr0-f196.google.com (mail-wr0-f196.google.com [209.85.128.196]) by dpdk.org (Postfix) with ESMTP id 799E31B00C for ; Mon, 18 Dec 2017 17:46:38 +0100 (CET) Received: by mail-wr0-f196.google.com with SMTP id s66so14780907wrc.9 for ; Mon, 18 Dec 2017 08:46:38 -0800 (PST) Content-Disposition: inline In-Reply-To: <20171218162443.12971-1-adrien.mazarguil@6wind.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This parameter allows specifying any non-NetVSC interface to use with tap sub-devices for development purposes. Signed-off-by: Adrien Mazarguil --- doc/guides/nics/hyperv.rst | 5 +++++ drivers/net/hyperv/hyperv.c | 26 +++++++++++++++++++------- 2 files changed, 24 insertions(+), 7 deletions(-) diff --git a/doc/guides/nics/hyperv.rst b/doc/guides/nics/hyperv.rst index 8f7a8b153..9b5220919 100644 --- a/doc/guides/nics/hyperv.rst +++ b/doc/guides/nics/hyperv.rst @@ -110,5 +110,10 @@ The following device parameters are supported: Same as ``iface`` except a suitable NetVSC interface is located using its MAC address. +- ``force`` [int] + + If nonzero, forces the use of specified interfaces even if not detected as + NetVSC. + Not specifying either ``iface`` or ``mac`` makes this PMD attach itself to all NetVSC interfaces found on the system. diff --git a/drivers/net/hyperv/hyperv.c b/drivers/net/hyperv/hyperv.c index bad224be9..d9d9bbcd5 100644 --- a/drivers/net/hyperv/hyperv.c +++ b/drivers/net/hyperv/hyperv.c @@ -62,6 +62,7 @@ #define HYPERV_DRIVER net_hyperv #define HYPERV_ARG_IFACE "iface" #define HYPERV_ARG_MAC "mac" +#define HYPERV_ARG_FORCE "force" #define HYPERV_PROBE_MS 1000 #define NETVSC_CLASS_ID "{f8615163-df3e-46c5-913f-f2d2f965ed0e}" @@ -504,6 +505,9 @@ hyperv_alarm(void *arg) * - struct rte_kvargs *kvargs: * Device arguments provided to current driver instance. * + * - int force: + * Accept specified interface even if not detected as NetVSC. + * * - unsigned int specified: * Number of specific netdevices provided as device arguments. * @@ -521,6 +525,7 @@ hyperv_netvsc_probe(const struct if_nameindex *iface, { const char *name = va_arg(ap, const char *); struct rte_kvargs *kvargs = va_arg(ap, struct rte_kvargs *); + int force = va_arg(ap, int); unsigned int specified = va_arg(ap, unsigned int); unsigned int *matched = va_arg(ap, unsigned int *); unsigned int i; @@ -567,9 +572,11 @@ hyperv_netvsc_probe(const struct if_nameindex *iface, if (!hyperv_iface_is_netvsc(iface)) { if (!specified) return 0; - WARN("interface \"%s\" (index %u) is not NetVSC, skipping", - iface->if_name, iface->if_index); - return 0; + WARN("interface \"%s\" (index %u) is not NetVSC, %s", + iface->if_name, iface->if_index, + force ? "using anyway (forced)" : "skipping"); + if (!force) + return 0; } /* Create interface context. */ ctx = calloc(1, sizeof(*ctx)); @@ -700,6 +707,7 @@ hyperv_vdev_probe(struct rte_vdev_device *dev) static const char *const hyperv_arg[] = { HYPERV_ARG_IFACE, HYPERV_ARG_MAC, + HYPERV_ARG_FORCE, NULL, }; const char *name = rte_vdev_device_name(dev); @@ -708,6 +716,7 @@ hyperv_vdev_probe(struct rte_vdev_device *dev) hyperv_arg); unsigned int specified = 0; unsigned int matched = 0; + int force = 0; unsigned int i; int ret; @@ -719,13 +728,15 @@ hyperv_vdev_probe(struct rte_vdev_device *dev) for (i = 0; i != kvargs->count; ++i) { const struct rte_kvargs_pair *pair = &kvargs->pairs[i]; - if (!strcmp(pair->key, HYPERV_ARG_IFACE) || - !strcmp(pair->key, HYPERV_ARG_MAC)) + if (!strcmp(pair->key, HYPERV_ARG_FORCE)) + force = !!atoi(pair->value); + else if (!strcmp(pair->key, HYPERV_ARG_IFACE) || + !strcmp(pair->key, HYPERV_ARG_MAC)) ++specified; } rte_eal_alarm_cancel(hyperv_alarm, NULL); /* Gather interfaces. */ - ret = hyperv_foreach_iface(hyperv_netvsc_probe, name, kvargs, + ret = hyperv_foreach_iface(hyperv_netvsc_probe, name, kvargs, force, specified, &matched); if (ret < 0) goto error; @@ -784,4 +795,5 @@ RTE_PMD_REGISTER_VDEV(HYPERV_DRIVER, hyperv_vdev); RTE_PMD_REGISTER_ALIAS(HYPERV_DRIVER, eth_hyperv); RTE_PMD_REGISTER_PARAM_STRING(net_hyperv, HYPERV_ARG_IFACE "= " - HYPERV_ARG_MAC "="); + HYPERV_ARG_MAC "= " + HYPERV_ARG_FORCE "="); -- 2.11.0