From: Ben Pfaff <blp@nicira.com>
Date: Fri, 26 Jun 2009 19:20:02 +0000 (-0700)
Subject: datapath: Remove redundant synchronize_rcu() call.
X-Git-Tag: v0.90.2~3
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=312b427dab326ffacfc64daaa57ed4d31595302b;p=sliver-openvswitch.git

datapath: Remove redundant synchronize_rcu() call.

There is no benefit to synchronizing twice, and it might cost us a lot of
time.
---

diff --git a/datapath/datapath.c b/datapath/datapath.c
index 2a9616137..e62eaea13 100644
--- a/datapath/datapath.c
+++ b/datapath/datapath.c
@@ -285,7 +285,6 @@ static void do_destroy_dp(struct datapath *dp, struct list_head *dp_devs)
 		dp_del_port(p, dp_devs);
 
 	rcu_assign_pointer(dps[dp->dp_idx], NULL);
-	synchronize_rcu();
 
 	/* Wait until no longer in use, then destroy it. */
 	synchronize_rcu();