Thanks for the review. Pushed.
On Tue, Apr 29, 2014 at 2:13 PM, Pritesh Kothari (pritkoth) <[email protected]> wrote: > works for me here. > > Acked-by: Pritesh Kothari <[email protected]> > > On Apr 29, 2014, at 1:22 PM, Andy Zhou <[email protected]> wrote: > >> Caught by clang-3.5. >> >> Reported-by: Simon Horman <[email protected]> >> Signed-off-by: Andy Zhou <[email protected]> >> --- >> 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 >> [email protected] >> http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
