[ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-05-17 Thread Eelco Chaudron
When OVs installs the flower rule, it only checks for the OK from the kernel. It does not check if the rule requested matches the one actually programmed. This change will add this check and warns the user if this is not the case. Signed-off-by: Eelco Chaudron --- lib/tc.c | 59 +++

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-08 Thread Ilya Maximets
On 5/17/21 3:20 PM, Eelco Chaudron wrote: > When OVs installs the flower rule, it only checks for the OK from the > kernel. It does not check if the rule requested matches the one > actually programmed. This change will add this check and warns the > user if this is not the case. > > Signed-off-by

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-08 Thread Marcelo Ricardo Leitner
On Thu, Jul 08, 2021 at 10:18:50PM +0200, Ilya Maximets wrote: > On 5/17/21 3:20 PM, Eelco Chaudron wrote: > > When OVs installs the flower rule, it only checks for the OK from the > > kernel. It does not check if the rule requested matches the one > > actually programmed. This change will add this

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-09 Thread Eelco Chaudron
On 8 Jul 2021, at 22:18, Ilya Maximets wrote: > On 5/17/21 3:20 PM, Eelco Chaudron wrote: >> When OVs installs the flower rule, it only checks for the OK from the >> kernel. It does not check if the rule requested matches the one >> actually programmed. This change will add this check and warns

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-09 Thread Ilya Maximets
On 7/9/21 10:35 AM, Eelco Chaudron wrote: > > > On 8 Jul 2021, at 22:18, Ilya Maximets wrote: > >> On 5/17/21 3:20 PM, Eelco Chaudron wrote: >>> When OVs installs the flower rule, it only checks for the OK from the >>> kernel. It does not check if the rule requested matches the one >>> actually

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-12 Thread Eelco Chaudron
On 9 Jul 2021, at 20:23, Ilya Maximets wrote: > On 7/9/21 10:35 AM, Eelco Chaudron wrote: >> >> >> On 8 Jul 2021, at 22:18, Ilya Maximets wrote: >> >>> On 5/17/21 3:20 PM, Eelco Chaudron wrote: When OVs installs the flower rule, it only checks for the OK from the kernel. It does not c

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-12 Thread Marcelo Ricardo Leitner
On Mon, Jul 12, 2021 at 10:28:15AM +0200, Eelco Chaudron wrote: > > > On 9 Jul 2021, at 20:23, Ilya Maximets wrote: > > > On 7/9/21 10:35 AM, Eelco Chaudron wrote: > >> > >> > >> On 8 Jul 2021, at 22:18, Ilya Maximets wrote: > >> > >>> On 5/17/21 3:20 PM, Eelco Chaudron wrote: > When OVs insta

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-07-14 Thread Eelco Chaudron
On 12 Jul 2021, at 14:54, Marcelo Ricardo Leitner wrote: > On Mon, Jul 12, 2021 at 10:28:15AM +0200, Eelco Chaudron wrote: >> >> >> On 9 Jul 2021, at 20:23, Ilya Maximets wrote: >> >>> On 7/9/21 10:35 AM, Eelco Chaudron wrote: On 8 Jul 2021, at 22:18, Ilya Maximets wrote: >>

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-05-21 Thread Marcelo Ricardo Leitner
On Mon, May 17, 2021 at 09:20:28AM -0400, Eelco Chaudron wrote: > When OVs installs the flower rule, it only checks for the OK from the > kernel. It does not check if the rule requested matches the one > actually programmed. This change will add this check and warns the > user if this is not the ca

Re: [ovs-dev] [PATCH] netdev-offload-tc: verify the flower rule installed

2021-10-27 Thread Ilya Maximets
On 7/14/21 13:18, Eelco Chaudron wrote: > > > On 12 Jul 2021, at 14:54, Marcelo Ricardo Leitner wrote: > >> On Mon, Jul 12, 2021 at 10:28:15AM +0200, Eelco Chaudron wrote: >>> >>> >>> On 9 Jul 2021, at 20:23, Ilya Maximets wrote: >>> On 7/9/21 10:35 AM, Eelco Chaudron wrote: > > >>>