Re: [ovs-dev] [PATCH 0/2] datapath: Optimize operations for OvS flow_stats.

2017-09-18 Thread Tonghao Zhang
Haha, I'm fine. We just make the ovs better. On Sat, Sep 16, 2017 at 1:45 AM, Greg Rose wrote: > On 09/10/2017 06:00 PM, Tonghao Zhang wrote: >> >> The linux kernel 4.13 has been released. I backport the patches of >> openvswitch here. >> >> Tonghao Zhang (2): >>

[ovs-dev] 答复: Re: [PATCH] ovn: Discard flows for non-local ports.

2017-09-18 Thread wang . qianyu
I agree with Han Zhou, This patch can reduce the flow to vswitchd, but can not reduce the flow calculation of ovn-controller. It may be better to move the check for local_lport_ids before the parse happens. Add a lport column in logical flow may be more efficient. Thanks. Han Zhou

Re: [ovs-dev] [PATCH] ovn: Discard flows for non-local ports.

2017-09-18 Thread Han Zhou
Thanks Russell for the quick work! On Mon, Sep 18, 2017 at 8:24 AM, Russell Bryant wrote: > @@ -301,6 +305,22 @@ consider_logical_flow(struct controller_ctx *ctx, > if (m->match.wc.masks.conj_id) { > m->match.flow.conj_id += *conj_id_ofs; > } > +

Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: Transition ukey on dp_ops error.

2017-09-18 Thread Greg Rose
On 09/06/2017 03:12 PM, Joe Stringer wrote: In most situations, we don't expect that a flow we've successfully dumped, which we intend to delete, cannot be deleted. However, to make this code more resilient to ensure that ukeys *will* transition in all cases (including an error at this stage),

Re: [ovs-dev] [PATCH 2/2] ofproto-dpif-ipfix: add interface Information Elements to flow key

2017-09-18 Thread Greg Rose
On 09/18/2017 03:01 AM, Weglicki, MichalX wrote: Hi Greg - comments inline marked [MW]. -Original Message- From: Greg Rose [mailto:gvrose8...@gmail.com] Sent: Saturday, September 16, 2017 12:45 AM To: Weglicki, MichalX Cc: d...@openvswitch.org; Darrell Ball

Re: [ovs-dev] why the max action length is 32K in kernel?

2017-09-18 Thread Ben Pfaff
On Mon, Sep 18, 2017 at 02:06:52PM -0700, Greg Rose wrote: > On 09/18/2017 01:32 PM, Ben Pfaff wrote: > >On Mon, Sep 18, 2017 at 01:27:52PM -0700, Greg Rose wrote: > >>On 09/18/2017 11:15 AM, Ben Pfaff wrote: > >>>On Mon, Sep 18, 2017 at 10:58:28AM -0700, Greg Rose wrote: > On 09/12/2017 08:37

Re: [ovs-dev] why the max action length is 32K in kernel?

2017-09-18 Thread Greg Rose
On 09/18/2017 01:32 PM, Ben Pfaff wrote: On Mon, Sep 18, 2017 at 01:27:52PM -0700, Greg Rose wrote: On 09/18/2017 11:15 AM, Ben Pfaff wrote: On Mon, Sep 18, 2017 at 10:58:28AM -0700, Greg Rose wrote: On 09/12/2017 08:37 PM, ychen wrote: in function nla_alloc_flow_actions(), there is a check

Re: [ovs-dev] why the max action length is 32K in kernel?

2017-09-18 Thread Ben Pfaff
On Mon, Sep 18, 2017 at 01:27:52PM -0700, Greg Rose wrote: > On 09/18/2017 11:15 AM, Ben Pfaff wrote: > >On Mon, Sep 18, 2017 at 10:58:28AM -0700, Greg Rose wrote: > >>On 09/12/2017 08:37 PM, ychen wrote: > >>>in function nla_alloc_flow_actions(), there is a check if action length is > >>>greater

Re: [ovs-dev] why the max action length is 32K in kernel?

2017-09-18 Thread Greg Rose
On 09/18/2017 11:15 AM, Ben Pfaff wrote: On Mon, Sep 18, 2017 at 10:58:28AM -0700, Greg Rose wrote: On 09/12/2017 08:37 PM, ychen wrote: in function nla_alloc_flow_actions(), there is a check if action length is greater than MAX_ACTIONS_BUFSIZE(32k), then kernel datapath flow will not be

Re: [ovs-dev] adding dpdk ports sharing same pci address to ovs-dpdk bridge

2017-09-18 Thread Darrell Ball
Thanks for confirming Devendra Adding Ciara There have been some offline discussions regarding the issue. From: devendra rawat Date: Monday, September 18, 2017 at 4:27 AM To: Kevin Traynor Cc: Darrel Ball ,

[ovs-dev] Multiple Subnets connecting LXC containers over Single GRE Tunnel conneting Two Physical Hosts Using Patch Ports

2017-09-18 Thread Gilbert Standen
Hi can anyone help with this problem? I summarized it here at my blog: https://sites.google.com/site/nandydandyoracle/openvswitch-ovs/networking-problem-1 Please let me know if you need more information or propose an alternative solution. Thanks!! Gil

Re: [ovs-dev] why the max action length is 32K in kernel?

2017-09-18 Thread Ben Pfaff
On Mon, Sep 18, 2017 at 10:58:28AM -0700, Greg Rose wrote: > On 09/12/2017 08:37 PM, ychen wrote: > >in function nla_alloc_flow_actions(), there is a check if action length is > >greater than MAX_ACTIONS_BUFSIZE(32k), then kernel datapath flow will not be > >installed, and packets will droppped.

Re: [ovs-dev] [PATCH 2/2] dpctl: init CT entry variable.

2017-09-18 Thread Greg Rose
On 09/13/2017 05:36 AM, antonio.fische...@intel.com wrote: ct_dpif_entry_uninit could potentially be called even if ct_dpif_dump_next failed. As ct_dpif_entry_uninit receives a pointer to a CT entry - and just checks it is not null - it's safer to init to zero any instantiated ct_dpif_entry

Re: [ovs-dev] [PATCH 1/2] dpctl: manage ret value when dumping CT entries.

2017-09-18 Thread Greg Rose
On 09/13/2017 05:36 AM, antonio.fische...@intel.com wrote: Manage error value returned by ct_dpif_dump_next. Signed-off-by: Antonio Fischetti --- lib/dpctl.c | 28 +--- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git

Re: [ovs-dev] why the max action length is 32K in kernel?

2017-09-18 Thread Greg Rose
On 09/12/2017 08:37 PM, ychen wrote: in function nla_alloc_flow_actions(), there is a check if action length is greater than MAX_ACTIONS_BUFSIZE(32k), then kernel datapath flow will not be installed, and packets will droppped. but in function xlate_actions(), there is such clause: if

Re: [ovs-dev] [PATCH] ovn: Discard flows for non-local ports.

2017-09-18 Thread Russell Bryant
On Mon, Sep 18, 2017 at 11:24 AM, Russell Bryant wrote: > Discard some OpenFlow flows that will never match. This includes > flows that match on a non-local inport in the ingress pipeline or a > non-local outport in the egress pipeline of a logical switch. > > This is most

[ovs-dev] [PATCH] ovn: Discard flows for non-local ports.

2017-09-18 Thread Russell Bryant
Discard some OpenFlow flows that will never match. This includes flows that match on a non-local inport in the ingress pipeline or a non-local outport in the egress pipeline of a logical switch. This is most useful for networks with a large number of ports or ACLs that use large address sets.

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Kavanagh, Mark B
>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] >On Behalf Of Kavanagh, Mark B >Sent: Monday, September 18, 2017 3:36 PM >To: Nitin Katiyar ; ovs-dev@openvswitch.org >Subject: Re: [ovs-dev] MTU in i40e dpdk driver > >>From: Nitin Katiyar

Re: [ovs-dev] data path flow addition error in ovs 2.5

2017-09-18 Thread Ben Pfaff
On Mon, Sep 18, 2017 at 01:51:58PM +0530, Prasannaa Vengatesan wrote: > I am using the following version of OVS (2.5.1) with datapath in kernel > mode. > > root@localhost:~# ovs-vsctl --version > ovs-vsctl (Open vSwitch) 2.5.1 > > > When I try to add a flow in the data path I get error. Why

Re: [ovs-dev] [PATCH V2 3/4] tc: Add header rewrite using tc pedit action

2017-09-18 Thread Simon Horman
On Mon, Sep 18, 2017 at 07:16:03AM +0300, Roi Dayan wrote: > From: Paul Blakey > > To be later used to implement ovs action set offloading. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > --- > lib/tc.c | 372 >

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Kavanagh, Mark B
>From: Nitin Katiyar [mailto:nitin.kati...@ericsson.com] >Sent: Monday, September 18, 2017 3:20 PM >To: Kavanagh, Mark B ; ovs-dev@openvswitch.org >Subject: RE: [ovs-dev] MTU in i40e dpdk driver > >Hi, >Yes, the tag is configured for VHU port so traffic from VM would be

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Aaron Conole
Nitin Katiyar writes: > Hi, > We are using OVS-DPDK (2.6 version) with Fortville NIC (configured in > 25G mode) being used as dpdk port. The setup involves 2 VMs running on > 2 different computes (destination VM in compute with 10G NIC while > originating VM is in

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Nitin Katiyar
Hi, Yes, the tag is configured for VHU port so traffic from VM would be tagged with vlan. Why is it different from 10G (ixgbe) driver? It should allow the packet matching with the configured MTU. Is it expected behavior with i40e driver? Thanks, Nitin -Original Message- From:

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Kavanagh, Mark B
>From: Nitin Katiyar [mailto:nitin.kati...@ericsson.com] >Sent: Monday, September 18, 2017 3:02 PM >To: Kavanagh, Mark B ; ovs-dev@openvswitch.org >Subject: RE: [ovs-dev] MTU in i40e dpdk driver > >Hi, >It is set to 2140. That should accommodate a max packet length of

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Nitin Katiyar
Hi, It is set to 2140. compute-0-4:~# ovs-vsctl get Interface dpdk1 mtu 2140 Regards, Nitin -Original Message- From: Kavanagh, Mark B [mailto:mark.b.kavan...@intel.com] Sent: Monday, September 18, 2017 7:26 PM To: Nitin Katiyar ; ovs-dev@openvswitch.org

Re: [ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Kavanagh, Mark B
>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] >On Behalf Of Nitin Katiyar >Sent: Monday, September 18, 2017 2:05 PM >To: ovs-dev@openvswitch.org >Subject: [ovs-dev] MTU in i40e dpdk driver > >Hi, >We are using OVS-DPDK (2.6 version) with Fortville NIC (configured

Re: [ovs-dev] ovs-vswitchd is resetting the MTU of a bridge when a patch port is deleted.

2017-09-18 Thread Daniel Alvarez Sanchez
Yes, thanks Numan for the patch :) Another option would be that ovn-controller sets explicitly the MTU to 1450. Not sure which of the two is the best or would have less side effects. Cheers, Daniel On Tue, Sep 12, 2017 at 10:43 AM, Numan Siddique wrote: > Hello, > > Daniel

[ovs-dev] MTU in i40e dpdk driver

2017-09-18 Thread Nitin Katiyar
Hi, We are using OVS-DPDK (2.6 version) with Fortville NIC (configured in 25G mode) being used as dpdk port. The setup involves 2 VMs running on 2 different computes (destination VM in compute with 10G NIC while originating VM is in compute with Fortville NIC). All the interfaces in the path

Re: [ovs-dev] does ovs bfd support flow based tunnel?

2017-09-18 Thread ychen
for flow-based tunnel: ovs-vsctl add-port br-int vxlan1 -- set interface vxlan1 type=vxlan options:remote_ip=flow options:key=flow options:local_ip=10.10.0.1 ovs-vsctl set interface vxlan1 bfd:enable=true when I enable bfd in such a vxlan interface , I can not capture any bfd packets in the

Re: [ovs-dev] adding dpdk ports sharing same pci address to ovs-dpdk bridge

2017-09-18 Thread devendra rawat
Hi Kevin, On Fri, Sep 8, 2017 at 12:24 AM, Kevin Traynor wrote: > On 09/07/2017 06:47 PM, Darrell Ball wrote: > > Adding disc...@openvswitch.org > > > > The related changes went into 2.7 > > > > > > > > On 9/7/17, 3:51 AM, "ovs-dev-boun...@openvswitch.org on behalf of >

[ovs-dev] Sushi Ebi 4L 8.6-9.0 cm

2017-09-18 Thread Bonesca Import & Export BV
  [ View in browser ]( http://r.newsletter.bonescamail.nl/7xa28juxqoatrf.html )         Special offer for Sushi Ebi 4L with expiry this month :   20 x 195 grs ( 30 x 6.5 grs) 8.6 - 9.0 cm   1 box € 3,75 10 box € 3,50 complete remaining stock (95 box) € 3,25 per tray!!     [ Click here

[ovs-dev] [PATCH 5/5] conntrack: update manual and usage for R/W parameter.

2017-09-18 Thread antonio . fischetti
Update manual and usage for R/W parameters commands. Signed-off-by: Antonio Fischetti --- lib/dpctl.man | 8 utilities/ovs-dpctl.c | 2 ++ 2 files changed, 10 insertions(+) diff --git a/lib/dpctl.man b/lib/dpctl.man index 675fe5a..836cc08 100644

[ovs-dev] [PATCH 4/5] conntrack: read current nr of connections.

2017-09-18 Thread antonio . fischetti
Read current number of connections managed by the CT module. Example: ovs-appctl dpctl/ct-get totconn Signed-off-by: Antonio Fischetti --- lib/conntrack.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/conntrack.c b/lib/conntrack.c index

[ovs-dev] [PATCH 3/5] conntrack: r/w clean-up interval.

2017-09-18 Thread antonio . fischetti
Read/Write conntrack clean-up interval used by the clean_thread_main() thread. Example: ovs-appctl dpctl/ct-set cleanup=4000 # Set a new value ovs-appctl dpctl/ct-get cleanup # Read Signed-off-by: Antonio Fischetti --- lib/conntrack.c | 27

[ovs-dev] [PATCH 1/5] conntrack: add commands to r/w conntrack parameters.

2017-09-18 Thread antonio . fischetti
Add infrastructure to implement: - dpctl/ct-get to read a current value of available conntrack parameters. - dpctl/ct-set to set a value to the available conntrack parameters. Add dpctl/ct-get to read current values of conntrack parameters. Add dpctl/ct-set to set a value to conntrack

[ovs-dev] International Scientific Publication Journals (EU)

2017-09-18 Thread International Scientific Publications (EU)
International Scientific Publications Journals (EU) We are writing to kindly inform you that you can publish your scientific papers in the peer-reviewed open access journals. Currently, we issue six scientific journals online: Agriculture & Food ISSN 1314-8591 (online) Ecology & Safety ISSN

[ovs-dev] data path flow addition error in ovs 2.5

2017-09-18 Thread Prasannaa Vengatesan
Hi, I am using the following version of OVS (2.5.1) with datapath in kernel mode. root@localhost:~# ovs-vsctl --version ovs-vsctl (Open vSwitch) 2.5.1 When I try to add a flow in the data path I get error. root@localhost:~# ovs-appctl dpctl/add-flow ovs-system "in_port(2)" 3 ovs-vswitchd:

Re: [ovs-dev] [PATCH net-next v9] openvswitch: enable NSH support

2017-09-18 Thread Yang, Yi
On Thu, Sep 14, 2017 at 05:09:02PM +0800, Jiri Benc wrote: > On Thu, 14 Sep 2017 16:37:59 +0800, Yi Yang wrote: > > OVS master and 2.8 branch has merged NSH userspace > > patch series, this patch is to enable NSH support > > in kernel data path in order that OVS can support > > NSH in compat mode

Re: [ovs-dev] [PATCH 02/13] netdev-dummy: Reorder elements in dummy_packet_stream structure.

2017-09-18 Thread Bodireddy, Bhanuprakash
Hi greg, >On 09/08/2017 10:59 AM, Bhanuprakash Bodireddy wrote: >> By reordering elements in dummy_packet_stream structure, sum holes > >Do you mean "the sum of the holes" can be reduced or do you mean "some >holes" >can be reduced? In this patch series "sum of the holes" means, the sum/total of