[ovs-git] Open vSwitch: netdev-dpdk: Fix memory leak in dpdk_do_tx_copy(). (master)

dev at openvswitch.org dev at openvswitch.org
Mon Jun 30 19:04:49 UTC 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  175cf4de3f785d7b0e4c9d834aaad3dcc11c18fe (commit)
       via  844f2d749ab02b59218e869483e78725869c199f (commit)
       via  b170db2aaa65a6a77f18667685ce94c70a24941d (commit)
      from  3c49da46d0bfcb7def07c5bfba8d2fc6cc3074dd (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 175cf4de3f785d7b0e4c9d834aaad3dcc11c18fe
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=175cf4de3f785d7b0e4c9d834aaad3dcc11c18fe
Author: Ryan Wilson <wryan at nicira.com>
		
netdev-dpdk: Fix memory leak in dpdk_do_tx_copy().
		
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
multiple packets are dropped in dpdk_do_tx_copy().

Signed-off-by: Ryan Wilson <wryan at nicira.com>
Acked-by: Daniele Di Proietto <ddiproietto at vmware.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


commit 844f2d749ab02b59218e869483e78725869c199f
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=844f2d749ab02b59218e869483e78725869c199f
Author: Ryan Wilson <wryan at nicira.com>
		
netdev-dpdk: Set current timestamp when flushing TX queue.
		
The current timestamp should be set every time the queue is flushed.
Thus, if DRAIN_TSC timer cycles have passed since the last timestamp,
the send queue should be flushed again.

Signed-off-by: Ryan Wilson <wryan at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


commit b170db2aaa65a6a77f18667685ce94c70a24941d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=b170db2aaa65a6a77f18667685ce94c70a24941d
Author: Ryan Wilson <wryan at nicira.com>
		
netdev-dpdk: Refactor dpdk_queue_flush().
		
This patch refactors dpdk_queue_flush() to reuse code in
dpdk_queue_pkts().

Signed-off-by: Ryan Wilson <wryan at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 lib/netdev-dpdk.c |   68 ++++++++++++++++++++++++-----------------------------
 1 file changed, 31 insertions(+), 37 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list