Re: [ovs-dev] [PATCH net v2 2/3] net/sched: flow_dissector: Fix matching on zone id for invalid conns

2021-12-10 Thread Jakub Kicinski
On Thu, 9 Dec 2021 09:57:33 +0200 Paul Blakey wrote: > @@ -238,10 +239,12 @@ void > skb_flow_dissect_ct(const struct sk_buff *skb, > struct flow_dissector *flow_dissector, > void *target_container, u16 *ctinfo_map, > - size_t mapsize, bool

[ovs-dev] [PATCH] docs: Re-work the documentation around CPU ISA optimizations.

2021-12-10 Thread Ilya Maximets
Few problems with a current documentation: 1. bridge.rst is the high-level documentation for the end user. Unit testing and complex implementation details are for developers, hence should not be there. Testing instructions for developers should be in testing.rst. Words in the doc

Re: [ovs-dev] [PATCH v1 01/18] python: add generic Key-Value parser

2021-12-10 Thread Eelco Chaudron
On 22 Nov 2021, at 12:22, Adrian Moreno wrote: > Most of ofproto and dpif flows are based on key-value pairs. These > key-value pairs can be represented in several ways, eg: key:value, > key=value, key(value). > > Add the following classes that allow parsing of key-value strings: > * KeyValue:

Re: [ovs-dev] [PATCH v3 ovn] ovn-trace: honor ct state in execute_ct_lb

2021-12-10 Thread Numan Siddique
On Thu, Dec 9, 2021 at 10:16 AM Dumitru Ceara wrote: > > On 12/9/21 14:54, Lorenzo Bianconi wrote: > > When performing CT_LB action in ovn-trace, take into account current > > connection tracking state provided by the user. > > > > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1980702

Re: [ovs-dev] [PATCH v4] dpcls: Change info-get function to fetch dpcls usage stats.

2021-12-10 Thread Eelco Chaudron
On 2 Dec 2021, at 11:05, Kumar Amber wrote: > Modified the dplcs info-get command output to include > the count for different dpcls implementations. > > $ovs-appctl dpif-netdev/subtable-lookup-prio-get > > Available dpcls implementations: > autovalidator (Use count: 1, Priority: 5) >

Re: [ovs-dev] [PATCH ovn 3/3] northd: support mix of stateless ACL with lower priority than stateful

2021-12-10 Thread Vladislav Odintsov
Thanks Dumitru. Regards, Vladislav Odintsov > On 10 Dec 2021, at 14:13, Dumitru Ceara wrote: > > On 12/9/21 17:12, Vladislav Odintsov wrote: >> Hi Dumitru, >> > > Hi Vladislav, > >> thanks for review & testing and pointing for the broken behaviour. >> I’ve missed such case, and I think this

Re: [ovs-dev] [PATCH ovn 3/3] northd: support mix of stateless ACL with lower priority than stateful

2021-12-10 Thread Dumitru Ceara
On 12/9/21 17:12, Vladislav Odintsov wrote: > Hi Dumitru, > Hi Vladislav, > thanks for review & testing and pointing for the broken behaviour. > I’ve missed such case, and I think this usecase should be of course > supported. Maybe, it should be listed in a ovn testcases for not loose it one