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