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

Reply via email to