[ovs-dev] [PATCH 3/5] datapath: Drop synchronize_rcu() in internal dev destroy.

Jesse Gross jesse at nicira.com
Fri Dec 10 20:42:55 UTC 2010


unregister_device() contains a call to synchronize_rcu(), so there
is no need to directly call it ourselves immediately beforehand.
We were relying on the call during unregistration anyways to stop
packets from being transmited on the device, so our version was
both misleading and had a performance penalty.

Signed-off-by: Jesse Gross <jesse at nicira.com>
---
 datapath/vport-internal_dev.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/datapath/vport-internal_dev.c b/datapath/vport-internal_dev.c
index c88fb3b..74a7dfd 100644
--- a/datapath/vport-internal_dev.c
+++ b/datapath/vport-internal_dev.c
@@ -219,8 +219,6 @@ static int internal_dev_destroy(struct vport *vport)
 	netif_stop_queue(netdev_vport->dev);
 	dev_set_promiscuity(netdev_vport->dev, -1);
 
-	synchronize_rcu();
-
 	unregister_netdevice(netdev_vport->dev);
 	vport_free(vport);
 
-- 
1.7.1





More information about the dev mailing list