Applied all three patches to master, thx for the review,
On Fri, Jun 13, 2014 at 5:41 PM, Alex Wang <al...@nicira.com> wrote:
> TXN_UNCOMMITTED should not be possible here (because we just committed
>> it). If you want to check for it (I probably wouldn't bother) then
>> I'd use an assertion:
>> > + /* Sets the 'status_txn_try_again' if the transaction
>> fails. */
>> > + if (status == TXN_SUCCESS || status == TXN_UNCOMMITTED) {
>> > + status_txn_try_again = false;
>> > + } else {
>> > + status_txn_try_again = true;
>> > + }
>>
>
>
>
> Thanks for catching this! It should be TXN_UNCHANGED.
>
>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
