[ovs-dev] [PATCH 3/3] ovsdb-idl: Enhance conditional monitoring API

Ben Pfaff blp at ovn.org
Thu Jan 5 00:00:08 UTC 2017


On Tue, Dec 20, 2016 at 01:47:17AM -0800, Andy Zhou wrote:
> To allow client to know when the conditional monitoring changes
> has been accepted by the OVSDB server and the 'idl' contents has
> been updated to match the new conditions.
> 
> Signed-off-by: Andy Zhou <azhou at ovn.org>

The API seems a little suboptimal, in that:
        uint64_t seqno = ovsdb_idl_set_condition(idl, tc, &cond);
        uint64_t seqno2 = ovsdb_idl_set_condition(idl, tc, &cond);
        if (seqno2 == UINT_MAX) {
            ovs_fatal(0, "condition unchanged");
        }
will always abort with "condition unchanged", I think.

Is there a simple solution?


More information about the dev mailing list