datapath: Remove redundant synchronize_rcu() call.
authorBen Pfaff <blp@nicira.com>
Fri, 26 Jun 2009 19:20:02 +0000 (12:20 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 8 Jul 2009 21:13:15 +0000 (14:13 -0700)
There is no benefit to synchronizing twice, and it might cost us a lot of
time.

datapath/datapath.c

index fc73cac3d8336a71842671212274a973e4ffbfe5..6df0843781a6e9e5223809d6d33d27eb86d25034 100644 (file)
@@ -284,7 +284,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();