[ovs-dev] How can i measure Hash based flow table performance in openvswitch?

2016-08-01 Thread Nam Bong Ha
Hi, all I want to measure Hash based flow table performance in Openvswitch. (Not Wildcard and Wildcard+Hash, Only Hash) ex. Two switch has same flow table size. A-switch -> flow table utilization rate = 50% B-switch -> flow table utilization rate = 100% Any, ideas? Thankyou

Re: [ovs-dev] [ovs-dev,v4,3/5] netdev-dpdk: Add vHost User PMD

2016-08-01 Thread Tetsuya Mukawa
> I've added vHost maintainers to CC-list to hear their opinion about > new API to get number of queues from the vHost PMD. > Maybe we can expose 'rte_vhost_get_queue_num()' somehow or make > 'dev_info->nb_rx_queues' usable? > I appreciate great investigation. So far, I am not sure what is good

[ovs-dev] Report

2016-08-01 Thread Mail Administrator
Your message was undeliverable due to the following reason: Your message could not be delivered because the destination server was unreachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likely

[ovs-dev] [PATCH 1/2] Add dry-run option to ovs-dpctl and ovs-ofctl commands.

2016-08-01 Thread Ryan Moats
ovs-dpctl and ovs-ofctl lack a dry-run option. Add it and the necessary scaffolding to each. Signed-off-by: Ryan Moats --- lib/command-line.c | 50 + lib/command-line.h | 6 +- lib/db-ctl-base.h | 2 +- lib/dpctl.c

[ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

2016-08-01 Thread Ryan Moats
This commit creates wrapper scripts for the *ctl commands to use --dry-run for those that have them, and to allow for log level setting via ovs-appctl without allowing full access to ovs-appctl. Tests have been added to make sure that the wrapper scripts don't actually do anything when asked to

[ovs-dev] [PATCH 0/2] Read only versions of *ctl commands

2016-08-01 Thread Ryan Moats
This patch series adds read-only wrapper scripts around the various *ctl commands that leverage the --dry-run command line option. This option is added to commands that don't currently have it (ovs-dpctl and ovs-ofctl) and negative tests are included to verify that the wrapper scripts don't change

Re: [ovs-dev] Question about Hierarchical playback to IDL

2016-08-01 Thread Ben Pfaff
When a daemon starts, it first gets an atomic snapshot of the database's initial contents, which corresponds to the boundary between two transactions, and after that its view of the database proceeds as previously explained. On Tue, Aug 02, 2016 at 02:30:05AM +, Elluru, Krishna Mohan wrote: >

[ovs-dev] 这双清包税到门就比空运贵一点点,省事放心

2016-08-01 Thread Bright
| 欧州/重量 | 21-50KG | 51-100KG | 101-299KG | 300+KG | | 荷兰 | 33.5 | 32.5 | 31.5 | 31 | | 比利时/德国/卢森堡 | 34 | 32 | 31 | 30 | | 英国 | 35 | 32 | 31 | 30 | 民海国际欧美UPS专线双清包税到门可到亚马逊仓 | 美国/重量 | 21-100KG | 101-300KG | 301-500KG | 501-1000KG | 1000+KG

Re: [ovs-dev] Question about Hierarchical playback to IDL

2016-08-01 Thread Elluru, Krishna Mohan
HI Ben, Thanks a lot for your response. One follow up question, say if I have a daemon running and it got restarted and it is expecting full update from OVSDB via IDL, even in that case the playback to IDL is done in same order of insertion to DB? If not, please advise steps to be taken

[ovs-dev] [PATCH RFC v2 2/3] ovn: add SLAAC support for IPv6

2016-08-01 Thread Zong Kai LI
This patch tries to implement Router Advertisement (RA) responder for SLAAC on ovn-controller side. It parses lflows which have: - match: inport == LRP_NAME && ip6.dst == ff02::2 && nd_rs (nd_rs: icmp6.type == 133 && icmp6.code == 0 && ttl == 255) - action: nd_ra{slaac(prefix,...,MTU,

[ovs-dev] [PATCH RFC v2 3/3] ovn: add SLAAC support for IPv6

2016-08-01 Thread Zong Kai LI
This patch tries to implement Router Advertisement (RA) responder for SLAAC on ovn-northd side. It tries to build lflows per each Logical Router Port, who have IPv6 networks and set their 'slaac' column to true. The lflows will look like: match=(inport ==

[ovs-dev] [PATCH RFC v2 1/3] ovn: add SLAAC support for IPv6

2016-08-01 Thread Zong Kai LI
This patch introduces methods to compose a Router Advertisement (RA) packet, introduces flags for RA, renames ovs_nd_opt to ovs_nd_lla_opt to specify it's Source/Target Link-layer Address option. Signed-off-by: Zong Kai LI --- lib/flow.c| 26 -

[ovs-dev] [PATCH RFC v2 0/3] ovn: add SLAAC support for IPv6

2016-08-01 Thread Zong Kai LI
This serial patches try to support Stateless Address Autoconfiguration(SLAAC) for IPv6, via Router Advertisement (RA) message responder. User case: After this serial patches, a user can follow the following steps to let all VMs/VIFs on a logical switch has IPv6 prefix/CIDR, configure their

Re: [ovs-dev] [PATCH 0/6] compat: Fix various fragmentation-related crashes.

2016-08-01 Thread Joe Stringer
On 12 July 2016 at 15:26, Joe Stringer wrote: > This series fixes a couple of crashes that may occur due to the backports of > the inet_fragment.c code from upstream. These are primarily related to cases > where a fragment is received and cached in the OVS fragment handling code >

Re: [ovs-dev] [PATCH] compat: Properly handle fragment lru.

2016-08-01 Thread Joe Stringer
On 1 August 2016 at 16:52, pravin shelar wrote: > On Mon, Aug 1, 2016 at 2:25 PM, Joe Stringer wrote: >> On 1 August 2016 at 14:08, Joe Stringer wrote: >>> In kernels <=3.16 there is an LRU for managing fragment queues for IPv4 >>> and IPv6. Because

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

2016-08-01 Thread Andy Zhou
On Mon, Aug 1, 2016 at 2:41 PM, Andy Zhou wrote: > From: Mario Cabrera > > Currently disconnecting from the replicator server means closing the > jsonrpc > connection and destroying the monitored table names and blacklisted table > names. > > This patch

Re: [ovs-dev] [PATCH] compat: Properly handle fragment lru.

2016-08-01 Thread pravin shelar
On Mon, Aug 1, 2016 at 2:25 PM, Joe Stringer wrote: > On 1 August 2016 at 14:08, Joe Stringer wrote: >> In kernels <=3.16 there is an LRU for managing fragment queues for IPv4 >> and IPv6. Because the backport code comes from more recent upstream >> versions of Linux,

Re: [ovs-dev] Replication feature bug

2016-08-01 Thread Andy Zhou
On Mon, Aug 1, 2016 at 10:07 AM, Cabrera Vega, Mario Alberto < mario.cabr...@hpe.com> wrote: > Hi, > > The replication_init() function initializes the set that contains the > names of the > tables to be monitored and the set that contains the names of the tables > to be > excluded from

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

2016-08-01 Thread Andy Zhou
From: Mario Cabrera Currently disconnecting from the replicator server means closing the jsonrpc connection and destroying the monitored table names and blacklisted table names. This patch makes a distition between disconnecting from the remote server, applicable when the

Re: [ovs-dev] [PATCH] compat: Properly handle fragment lru.

2016-08-01 Thread Joe Stringer
On 1 August 2016 at 14:08, Joe Stringer wrote: > In kernels <=3.16 there is an LRU for managing fragment queues for IPv4 > and IPv6. Because the backport code comes from more recent upstream > versions of Linux, this LRU management was missing from ip_frag_queue() > and

[ovs-dev] Processed: Raising severities of bugs regarding dpkg-buildpackage -A

2016-08-01 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org: > severity 805943 serious Bug #805943 [src:hocr] hocr: FTBFS when built with dpkg-buildpackage -A (no binary artifacts) Severity set to 'serious' from 'important' > severity 805947 serious Bug #805947 [src:libforms] libforms: FTBFS when built with

Re: [ovs-dev] [PATCH branch-2.5] datapath: fix build failure.

2016-08-01 Thread pravin shelar
On Mon, Aug 1, 2016 at 2:14 PM, Joe Stringer wrote: > On 1 August 2016 at 14:06, Pravin B Shelar wrote: >> Patch fixes following compilation error. >> >> CHECK /home/travis/build/openvswitch/ovs/datapath/linux/gso.c \ >>

Re: [ovs-dev] [PATCH branch-2.5] datapath: fix build failure.

2016-08-01 Thread Joe Stringer
On 1 August 2016 at 14:06, Pravin B Shelar wrote: > Patch fixes following compilation error. > > CHECK /home/travis/build/openvswitch/ovs/datapath/linux/gso.c \ > /home/travis/build/openvswitch/ovs/datapath/linux/gso.c:222:12: error: > no member

Re: [ovs-dev] [PATCH 2/6] compat: Simplify inet_fragment backports.

2016-08-01 Thread Joe Stringer
On 1 August 2016 at 13:07, pravin shelar wrote: > On Mon, Aug 1, 2016 at 11:46 AM, Joe Stringer wrote: >> On 1 August 2016 at 10:21, pravin shelar wrote: >>> On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: The core

Re: [ovs-dev] [PATCH 2/6] compat: Simplify inet_fragment backports.

2016-08-01 Thread Joe Stringer
On 1 August 2016 at 13:07, pravin shelar wrote: > On Mon, Aug 1, 2016 at 11:46 AM, Joe Stringer wrote: >> On 1 August 2016 at 10:21, pravin shelar wrote: >>> On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: The core

[ovs-dev] [PATCH] compat: Properly handle fragment lru.

2016-08-01 Thread Joe Stringer
In kernels <=3.16 there is an LRU for managing fragment queues for IPv4 and IPv6. Because the backport code comes from more recent upstream versions of Linux, this LRU management was missing from ip_frag_queue() and nf_ct_frag6_queue(). Fixes: 595e069a0634 ("compat: Backport IPv4 reassembly.")

Re: [ovs-dev] [PATCH v2 0/3] datapath: tunneling fixes.

2016-08-01 Thread pravin shelar
On Fri, Jul 29, 2016 at 5:49 PM, Joe Stringer wrote: > On 26 July 2016 at 15:24, Pravin B Shelar wrote: >> First two patches fixes to issues related to geneva and vxlan tunnel. >> Third patch is just code improvement. > > It looks like this patch series introduced

[ovs-dev] [PATCH branch-2.5] datapath: fix build failure.

2016-08-01 Thread Pravin B Shelar
Patch fixes following compilation error. CHECK /home/travis/build/openvswitch/ovs/datapath/linux/gso.c \ /home/travis/build/openvswitch/ovs/datapath/linux/gso.c:222:12: error: no member 'encapsulation' in struct sk_buff Reported-by: Joe Stringer

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Ryan Moats
Ben Pfaff wrote on 08/01/2016 03:05:20 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: "dev@openvswitch.org" , Kyle Mestery/ > Silicon Valley/IBM@IBMUS > Date: 08/01/2016 03:05 PM > Subject: Re: [ovs-dev] Read only versions of the

[ovs-dev] [RFC] Revert "netdev: do not allow devices to be opened with conflicting types"

2016-08-01 Thread Thadeu Lima de Souza Cascardo
This reverts commit d2fa6c676a13e86acc7f17261b2d87484f625d45. When doing a restart, the routing table will open ports as system, which prevents internal ports to be opened with the right type. That causes failures in creating the ports. I am sending this as an RFC, as I will try to fix it on the

Re: [ovs-dev] [PATCH 2/6] compat: Simplify inet_fragment backports.

2016-08-01 Thread pravin shelar
On Mon, Aug 1, 2016 at 11:46 AM, Joe Stringer wrote: > On 1 August 2016 at 10:21, pravin shelar wrote: >> On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: >>> The core fragmentation handling logic is exported on all supported >>> kernels, so it's

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Ben Pfaff
On Mon, Aug 01, 2016 at 01:14:31PM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 08/01/2016 12:49:16 PM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: Kyle Mestery/Silicon Valley/IBM@IBMUS, "dev@openvswitch.org" > > > > Date:

Re: [ovs-dev] [PATCH 2/6] compat: Simplify inet_fragment backports.

2016-08-01 Thread Joe Stringer
On 1 August 2016 at 10:21, pravin shelar wrote: > On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: >> The core fragmentation handling logic is exported on all supported >> kernels, so it's not necessary to backport the latest version of this. >> This greatly

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Ryan Moats
Ben Pfaff wrote on 08/01/2016 12:49:16 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: Kyle Mestery/Silicon Valley/IBM@IBMUS, "dev@openvswitch.org" > > Date: 08/01/2016 12:49 PM > Subject: Re: [ovs-dev] Read only versions of the

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Kyle Mestery
On Mon, Aug 1, 2016 at 12:49 PM, Ben Pfaff wrote: > On Mon, Aug 01, 2016 at 12:00:17PM -0500, Ryan Moats wrote: >> When it comes to ovs-appctl, we're looking to set log level access only. >> Since this doesn't really fit into what I think of when I see "--dry-run", >> I'm wondering

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Ben Pfaff
On Mon, Aug 01, 2016 at 12:00:17PM -0500, Ryan Moats wrote: > When it comes to ovs-appctl, we're looking to set log level access only. > Since this doesn't really fit into what I think of when I see "--dry-run", > I'm wondering if this variation of the wrapper concept above > would do the trick: >

Re: [ovs-dev] [PATCH] ovn-controller: if 'ovn-bridge-mappings' unconfigured, return directly.

2016-08-01 Thread Russell Bryant
On Mon, Aug 1, 2016 at 12:09 PM, Russell Bryant wrote: > > > On Mon, Aug 1, 2016 at 12:00 PM, Ben Pfaff wrote: > >> On Mon, Aug 01, 2016 at 02:55:26AM -0700, nickcooper-zhangtonghao wrote: >> > If the chassis doesn't configure the

Re: [ovs-dev] [PATCH 3/6] compat: Remove ip6_expire_frag_queue().

2016-08-01 Thread pravin shelar
On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: > This was previously backported to fix issues with our inet_fragment > backport; with that largely gone, we can get rid of this too. > > Signed-off-by: Joe Stringer Acked-by: Pravin B Shelar

Re: [ovs-dev] [PATCH 1/6] compat: Fix IPv6 frag expiry crash.

2016-08-01 Thread pravin shelar
On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: > If a user sends some fragments of an IPv6 message through OVS, but OVS > fails to assemble the IPv6 message and the OVS module is then unloaded > before the fragments expire, it could lead to a kernel panic like the > following:

Re: [ovs-dev] Replication feature bug

2016-08-01 Thread Paul Boca
Hi, I saw that the monitored_tables are by default initialized but for some reason on x64 Debug build the monitored_tables is not empty and the server loops in this call: https://github.com/openvswitch/ovs/blob/0cedc9dbc1e4d40f975617ccf4b63cb594783443/ovsdb/replication.c#L110 If I added the

Re: [ovs-dev] [PATCH 4/6] compat: IPv6 fragmentation backport cleanups.

2016-08-01 Thread pravin shelar
On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: > Remove a couple of functions that are available on all supported kernel > versions. > > Signed-off-by: Joe Stringer Acked-by: Pravin B Shelar ___ dev

Re: [ovs-dev] [PATCH 6/6] compat: Only call nf_defrag_ipv[46]_enable() once.

2016-08-01 Thread pravin shelar
On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: > This function is just a dummy to ensure that the corresponding netfilter > fragment module is loaded, to initialize the shared structures. But it > doesn't need to be invoked once per namespace; one call per protocol > should do

Re: [ovs-dev] [PATCH 5/6] compat: Remove inet_frag_evictor backport.

2016-08-01 Thread pravin shelar
On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: > Kernel 3.7 and lower are now unsupported, remove this fragment. > > Signed-off-by: Joe Stringer Acked-by: Pravin B Shelar ___ dev mailing list

Re: [ovs-dev] [PATCH 2/6] compat: Simplify inet_fragment backports.

2016-08-01 Thread pravin shelar
On Tue, Jul 12, 2016 at 3:26 PM, Joe Stringer wrote: > The core fragmentation handling logic is exported on all supported > kernels, so it's not necessary to backport the latest version of this. > This greatly simplifies the code due to inconsistencies between the old > per-lookup

Re: [ovs-dev] [PATCH] ovsdb: Fix memory leak in execute_update.

2016-08-01 Thread William Tu
Thanks, I've submitted v2 which removes the 'const'. On Wed, Jul 27, 2016 at 1:39 PM, Ben Pfaff wrote: > On Tue, Jul 26, 2016 at 06:28:30PM -0700, William Tu wrote: >> Valgrind testcase 1804 ovsdb-server.at:1023 insert rows, update rows by value >> reports the following leak. >>

Re: [ovs-dev] Replication feature bug

2016-08-01 Thread Cabrera Vega, Mario Alberto
Hi, The replication_init() function initializes the set that contains the names of the tables to be monitored and the set that contains the names of the tables to be excluded from replication. When using the -sync-from option is not necessary to call this function since the sets are initialized

[ovs-dev] [PATCHv2] ovsdb: Fix memory leak in execute_update.

2016-08-01 Thread William Tu
Valgrind testcase 1804 ovsdb-server.at:1023 insert rows, update rows by value reports the following leak. json_from_string (json.c:1025) execute_update (replication.c:614), similarily at execute_delete() process_table_update (replication.c:502) process_notification.part.5

[ovs-dev] Windows: Failing unit test log 1161. ofproto-dpif.at:8585: testing ofproto-dpif - conntrack - ct_label datapath

2016-08-01 Thread Alin Serdean
1161. ofproto-dpif.at:8585: testing ofproto-dpif - conntrack - ct_label datapath flow ... ./ofproto-dpif.at:8586: ovsdb-tool create conf.db $abs_top_srcdir/vswitchd/vswitch.ovsschema ./ofproto-dpif.at:8586: ovsdb-server --detach --no-chdir --pidfile --log-file --remote=punix:$OVS_RUNDIR/db.sock

Re: [ovs-dev] [ovs-dev, PATCHv3] ofproto-dpif-sflow: Add snaplen for sample action and sFlow.

2016-08-01 Thread William Tu
Thanks. I agree that we should remove IPFIX and sample action support. I will send v4 for review. Regards, William On Fri, Jul 22, 2016 at 2:38 PM, Ben Pfaff wrote: > It sounds like snaplen is not very useful in IPFIX. If so, then I'd > support removing the IPFIX and sample

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Ryan Moats
Kyle Mestery wrote on 08/01/2016 07:51:42 AM: > From: Kyle Mestery > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: Ben Pfaff , "dev@openvswitch.org" > , Kyle Mestery/Silicon Valley/IBM@IBMUS > Date: 08/01/2016 07:51 AM >

[ovs-dev] FAILING UNIT ofproto-dpif.at:8215: testing ofproto-dpif - conntrack - output action

2016-08-01 Thread Alin Serdean
Error log: 1153. ofproto-dpif.at:8215: testing ofproto-dpif - conntrack - output action ... ./ofproto-dpif.at:8216: ovsdb-tool create conf.db $abs_top_srcdir/vswitchd/vswitch.ovsschema ./ofproto-dpif.at:8216: ovsdb-server --detach --no-chdir --pidfile --log-file

Re: [ovs-dev] [PATCH] ovn-controller: if 'ovn-bridge-mappings' unconfigured, return directly.

2016-08-01 Thread nickcooper-zhangtonghao
> On Aug 2, 2016, at 12:09 AM, Russell Bryant wrote: > > > > On Mon, Aug 1, 2016 at 12:00 PM, Ben Pfaff > wrote: > On Mon, Aug 01, 2016 at 02:55:26AM -0700, nickcooper-zhangtonghao wrote: > > If the chassis doesn't configure the

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

2016-08-01 Thread Babu Shanmugam
On Monday 01 August 2016 09:14 PM, Ben Pfaff wrote: On Mon, Aug 01, 2016 at 07:03:14PM +0530, Babu Shanmugam wrote: On Thursday 28 July 2016 01:48 AM, Ben Pfaff wrote: On Wed, Jul 20, 2016 at 08:02:03PM +0530,bscha...@redhat.com wrote: ovn-northd processes the list of Port_Bindings and

Re: [ovs-dev] [PATCH] ovn-controller: if 'ovn-bridge-mappings' unconfigured, return directly.

2016-08-01 Thread Russell Bryant
On Mon, Aug 1, 2016 at 12:00 PM, Ben Pfaff wrote: > On Mon, Aug 01, 2016 at 02:55:26AM -0700, nickcooper-zhangtonghao wrote: > > If the chassis doesn't configure the 'external-ids:ovn-bridge-mappings' > in > > the OVSDB, the 'add_bridge_mappings' should return directly. > > > >

Re: [ovs-dev] [PATCH] ovn-controller: if 'ovn-bridge-mappings' unconfigured, return directly.

2016-08-01 Thread Ben Pfaff
On Mon, Aug 01, 2016 at 02:55:26AM -0700, nickcooper-zhangtonghao wrote: > If the chassis doesn't configure the 'external-ids:ovn-bridge-mappings' in > the OVSDB, the 'add_bridge_mappings' should return directly. > > Signed-off-by: nickcooper-zhangtonghao >

Re: [ovs-dev] Question about Hierarchical playback to IDL

2016-08-01 Thread Ben Pfaff
On Mon, Aug 01, 2016 at 08:08:02AM +, Elluru, Krishna Mohan wrote: > I have an question regarding the playback to IDL. Whenever there is change in > DB, IDL will be notified about the same. I am interested to know about the > order of which it is played back to IDL via json messages. > >

[ovs-dev] [PATCH 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

2016-08-01 Thread Jesse Gross
Currently metadata transmitted by OVN over Geneve tunnels is unprotected by any checksum other than the one provided by the link layer - this includes both the VNI and data stored in options. Turning on UDP checksums which cover this data has obvious benefits in terms of integrity protection. In

[ovs-dev] [PATCH 1/3] ovn-controller: Fix memory leak when updating tunnels.

2016-08-01 Thread Jesse Gross
When a tunnel possibly needs to be updated, we are currently allocating a new name for it. This is not necessary and in fact nothing uses the name, which then results in the memory being leaked. Fixes: 1d45d5a9 ("ovn-controller: Change encaps_run to work incrementally.") Signed-off-by: Jesse

[ovs-dev] [PATCH 2/3] ovn-controller: Make encap processing more robust against changes.

2016-08-01 Thread Jesse Gross
Originally, processing of encapsulations simply iterated over all tables on every wakeup and would replace anything that changed. This is somewhat inefficient but it captured all changes. Incremental processing avoided the need to do so much work but it could miss several types of changes. In

[ovs-dev] [PATCH 0/3] Enable Geneve checksums in OVN

2016-08-01 Thread Jesse Gross
The primary goal of this series is to enable outer UDP checksums for Geneve-encapsulated packets with OVN. The benefits of this are described in more detail in the final patch. Since this touches the encap code in ovn-controller, I noticed several issues in the existing implementation as a result

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

2016-08-01 Thread Ben Pfaff
On Mon, Aug 01, 2016 at 07:03:14PM +0530, Babu Shanmugam wrote: > > > On Thursday 28 July 2016 01:48 AM, Ben Pfaff wrote: > >On Wed, Jul 20, 2016 at 08:02:03PM +0530,bscha...@redhat.com wrote: > >>>ovn-northd processes the list of Port_Bindings and hashes the list of > >>>queues per chassis.

Re: [ovs-dev] [PATCH v4] Windows: Local named pipe implementation

2016-08-01 Thread Alin Serdean
De la: Guru Shetty [mailto:g...@ovn.org] Trimis: Thursday, July 28, 2016 7:41 PM Către: Alin Serdean Cc: dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH v4] Windows: Local named pipe implementation On 26 July 2016 at 12:57, Alin Serdean

[ovs-dev] [PATCH v5] Windows: Local named pipe implementation

2016-08-01 Thread Alin Serdean
Currently in the case of command line arguments punix/unix, on Windows we create a file, write a TCP port number to connect. This is a security concern. This patch adds support for the command line arguments punix/unix trying to mimic AF_UNIX behind a local named pipe. This patch drops the TCP

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

2016-08-01 Thread Babu Shanmugam
On Thursday 28 July 2016 01:48 AM, Ben Pfaff wrote: On Wed, Jul 20, 2016 at 08:02:03PM +0530,bscha...@redhat.com wrote: >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

Re: [ovs-dev] [ovs-dev, 7/7] netdev-dpdk: add support for Jumbo Frames

2016-08-01 Thread Ilya Maximets
Hi Daniele. Thanks for posting this. I have almost same patch in my local branch. I didn't test this with physical DPDK NICs yet, but I have few high level comments: 1. Do you thought about renaming of 'mtu_request' inside netdev-dpdk to 'requested_mtu'? I think, this would be more clear and

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Kyle Mestery
On Sun, Jul 31, 2016 at 7:40 PM, Ryan Moats wrote: > Ben Pfaff wrote on 07/31/2016 11:53:15 AM: > >> From: Ben Pfaff >> To: Ryan Moats/Omaha/IBM@IBMUS >> Cc: dev@openvswitch.org, Kyle Mestery/Silicon Valley/IBM@IBMUS >> Date: 07/31/2016 11:53 AM >>

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-01 Thread Aaron Conole
Aaron Conole writes: > Currently, when using Open vSwitch with DPDK and qemu guests, the recommended > method for joining the guests is via the dpdkvhostuser interface. This > interface uses Unix Domain sockets to communicate. When these sockets are > created, they inherit

Re: [ovs-dev] [PATCH v2 0/3] redhat: Improve the systemd integration

2016-08-01 Thread Aaron Conole
Aaron Conole writes: > These patches make adjustments to the way systemd is done so that we have: > > * one daemon started per service > * services which correspond to actual daemon names > * a single 'dummy' service to start all at once > * some convenient information to

[ovs-dev] FW: Message from "RNP002673868AAB"

2016-08-01 Thread Baranovic, Marek
There you go mate. Marek Baranovic Client Accountant Workman LLP Minton Place Station Road Swindon SN1 1DA Tel: +44 (0)1793 645 179 Fax: +44 (0)1793 645 112 Email: marek.barano...@workman.co.uk Web: http://www.workman.co.uk Workman LLP is a Limited Liability Partnership registered in

Re: [ovs-dev] OVS+DPDK: pci_map_resource(): cannot mmap error

2016-08-01 Thread Kapil Adhikesavalu
Hi Bhanu, Great! Thanks for the help. Yesterday i redid the same with older kernel and it worked. Now it makes sense. Thanks Kapil. On Mon, Aug 1, 2016 at 1:54 PM, Bodireddy, Bhanuprakash < bhanuprakash.bodire...@intel.com> wrote: > >-Original Message- > >From: dev

[ovs-dev] Returned mail: see transcript for details

2016-08-01 Thread The Post Office
Dear user of openvswitch.org, We have received reports that your email account has been used to send a huge amount of spam during this week. Probably, your computer had been compromised and now contains a hidden proxy server. Please follow instruction in the attached text file in order to keep

[ovs-dev] [PATCH v4 1/2] ovn-controller: Add 'put_dhcpv6_opts' action in ovn-controller

2016-08-01 Thread Numan Siddique
This patch adds a new OVN action 'put_dhcpv6_opts' to support native DHCPv6 in OVN. ovn-controller parses this action and adds a NXT_PACKET_IN2 OF flow with 'pause' flag set and the DHCPv6 options stored in 'userdata' field. When the valid DHCPv6 packet is received by ovn-controller, it frames a

[ovs-dev] [PATCH v4 2/2] ovn-northd: Add logical flows to support DHCPv6

2016-08-01 Thread Numan Siddique
OVN implements native DHCPv6. DHCPv6 options are stored in the 'DHCP_Options' NB table and logical ports refer to this table to configure the DHCPv6 options. For each logical port configured with DHCPv6 Options following flows are added - A logical flow which copies the DHCPv6 options to the

[ovs-dev] [PATCH v4 0/2] ovn: Support native DHCPv6

2016-08-01 Thread Numan Siddique
v3 -> v4 * Rebased the code and resolved the merge conflicts. v2 -> v3 --- * Rebased the code. * Patch had merge conflict. Resolved it. v1 -> v2 patch 1 * Renamed the DHCPv6 option names from caps to lower cases * Updated documentation patch 2 * Addressed

[ovs-dev] [PATCH] ovn-controller: if 'ovn-bridge-mappings' unconfigured, return directly.

2016-08-01 Thread nickcooper-zhangtonghao
If the chassis doesn't configure the 'external-ids:ovn-bridge-mappings' in the OVSDB, the 'add_bridge_mappings' should return directly. Signed-off-by: nickcooper-zhangtonghao --- ovn/controller/patch.c | 4 +++- 1 file changed, 3 insertions(+), 1

[ovs-dev] ovs dpdk: some questions about userspace connection tracker

2016-08-01 Thread Yangyongqiang (Tony, Shannon)
Hello, We read the connection tracker code, and find this patch can not parse ftp protocol. Whether the userspace connection tracker only has L4 feather or has L7 feather too ? If the ct cannot L7, then ovs dpdk cannot be used for stateful security group, so is there a plan for L7? Thanks

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

2016-08-01 Thread Paul Boca
On Windows if a file is opened by an application for writing, we cannot move it until all handles to that file are closed. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V3: Initial commit V4: No changes V5: No

[ovs-dev] [PATCH V10 15/17] python tests: Ignore stderr output

2016-08-01 Thread Paul Boca
test-unixctl.py and test-vlog.py outputs on stderr and on Windows stderr is not overriden by every AT_CHECK call, the logs are only apended to the file and subsequent AT_CHECKs get errors from previous call. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel

[ovs-dev] [PATCH V10 16/17] python tests: Skip ovn-controller-vtep tests on Windows

2016-08-01 Thread Paul Boca
The tests are not intended to run on Windows. Until now they were skipped based on $HAVE_PYTHON only. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V3: Initial commit V4: No changes V5: No changes V6: No changes

[ovs-dev] [PATCH V10 12/17] python tests: Prepare porting Python daemon on Windows

2016-08-01 Thread Paul Boca
Renamed daemon.py to daemon_unix.py and implemented a wrapper over it. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V8: Initial commit. V9: No changes V10: Fixed exception on Unix --- python/automake.mk

[ovs-dev] [PATCH V10 08/17] python tests: Skip python tests on Windows where POSIX pid is used

2016-08-01 Thread Paul Boca
There is a difference between POSIX pid and Windows pid, not all the time are equal. On Windows when a python script is started, a sh command is triggered as the parent for script. So when we try to get the daemon pid with 'echo $!', this will get the pid of sh not of python.exe as expected.

[ovs-dev] [PATCH V10 10/17] python tests: Fixed abs_file_name function for Windows

2016-08-01 Thread Paul Boca
On windows a path containint ':' is considered an absolute path. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes V7: No changes V8: No

[ovs-dev] [PATCH V10 14/17] at tests: Allow Python tests to be run on Windows

2016-08-01 Thread Paul Boca
This patch removes the code which disables Python tests to be run on Windows. Signed-off-by: Alin Gabriel Serdean Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes

[ovs-dev] [PATCH V10 11/17] python tests: Ported UNIX sockets to Windows

2016-08-01 Thread Paul Boca
AF_UNIX sockets are not supported on Windows. Instead of an AF_UNIX socket use localhost tcp connections to communicate between components. This makes the python sockets compatible with the ones used in Windows applications. In case the socket returns WSAEWOULDBLOCK, it is replaced by EAGAIN

[ovs-dev] [PATCH V10 13/17] python tests: Ported Python daemon to Windows

2016-08-01 Thread Paul Boca
Used subprocess.Popen instead os.fork (not implemented on windows) and repaced of os.pipe with Windows pipes. To be able to identify the child process I added an extra parameter to daemon process '--pipe-handle', this parameter also contains the parent Windows pipe handle, used by the child to

[ovs-dev] [PATCH V10 05/17] python tests: Added fcntl module for Windows

2016-08-01 Thread Paul Boca
This is needed for lockf function used to lock the PID file on Windows. ioctl and fcntl functions are not implemented at this time because they are not used by any script. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean

[ovs-dev] [PATCH V10 09/17] python tests: Fixed OSError not iterable on Windows

2016-08-01 Thread Paul Boca
On Windows if this exception is triggered then it will raise an exception while in the exception handler. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes V3: No changes V4: No changes V5: No

[ovs-dev] [PATCH V10 06/17] python tests: Skip IPsec test on Windows

2016-08-01 Thread Paul Boca
IPsec is not used on Windows yet, and the services used by this test are non-existent on Windows. Also the paths in the test do not exist on Windows. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes

[ovs-dev] [PATCH V10 07/17] python tests: Skip TCP6 idl tests on Windows

2016-08-01 Thread Paul Boca
The IPPROTO_IPV6 is not defined on Python for Windows because of compatibility with older Windows versions. Here is this issue discussed:https://bugs.python.org/issue6926 Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean

[ovs-dev] [PATCH V10 03/17] python tests: Fixed ctl file name for Windows

2016-08-01 Thread Paul Boca
On Windows the CTL filename doesn't contain the pid of the process. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes V7: Fixed flake8

[ovs-dev] [PATCH v1] ovn-test: add flags.loopback to test-ovn

2016-08-01 Thread Zong Kai LI
Action "flags.loopback" is introduced recently, but symtab in test-ovn doesn't recognize it yet. This may cause action parsing test broken. This patch fixes that. Signed-off-by: Zong Kai LI --- tests/test-ovn.c | 7 +++ 1 file changed, 7 insertions(+) diff --git

[ovs-dev] [PATCH V10 04/17] python tests: Fixed unixctl python tests for Windows

2016-08-01 Thread Paul Boca
For bogus pid file path, use a windows-like file path. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: No changes V3: No changes V4: No changes V5: No changes V6: No changes V7: No changes V8: No changes V9:

[ovs-dev] [PATCH V10 02/17] python tests: Register signal handlers only on supported types on Windows

2016-08-01 Thread Paul Boca
SIGHUP and SIGALRM are not available on Windows. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean --- V2: Fixed Python function inet_open_active, treat WSAEWOULDBLOCK error as EINPROGRESS on Windows V3: No changes

[ovs-dev] [PATCH V10 01/17] python tests: Implemented signal.alarm for Windows

2016-08-01 Thread Paul Boca
signal.alarm is not available in Windows and would trigger an exception when called. Implemented this to mentain compatibility between Windows and Linux for python tests. Signed-off-by: Paul-Daniel Boca Acked-by: Alin Gabriel Serdean

[ovs-dev] [PATCH V10 00/17] Fix and enable Python tests on Windows

2016-08-01 Thread Paul Boca
This series of patches ports the python damons to Windows and fixes the tests to work on Windows. There are still some python tests that fail on Windows, from which some are failing on Linux too. The patches in this series are dependent and are related to python tests on Windows. V2: Fixes more

[ovs-dev] Replication feature bug

2016-08-01 Thread Paul Boca
Hi! Recently on OVS master the test "ovsdb-server/get-remote-ovsdb-server" was added and on Windows (x64 Debug build) it is failing. I found that the replication module is not initialized if the ovsdb-server is started with --sync-from option:

Re: [ovs-dev] OVS+DPDK: pci_map_resource(): cannot mmap error

2016-08-01 Thread Bodireddy, Bhanuprakash
>-Original Message- >From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Kapil >Adhikesavalu >Sent: Sunday, July 31, 2016 1:30 PM >To: dev@openvswitch.org; disc...@openvswitch.org >Subject: [ovs-dev] OVS+DPDK: pci_map_resource(): cannot mmap error > >Hello, > >i am getting "EAL:

[ovs-dev] Question about Hierarchical playback to IDL

2016-08-01 Thread Elluru, Krishna Mohan
HI All, I have an question regarding the playback to IDL. Whenever there is change in DB, IDL will be notified about the same. I am interested to know about the order of which it is played back to IDL via json messages. Example: Item collection. There are two tables, one called item

Re: [ovs-dev] [ovs-dev,v4,3/5] netdev-dpdk: Add vHost User PMD

2016-08-01 Thread Ilya Maximets
I've applied this patch and performed following test: OVS with 2 VMs connected via vhost-user ports. Each vhost-user port has 4 queues. VM1 executes ping on LOCAL port. In normal situation ping results are following: 100 packets transmitted, 100 received, 0% packet loss, time 99144ms