From: Milan Lenco Date: Thu, 22 Jun 2017 07:10:07 +0000 (+0200) Subject: memif: If rx/tx_queues are unset, use default values X-Git-Tag: v17.10-rc1~397 X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=5fa72c8f1f8ec3304f6c8dc51662416b3b16936f;p=vpp.git memif: If rx/tx_queues are unset, use default values The standard behaviour for binary API is that zeroed arguments are treated as if the default values were set. This way the configurator does not need to know what default values are. This, however, wasn't the case for rx_queues and tx_queues in memif_create binary API message. Change-Id: Ib588b472823f6bbb5ef12a798d68b0dda6dd2139 Signed-off-by: Milan Lenco --- diff --git a/src/plugins/memif/memif_api.c b/src/plugins/memif/memif_api.c index 533e84822f2..65a4a09a09e 100644 --- a/src/plugins/memif/memif_api.c +++ b/src/plugins/memif/memif_api.c @@ -133,10 +133,20 @@ vl_api_memif_create_t_handler (vl_api_memif_create_t * mp) /* role */ args.is_master = (mp->role == 0); + + /* rx/tx queues */ if (args.is_master == 0) { - args.rx_queues = mp->rx_queues; - args.tx_queues = mp->tx_queues; + args.rx_queues = MEMIF_DEFAULT_RX_QUEUES; + args.tx_queues = MEMIF_DEFAULT_TX_QUEUES; + if (mp->rx_queues) + { + args.rx_queues = mp->rx_queues; + } + if (mp->tx_queues) + { + args.tx_queues = mp->tx_queues; + } } /* ring size */