Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

The discussion about the ‘Area’ prefix has come up again, even after Ben had commented about it and after I had pointed folks to the submitting-patches.rst, which allows flexibility in choosing an ‘Area’ prefix by the patch submitter. In this thread, it was again suggested that the use of

Re: [ovs-dev] [PATCH 0/4] netdev-dpdk: Support multi-queue for ovs-dpdk QoS

Thanks for your attention. My work is on network of IaaS, and I want use it for network nodes, use openflow set_queue as a filter to limit egress SNATed floating IP traffic rate on dpdk port for vpc user. I also want to expand ingress policing to limit ingress DNATed traffic for vpc user, but

Re: [ovs-dev] [PATCH RFC v2] Conntrack: Avoid recirculation for established connections.

I did lose track of this patch; sorry. I see Sugesh has added some other comments, but I’ll respond to the previous comments to speed things along. -Original Message- From: "Fischetti, Antonio" Date: Tuesday, June 6, 2017 at 9:14 AM To: Darrell Ball

Re: [ovs-dev] [PATCH] rhel/systemd: Set ovs-vswitchd timeout to 5 minutes

On Thu, Jul 13, 2017 at 12:27 PM, Timothy M. Redaelli wrote: > On 07/13/2017 04:51 PM, Aaron Conole wrote: >> During initialization, it's possible that the startup time takes longer >> than the systemd default provided. Set this to be 5 minutes. If we >> take longer than 5

[ovs-dev] [PATCH] docs: Refer to correct package name for sphinx theme.

Update the log message emitted when the OVS sphinx theme is not found to reference the name of the package to be installed via pip: ovs-sphinx-theme. Signed-off-by: Russell Bryant --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [ovs-dev] [PATCH] rhel: Fix creation of symlink for ocf script

On Thu, Jul 20, 2017 at 3:41 AM, Numan Siddique wrote: > On Wed, Jul 19, 2017 at 7:00 PM, Aaron Conole wrote: > >> Timothy Redaelli writes: >> >> > The policy is to use %files to track installed files. >> > >> > If %files is not

Re: [ovs-dev] [PATCH 1/2] NEWS: Add OVN L3 Gateway HA.

Thanks for the review. I applied this to master. Patch 2 of this series needs a review, as well. On Wed, Jul 19, 2017 at 12:50 AM, Miguel Angel Ajo Pelayo wrote: > Acked-by: Miguel Angel Ajo > > El 16 jul. 2017 9:56 p. m., "Russell Bryant"

Re: [ovs-dev] [PATCH] ovn-architecture: Add notes on L3 gateway HA.

Thanks for the review. I applied this to master. On Wed, Jul 19, 2017 at 4:13 AM, Miguel Angel Ajo Pelayo wrote: > Sounds great, thank you! > > Acked-by: Miguel Angel Ajo > > On Sun, Jul 16, 2017 at 10:09 PM, Russell Bryant wrote: >>

Re: [ovs-dev] [PATCH v2] odp-execute: Reuse rss hash in OVS_ACTION_ATTR_HASH.

> Acked-by: Darrell Ball Thanks llya for the patch and Darrell for the review! Pushed to master. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [clone optmization 6/7] xlate: Refactor compose_clone() API

Create a new function that hides the details of netlink encoding for the translated clone action. Signed-off-by: Andy Zhou --- ofproto/ofproto-dpif-xlate.c | 76 +++- tests/ofproto-dpif.at| 7 2 files changed, 39

[ovs-dev] [clone optmization 5/7] ofproto-dpif: Remove ofprto/tnl-push-pop command.

Use dpif/set-dp-features command instead. Signed-off-by: Andy Zhou --- ofproto/ofproto-dpif.c | 34 -- tests/ofproto-macros.at | 5 ++--- 2 files changed, 2 insertions(+), 37 deletions(-) diff --git a/ofproto/ofproto-dpif.c

[ovs-dev] [clone optmization 4/7] ofproto-dpif: Remove dpif/disable-truncate command.

Use 'dpif/set-dp-features' command instead. Signed-off-by: Andy Zhou --- ofproto/ofproto-dpif.c | 21 - tests/system-traffic.at | 8 ++-- 2 files changed, 2 insertions(+), 27 deletions(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c

[ovs-dev] [clone optmization 2/7] ofproto-dpif: Add boottime support field.

When changing support fields, it may be unsafe to set support level beyond what datapath can support. This patch introduce the notion of boot time support and runtime support fields. Boot time support are set only once during ofproto start up phase, and not changed during runtime. The runtime

[ovs-dev] [clone optmization 1/7] ofproto-dpif: A new command for changing dpif support fields

dpif support fields contain various datapath capabilities detected by ofproto at start up time. Usually those fields are read-only, not intended to be changed at runtime. However in practice, when writing tests or running experiments, it becomes necessary to set those fields to emulate different

[ovs-dev] [clone optmization 0/7] Clone optimization

I respined the patches based on review feedback at: http://mail.openvswitch.org/pipermail/ovs-dev/2017-June/333502.html Patch 1-5: Implements a generic command that can defeature and restore the datapath support level. They are not directly related to the main objective

[ovs-dev] ovs-dpdk: dpdk pdump shows arp reply packet egress but seems disappear over the wire

Hi I have two VMs (Ubuntu 14.04, BIGIP Virtual Edition) running over the ovs-dpdk vhostuserport, everything works for Ubuntu 14.04 VM, but no network connectivity for BIGIP Virtual Edition, I used DPDK pdump to capture packet, I see arp reply egress over the dpdk port, but the arp reply not seen

Re: [ovs-dev] [patch_v1 2/2] System Tests: Improve reliability of an icmp test.

-Original Message- From: on behalf of Joe Stringer Date: Thursday, July 20, 2017 at 11:17 AM To: Darrell Ball Cc: ovs dev Subject: Re: [ovs-dev] [patch_v1 2/2] System Tests: Improve reliability of

[ovs-dev] [patch_v2 2/2] System-tests: Improve reliability of an icmp test.

One SNAT test is based on a single ping being successful; to make the result more predictable, static arp binding is now used. To put less stress on the stack a single arp binding is used for the reverse direction mapping. This does not change the goal of the test, but significantly increases the

[ovs-dev] [patch_v2 1/2] System-tests: Allow SNAT address variability retries.

Three of the SNAT tests allow for wget retries, which occasionally happen. However, these tests did not allow for SNAT address variability for the retries, which is now tolerated. Signed-off-by: Darrell Ball --- tests/system-traffic.at | 6 +++--- 1 file changed, 3

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

On 20 July 2017 at 11:17, Darrell Ball wrote: > Please don’t update the prefix OK. As discussed offline, I'll wait for Darrell to respin this series. ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [patch_v1 2/2] System Tests: Improve reliability of an icmp test.

On 16 July 2017 at 11:27, Darrell Ball wrote: > One SNAT test is based on a single ping being successful; > to make the result more predictable, static arp binding is now used. > Occasionally, tracing shows the reply side stack does not respond, > but this is much less common

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

Please don’t update the prefix -Original Message- From: on behalf of Joe Stringer Date: Thursday, July 20, 2017 at 11:15 AM To: Darrell Ball Cc: ovs dev Subject: Re: [ovs-dev] [patch_v1 1/2] System

Re: [ovs-dev] [patch_v1 1/2] System Tests: Allow SNAT address variability retries.

On 16 July 2017 at 11:27, Darrell Ball wrote: > Three of the SNAT tests allow for wget retries, which occasionally > happen. However, these tests did not allow for SNAT address > variability for the retries, which is now tolerated. > > Signed-off-by: Darrell Ball

[ovs-dev] [PATCH 2/2] ofproto-dpif.at: Add bonding down/up rebalancing test.

Add regression test which checks rebalancing of tcp balanced bonding after link state changes of one of the ports. Signed-off-by: Ilya Maximets --- To work correctly this unit test requires fix for '--len' option of netdev-dummy/receive appctl command: *

[ovs-dev] [PATCH 1/2] bond: Fix broken rebalancing after link state changes.

There are 3 constraints for moving hashes from one slave to another: 1. The load difference is larger than ~3% of one slave's load. 2. The load difference between slaves exceeds 10 bytes. 3. Moving of the hash makes the load difference lower by > 10%. In current

[ovs-dev] [PATCH 0/2] bond: Fix broken rebalancing after link state changes.

See commit message in patch #1 for bug description. Issue can be trigered by unit test from patch #2, but it requires fix [1] for '--len' option to be applied to work correctly. Actual fix in patch #1 doesn't have that dependency. [1] [PATCH 0/3] Fix '--len' option for netdev-dummy/receive.

Re: [ovs-dev] [PATCH v7 1/1] ovn: l3ha, CLI for logical router port gateway chassis

ack, ok, from that point of view, current name makes more sense IMO. Thank you. On Thu, Jul 20, 2017 at 6:22 PM, Anil Venkata wrote: > Thanks Miguel. > lrp-set-gateway-chassis is also used for updating the priority. > User can initially set priority to some value and

Re: [ovs-dev] [PATCH v7 1/1] ovn: l3ha, CLI for logical router port gateway chassis

Thanks Miguel. lrp-set-gateway-chassis is also used for updating the priority. User can initially set priority to some value and later can update with another value. For example, user can add chassis with priority 5, later can update it to 10 ovn-nbctl lrp-set-gateway-chassis lrp0 chassis1 5

[ovs-dev] [PATCH 3/4] dpif-netdev: Add rxq prioritization

If an rxq is marked as 'prioritized' then keep reading from this queue until there are no packets available. Only then proceed to other queues. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 24 ++-- 1 file changed, 18 insertions(+), 6

[ovs-dev] [PATCH 4/4] docs: Document ingress scheduling feature

Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 31 +++ vswitchd/vswitch.xml | 31 +++ 2 files changed, 62 insertions(+) diff --git a/Documentation/howto/dpdk.rst

[ovs-dev] [PATCH 2/4] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

Ingress scheduling configuration is given effect by way of Flow Director filters. A small subset of the ingress scheduling possible is implemented in this patch. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 + lib/dpif-netdev.c

Re: [ovs-dev] [PATCH v7 1/1] ovn: l3ha, CLI for logical router port gateway chassis

One last comment, sorry to come late with more comments: if lrp-set-gateway-chassis is additive, shouldn't we rename it to: lrp-add-gateway-chassis ? Thanks, and best regards On Tue, Jul 18, 2017 at 8:05 AM, Venkata Anil Kommaddi wrote: > From: Venkata Anil

Re: [ovs-dev] [PATCH RFC v2] Conntrack: Avoid recirculation for established connections.

Thanks Sugesh for your comments, pls see replies inline. -Antonio > -Original Message- > From: Chandran, Sugesh > Sent: Friday, June 23, 2017 4:20 PM > To: Fischetti, Antonio ; d...@openvswitch.org > Subject: RE: [ovs-dev] [PATCH RFC v2] Conntrack: Avoid

[ovs-dev] [RFC PATCH v2] net-dpdk: Introducing TX TCP/UDP HW checksum offload support for DPDK pnic

This patch introduce TX tcp/udp-checksum offload support for DPDK pnic. The feature is disabled by default and can be enabled by setting tx-checksum-offload, which like: ovs-vsctl set Interface dpdk-eth3 \ options:tx-checksum-offload=true Signed-off-by: Zhenyu Gao ---

[ovs-dev] DPDK Hierarchical Scheduler support

Are there any plans to support DPDK Hierarchical Scheduler (rte_sched) in OVS? ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH 3/4] Support to show multi-queue qos info

Signed-off-by: zhaozhanxu --- lib/netdev-dpdk.c | 147 -- 1 file changed, 142 insertions(+), 5 deletions(-) diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index 49816ca..9d6a87c 100644 --- a/lib/netdev-dpdk.c +++

[ovs-dev] [PATCH 2/4] Support multi-queue rate limit function for ovs-dpdk Qos.

Signed-off-by: zhaozhanxu --- lib/netdev-dpdk.c | 40 ++-- 1 file changed, 38 insertions(+), 2 deletions(-) diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index f0d9842..49816ca 100644 --- a/lib/netdev-dpdk.c +++ b/lib/netdev-dpdk.c @@

[ovs-dev] [PATCH 4/4] Modify QoS configure documents for ovs-dpdk

Signed-off-by: zhaozhanxu --- Documentation/howto/dpdk.rst | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst index af01d3e..bb60f83 100644 --- a/Documentation/howto/dpdk.rst +++

[ovs-dev] [PATCH 1/4] Support multi-queue for ovs-dpdk QoS.

Original command is as below: $ ovs-vsctl set port vhost-user0 qos=@newqos -- \ --id=@newqos create qos type=egress-policer other-config:cir=4600 \ other-config:cbs=2048` Now it can support command as below: $ ovs-vsctl set port vhost-user0 qos=@newqos -- \ --id=@newqos

[ovs-dev] [PATCH 0/4] netdev-dpdk: Support multi-queue for ovs-dpdk QoS

Original command is as below: $ ovs-vsctl set port vhost-user0 qos=@newqos -- \ --id=@newqos create qos type=egress-policer other-config:cir=4600 \ other-config:cbs=2048` Now it can support command as below: $ ovs-vsctl set port vhost-user0 qos=@newqos -- \

Re: [ovs-dev] [PATCH v2 0/3] basic encap/decap

Hi Zoltan, Could you please also update the NEWS with the new features. See proposal below. Thanks, Jan diff --git a/NEWS b/NEWS index 4aa937f..1f57a1e 100644 --- a/NEWS +++ b/NEWS @@ -56,11 +56,17 @@ Post-v2.7.0 * The "learn" action now supports a "limit" option (see ovs-ofctl(8)).

Re: [ovs-dev] [PATCH] rhel: Fix creation of symlink for ocf script

On Wed, Jul 19, 2017 at 7:00 PM, Aaron Conole wrote: > Timothy Redaelli writes: > > > The policy is to use %files to track installed files. > > > > If %files is not used the resulting file is not owned by any package. > > > > Before this commit: > > #

Re: [ovs-dev] [PATCH v4 0/3] tunneling : Improving tunneling performance by avoiding dp recirc.

Thankyou Joe! Regards _Sugesh > -Original Message- > From: Joe Stringer [mailto:j...@ovn.org] > Sent: Wednesday, July 19, 2017 10:38 PM > To: Chandran, Sugesh > Cc: ovs dev ; Andy Zhou ; Zoltán > Balogh

[ovs-dev] Sync on PTAP, EXT-382 and NSH - Minutes of Meeting Wed 2017-07-19 17:00 CET

Participants: Zoltan, Jan Patch series * Generic Encap/Decap (EXT 382) for Ethernet o v2 series submitted as discussed o Ben P already merged the PTAP bugfix patch (1/3) o Jan identified lack of check on VLAN tags in translation of decap() for Ethernet. Tagged packets

Re: [ovs-dev] [PATCH v1 1/1] ovn: l3ha fix unexpected ARP requests on pcap during tests

Please, ignore this patch, it doesn't help. On Wed, Jul 19, 2017 at 12:07 PM, Miguel Angel Ajo wrote: > For testing l3ha datapath we have an external port on a separate sim > instance (ext1), and we send an UDP packet that we expect to see > going through the specific