Re: [ovs-dev] [PATCH] vagrant: Add FreeBSD 10.2 box support.

2016-07-04 Thread William Tu
oh, I forgot to add the new file to EXTRA_DIST. Thanks for fixing it! On Sun, Jul 3, 2016 at 10:28 AM, Ben Pfaff wrote: > On Wed, Jun 29, 2016 at 10:35:00AM -0700, William Tu wrote: >> Add FreeBSD 10.2 vagrant file "Vagrantfile-FreeBSD". Users can run >>

Re: [ovs-dev] [question] How to invalidate a dp-flow which was generated by of-flow ?

2016-07-04 Thread Ben Pfaff
On Tue, Jul 05, 2016 at 11:09:26AM +0800, Zhang Haoyu wrote: > How to invalidate a dp-flow which was generated by of-flow, like > recirc_id(0),in_port(2),eth(src=50:1c:bf:71:13:22,dst=50:1c:bf:71:13:22),eth_type(0x9000), > packets:27, bytes:1620, used:8.822s, actions:drop Datapath flows are an

Re: [ovs-dev] Compiler warnings with gcc 6.1 (On Fedora 24)

2016-07-04 Thread Numan Siddique
> > > > OK. > > If you have time and the inclination, please feel free to experiment a > bit with the definition of BUILD_ASSERT_TYPE to see if you can make it > work without warnings. > ​Thanks for the pointers. I will give it a try. Regards Numan ​

[ovs-dev] [question] How to invalidate a dp-flow which was generated by of-flow ?

2016-07-04 Thread Zhang Haoyu
Hi all How to invalidate a dp-flow which was generated by of-flow, like recirc_id(0),in_port(2),eth(src=50:1c:bf:71:13:22,dst=50:1c:bf:71:13:22),eth_type(0x9000), packets:27, bytes:1620, used:8.822s, actions:drop Thanks, Zhang Haoyu ___ dev mailing

[ovs-dev] DELIVERY FAILED

2016-07-04 Thread MAILER-DAEMON
The original message was received at Tue, 5 Jul 2016 10:21:18 +0800 from openvswitch.org [214.81.236.92] - The following addresses had permanent fatal errors - - Transcript of session follows - while talking to openvswitch.org.: >>> MAIL

[ovs-dev] Returned mail: Data format error

2016-07-04 Thread Bounced mail
¯ÁŽD}D0ƒÅ©ŒÅR¶Ì¼\A¸zÁù¬:K¡ŸÌ0w%s]Fd$JÉá¯OÙfGÆ0ýGó>†ˆqdr˜Ü˛ú2I­ûH‹7æø—… ˜Îgf]}<Èv”iÞtÝ'»Àò{Ú ðµ»`Tè §ÌT¿½X\5m!K¤¸Ú-Þ¤|•­Þ®?rÅâ,£áWvGlvê™|µl“ÜnA¯Å  ÌÇ)q°\&|*˜o¼”$^>Tiy[ÎÃ,å%ʟî–Ìô4²“ S«‹íc8G7$>Ÿ°Ò"”½ûä 3?1Ð Ë^<5 œiD(u`¦AO‘^eši£Æè΅

Re: [ovs-dev] v22 arrived a bit early (was Re: [ovs-dev, v21, 2/8] Persist ovn flow tables)

2016-07-04 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 05:20:38PM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 07/03/2016 11:51:17 PM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 07/03/2016 11:51 PM > > Subject: Re: [ovs-dev,v21,2/8] Persist ovn

Re: [ovs-dev] [PATCH v2] ipfix: Add ingress and egress interface in exporting flows

2016-07-04 Thread Ben Pfaff
The odp_ports aren't meaningful at all outside of OVS, so they're almost worthless. How do you intend for the receiver to interpret them? If the IPFIX record doesn't otherwise say what bridge the ofp_ports correspond to, then I suggest adding some way to identify the bridge; for example, you

Re: [ovs-dev] [PATCH v2] ipfix: Add ingress and egress interface in exporting flows

2016-07-04 Thread Daniel Ye
Hi Ben, Thanks for review. In my opinion, the ofp_port may be the same on different bridge. If so, we still don’t differentiate the flows in this scenario. That’s why I choose to use odd_port. Bests, Daniel > On Jul 5, 2016, at 5:22 AM, Ben Pfaff wrote: > > On Thu, Jun 30, 2016

[ovs-dev] v22 arrived a bit early (was Re: [ovs-dev, v21, 2/8] Persist ovn flow tables)

2016-07-04 Thread Ryan Moats
Ben Pfaff wrote on 07/03/2016 11:51:17 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/03/2016 11:51 PM > Subject: Re: [ovs-dev,v21,2/8] Persist ovn flow tables > > On Sun, Jul 03, 2016 at 08:03:54PM -0500, Ryan Moats

[ovs-dev] [PATCH v22 2/2] Add incremental processing to lflow_run and physical_run

2016-07-04 Thread Ryan Moats
This code changes to allow incremental processing of the logical flow and physical binding tables whenver possible. Note: flows created by physical_run for multicast_groups are *NOT* handled incrementally due to to be solved issues with GWs and local routers. Signed-off-by: Ryan Moats

[ovs-dev] [PATCH v22 1/2] Persist ovn flow tables

2016-07-04 Thread Ryan Moats
Ensure that ovn flow tables are persisted so that changes to them chan be applied incrementally - this is a prereq for making lflow_run and physical_run incremental. As part of this change, add a one-to-many hindex for finding desired flows by their parent's UUID. Also extend the mapping by

[ovs-dev] [PATCH v22 0/2] Add incremental processing

2016-07-04 Thread Ryan Moats
This patch set has the residual changes that didn't merge from v21. Unlike previous versions, the persist ovn flow tables patch changes semantics of the desired physical flow hmap table from one-to-one to one-to-many, and uses the logical parent's UUID to disambiguate. This should provide

Re: [ovs-dev] [PATCH] ovn: improve OVN tutorial.

2016-07-04 Thread Ben Pfaff
Russell, would you mind reviewing this? On Mon, Jul 04, 2016 at 08:03:49AM -0700, Ben Pfaff wrote: > From: nickcooper-zhangtonghao > > Improve the tutorial of the basic OVN features. The addresses and > port_security > columns of the logical port is

Re: [ovs-dev] [ovs-discuss] OVS fdb - MAC learning Subscription

2016-07-04 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 09:16:22PM +0530, Kapil Adhikesavalu wrote: > Hi Ben, > > Thanks for the information. This is exactly what I was looking for. > > My implementation involves around 3K dynamic MACs. > > Couple of questions, > > 1. MAC eviction on table full. I took a look at NORMAL

Re: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID file

2016-07-04 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 06:02:31PM +, Alin Serdean wrote: > > > I'm still not sure that real locking is actually needed for Windows. > > > I believe that, on Windows, the functionality of a lock file only > > > requires holding the file open. For a program to detect whether the > > > lock

Re: [ovs-dev] [PATCH v2] ipfix: Add ingress and egress interface in exporting flows

2016-07-04 Thread Ben Pfaff
On Thu, Jun 30, 2016 at 08:48:43PM -0700, Daniel Benli Ye wrote: > In virtual evironment, IPFIX is unable to differentiate flows > between pair of VMs on different virtual network if their IP/mac > are same. > > Network: > VM1 < VNI1 > VM3 > VM2 < VNI2 > VM4 > > In

[ovs-dev] Toffbqcv

2016-07-04 Thread Automatic Email Delivery Software
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-04 Thread Aaron Rosen
Thanks for the review. Response inline On Sun, Jul 3, 2016 at 12:04 PM, Ben Pfaff wrote: > On Thu, Jun 30, 2016 at 02:27:19PM -0700, Aaron Rosen wrote: > > This patch exposes the c function expr_parse_string() to be called via > > python. The motivation for this is so that clients

[ovs-dev] [PATCH] bridge: open_type should be used for netdev_open

2016-07-04 Thread Thadeu Lima de Souza Cascardo
ofproto_port_open_type should be used for netdev_open, but not for other tests. For example, STP/RSTP check for interfaces of internal type, but that check will fail when the netdev datapath is used. The same thing goes for setting MAC address of internal Interfaces. That fails for the netdev

Re: [ovs-dev] [PATCH] ovsdb: Fix OVSDB disconnect replication bug

2016-07-04 Thread Cabrera Vega, Mario Alberto
Here's the pull request: https://github.com/openvswitch/ovs/pull/140 -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sent: Saturday, July 2, 2016 9:42 PM To: Cabrera Vega, Mario Alberto Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH] ovsdb: Fix

Re: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID file

2016-07-04 Thread Alin Serdean
> > I'm still not sure that real locking is actually needed for Windows. > > I believe that, on Windows, the functionality of a lock file only > > requires holding the file open. For a program to detect whether the > > lock file is locked, it can just try to delete it; on Windows, I > > believe

[ovs-dev] Emailing: Picture (06).png

2016-07-04 Thread dev
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [ovs-discuss] OVS fdb - MAC learning Subscription

2016-07-04 Thread Kapil Adhikesavalu
Hi Ben, Thanks for the information. This is exactly what I was looking for. My implementation involves around 3K dynamic MACs. Couple of questions, 1. MAC eviction on table full. I took a look at NORMAL action Mac learning code. It had implemented MAC eviction by giving fair share of entries

Re: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID file

2016-07-04 Thread Alin Serdean
> > I used only shared lock, in order to be compatible with Python tests, > > which try to acquire the lock exclusively. On Windows if the exclusive > > lock is used, than the read access is denied too for other instances of this > file. > > > > Signed-off-by: Paul-Daniel Boca

Re: [ovs-dev] Compiler warnings with gcc 6.1 (On Fedora 24)

2016-07-04 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 11:03:34AM +0530, Numan Siddique wrote: > On Mon, Jul 4, 2016 at 10:47 AM, Ben Pfaff wrote: > > > On Mon, Jul 04, 2016 at 10:17:24AM +0530, Numan Siddique wrote: > > > I am seeing the below compiler warnings with GCC version 6.1 on Fedora > > 24. > > > > > >

Re: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID file

2016-07-04 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 04:27:49PM +, Paul Boca wrote: > If the PID file isn't locked then appctl.py detects it as stale and > bails out without doing anything. Because of this lots of Python tests fail. > Also this protects the PID file from being overwritten. > > I used only shared lock, in

[ovs-dev] [PATCH] ovn: improve OVN tutorial.

2016-07-04 Thread Ben Pfaff
From: nickcooper-zhangtonghao Improve the tutorial of the basic OVN features. The addresses and port_security columns of the logical port is described in more detail. Signed-off-by: nickcooper-zhangtonghao ---

Re: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID file

2016-07-04 Thread Paul Boca
Hi Alin! Thanks for review. You are right with all the observations. I will send another patch that will fix them. Thanks, Paul > -Original Message- > From: Alin Serdean > Sent: Monday, July 4, 2016 5:41 PM > To: Paul Boca; dev@openvswitch.org > Subject: RE: [PATCH V5] windows: Added

Re: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID file

2016-07-04 Thread Alin Serdean
> -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Friday, July 1, 2016 7:28 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH V5] windows: Added lockf function and lock PID > file > > If the PID file isn't locked then appctl.py

Re: [ovs-dev] [PATCH V5 16/17] tests: Skip vlog tests that try to move opened file

2016-07-04 Thread Alin Serdean
> -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Friday, July 1, 2016 7:27 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH V5 16/17] tests: Skip vlog tests that try to move > opened file > > m4_define([VLOG_REOPEN_PYN], >

[ovs-dev] Scanned image

2016-07-04 Thread operator6684
Image data has been attached to this email. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH V2] ovn test: Skip "send gratuitous arp on localnet" if no Python installed

2016-07-04 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean Thanks, Alin. > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Monday, July 4, 2016 1:01 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH V2] ovn test: Skip "send

[ovs-dev] Scanned image

2016-07-04 Thread scan3126
Image data has been attached to this email. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] OVN: Fix ARP request flow in router ingress table

2016-07-04 Thread Chandra S Vejendla
TPA in arp requests generated for unknown MAC-to-IP bindings is currently set to DST_IP of the original packet. These arps will not be resolved when the DST_IP is rechable via the default gateway. This patch fixes the issue by setting the TPA to reg0. In routing stage reg0 is set to IP of the

[ovs-dev] Scanned image

2016-07-04 Thread operator408
Image data has been attached to this email. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] ovn: Support chassis option in "l2gateway" logical ports

2016-07-04 Thread Numan Siddique
ovn-controller will now bind the l2gateway logical ports. Signed-Off-by: Numan Siddique --- ovn/TODO | 9 - ovn/controller/binding.c | 33 +++-- ovn/ovn-nb.xml | 7 +++ ovn/ovn-sb.xml | 6 ++

Re: [ovs-dev] [RFC Patch] dpif-netdev: dpcls per in_port with sorted subtables

2016-07-04 Thread Jarno Rajahalme
I like this RFC patch and suggest you send a non-RFC patch after taking care of the following: - There are numerous coding style violations. I've done my best to note them below, but you should also read 'CodingStyle.md'. - The subject line should have 'Patch' in all upper-case and have the

[ovs-dev] [PATCH v5 2/2] DSCP marking on packets egressing VIF interface

2016-07-04 Thread bschanmu
ovn-northd sets 'ip.dscp' to the DSCP value Signed-off-by: Babu Shanmugam --- ovn/controller/lflow.c | 2 +- ovn/northd/ovn-northd.c | 4 ovn/ovn-nb.xml | 6 ++ ovn/ovn-sb.xml | 5 + 4 files changed, 16 insertions(+), 1 deletion(-) diff

[ovs-dev] [PATCH v5 1/2] Check and allocate free qdisc queue id for ports with qos parameters

2016-07-04 Thread bschanmu
ovn-northd processes the list of Port_Bindings and hashes the list of queues per chassis. When it finds a port with qos_parameters and without a queue_id, it allocates a free queue for the chassis that this port belongs. The queue_id information is stored in the options field of Port_binding

[ovs-dev] [PATCH v5 0/2] QOS updates with DSCP support

2016-07-04 Thread bschanmu
v4->v5 - Rebased v4 along with changes from Ben Pfaff Babu Shanmugam (2): Check and allocate free qdisc queue id for ports with qos parameters DSCP marking on packets egressing VIF interface ovn/controller/binding.c | 230 +--

Re: [ovs-dev] [PATCH v2 19/24] datapath: backport: udp_offload: Set encapsulation before inner completes.

2016-07-04 Thread Jarno Rajahalme
I'm not really familiar with these backports, but is it so that the main change the upstream patch introduced to udp_gro_complete() is not applicable to the backports? Jarno > On Jun 30, 2016, at 1:06 AM, Pravin B Shelar wrote: > > Upstream commit: >commit

Re: [ovs-dev] [PATCH v3] dpif-netdev: Remove PMD latency on seq_mutex

2016-07-04 Thread Jarno Rajahalme
> On Jun 29, 2016, at 4:58 PM, Flavio Leitner wrote: > > On Wed, Jun 29, 2016 at 05:04:06AM -0700, Jarno Rajahalme wrote: >> >>> On Jun 23, 2016, at 11:40 AM, Flavio Leitner wrote: >>> ... > >>> + >>> +ovs_assert(!single_threaded()); >>> +perthread

[ovs-dev] [RFC PATCH 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-04 Thread Ciara Loftus
This commit introduces support for DPDK 16.07 and consequently breaks compatibility with DPDK 16.04. DPDK 16.07 introduces some changes to various APIs. These have been updated in OVS, including: * xstats API: changes to structure of xstats * vhost API: replace virtio-net references with 'vid'

[ovs-dev] [RFC PATCH 0/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-04 Thread Ciara Loftus
This RFC patch provides initial support for DPDK 16.07-rc1 (release candidate 1) which was tagged July 4th. I plan to submit a full patch once the 16.07 release is final, hopefully later this month. This RFC patch can be used in meantime to test with the DPDK master branch or 16.07-rc1 tag. The

[ovs-dev] [PATCH V2] ovn test: Skip "send gratuitous arp on localnet" if no Python installed

2016-07-04 Thread Paul Boca
This test needs Python in order to run. Signed-off-by: Paul-Daniel Boca --- V2: Skip this test only if Python is not installed. Let it run if the OS is Windows. --- tests/ovn.at | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/ovn.at b/tests/ovn.at

[ovs-dev] [PATCH] datapath-windows: Fixed packet limits on STT protocol

2016-07-04 Thread Paul Boca
"If the length of the payload to be encapsulated exceeds 64KB, or if the offset to the L4 header exceeds 255 bytes, then it will not be possible to offload the packet to the NIC for segmentation." (STT RFC) In this case the packet needs to be segmented by us, before sending it. Signed-off-by:

Re: [ovs-dev] [PATCH v8 2/2] INSTALL.DPDK: Refactor DPDK install guide, add ADVANCED doc

2016-07-04 Thread Bodireddy, Bhanuprakash
>-Original Message- >From: Lance Richardson [mailto:lrich...@redhat.com] >Sent: Sunday, July 3, 2016 4:17 PM >To: Bodireddy, Bhanuprakash >Cc: dev@openvswitch.org; daniele di proietto > >Subject: Re: [ovs-dev] [PATCH v8 2/2]

Re: [ovs-dev] [PATCH v1 1/1] INSTALL.DPDK: Flag DPDK firmware requirements.

2016-07-04 Thread Bodireddy, Bhanuprakash
Please note that Firmware requirements information is also captured in Refactored install guide v8. http://openvswitch.org/pipermail/dev/2016-July/074373.html http://openvswitch.org/pipermail/dev/2016-July/074374.html Regards, Bhanu Prakash. >-Original Message- >From: dev

Re: [ovs-dev] [PATCH v1] INSTALL.DPDK: Update vhost multiqueue instructions.

2016-07-04 Thread Bodireddy, Bhanuprakash
>-Original Message- >From: Stokes, Ian >Sent: Wednesday, June 29, 2016 2:32 PM >To: dev@openvswitch.org >Cc: Bodireddy, Bhanuprakash >Subject: RE: [PATCH v1] INSTALL.DPDK: Update vhost multiqueue instructions. > >Hi All, > >Just a gentle reminder as

Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory in ovsdb

2016-07-04 Thread Wojciechowicz, RobertX
Hi, > -Original Message- > From: Ben Pfaff [mailto:b...@ovn.org] > Sent: Saturday, July 2, 2016 2:49 AM > To: Wojciechowicz, RobertX > Cc: dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory > in ovsdb > > On

[ovs-dev] Mail System Error - Returned Mail

2016-07-04 Thread Post Office
Ðò2ºÔd*,3zþþÓRGÒ°®Ü>7ûZnuëÌ£p®’˜G1÷Qj–<µ.éGВb©r£(í𙻓þÛë†(´Žæ^Ê°F‚¾qÓ\Öi,ØÔÊØB×k¢®”,õ’SÝÔ\ÙúLuŽ¡¿·±Àбʛç#ù˜‹³"µ£[^©c9ëƒ5š2í­© ùFýµÑˆ÷lLE#uœú‚rOíÚê  §a“®U°‰ì¬um;»ÎªÄz¥àDv½£®BŸôÝÙ¥(‚㯹`˜´ÚﱈúÝ^$ÄÍ3(£¹œ”¦´5½ô5¼q¾K«-nk4“ŽÈS”Ù¥bä¿c(¯;÷Bqµúíu–6Öì'èt¡';êTØ9ԗ–vÅðiù`«ëEôŽ×sš–Xîj5ñIbÓC&/j