From: Wei Hu (Xavier) Date: Mon, 25 Nov 2019 09:00:53 +0000 (+0800) Subject: net/hns3: fix checking enough Tx BDs X-Git-Url: http://git.droids-corp.org/?p=dpdk.git;a=commitdiff_plain;h=8f64f2846d8b42de638aa0318c8c9f5ac784d926 net/hns3: fix checking enough Tx BDs In .tx_pkt_burst ops implementation function of hns3 PMD driver, there is one check whether there are enough BDs in the TX queue. If not, driver will stop sending the packets. Currently in the 'for' process loop, the next_to_use member of TX queue is not updated in time after processing BDs of one packet, which results in the invalid action of checking whether there are enough BDs and failure in sending packets. This patch fixes it by moving the assignment statment of the next_to_use member of TX queue to the place after porcessing TX BDs in the 'for' loop. Fixes: bba636698316 ("net/hns3: support Rx/Tx and related operations") Cc: stable@dpdk.org Signed-off-by: Hongbo Zheng Signed-off-by: Huisong Li Signed-off-by: Wei Hu (Xavier) --- diff --git a/drivers/net/hns3/hns3_rxtx.c b/drivers/net/hns3/hns3_rxtx.c index 34cb7faf9e..8166447131 100644 --- a/drivers/net/hns3/hns3_rxtx.c +++ b/drivers/net/hns3/hns3_rxtx.c @@ -1649,6 +1649,7 @@ hns3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts) } while (m_seg != NULL); nb_hold += i; + txq->next_to_use = tx_next_use; } end_of_tx: @@ -1656,7 +1657,6 @@ end_of_tx: if (likely(nb_tx)) { hns3_queue_xmit(txq, nb_hold); txq->next_to_clean = tx_next_clean; - txq->next_to_use = tx_next_use; txq->tx_bd_ready = tx_bd_ready - nb_hold; }