Thanks for the fix!

Acked-by: Daniele Di Proietto <ddiproie...@vmware.com>

On 9/15/14, 1:34 PM, "Alex Wang" <al...@nicira.com> wrote:

>With the separation of tx queue and rx queue configuration
>in netdev-dpdk module, the netdev_dpdk_get_config() can no
>longer report 'n_rxq' as tx queue configuration.
>
>This commit fixes the above issue.
>
>Reported-by: Daniele Di Proietto <ddiproie...@vmware.com>
>Signed-off-by: Alex Wang <al...@nicira.com>
>---
> lib/netdev-dpdk.c |    5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
>diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
>index 0a6a398..e66eb6e 100644
>--- a/lib/netdev-dpdk.c
>+++ b/lib/netdev-dpdk.c
>@@ -593,9 +593,8 @@ netdev_dpdk_get_config(const struct netdev *netdev_,
>struct smap *args)
> 
>     ovs_mutex_lock(&dev->mutex);
> 
>-    /* XXX: Allow to configure number of queues. */
>-    smap_add_format(args, "configured_rx_queues", "%u", netdev_->n_rxq);
>-    smap_add_format(args, "configured_tx_queues", "%u", netdev_->n_rxq);
>+    smap_add_format(args, "configured_rx_queues", "%d", netdev_->n_rxq);
>+    smap_add_format(args, "configured_tx_queues", "%d", netdev_->n_txq);
>     ovs_mutex_unlock(&dev->mutex);
> 
>     return 0;
>-- 
>1.7.9.5
>
>_______________________________________________
>dev mailing list
>dev@openvswitch.org
>https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailman/
>listinfo/dev&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=MV9BdLjtFIdhBDBaw5z%2BU
>6SSA2gAfY4L%2F1HCy3VjlKU%3D%0A&m=YI2MU7%2BHXQreVVVOV%2BWL1k5hDbwUSrD%2BEc4
>YECUIUgE%3D%0A&s=7b2663a8324d8e386082e05ff2a8093f2e211c0c2f3c182105b546285
>5cdd170

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to