Re: [ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-03-06 Thread Ales Musil
On Fri, Mar 3, 2023 at 4:00 PM Roi Dayan wrote: > > > On 01/03/2023 14:01, Ales Musil wrote: > > On Wed, Mar 1, 2023 at 12:34 PM Roi Dayan wrote: > > > >> > >> > >> On 01/03/2023 12:53, Roi Dayan via dev wrote: > >>> > >>> > >>> On 16/01/2023 13:45, Ales Musil wrote: > Currently, the CT

Re: [ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-03-03 Thread Roi Dayan via dev
On 01/03/2023 14:01, Ales Musil wrote: > On Wed, Mar 1, 2023 at 12:34 PM Roi Dayan wrote: > >> >> >> On 01/03/2023 12:53, Roi Dayan via dev wrote: >>> >>> >>> On 16/01/2023 13:45, Ales Musil wrote: Currently, the CT can be flushed by dpctl only by specifying the whole 5-tuple. This

Re: [ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-03-01 Thread Ales Musil
On Wed, Mar 1, 2023 at 12:34 PM Roi Dayan wrote: > > > On 01/03/2023 12:53, Roi Dayan via dev wrote: > > > > > > On 16/01/2023 13:45, Ales Musil wrote: > >> Currently, the CT can be flushed by dpctl only by specifying > >> the whole 5-tuple. This is not very convenient when there are > >> only

Re: [ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-03-01 Thread Roi Dayan via dev
On 01/03/2023 12:53, Roi Dayan via dev wrote: > > > On 16/01/2023 13:45, Ales Musil wrote: >> Currently, the CT can be flushed by dpctl only by specifying >> the whole 5-tuple. This is not very convenient when there are >> only some fields known to the user of CT flush. Add new struct >>

Re: [ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-03-01 Thread Roi Dayan via dev
On 16/01/2023 13:45, Ales Musil wrote: > Currently, the CT can be flushed by dpctl only by specifying > the whole 5-tuple. This is not very convenient when there are > only some fields known to the user of CT flush. Add new struct > ofputil_ct_match which represents the generic filtering that

Re: [ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-01-16 Thread 0-day Robot
Bleep bloop. Greetings Ales Musil, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 106 characters long (recommended limit is 79) #619 FILE: lib/dpctl.man:305:

[ovs-dev] [PATCH v7 1/2] ofp, dpif: Allow CT flush based on partial match

2023-01-16 Thread Ales Musil
Currently, the CT can be flushed by dpctl only by specifying the whole 5-tuple. This is not very convenient when there are only some fields known to the user of CT flush. Add new struct ofputil_ct_match which represents the generic filtering that can be done for CT flush. The match is done only on