Re: [ovs-dev] OVS with Bifurcation Example

2022-02-07 Thread Tonghao Zhang
On Sat, Feb 5, 2022 at 8:59 PM Lazuardi Nasution wrote: > > Hi, > > Is there any example about how to use OVS with Bifurcation driver? I have > read that with Intel NIC I should create specific VF for that, but with > Mellanox NIC it is not required to have VF. Hi there is a patchset, I hope it ca

Re: [ovs-dev] Datapath downtime during OVN upgrade

2022-02-07 Thread Numan Siddique
On Thu, Feb 3, 2022 at 5:22 PM Vladislav Odintsov wrote: > > UPD2: > > I see that only snat/dnat ct zones are deleted and assigned again (link [1] > from previous mail). > I’ve added more logging and see that local datapaths hmaps is empty on one > run and then snat/dnat zones are removed and on

Re: [ovs-dev] [PATCH ovn] Add support for port binding migration

2022-02-07 Thread Mark Michelson
Hi Ihar, In addition to the findings in-line down below, I have some high-level questions. Why does the port announces its readiness with a RARP packet? Presumably, a CMS will set the migration-destination option on a logical switch port, then will do something that prompts the the device t

[ovs-dev] ovs/ipsec: StrongSwan report connection update failures to ovs logs

2022-02-07 Thread Mohammad Heib
Currently when the user adds an IPsec tunnel port to the ovs bridge the ovs-monitor-ipsec script will add this tunnel IPsec-related configuration to the appropriate file and submit a request to start the IPsec connection for this port and ignores the request output which can contain an error messag

[ovs-dev] [PATCH V2 3/3] netdev-offload-dpdk: Add geneve header pattern match

2022-02-07 Thread Eli Britstein via dev
Add support for matching on geneve header. Signed-off-by: Eli Britstein Reviewed-by: Nir Anteby Acked-by: Michael Santana --- NEWS | 2 ++ lib/netdev-offload-dpdk.c | 58 +++ 2 files changed, 60 insertions(+) diff --git a/NEWS b/NEWS i

[ovs-dev] [PATCH V2 0/3] Support geneve offloads

2022-02-07 Thread Eli Britstein via dev
Add support for dpdk geneve tunnel offloads. v2-v1: - Rebase. GitHub actions: v1: https://github.com/elibritstein/OVS/actions/runs/1633157455 v2: https://github.com/elibritstein/OVS/actions/runs/1807027490 Eli Britstein (3): netdev-dpdk: Add flow_api support for netdev geneve vports netdev-o

[ovs-dev] [PATCH V2 2/3] netdev-offload-dpdk: Support tnl_pop for geneve tunnel

2022-02-07 Thread Eli Britstein via dev
Signed-off-by: Eli Britstein Reviewed-by: Nir Anteby Acked-by: Michael Santana --- lib/netdev-offload-dpdk.c | 13 + 1 file changed, 13 insertions(+) diff --git a/lib/netdev-offload-dpdk.c b/lib/netdev-offload-dpdk.c index 94dc6a9b7..edd4e009d 100644 --- a/lib/netdev-offload-dpdk.c

[ovs-dev] [PATCH V2 1/3] netdev-dpdk: Add flow_api support for netdev geneve vports

2022-02-07 Thread Eli Britstein via dev
Add the acceptance of geneve devices to netdev_dpdk_flow_api_supported() API, to allow offloading of DPDK geneve devices. Signed-off-by: Eli Britstein Reviewed-by: Nir Anteby Acked-by: Michael Santana --- lib/netdev-dpdk.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/l

[ovs-dev] [PATCH V2 1/1] dpif-netdev: Keep orig_in_port as a field of the flow

2022-02-07 Thread Eli Britstein via dev
A flow may be modified after its initial offload failed. In this case, according to [1], the modification is handled as a flow add. For a vport flow "add", the orig_in_port should be provided. Keep that field in the flow struct, so it can be provided in the flow modification use case. [1] 0d25621e

[ovs-dev] [PATCH V4 2/2] netdev-offload-dpdk: Fix ethernet type for VLANs

2022-02-07 Thread Eli Britstein via dev
For VLANs, the match of ethernet type should be specified in inner_type field of the vlan match, and not type field in ethernet match. Fix it. Fixes: e8a2b5bf92bb ("netdev-dpdk: implement flow offload with rte flow") Signed-off-by: Eli Britstein Reviewed-by: Salem Sol --- lib/netdev-offload-dpd

[ovs-dev] [PATCH V4 1/2] netdev-offload-dpdk: Use has_vlan match attribute

2022-02-07 Thread Eli Britstein via dev
DPDK 20.11 introduced an ability to specify existance/non-existance of VLAN tag by [1]. Use this attribute. [1]: 09315fc83861 ("ethdev: add VLAN attributes to ethernet and VLAN items") Signed-off-by: Eli Britstein Reviewed-by: Salem Sol --- lib/netdev-offload-dpdk.c | 16 1 fi

Re: [ovs-dev] RFC for adding P4 Support in OVS

2022-02-07 Thread Mike Pattrick
On Thu, Jan 20, 2022 at 6:52 PM Limaye, Namrata wrote: > > Hi, > > We are working on adding P4 support to OVS and we have recently open-sourced > the patches on the IPDK github - >https://github.com/ipdk-io/ovs/tree/ovs-with-p4 Hello Namrata, I've been trying to compile this, b

Re: [ovs-dev] [PATCH v1] system-dpdk.at: Add warning log in mfex fuzzy test.

2022-02-07 Thread David Marchand
On Mon, Feb 7, 2022 at 12:21 PM Kumar Amber wrote: > > Some specific warning are seen on various systems > which may not be visible on others but good to add > such logs to test to avoid test-case failure. > > Thw warning only effects the fuzzy tests due to > more than 1000+ flows being offloading

[ovs-dev] [PATCH net 1/1] openvswitch: Fix setting ipv6 fields causing hw csum failure

2022-02-07 Thread Paul Blakey via dev
Ipv6 ttl, label and tos fields are modified without first pulling/pushing the ipv6 header, which would have updated the hw csum (if available). This might cause csum validation when sending the packet to the stack, as can be seen in the trace below. Fix this by calling postpush/postpull checksum c

Re: [ovs-dev] [PATCH v3] dpif-netdev: fix vlan and ipv4 parsing in avx512

2022-02-07 Thread Eelco Chaudron
On 7 Feb 2022, at 12:41, Amber, Kumar wrote: > Hi Eelco, > > I have created a patch to add the warning to the system log > http://patchwork.ozlabs.org/project/openvswitch/patch/20220207110008.2054074-1-kumar.am...@intel.com/ > > I am not able to reproduce the warning, it will be good if you can

Re: [ovs-dev] [PATCH v1] system-dpdk.at: Add warning log in mfex fuzzy test.

2022-02-07 Thread Eelco Chaudron
On 7 Feb 2022, at 12:00, Kumar Amber wrote: > Some specific warning are seen on various systems > which may not be visible on others but good to add > such logs to test to avoid test-case failure. > > Thw warning only effects the fuzzy tests due to > more than 1000+ flows being offloading simul

Re: [ovs-dev] [PATCH v3] dpif-netdev: fix vlan and ipv4 parsing in avx512

2022-02-07 Thread Amber, Kumar
Hi Eelco, I have created a patch to add the warning to the system log http://patchwork.ozlabs.org/project/openvswitch/patch/20220207110008.2054074-1-kumar.am...@intel.com/ I am not able to reproduce the warning, it will be good if you can test it and let me know if something more needs to be Add

[ovs-dev] [PATCH] Documentation: Update USDT documentation to include systemtap dependency

2022-02-07 Thread Eelco Chaudron
Update the documentation to include details on SystemTap dependency when enabling USDT probes. Suggested-by: Adrian Moreno Signed-off-by: Eelco Chaudron --- Documentation/topics/usdt-probes.rst |5 + 1 file changed, 5 insertions(+) diff --git a/Documentation/topics/usdt-probes.rst b/D

[ovs-dev] [PATCH v1] system-dpdk.at: Add warning log in mfex fuzzy test.

2022-02-07 Thread Kumar Amber
Some specific warning are seen on various systems which may not be visible on others but good to add such logs to test to avoid test-case failure. Thw warning only effects the fuzzy tests due to more than 1000+ flows being offloading simultanously. Suggested-by: Eelco Chaudron Signed-off-by: Kum

Re: [ovs-dev] [PATCH v3 2/2] ofproto-dpif: avoid unneccesary backer revalidation

2022-02-07 Thread Eelco Chaudron
On 6 Feb 2022, at 4:58, lic121 wrote: >> If lldp didn't change, we are not supposed to trigger backer >> revalidation. >> >> Without this patch, bridge_reconfigure() always trigger udpif >> revalidator because of lldp. >> >> Signed-off-by: lic121 >> Co-authored-by: Eelco Chaudron >> --- >> lib