On 10/7/25 2:40 PM, Ales Musil wrote:
> 
> 
> On Tue, Oct 7, 2025 at 2:32 PM Ilya Maximets <[email protected] 
> <mailto:[email protected]>> wrote:
> 
>     On 10/7/25 1:57 PM, Ales Musil via dev wrote:
>     > Add a way to assert IDL txn as read only, that is useful in cases when
>     > we want to ensure that the program doesn't write anything into the
>     > txn. It is done via assert because this is considered a bug which
>     > should be fixed in the IDL caller.
>     >
>     > Signed-off-by: Ales Musil <[email protected] <mailto:[email protected]>>
>     > ---
>     > v4: Remove acks.
>     >     Fix write to index row.
> 
>     I'm a little confused by this.  ovsdb_idl_txn_write__() should not be
>     used for writing into index rows.  Index rows supposed to be updated
>     via ovsdb_idl_index_write() instead.  What am I missing here?
> 
> 
> 
> Yeah I will be sending fix for ovn about this, there were places where
> set was used instead of index_set.

I guess, that just confirms that assertion is useful. :)

> I've got confused in the process.
> So I guess we can still go with the v3?

I'll take a look at v3.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to