Yes, your patch also solved my problem.

  Thanks,
  Ken

 On Thu, 23 Jan 2014 17:24:27 +0000,
 [email protected] wrote
 in E-mail "Re: [ovs-discuss] About multiple ovs-vsctl execution":

> >   When two ovs-vsctl update map type column at same time,
> >   one ovs-vsctl's update will be discarded although all
> >   ovs-vsctl succeeded.
> 
> Good catch!
> 
> Does the following simpler patch also solve the problem for you?
> 
> Thanks,
> 
> Ben.
> 
> diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c
> index 528b40c..d09775f 100644
> --- a/utilities/ovs-vsctl.c
> +++ b/utilities/ovs-vsctl.c
> @@ -3352,6 +3352,8 @@ set_column(const struct vsctl_table_class *table,
>  
>          ovsdb_datum_union(&datum, ovsdb_idl_read(row, column),
>                            &column->type, false);
> +
> +        ovsdb_idl_txn_verify(row, column);
>          ovsdb_idl_txn_write(row, column, &datum);
>      } else {
>          struct ovsdb_datum datum;
_______________________________________________
discuss mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to