Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-04-06 Thread Ilya Maximets
On 4/5/22 12:33, Ilya Maximets wrote: > On 4/5/22 12:24, Eelco Chaudron wrote: >> >> >> On 4 Apr 2022, at 21:05, Ilya Maximets wrote: >> >>> On 3/30/22 11:35, Eelco Chaudron wrote: On 29 Mar 2022, at 13:06, Ilya Maximets wrote: > On 2/25/22 13:29, Marcelo Ricardo Leitner wro

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-04-05 Thread Ilya Maximets
On 4/5/22 12:24, Eelco Chaudron wrote: > > > On 4 Apr 2022, at 21:05, Ilya Maximets wrote: > >> On 3/30/22 11:35, Eelco Chaudron wrote: >>> >>> >>> On 29 Mar 2022, at 13:06, Ilya Maximets wrote: >>> On 2/25/22 13:29, Marcelo Ricardo Leitner wrote: > On Thu, Feb 24, 2022 at 03:20:23PM +0

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-04-05 Thread Eelco Chaudron
On 4 Apr 2022, at 21:05, Ilya Maximets wrote: > On 3/30/22 11:35, Eelco Chaudron wrote: >> >> >> On 29 Mar 2022, at 13:06, Ilya Maximets wrote: >> >>> On 2/25/22 13:29, Marcelo Ricardo Leitner wrote: On Thu, Feb 24, 2022 at 03:20:23PM +0100, Eelco Chaudron wrote: > > > On 23 Feb

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-04-04 Thread Ilya Maximets
On 3/30/22 11:35, Eelco Chaudron wrote: > > > On 29 Mar 2022, at 13:06, Ilya Maximets wrote: > >> On 2/25/22 13:29, Marcelo Ricardo Leitner wrote: >>> On Thu, Feb 24, 2022 at 03:20:23PM +0100, Eelco Chaudron wrote: On 23 Feb 2022, at 17:23, Marcelo Ricardo Leitner wrote:

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-03-31 Thread Marcelo Ricardo Leitner
On Tue, Mar 29, 2022 at 01:06:36PM +0200, Ilya Maximets wrote: > On 2/25/22 13:29, Marcelo Ricardo Leitner wrote: > > On Thu, Feb 24, 2022 at 03:20:23PM +0100, Eelco Chaudron wrote: > >> > >> > >> On 23 Feb 2022, at 17:23, Marcelo Ricardo Leitner wrote: > >> > >>> On Tue, Feb 22, 2022 at 04:26:10PM

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-03-30 Thread Eelco Chaudron
On 29 Mar 2022, at 13:06, Ilya Maximets wrote: > On 2/25/22 13:29, Marcelo Ricardo Leitner wrote: >> On Thu, Feb 24, 2022 at 03:20:23PM +0100, Eelco Chaudron wrote: >>> >>> >>> On 23 Feb 2022, at 17:23, Marcelo Ricardo Leitner wrote: >>> On Tue, Feb 22, 2022 at 04:26:10PM +0100, Eelco Chaud

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-03-29 Thread Ilya Maximets
On 2/25/22 13:29, Marcelo Ricardo Leitner wrote: > On Thu, Feb 24, 2022 at 03:20:23PM +0100, Eelco Chaudron wrote: >> >> >> On 23 Feb 2022, at 17:23, Marcelo Ricardo Leitner wrote: >> >>> On Tue, Feb 22, 2022 at 04:26:10PM +0100, Eelco Chaudron wrote: This patch checks for none offloadable ct_

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-26 Thread Roi Dayan via dev
On 2022-02-24 4:24 PM, Eelco Chaudron wrote: On 23 Feb 2022, at 12:35, Roi Dayan wrote: On 2022-02-23 11:31 AM, Eelco Chaudron wrote: On 23 Feb 2022, at 8:09, Roi Dayan wrote: On 2022-02-22 5:26 PM, Eelco Chaudron wrote: This patch checks for none offloadable ct_state match flag combi

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-25 Thread Marcelo Ricardo Leitner
On Thu, Feb 24, 2022 at 03:20:23PM +0100, Eelco Chaudron wrote: > > > On 23 Feb 2022, at 17:23, Marcelo Ricardo Leitner wrote: > > > On Tue, Feb 22, 2022 at 04:26:10PM +0100, Eelco Chaudron wrote: > >> This patch checks for none offloadable ct_state match flag combinations. > >> If they exist force

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-24 Thread Eelco Chaudron
On 23 Feb 2022, at 12:35, Roi Dayan wrote: > On 2022-02-23 11:31 AM, Eelco Chaudron wrote: >> >> >> On 23 Feb 2022, at 8:09, Roi Dayan wrote: >> >>> On 2022-02-22 5:26 PM, Eelco Chaudron wrote: This patch checks for none offloadable ct_state match flag combinations. If they exist force

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-24 Thread Eelco Chaudron
On 23 Feb 2022, at 17:23, Marcelo Ricardo Leitner wrote: > On Tue, Feb 22, 2022 at 04:26:10PM +0100, Eelco Chaudron wrote: >> This patch checks for none offloadable ct_state match flag combinations. >> If they exist force the +trk flag down to TC Flower >> >> Signed-off-by: Eelco Chaudron >> --

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-23 Thread Marcelo Ricardo Leitner
On Tue, Feb 22, 2022 at 04:26:10PM +0100, Eelco Chaudron wrote: > This patch checks for none offloadable ct_state match flag combinations. > If they exist force the +trk flag down to TC Flower > > Signed-off-by: Eelco Chaudron > --- > v3: > - Instead of warning about an invalid flag combination f

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-23 Thread Roi Dayan via dev
On 2022-02-23 11:31 AM, Eelco Chaudron wrote: On 23 Feb 2022, at 8:09, Roi Dayan wrote: On 2022-02-22 5:26 PM, Eelco Chaudron wrote: This patch checks for none offloadable ct_state match flag combinations. If they exist force the +trk flag down to TC Flower Signed-off-by: Eelco Chaudron

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-23 Thread Eelco Chaudron
On 23 Feb 2022, at 8:09, Roi Dayan wrote: > On 2022-02-22 5:26 PM, Eelco Chaudron wrote: >> This patch checks for none offloadable ct_state match flag combinations. >> If they exist force the +trk flag down to TC Flower >> >> Signed-off-by: Eelco Chaudron >> --- >> v3: >> - Instead of warnin

Re: [ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-22 Thread Roi Dayan via dev
On 2022-02-22 5:26 PM, Eelco Chaudron wrote: This patch checks for none offloadable ct_state match flag combinations. If they exist force the +trk flag down to TC Flower Signed-off-by: Eelco Chaudron --- v3: - Instead of warning about an invalid flag combination fix it. lib/netdev-offlo

[ovs-dev] [PATCH v3 8/9] netdev-offload-tc: Check for none offloadable ct_state flag combination

2022-02-22 Thread Eelco Chaudron
This patch checks for none offloadable ct_state match flag combinations. If they exist force the +trk flag down to TC Flower Signed-off-by: Eelco Chaudron --- v3: - Instead of warning about an invalid flag combination fix it. lib/netdev-offload-tc.c |6 ++ 1 file changed, 6 insertions(