[ovs-dev] [MASTER 3/3] bridge: Make ovs-vswitchd run again if status_txn commit fails.

Alex Wang alexw at nicira.com
Sat Jun 14 00:41:02 UTC 2014


>
> 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140613/31f35e1b/attachment-0005.html>


More information about the dev mailing list