Re: [ovs-dev] [PATCH 2/4] dpif-netdev: Fix non-local numa selection.

2022-03-01 Thread Mike Pattrick
On Tue, Mar 1, 2022 at 12:25 PM Kevin Traynor wrote: > > In the event of no pmd cores available on the local numa for > an rxq to be assigned to, a pmd core from a non-local numa is > selected. > > If there are more than one non-local numas with pmd cores they > are RR through and checked if they

Re: [ovs-dev] [PATCH 1/4] dpif-netdev: Fix typo in function name.

2022-03-01 Thread Mike Pattrick
On Tue, Mar 1, 2022 at 12:25 PM Kevin Traynor wrote: > > Rename pmd_reblance_dry_run_needed() to > pmd_rebalance_dry_run_needed(). > > Fixes: a83a406096e9 ("dpif-netdev: Sync PMD ALB state with user commands.") > > Signed-off-by: Kevin Traynor > --- Acked-by: Mike Pattrick

Re: [ovs-dev] [PATCH net v4 1/1] net/sched: act_ct: Fix flow table lookup failure with no originating ifindex

2022-03-01 Thread Pablo Neira Ayuso
On Mon, Feb 28, 2022 at 11:23:49AM +0200, Paul Blakey wrote: > After cited commit optimizted hw insertion, flow table entries are > populated with ifindex information which was intended to only be used > for HW offload. This tuple ifindex is hashed in the flow table key, so > it must be filled for

Re: [ovs-dev] [PATCH v1] bond: Improve bond and lacp visibility

2022-03-01 Thread 0-day Robot
Bleep bloop. Greetings Mike Pattrick, 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 88 characters long (recommended limit is 79) #29 FILE: lib/lacp.c:391:

Re: [ovs-dev] [PATCH v2] ovs-monitor-ipsec: Add force-encapsulation option to force NAT-T

2022-03-01 Thread Mike Pattrick
On Tue, Mar 1, 2022 at 10:53 AM Andreas Karis wrote: > > Both LibreSwan and OpenSwan allow administrators to unconditionally > force enable NAT-T for ESP. This may help to surmount restrictive > firewalls in scenarios where IP protocol number 50 is blocked, but where > NAT autodetection fails.

[ovs-dev] [PATCH v1] bond: Improve bond and lacp visibility

2022-03-01 Thread Mike Pattrick
Add additional logging for debug and info level with a focus on code related to bond members coming up, go down, and changing. Several existing log messages were modified to handle sub 1kB log messages with more grace. Instead of reporting 0kB of traffic load shifting from one member to another,

Re: [ovs-dev] [PATCH] ovs-monitor-ipsec: Add force-encapsulation option to force NAT-T

2022-03-01 Thread Andreas Karis
Thank you, I posted a new version. On Fri, Feb 25, 2022 at 10:17 PM Mike Pattrick wrote: > On Thu, 2022-01-20 at 16:33 +0100, Andreas Karis wrote: > > Both LibreSwan and OpenSwan allow administrators to unconditionally > > force enable NAT-T for ESP. This may help to surmount restrictive > >

[ovs-dev] [PATCH 4/4] alb.ut: Add tests for cross-numa polling.

2022-03-01 Thread Kevin Traynor
PMD auto load balance currently only operates when the polling pmd core will not change numa after reassignment. Add a unit test for this. Signed-off-by: Kevin Traynor --- tests/alb.at | 46 ++ 1 file changed, 46 insertions(+) diff --git

[ovs-dev] [PATCH 2/4] dpif-netdev: Fix non-local numa selection.

2022-03-01 Thread Kevin Traynor
In the event of no pmd cores available on the local numa for an rxq to be assigned to, a pmd core from a non-local numa is selected. If there are more than one non-local numas with pmd cores they are RR through and checked if they have non-isolated pmds. When successfully finding a non-local

[ovs-dev] [PATCH 3/4] dpif-netdev: Fix PMD auto load balance with pmd-rxq-isolate.

2022-03-01 Thread Kevin Traynor
There are currently some checks for cross-numa polling cases to ensure that they won't effect the accuracy of the PMD ALB. If an rxq is pinned to a pmd core by the user it will not be reassigned by OVS, so even if it is non-local numa polled it will not impact PMD ALB accuracy. To establish

[ovs-dev] [PATCH 0/4] ALB/Rxq scheduling cross-numa polling fixes.

2022-03-01 Thread Kevin Traynor
Some fixes found in the area of cross-numa polling during recent code inspection and testing. GHA: https://github.com/kevintraynor/ovs/actions/runs/1917535875 Kevin Traynor (4): dpif-netdev: Fix typo in function name. dpif-netdev: Fix non-local numa selection. dpif-netdev: Fix PMD auto

[ovs-dev] [PATCH 1/4] dpif-netdev: Fix typo in function name.

2022-03-01 Thread Kevin Traynor
Rename pmd_reblance_dry_run_needed() to pmd_rebalance_dry_run_needed(). Fixes: a83a406096e9 ("dpif-netdev: Sync PMD ALB state with user commands.") Signed-off-by: Kevin Traynor --- lib/dpif-netdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/dpif-netdev.c

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Jeffrey Walton
On Tue, Mar 1, 2022 at 7:54 AM Dumitru Ceara wrote: > > On 3/1/22 13:44, Adrian Moreno wrote: > > > > > > On 3/1/22 12:59, Dumitru Ceara wrote: > >> On 3/1/22 12:54, Adrian Moreno wrote: > >>> > >>> > >>> On 3/1/22 10:35, Dumitru Ceara wrote: > On 3/1/22 10:07, Adrian Moreno wrote: > > >

[ovs-dev] [PATCH v2] ovs-monitor-ipsec: Add force-encapsulation option to force NAT-T

2022-03-01 Thread Andreas Karis
Both LibreSwan and OpenSwan allow administrators to unconditionally force enable NAT-T for ESP. This may help to surmount restrictive firewalls in scenarios where IP protocol number 50 is blocked, but where NAT autodetection fails. Add a switch --force-encapsulation to expose this feature to users

[ovs-dev] [PATCH v2] system-dpdk: Fix mfex autovalidator tests.

2022-03-01 Thread Kumar Amber
AVX512 DPIF must be active in order for the MFEX AutoValidator to be executed. If the DPIF-AVX512 is not available, the unit test is skipped, as the scalar DPIF does not use the MFEX function-pointer based optimizations. Fixes: 50be6715c0 ("test/sytem-dpdk: Add unit test for mfex autovalidator")

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Dumitru Ceara
On 3/1/22 13:44, Adrian Moreno wrote: > > > On 3/1/22 12:59, Dumitru Ceara wrote: >> On 3/1/22 12:54, Adrian Moreno wrote: >>> >>> >>> On 3/1/22 10:35, Dumitru Ceara wrote: On 3/1/22 10:07, Adrian Moreno wrote: > > > On 2/25/22 18:14, Dumitru Ceara wrote: >> For example is

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Adrian Moreno
On 3/1/22 12:59, Dumitru Ceara wrote: On 3/1/22 12:54, Adrian Moreno wrote: On 3/1/22 10:35, Dumitru Ceara wrote: On 3/1/22 10:07, Adrian Moreno wrote: On 2/25/22 18:14, Dumitru Ceara wrote: For example is parsing the OVN "eth.dst[40] = 1;" action, which triggered the following warning

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Dumitru Ceara
On 3/1/22 12:54, Adrian Moreno wrote: > > > On 3/1/22 10:35, Dumitru Ceara wrote: >> On 3/1/22 10:07, Adrian Moreno wrote: >>> >>> >>> On 2/25/22 18:14, Dumitru Ceara wrote: For example is parsing the OVN "eth.dst[40] = 1;" action, which triggered the following warning from

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Adrian Moreno
On 3/1/22 10:35, Dumitru Ceara wrote: On 3/1/22 10:07, Adrian Moreno wrote: On 2/25/22 18:14, Dumitru Ceara wrote: For example is parsing the OVN "eth.dst[40] = 1;" action, which triggered the following warning from UndefinedBehaviorSanitizer:    lib/meta-flow.c:3210:9: runtime error:

Re: [ovs-dev] [PATCH v4 5/7] ofp-actions: Use aligned structures when decoding ofp actions.

2022-03-01 Thread Dumitru Ceara
On 3/1/22 09:30, Adrian Moreno wrote: > Hi Dumitru, > Hi Adrian, Thanks for the review! > On 2/25/22 18:14, Dumitru Ceara wrote: >> Some openflow actions can be misaligned, e.g., actions whithin OF 1.0 >> replies to statistics reply messages which have a header of 12 bytes >> and no additional

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Dumitru Ceara
On 3/1/22 10:07, Adrian Moreno wrote: > > > On 2/25/22 18:14, Dumitru Ceara wrote: >> For example is parsing the OVN "eth.dst[40] = 1;" action, which >> triggered the following warning from UndefinedBehaviorSanitizer: >> >>    lib/meta-flow.c:3210:9: runtime error: member access within >>

Re: [ovs-dev] [RFC] tc: use pedit add actions for ipv4 and ipv6 dec ttl

2022-03-01 Thread Pieter Jansen van Vuuren
On 01/03/2022 08:33, Eelco Chaudron wrote: > CAUTION: This message has originated from an External Source. Please use > proper judgment and caution when opening attachments, clicking links, or > responding to this email. > > > On 28 Feb 2022, at 19:01, Pieter Jansen van Vuuren wrote: > >>

Re: [ovs-dev] [PATCH v4 3/7] ofp-actions: Ensure aligned accesses when setting masked fields.

2022-03-01 Thread Adrian Moreno
On 2/25/22 18:14, Dumitru Ceara wrote: For example is parsing the OVN "eth.dst[40] = 1;" action, which triggered the following warning from UndefinedBehaviorSanitizer: lib/meta-flow.c:3210:9: runtime error: member access within misaligned address 0x00de4e36 for type 'const union

Re: [ovs-dev] [RFC] tc: use pedit add actions for ipv4 and ipv6 dec ttl

2022-03-01 Thread Eelco Chaudron
On 28 Feb 2022, at 19:01, Pieter Jansen van Vuuren wrote: > On 25/02/2022 13:20, Eelco Chaudron wrote: >> CAUTION: This message has originated from an External Source. Please use >> proper judgment and caution when opening attachments, clicking links, or >> responding to this email. >> >> >>

Re: [ovs-dev] [PATCH v4 5/7] ofp-actions: Use aligned structures when decoding ofp actions.

2022-03-01 Thread Adrian Moreno
Hi Dumitru, On 2/25/22 18:14, Dumitru Ceara wrote: Some openflow actions can be misaligned, e.g., actions whithin OF 1.0 replies to statistics reply messages which have a header of 12 bytes and no additional padding. Also, buggy controllers might incorrectly encode actions. When decoding