On 6/28/2019 10:22 AM, Ian Stokes wrote:
On 6/25/2019 3:57 PM, Kevin Traynor wrote:
Fix minor issue of one possible additional retry.

Fixes: c6ec9d176dbf ("netdev-dpdk: Fix vHost stats.")
Signed-off-by: Kevin Traynor <[email protected]>
Acked-by: Eelco Chaudron <[email protected]>
---
  lib/netdev-dpdk.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index 4856c56aa..0f3b9c6f4 100644
--- a/lib/netdev-dpdk.c
+++ b/lib/netdev-dpdk.c
@@ -2353,5 +2353,5 @@ __netdev_dpdk_vhost_send(struct netdev *netdev, int qid,
              break;
          }
-    } while (cnt && (retries++ <= VHOST_ENQ_RETRY_NUM));
+    } while (cnt && (retries++ < VHOST_ENQ_RETRY_NUM));
      rte_spinlock_unlock(&dev->tx_q[qid].tx_lock);

Looks OK to me, can be backported to 2.7 as far as I can see as well. Applied an pushed.


Apologies, just spotted I've replied on the v2 thread, should be the v3.

Ian
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to