[dpdk-dev] [PATCH v2 5/7] net/mlx4: remove unnecessary variables in Tx burst

[Ophir Munk]

From: Matan Azrad <ma...@mellanox.com>

Remove usage of variables which doesn't add new information for
performance improvement.

Signed-off-by: Matan Azrad <ma...@mellanox.com>
---
 drivers/net/mlx4/mlx4_rxtx.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/net/mlx4/mlx4_rxtx.c b/drivers/net/mlx4/mlx4_rxtx.c
index 014a6d3..e8d9a35 100644
--- a/drivers/net/mlx4/mlx4_rxtx.c
+++ b/drivers/net/mlx4/mlx4_rxtx.c
@@ -285,8 +285,6 @@ mlx4_tx_burst(void *dpdk_txq, struct rte_mbuf **pkts, 
uint16_t pkts_n)
        struct txq *txq = (struct txq *)dpdk_txq;
        unsigned int elts_head = txq->elts_head;
        const unsigned int elts_n = txq->elts_n;
-       unsigned int elts_comp = 0;
-       unsigned int bytes_sent = 0;
        unsigned int i;
        unsigned int max;
        struct mlx4_sq *sq = &txq->msq;
@@ -498,8 +496,7 @@ mlx4_tx_burst(void *dpdk_txq, struct rte_mbuf **pkts, 
uint16_t pkts_n)
                                                       MLX4_BIT_WQE_OWN : 0));
                sq->head += nr_txbbs;
                elt->buf = buf;
-               bytes_sent += buf->pkt_len;
-               ++elts_comp;
+               txq->stats.obytes += buf->pkt_len;
                elts_head = elts_head_next;
        }
        /* Take a shortcut if nothing must be sent. */
@@ -507,13 +504,12 @@ mlx4_tx_burst(void *dpdk_txq, struct rte_mbuf **pkts, 
uint16_t pkts_n)
                return 0;
        /* Increment send statistics counters. */
        txq->stats.opackets += i;
-       txq->stats.obytes += bytes_sent;
        /* Make sure that descriptors are written before doorbell record. */
        rte_wmb();
        /* Ring QP doorbell. */
        rte_write32(txq->msq.doorbell_qpn, txq->msq.db);
        txq->elts_head = elts_head;
-       txq->elts_comp += elts_comp;
+       txq->elts_comp += i;
        return i;
 }
 
-- 
2.7.4