There is a inconsistent ordering in function ovs_vport_cmd_set()
between upstream and out of tree ovs module. Following patch
fixes it by releasing lock before calling ovs_notify.

Signed-off-by: Pravin B Shelar <pshe...@nicira.com>
---
 datapath/datapath.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/datapath/datapath.c b/datapath/datapath.c
index a8bb5b8..3cb58b0 100644
--- a/datapath/datapath.c
+++ b/datapath/datapath.c
@@ -2130,9 +2130,8 @@ static int ovs_vport_cmd_set(struct sk_buff *skb, struct 
genl_info *info)
                                      info->snd_seq, 0, OVS_VPORT_CMD_NEW);
        BUG_ON(err < 0);
 
-       ovs_notify(reply, info, &ovs_dp_vport_multicast_group);
-
        ovs_unlock();
+       ovs_notify(reply, info, &ovs_dp_vport_multicast_group);
        return 0;
 
 exit_free:
-- 
1.7.1

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to