From: Yury Kylulin Date: Mon, 29 Aug 2016 16:50:47 +0000 (+0300) Subject: net/ixgbe: fix mbuf leak during Rx queue release X-Git-Tag: spdx-start~5895 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=f853230920a9be609a87e4cb7335c359c4945744;p=dpdk.git net/ixgbe: fix mbuf leak during Rx queue release For the vector PMD, release all mbufs from the Rx queue if no packets are received after device start. Fixes: 11b220c6498d ("ixgbe: fix release queue mbufs") Signed-off-by: Yury Kylulin Acked-by: Wenzhuo Lu --- diff --git a/drivers/net/ixgbe/ixgbe_rxtx_vec_common.h b/drivers/net/ixgbe/ixgbe_rxtx_vec_common.h index 62b82013f6..3c3c0095bc 100644 --- a/drivers/net/ixgbe/ixgbe_rxtx_vec_common.h +++ b/drivers/net/ixgbe/ixgbe_rxtx_vec_common.h @@ -204,8 +204,20 @@ _ixgbe_rx_queue_release_mbufs_vec(struct ixgbe_rx_queue *rxq) return; /* free all mbufs that are valid in the ring */ - for (i = rxq->rx_tail; i != rxq->rxrearm_start; i = (i + 1) & mask) - rte_pktmbuf_free_seg(rxq->sw_ring[i].mbuf); + if (rxq->rxrearm_nb == 0) { + for (i = 0; i < rxq->nb_rx_desc; i++) { + if (rxq->sw_ring[i].mbuf != NULL) + rte_pktmbuf_free_seg(rxq->sw_ring[i].mbuf); + } + } else { + for (i = rxq->rx_tail; + i != rxq->rxrearm_start; + i = (i + 1) & mask) { + if (rxq->sw_ring[i].mbuf != NULL) + rte_pktmbuf_free_seg(rxq->sw_ring[i].mbuf); + } + } + rxq->rxrearm_nb = rxq->nb_rx_desc; /* set all entries to NULL */