Re: [ovs-dev] [PATCH v2 3/5] netdev-dpdk: Fix memory leak in dpdk_do_tx_copy().

2014-06-30 Thread Pravin Shelar
On Mon, Jun 30, 2014 at 9:58 AM, Daniele Di Proietto wrote: > This makes a lot of sense, thanks! > > Acked-by: Daniele Di Proietto > > On Jun 26, 2014, at 6:16 PM, Ryan Wilson wrote: > >> This patch fixes a bug where rte_pktmbuf_alloc() would fail and >> packets which succeeded to allocate memor

Re: [ovs-dev] [PATCH v2 3/5] netdev-dpdk: Fix memory leak in dpdk_do_tx_copy().

2014-06-30 Thread Daniele Di Proietto
This makes a lot of sense, thanks! Acked-by: Daniele Di Proietto On Jun 26, 2014, at 6:16 PM, Ryan Wilson wrote: > This patch fixes a bug where rte_pktmbuf_alloc() would fail and > packets which succeeded to allocate memory with rte_pktmbuf_alloc() > would not be sent and leak memory. > > Als

[ovs-dev] [PATCH v2 3/5] netdev-dpdk: Fix memory leak in dpdk_do_tx_copy().

2014-06-26 Thread Ryan Wilson
This patch fixes a bug where rte_pktmbuf_alloc() would fail and packets which succeeded to allocate memory with rte_pktmbuf_alloc() would not be sent and leak memory. Also, as a byproduct of using a local variable to record dropped packets, this reduces the locking of the netdev's mutex when multi