DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] netvsc: free ring with rte_ring_free
@ 2018-08-01 23:18 Stephen Hemminger
  2018-08-04 23:53 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2018-08-01 23:18 UTC (permalink / raw)
  To: dev; +Cc: Stephen Hemminger, Stephen Hemminger

The internal receive ring should be freed with rte_ring_free
not rte_free.

Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device")
Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
---
 drivers/net/netvsc/hn_rxtx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/netvsc/hn_rxtx.c b/drivers/net/netvsc/hn_rxtx.c
index 6d2f41c4c011..400598a621f1 100644
--- a/drivers/net/netvsc/hn_rxtx.c
+++ b/drivers/net/netvsc/hn_rxtx.c
@@ -774,7 +774,7 @@ hn_dev_rx_queue_setup(struct rte_eth_dev *dev,
 	return 0;
 
 fail:
-	rte_free(rxq->rx_ring);
+	rte_ring_free(rxq->rx_ring);
 	rte_free(rxq->event_buf);
 	rte_free(rxq);
 	return -ENOMEM;
@@ -790,7 +790,7 @@ hn_dev_rx_queue_release(void *arg)
 	if (!rxq)
 		return;
 
-	rte_free(rxq->rx_ring);
+	rte_ring_free(rxq->rx_ring);
 	rxq->rx_ring = NULL;
 	rxq->mb_pool = NULL;
 
-- 
2.18.0

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] netvsc: free ring with rte_ring_free
  2018-08-01 23:18 [dpdk-dev] [PATCH] netvsc: free ring with rte_ring_free Stephen Hemminger
@ 2018-08-04 23:53 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2018-08-04 23:53 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: dev, Stephen Hemminger

02/08/2018 01:18, Stephen Hemminger:
> The internal receive ring should be freed with rte_ring_free
> not rte_free.
> 
> Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device")
> Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-08-04 23:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-01 23:18 [dpdk-dev] [PATCH] netvsc: free ring with rte_ring_free Stephen Hemminger
2018-08-04 23:53 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).