Thanks for the review. Pushed.
On Tue, Apr 29, 2014 at 2:13 PM, Pritesh Kothari (pritkoth) <pritk...@cisco.com> wrote: > works for me here. > > Acked-by: Pritesh Kothari <pritesh.koth...@cisco.com> > > On Apr 29, 2014, at 1:22 PM, Andy Zhou <az...@nicira.com> wrote: > >> Caught by clang-3.5. >> >> Reported-by: Simon Horman <ho...@verge.net.au> >> Signed-off-by: Andy Zhou <az...@nicira.com> >> --- >> ofproto/bond.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/ofproto/bond.c b/ofproto/bond.c >> index f5a9d47..c1a8448 100644 >> --- a/ofproto/bond.c >> +++ b/ofproto/bond.c >> @@ -1180,10 +1180,11 @@ bond_rebalance(struct bond *bond) >> e->tx_bytes /= 2; >> } >> >> -done: >> if (use_recirc && rebalanced) { >> bond_update_post_recirc_rules(bond,true); >> } >> + >> +done: >> ovs_rwlock_unlock(&rwlock); >> } >> >> -- >> 1.9.1 >> >> _______________________________________________ >> dev mailing list >> dev@openvswitch.org >> http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev