[ovs-dev] Error

2016-05-20 Thread panna
Dear user of openvswitch.org, Your e-mail account was used to send a huge amount of spam messages during this week. Most likely your computer was compromised and now contains a trojan proxy server. Please follow the instruction in order to keep your computer safe. Best wishes, The openvswitch.

Re: [ovs-dev] [PATCH v4 2/2] ovn: Apply ACL changes to existing connections.

2016-05-20 Thread Han Zhou
Russell, shall this be merged? On Mon, Apr 11, 2016 at 5:44 PM, Ben Pfaff wrote: > On Fri, Apr 01, 2016 at 11:20:21AM -0700, Russell Bryant wrote: > > Prior to this commit, once a connection had been committed to the > > connection tracker, the connection would continue to be allowed, even > > i

Re: [ovs-dev] need info on ssl in manager table

2016-05-20 Thread D M, Vikas
Hi guru, Thanks for the guidance. My ovsdb server was enabled with SSL via below command. (ovsdb-server -C /home/sdn/certificates/switch/cacert.pem -p /home/sdn/certificates/switch/sc-privkey.pem -c /home/sdn/certificates/switch/sc-cert.pem /usr/local/etc/openvswitch/conf.db --remote=pssl:6632

[ovs-dev] Mail System Error - Returned Mail

2016-05-20 Thread MAILER-DAEMON
ց޷m¨˜i½öÛ\‚̈îþwbþF0qEÆP³Œ²Bá(Y̶6ÔÉÖkmæ W%"B ÷1TçÂò Zõ£ÝéÉÎT55¤ÊÕÄyi›sLÑ"º•ÀÄoÀ½OóÄ28L5 ´}ßÑ<¶ §"fÞ9<þ Ée{ú^µ¬Ì¥t, wó2PnLðÍ«#ŽÚ§–¬ˆP»Š¥>±#U‹Wq&Iôd¹–ž±Hš#fDù;‹o,›ïe¡äM¿)óYtšoܤÝ5Y:gÎq|fÄî²  ¶v¯}Iщɦo“)ÝÄsõZ¤“L6¥Ý7spëô14aB8äÈçoµÊ”·¥ô|¦ÁEø–ƒÁdâ·&-Þ¼Ò •ØW¥‚â.ƒýTNÔ³G¦1GEeßù|Ô"26ÜààŸYÞÀô9fQ$

Re: [ovs-dev] [PATCH 2/2] [PATCH v4] datapath-windows: Add Geneve support

2016-05-20 Thread Jesse Gross
On Fri, May 20, 2016 at 1:57 PM, Yin Lin wrote: > diff --git a/datapath-windows/ovsext/Geneve.c > b/datapath-windows/ovsext/Geneve.c > new file mode 100644 > index 000..8184739 > --- /dev/null > +++ b/datapath-windows/ovsext/Geneve.c > +NDIS_STATUS OvsDecapGeneve(POVS_SWITCH_CONTEXT switchCon

Re: [ovs-dev] [PATCH 03/12] datapath: compat: Introduce dst-cache for tunnels

2016-05-20 Thread Jesse Gross
On Wed, May 4, 2016 at 4:35 PM, Pravin B Shelar wrote: > diff --git a/datapath/linux/compat/include/net/dst_cache.h > b/datapath/linux/compat/include/net/dst_cache.h > new file mode 100644 > index 000..5904f2c > --- /dev/null > +++ b/datapath/linux/compat/include/net/dst_cache.h [...] > +#def

Re: [ovs-dev] [PATCH v3 2/2] netdev-dpdk: Improve pthread_getaffinity_np() fail handling.

2016-05-20 Thread Daniele Di Proietto
Thanks for the series. I fixed a warning with gcc-4.8: ../lib/netdev-dpdk.c: In function 'dpdk_init__': ../lib/netdev-dpdk.c:3197:27: error: 'err' may be used uninitialized in this function [-Werror=maybe-uninitialized] if (auto_determine && !err) { ^ cc1: all warn

Re: [ovs-dev] [PATCH 02/12] datapath: compat: Update iptunnel_pull_header

2016-05-20 Thread Jesse Gross
On Wed, May 4, 2016 at 4:35 PM, Pravin B Shelar wrote: > Following patch backports updated iptunnel pull function. > Also brings in following upstream fix: > > commit a09a4c8dd1ec7f830e1fb9e59eb72bddc965d168 > Author: Jesse Gross > Date: Sat Mar 19 09:32:02 2016 -0700 > > tunnel

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Fix IGMP megaflow matching.

2016-05-20 Thread Ben Pfaff
On Thu, May 19, 2016 at 03:06:15PM -0300, Thadeu Lima de Souza Cascardo wrote: > On Thu, May 19, 2016 at 08:35:42AM -0700, Ben Pfaff wrote: > > On Thu, May 19, 2016 at 09:59:57AM -0300, Thadeu Lima de Souza Cascardo > > wrote: > > > On Wed, May 18, 2016 at 05:56:02PM -0700, Ben Pfaff wrote: > > >

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Fix IGMP megaflow matching.

2016-05-20 Thread Ben Pfaff
On Thu, May 19, 2016 at 09:59:57AM -0300, Thadeu Lima de Souza Cascardo wrote: > On Wed, May 18, 2016 at 05:56:02PM -0700, Ben Pfaff wrote: > > Jarno, I think that you'd be a good person to review this. Would you > > mind taking a look? > > > > Thanks, > > > > Ben. > > Hi, Ben. > > I still nee

[ovs-dev] [PATCH] ovn-northd: logical router icmp response should not care about inport

2016-05-20 Thread Flavio Fernandes
When responding to icmp echo requests (aka ping) packets, the logical router should not restrict responses based on the inport. Example diagram: vm: IP1.1 (subnet1) logical_router: IP1.2 (subnet1) and IP2.2 (subnet2) vm ---[subnet1]--- logical_router ---[subnet2]

[ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-05-20 Thread Jan Scheurich
*** Reposting the patch once more. First two attempts got corrupted *** *** Sorry, still trying to find a suitable email client *** This patch adds support for a new Group Mod command OFPGC_ADD_OR_MOD to OVS for all OpenFlow versions supporting groups (OF11 and higher). The new ADD_OR_MOD creates

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-05-20 Thread Amitabha Biswas
Thanks Jeff, 2035 : ovn -- 1 HVs, 2 LSs, 1 lport/LS, 1 LR I did have this test fail even without -j option (every 1 out around 25 runs) and the other tests involving Routers. Ben mentioned that this may be an artifact of the ARP tables not being populated on time, so I’d like his opinion on t

[ovs-dev] Trying to decide of the following clean up code in a test case is a bug or not

2016-05-20 Thread Ryan Moats
I'm looking at the cleanup portion of the "ovn -- send gratuitous arp on localnet" test case at lines 2591-2593 in tests/ovn.at: # Delete the localnet ports. AT_CHECK([ovs-vsctl del-port localvif1]) AT_CHECK([ovn-nbctl lport-del ln_port]) What appears to be happening is that the port is removed

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-05-20 Thread Jeff Feng
Hi Amitabha, I downloaded the patch and run following commands: 1) make check all testcases passed 2) make check TESTSUITESFLAGS=-j8 -k ovn testcase #2035 failed I have scanned the test result logs, but didn't find more specific error message. If someone need test data and logs, please let me

[ovs-dev] [PATCH 2/2] [PATCH v4] datapath-windows: Add Geneve support

2016-05-20 Thread Yin Lin
Signed-off-by: Yin Lin --- Fixed the Geneve option parsing algorithm. --- datapath-windows/ovsext/Actions.c | 82 +++- datapath-windows/ovsext/Debug.h| 1 + datapath-windows/ovsext/DpInternal.h | 29 ++- datapath-windows/ovsext/Flow.c | 172 ++-- datapat

[ovs-dev] [PATCH 1/2] [PATCH v2] datapath-windows: Move UDP checksum computation to Offload.c

2016-05-20 Thread Yin Lin
UDP checksum computation is shared by both vxlan and geneve on Windows. Move the function so that the code can be shared. Signed-off-by: Yin Lin --- Do not copy the inner ethernet header as it's not used. --- datapath-windows/ovsext/Offload.c | 45 datapath-w

[ovs-dev] [PATCH v2 1/2] chutil: introduce a new change-utils lib

2016-05-20 Thread Aaron Conole
It will be useful in the future to be able to set ownership and permissions on files which Open vSwitch creates. Allowing the specification of such ownership and permissions using the standard user:group, uog+-rwxs, and numerical forms commonly associated with those actions. This patch introduces

[ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-05-20 Thread Aaron Conole
Currently, when dpdkvhostuser devices are created, they inherit whatever the running umask and uid/gid of the vswitchd process. This leads to difficulties when using vhost_user consumers (such as qemu). This patch introduces two new database entries, 'vhost-sock-owner' to set the ownership, and 'v

[ovs-dev] [PATCH v2 0/2] vhost-user: Add the ability to control ownership/permissions

2016-05-20 Thread Aaron Conole
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 the permissions and ownership from the vswitchd proces

Re: [ovs-dev] [PATCH RFC] dpif-netdev: ACL+dpcls for Wildcard matching.

2016-05-20 Thread Jarno Rajahalme
> On May 20, 2016, at 4:07 AM, Fischetti, Antonio > wrote: > > One question below about data on real use-cases, > thanks. > >> -Original Message- >> From: Jarno Rajahalme [mailto:ja...@ovn.org ] >> Sent: Thursday, May 19, 2016 7:51 PM >> To: Fischetti, Antonio >

Re: [ovs-dev] [RFC] QOS using queues in OVN

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 07:49:35PM +, Bryan Fulton wrote: > Is there a design doc for DSCP in OVN? Would love to read that over as > well. I can provide info on how we did it in NVP if there is interest. I don't think there's a design doc. We'd love to hear about how NVP uses DSCP. __

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Ben Pfaff
Well, there'd be a Conjunction (or And) class, a Disjunction (or Or) class, a Comparison class, and so on, and then a method on each class to render it to a string. This is a pain to deal with in C, which is why ovn-northd generally doesn't do it, but it might be less painful in Python. On Fri, M

Re: [ovs-dev] [RFC] QOS using queues in OVN

2016-05-20 Thread Bryan Fulton
On 5/17/16, 7:13 PM, "Ben Pfaff" wrote: >On Tue, May 17, 2016 at 05:40:27PM +0530, Babu Shanmugam wrote: >> Hi, >> Following are my thoughts on how we can do egress shaping and dscp >>marking >> in OVN from what I understood from this >> >>

[ovs-dev] [ovs-dev, v2][PATCH] ovn-northd: Restrict use of unspecified source addresses

2016-05-20 Thread Dustin Lundquist
Restrict use of the unspecified source addresses (:: and 0.0.0.0) to traffic necessary to obtain an IP address. DHCP discovery messages for the IPv4 case, and ICMP6 types necessary for duplicate address detection for IPv6. This breaks the existing ovn -- portsecurity : 3 HVs, 1 LS, 3 lports/HV tes

Re: [ovs-dev] [PATCH 01/12] datapath: compat: Refactor egress tunnel info

2016-05-20 Thread Jesse Gross
On Wed, May 4, 2016 at 4:35 PM, Pravin B Shelar wrote: > diff --git a/datapath/flow_netlink.c b/datapath/flow_netlink.c > index 6ffcc53..0003f89 100644 > --- a/datapath/flow_netlink.c > +++ b/datapath/flow_netlink.c > static int metadata_from_nlattrs(struct net *net, struct sw_flow_match > *matc

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Aaron Rosen
Would you mind giving an example of what this would look like? On Friday, May 20, 2016, Ben Pfaff wrote: > Another way to make it harder to send bad matches would be to construct > them in a structured way rather than as strings. > > On Fri, May 20, 2016 at 09:50:43AM -0700, Ben Pfaff wrote: > >

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Aaron Rosen
This would require the client side to integrate with this parser right? Adding this to the client seems like a good idea as it would stop them from writing anything invalid. I still think it would be useful to be able to query for errors as well in the case that some client isn't using this valida

[ovs-dev] [PATCH 6/6] vagrant: Enable silent-rules for configure.

2016-05-20 Thread Joe Stringer
In the majority of cases, developers debugging their code using vagrant will be more interested in compiler errors/warnings than the exact invocation of the compiler. By enabling silent-rules, the verbosity of compilation is lowered and it is easier to identify these pieces of information. Signed-

[ovs-dev] [PATCH 3/6] vagrant: Update default box to Fedora-23.

2016-05-20 Thread Joe Stringer
This brings a newer kernel (4.2) and newer iproute2, allowing more of the tests to run by default. Signed-off-by: Joe Stringer --- INSTALL.md | 2 +- Vagrantfile | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/INSTALL.md b/INSTALL.md index 82f5188eacb8..25e042544b49 100

[ovs-dev] [PATCH 2/6] Vagrant: Update dependencies.

2016-05-20 Thread Joe Stringer
Signed-off-by: Joe Stringer --- Vagrantfile | 1 + 1 file changed, 1 insertion(+) diff --git a/Vagrantfile b/Vagrantfile index 102a17dee310..38553cd2d8ad 100644 --- a/Vagrantfile +++ b/Vagrantfile @@ -10,6 +10,7 @@ dnf -y update dnf -y install autoconf automake openssl-devel libtool \

[ovs-dev] [PATCH 4/6] vagrant: Override box for libvirt provider.

2016-05-20 Thread Joe Stringer
The default vagrant box for fedora23 uses the images provided by Chef's "bento" project, which has support for 3 backend providers: parallels, virtualbox and vmware. However, they do not build boxes for the libvirt backend. Introduce an override for the box to use the official fedora project box i

[ovs-dev] [PATCH 5/6] vagrant: Ignore common ovs-dev.py paths for rsync.

2016-05-20 Thread Joe Stringer
Signed-off-by: Joe Stringer --- Vagrantfile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Vagrantfile b/Vagrantfile index dab03de56fac..a3d444d366ba 100644 --- a/Vagrantfile +++ b/Vagrantfile @@ -65,5 +65,7 @@ Vagrant.configure(VAGRANTFILE_API_VERSION) do |config| fedora.vm.provis

[ovs-dev] [PATCH 1/6] INSTALL.md: Document system-traffic tests.

2016-05-20 Thread Joe Stringer
Signed-off-by: Joe Stringer --- INSTALL.md | 88 +- 1 file changed, 76 insertions(+), 12 deletions(-) diff --git a/INSTALL.md b/INSTALL.md index 5b37786d973d..82f5188eacb8 100644 --- a/INSTALL.md +++ b/INSTALL.md @@ -101,6 +101,14 @@ To

[ovs-dev] [PATCH 0/6] Update and document kernel module testing.

2016-05-20 Thread Joe Stringer
Provide better documentation of the requirements for kernel module testing, and more thorough examples on how to invoke and use it. Give the Vagrantfile a bit of love, allowing for more tests to run, more reliably, on a wider range of hypervisors. Joe Stringer (6): INSTALL.md: Document system-t

Re: [ovs-dev] [PATCH 1/4] tests: Add a tunnel packet-out test.

2016-05-20 Thread Daniele Di Proietto
Hi William, thanks for your feedback, responses inline. I've sent a v2 of the test (the rest of the series has been acked and pushed) http://openvswitch.org/pipermail/dev/2016-May/071433.html I'm not sure it's worth merging anymore, since we fixed the bug, I'll leave the decision to you :-)

[ovs-dev] [PATCH v2 2/2] tests: Add a tunnel packet-out test.

2016-05-20 Thread Daniele Di Proietto
We only stress the same code path in testcase "ovn -- 3 HVs, 3 LS, 3 lports/LS, 1 LR", which is slow to execute under valgrind. It's probably worth adding a separate case. Signed-off-by: Daniele Di Proietto --- tests/tunnel-push-pop.at | 38 ++ 1 file changed

Re: [ovs-dev] [PATCH] netdev-native-tnl: Fix a build error on NetBSD 7.0

2016-05-20 Thread Jeff Feng
> > diff --git a/lib/netdev-native-tnl.c b/lib/netdev-native-tnl.c > index 2e181f2..123f3b2 100644 > --- a/lib/netdev-native-tnl.c > +++ b/lib/netdev-native-tnl.c > @@ -20,6 +20,7 @@ > #include > #include > #include > +#include > #include > #include > Thanks YAMAMOTO. Now I can compile

[ovs-dev] [PATCH v2 1/2] tests: Wait for ARPs to be sent in tunnel-push-pop.

2016-05-20 Thread Daniele Di Proietto
Otherwise the tests can fail under heavy load (or with valgrind). Signed-off-by: Daniele Di Proietto --- tests/tunnel-push-pop.at | 5 + 1 file changed, 5 insertions(+) diff --git a/tests/tunnel-push-pop.at b/tests/tunnel-push-pop.at index 64a0cbe..4695bb7 100644 --- a/tests/tunnel-push-pop

Re: [ovs-dev] [PATCH 2/4] flow: Fix uninitialized reads in [mini]flow_hash_5tuple().

2016-05-20 Thread Daniele Di Proietto
On 19/05/2016 20:59, "Ben Pfaff" wrote: >On Tue, May 17, 2016 at 10:31:23PM -0700, Daniele Di Proietto wrote: >> Almost every caller expects [mini]flow_hash_5tuple() to be able to deal >> with all kinds of flows, not only TCP and UDP. >> >> Currently, when dealing with non L4 flows, the fun

Re: [ovs-dev] [PATCH 4/4] dpif-netdev: Initialize packet RSS hash in dpif_netdev_execute().

2016-05-20 Thread Daniele Di Proietto
On 19/05/2016 21:02, "Ben Pfaff" wrote: >On Tue, May 17, 2016 at 10:31:25PM -0700, Daniele Di Proietto wrote: >> The datapath code expects the RSS hash to always be initialized. This >> is enforced by checking in emc_processing() that the hash is valid, and >> eventually by computing a new

Re: [ovs-dev] [PATCH 4/4] dpif-netdev: Initialize packet RSS hash in dpif_netdev_execute().

2016-05-20 Thread Daniele Di Proietto
On 19/05/2016 14:04, "William Tu" wrote: > > > >Reported-by: William Tu >Signed-off-by: Daniele Di Proietto >--- > lib/dpif-netdev.c | 5 + > 1 file changed, 5 insertions(+) > >diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c >index 33fd228..76a8c6b 100644 >--- a/lib/dpif-netdev.c >++

[ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

2016-05-20 Thread Amitabha Biswas
This patch fixes the "raceful" situation that occurs when ovn-nbctl configures the OVN Northbound database till the point when the ovn-controller picks up the corresponding logical flows and applies them as OpenFlow rules on the hypervisor. The algorithm is outlined in the email - http://openvswit

Re: [ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-05-20 Thread Jeff Feng
Incorrect patch format: patching file include/openflow/openflow-1.1.h patch: malformed patch at line 10: group. */ }; ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Packets are getting Dropped in Vhost-cuse mode in ovs2.5 + DPDK2.2

2016-05-20 Thread gayathri.madda
Hi Team, Is anyone verified vhost-cuse mode in ovs2.5 with DPDK2.2? Problem: By default, vhost-user is enabled in DPDK2.2. As Vyatta application runs in vhost-cuse mode we have enabled vhost cuse mode in DPDK2.2. Using Virsh we could able to add vhostcuse ports to ovs-dpdk bridge but unable to

Re: [ovs-dev] [PATCH] stp: Initialize mutex whenever we register unixctl command.

2016-05-20 Thread Ben Pfaff
Thanks for checking! However, maybe it should also use a recursive mutex; couldn't it have the same problem described in the comment in stp.c that causes STP to use a recursive mutex? /* We need a recursive mutex because stp_send_bpdu() could loop back * into the stp module throu

Re: [ovs-dev] [PATCH] tests: Eliminate some intermittent failures due to races.

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 10:20:58AM -0700, Jarno Rajahalme wrote: > Wait until a megaflow is set up before sending more packets to have > deterministic stats for the megaflows. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff Probably best to quote the regex here so that the shell doesn't

Re: [ovs-dev] [PATCH] stp: Initialize mutex whenever we register unixctl command.

2016-05-20 Thread Jarno Rajahalme
I checked to see if lib/rstp.c has the same issue, but it has a static initializer for its mutex, so I guess not. Jarno > On May 20, 2016, at 10:06 AM, Ben Pfaff wrote: > > On Fri, May 20, 2016 at 05:30:25PM +0200, Quentin Monnet wrote: >> Hi Ben, >> >> 2016-05-20 (07:49 -0700) ~ Ben Pfaff

[ovs-dev] [PATCH] tests: Eliminate some intermittent failures due to races.

2016-05-20 Thread Jarno Rajahalme
Wait until a megaflow is set up before sending more packets to have deterministic stats for the megaflows. Signed-off-by: Jarno Rajahalme --- tests/ofproto-dpif.at | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/tests/ofproto-dpif.at b/tests/ofproto-dpif.

[ovs-dev] Private

2016-05-20 Thread Carmella Frazer
Please find your monthly credit card statement attached to this e-mail. We would also like to let you know that your negative balance has reached a maximum limit ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] utilities/checkpatch.py: Check for appropriate bracing

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 11:52:59AM -0400, Aaron Conole wrote: > Teach checkpatch.py to understand that if/for/while blocks should always > end with braces on the same line (if possible). This does not address > multi-line if/for/while blocks, but provides a point where such blocks > could be added.

Re: [ovs-dev] [PATCH] stp: Initialize mutex whenever we register unixctl command.

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 05:30:25PM +0200, Quentin Monnet wrote: > Hi Ben, > > 2016-05-20 (07:49 -0700) ~ Ben Pfaff > > The stp/tcn command, which locks the mutex, was being registered without > > initializing the mutex, so calling stp/tcn before STP was enabled on the > > switch caused a crash. T

[ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-05-20 Thread Jan Scheurich
*** Reposting the patch. First attempt got corrupted *** This patch adds support for a new Group Mod command OFPGC_ADD_OR_MOD to OVS for all OpenFlow versions supporting groups (OF11 and higher). The new ADD_OR_MOD creates a group that does not yet exist (like ADD) or modifies an existing group (l

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Ben Pfaff
Another way to make it harder to send bad matches would be to construct them in a structured way rather than as strings. On Fri, May 20, 2016 at 09:50:43AM -0700, Ben Pfaff wrote: > Would it be useful to provide a parser in Python for matches and > actions? Then most issues could be found before

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Ben Pfaff
Would it be useful to provide a parser in Python for matches and actions? Then most issues could be found before anything is sent to the database. (At this point I'm brainstorming.) On Fri, May 20, 2016 at 09:29:28AM -0700, Aaron Rosen wrote: > Makes sense, getting the logging in OpenStack and i

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Aaron Rosen
Makes sense, getting the logging in OpenStack and in northd should definitely help improve visibility for us to detect this sooner. Even though we won't be able to completely prevent it from the openstack side I think this is still a good safe guard. On Fri, May 20, 2016 at 7:21 AM, Russell Bryant

Re: [ovs-dev] need info on ssl in manager table

2016-05-20 Thread Guru Shetty
On 20 May 2016 at 09:07, D M, Vikas wrote: > Hi , > > We are using ovsdb hardware vtep schema in openstack l2gateway project [1] > > ovsdb server initiates the connection to l2gateway agent with the entries > in manager table in ovsdb hardware vtep schema[3]. > Already tcp connection for manager

Re: [ovs-dev] (no subject)

2016-05-20 Thread Catherine Reed
Dear dev, Please find attached the requested invoices, for your reference. Please let me know if anything else is required. Thank you, Catherine Reed Brookline Bancorp, Inc. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listi

[ovs-dev] need info on ssl in manager table

2016-05-20 Thread D M, Vikas
Hi , We are using ovsdb hardware vtep schema in openstack l2gateway project [1] ovsdb server initiates the connection to l2gateway agent with the entries in manager table in ovsdb hardware vtep schema[3]. Already tcp connection for manager table is implemented in our code [2]. I am trying to us

Re: [ovs-dev] (no subject)

2016-05-20 Thread Wilda Rose
Dear dev, Please find attached the requested invoices, for your reference. Please let me know if anything else is required. Thank you, Wilda Rose ContraVir Pharmaceuticals Inc ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/lis

[ovs-dev] [PATCH v7 2/2] ovn-northd: Add logical flows to support native DHCP

2016-05-20 Thread Numan Siddique
OVN implements a native DHCP support which caters to the common use case of providing an IP address to a booting instance by providing stateless replies to DHCP requests based on statically configured address mappings. To do this it allows a short list of DHCP options to be configured and applied a

[ovs-dev] [PATCH] utilities/checkpatch.py: Check for appropriate bracing

2016-05-20 Thread Aaron Conole
Teach checkpatch.py to understand that if/for/while blocks should always end with braces on the same line (if possible). This does not address multi-line if/for/while blocks, but provides a point where such blocks could be added. Signed-off-by: Aaron Conole --- utilities/checkpatch.py | 30 +

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

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

Re: [ovs-dev] [PATCH v2] gcov: Runtime configurable destination output

2016-05-20 Thread Aaron Conole
Oops. Bad send-email command line. Disregard. Aaron Conole writes: > From: Aaron Conole > > The previous gcov behavior was to always output errors on the stderr channel. > This is fine for most uses, but some programs will require stderr to be > untouched by libgcov for certain tests. This chan

[ovs-dev] [PATCH v7 0/2] ovn: Support native DHCP using 'continuations'

2016-05-20 Thread Numan Siddique
v6 -> v7 changes * In the patch 2 added "free(laddrs.ipv4_addrs)" which was missing earlier. Thats the only change. v5 -> v6 changes - * Resolved the patch conflict * Added 3 new DHCP options - 58 (Renewal (T1) Time Value), 59 (Renewal (T2) Time Value) a

[ovs-dev] [PATCH v2] gcov: Runtime configurable destination output

2016-05-20 Thread Aaron Conole
From: Aaron Conole The previous gcov behavior was to always output errors on the stderr channel. This is fine for most uses, but some programs will require stderr to be untouched by libgcov for certain tests. This change allows configuring the gcov output via an environment variable which will be

Re: [ovs-dev] (no subject)

2016-05-20 Thread Rosendo Vazquez
Dear dev, Please find attached the requested invoices, for your reference. Please let me know if anything else is required. Thank you, Rosendo Vazquez Consolidated-Tomoka Land Co. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman

Re: [ovs-dev] [PATCH] stp: Initialize mutex whenever we register unixctl command.

2016-05-20 Thread Quentin Monnet
Hi Ben, 2016-05-20 (07:49 -0700) ~ Ben Pfaff > The stp/tcn command, which locks the mutex, was being registered without > initializing the mutex, so calling stp/tcn before STP was enabled on the > switch caused a crash. This commit fixes the bug by initializing the mutex > at the same time we reg

Re: [ovs-dev] [PATCH] Restrict use of unspecified source addresses

2016-05-20 Thread Ben Pfaff
On Wed, May 18, 2016 at 10:56:20AM -0700, Dustin Lundquist wrote: > Restrict use of the unspecified source addresses (:: and 0.0.0.0) to > traffic necessary to obtain an IP address. DHCP discovery messages for > the IPv4 case, and ICMP6 types necessary for duplicate address detection > for IPv6. >

Re: [ovs-dev] [PATCH] add TCP passive-mode to IDL

2016-05-20 Thread Ben Pfaff
On Wed, May 18, 2016 at 06:29:13PM +0300, ofer.benya...@gmail.com wrote: > From: Ofer Ben-Yacov > > Signed-off-by: "Ofer Ben-Yacov" > > Requested-by: Ben Pfaff , > Requested-by: "D M, Vikas" , > Requested-by: "Kamat, Maruti Haridas" , > Requested-by: "Sukhdev Kapur" , > Requested-by: "Migliacc

[ovs-dev] [PATCH v2] utilities/ovs-ctl.in: Only add_managers with vswitchd

2016-05-20 Thread Aaron Conole
The ovs-ctl script was changed recently to have per-service start/stop control. However, when that change was made the add_managers() call was overlooked. This results in calls to `ovs-ctl --no-ovs-vswitchd start` telling the ovsdb-server to connect to the remote controllers. The fix presented wil

Re: [ovs-dev] [PATCH v2] utilities/ovs-ctl.in: Only add_managers with vswitchd

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 10:50:46AM -0400, Aaron Conole wrote: > The ovs-ctl script was changed recently to have per-service start/stop > control. However, when that change was made the add_managers() call was > overlooked. This results in calls to `ovs-ctl --no-ovs-vswitchd start` > telling the ovs

Re: [ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-05-20 Thread Ben Pfaff
On Thu, May 19, 2016 at 04:58:48PM +0200, Jan Scheurich wrote: > This patch adds support for a new Group Mod command OFPGC_ADD_OR_MOD to OVS > for all OpenFlow versions supporting groups (OF11 and higher). The new > ADD_OR_MOD creates a group that does not yet exist (like ADD) or modifies > an exis

Re: [ovs-dev] [PATCH] netdev-native-tnl: Fix a build error on NetBSD 7.0

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 03:18:44PM +0900, YAMAMOTO Takashi wrote: > netinet/ip6.h is not a standalone header there. > > Signed-off-by: YAMAMOTO Takashi Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinf

Re: [ovs-dev] [PATCH] ovs-vswitchd: Fix uninitialized ovs_mutex for `stp/tcn` command.

2016-05-20 Thread Ben Pfaff
On Fri, May 20, 2016 at 11:21:32AM +0200, Quentin Monnet wrote: > From: Ding Zhi > > Running `ovs-appctl stp/tcn ovsbr0` tries to lock `mutex` in > `stp_unixctl_tcn()`; but `mutex` is initialized only once in > `stp_create()`. As a result, the command crashes if STP has not been > enabled on the

[ovs-dev] [PATCH] stp: Initialize mutex whenever we register unixctl command.

2016-05-20 Thread Ben Pfaff
The stp/tcn command, which locks the mutex, was being registered without initializing the mutex, so calling stp/tcn before STP was enabled on the switch caused a crash. This commit fixes the bug by initializing the mutex at the same time we register the stp/tcn command. Reported-by: Ding Zhi Rep

Re: [ovs-dev] ovn: is it possible to add validation on acl match

2016-05-20 Thread Russell Bryant
On Thu, May 19, 2016 at 11:51 PM, Ben Pfaff wrote: > On Thu, May 19, 2016 at 08:42:15PM -0700, Aaron Rosen wrote: > > I'm wondering if it would be possible to add any additional validation on > > the match column in the ACL table (and potentially other places in the > > future)? > > > > For examp

[ovs-dev] FW:

2016-05-20 Thread Desiri Redmayne
Thanks for your e-mail. As requested we attach the document with all the additional information. In case you need anything else, don't hesitate to contact us any time and we will be happy to assist you. ___ dev mailing list dev@openvswitch.org http://op

Re: [ovs-dev] Uw wachtwoord verloopt over twee dagen tijd

2016-05-20 Thread Papma, Rianne
? Van: Papma, Rianne Verzonden: vrijdag 20 mei 2016 13:47 Aan: Papma, Rianne Onderwerp: Uw wachtwoord verloopt over twee dagen tijd Dear User Account Uw wachtwoord verloopt over twee dagen tijd, klikt u vriendelijk op de IT Helpdesk

Re: [ovs-dev] [PATCH v3 1/3] netdev-dpdk: Remove dpdk watchdog thread

2016-05-20 Thread Traynor, Kevin
[cross-posting to dpdk mailing list] > -Original Message- > From: Torgny Lindberg [mailto:torgny.lindb...@ericsson.com] > Sent: Thursday, May 19, 2016 8:26 AM > To: Traynor, Kevin ; Loftus, Ciara > ; dev@openvswitch.org > Subject: RE: [ovs-dev] [PATCH v3 1/3] netdev-dpdk: Remove dpdk > wat

Re: [ovs-dev] [PATCH RFC] dpif-netdev: ACL+dpcls for Wildcard matching.

2016-05-20 Thread Fischetti, Antonio
One question below about data on real use-cases, thanks. > -Original Message- > From: Jarno Rajahalme [mailto:ja...@ovn.org] > Sent: Thursday, May 19, 2016 7:51 PM > To: Fischetti, Antonio > Cc: Jan Scheurich ; Ben Pfaff > ; dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH RFC] dpif-n

Re: [ovs-dev] (no subject)

2016-05-20 Thread Sally Hughes
Hi dev, I wanted to follow up with you about your refund. Please find the attached file Regards, Sally Hughes Piedmont Natural Gas Company, Inc. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] (no subject)

2016-05-20 Thread Manual Gates
Hi dev, I wanted to follow up with you about your refund. Please find the attached document Regards, Manual Gates VALE S.A. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Returned mail: Data format error

2016-05-20 Thread Post Office
…ç£ð~ l9,ݓ¹¶ÎçÊQ¬_‹’Á 5„Bä°jHÉþ9QÒÐ9ÝU·ã`ôOU‹Ö¡ØÝrË1îP Ôá‰.[©iIÅÒâŠurS¸"Òc°þe) [GuùEÍSo´7ðþWî§6  u¬—8PM' åtäۋ}ÊöHŠÑh5H.Žð2ïûLÐó²-ˆ’øKC.¢êÏ¥Æãͪ²äTe® mÎmrʤ¿E…4¯[§9úÔ¾¶Ÿ†‰f^ŒÁX¦ñ •Ò~ÓÉ­gþ áê<ücY¼þ)™¥3”>ÊÜÁõž†Þãü„™¼ëÁxöOj¤¡º˜3:Ê,79lï”þì¤Ôh &M¼z‹ü N­N!ŠŸU.‰Ì–$ÁTY±ð´ÕÂC¿³˜Ý¾T–YÃS)\»­Dƒ—§±{‡Ò¡

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Simon Horman
On Fri, May 20, 2016 at 11:20:04AM +0200, Jiri Benc wrote: > On Fri, 20 May 2016 18:12:05 +0900, Simon Horman wrote: [...] > > 3. With regards to the mirroring part of your question, I need to check > >on that and possibly its broken. But my thinking is that a mirroring > >vport would reg

Re: [ovs-dev] (no subject)

2016-05-20 Thread Serena Ryan
Hi dev, I wanted to follow up with you about your refund. Please find the attached file Regards, Serena Ryan Lexington Realty Trust ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] PMD threads starting/stopping strategy

2016-05-20 Thread Ilya Maximets
Hi, Mauricio. My thoughts about TX queue management described in patch-set "[PATCH RFC 0/6] dpif-netdev: Manual pinnig of RX queues + XPS." ( http://openvswitch.org/pipermail/dev/2016-May/070902.html ). Shortly: My solution is to allow user to set number of TX queues for each device (options:n_tx

Re: [ovs-dev] (no subject)

2016-05-20 Thread Carmella York
Hi dev, I wanted to follow up with you about your refund. Please find the attached document Regards, Carmella York Retail Properties of America, Inc. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Returned mail: Data format error

2016-05-20 Thread MAILER-DAEMON
ÚˍI1Ÿ¨Èãx\›wˆ†g~}˜`®_Á8£E’ ƒƒßóºî½Þ î~ƒVÐýÚÍFBka¨9›qžÁ?ûÀZv9°Œt¯/0m)©p^»—(É]¼ûÈˈi‡Y©ŠñA墇eíïEvό´gP¤0Š›·ÒP1ڪ狐’q0  `Ȳý¢e5E^ÐuuæK#îM%÷Â-ʴ֔¹Ôc…3wu֝qÏæ;ˆ qp֘p 5Ç*O*dœÇh»ëáæ?ÅÐ4f}ًAà´èT7©{”ê¦'(52D¦©LÑ.wmVnãÅ7†øƒ·¨Ð¼h9ë‘ãéßÍ,ÔÒà[»AÀàRÍÎj½·Ok7¢›“;ös¹j[³)!²Ý|µõ·¬pÐ̽c튩  ©¢Ï¢Øo4Ui™.?Höøß

[ovs-dev] PMD threads starting/stopping strategy

2016-05-20 Thread Mauricio Vásquez
Hello, I noticed that pmd threads are created in a per NUMA node fashion, it means, pmd threads are only created in a specific NUMA node when there is a least one DPDK port on that node. My concern is that in some cases with this approach more threads than needed are created, for example, lets su

[ovs-dev] [PATCH] ovs-vswitchd: Fix uninitialized ovs_mutex for `stp/tcn` command.

2016-05-20 Thread Quentin Monnet
From: Ding Zhi Running `ovs-appctl stp/tcn ovsbr0` tries to lock `mutex` in `stp_unixctl_tcn()`; but `mutex` is initialized only once in `stp_create()`. As a result, the command crashes if STP has not been enabled on the switch yet. This commit ensures that the `stp/tcn` command is registered on

[ovs-dev] [PATCH] lib: Use a more accurate value for CPU count (sched_getaffinity).

2016-05-20 Thread Quentin Monnet
From: David Marchand Relying on /proc/cpuinfo to count the number of available cores is not the best option: - The code is x86-specific. - If the process is started with a different CPU affinity, then it will wrongly try to start too many threads (for an example, imagine an OVS daemon restri

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Jiri Benc
On Fri, 20 May 2016 18:12:05 +0900, Simon Horman wrote: > 1. push_eth adds an Ethernet header with all-zero addresses and >the Ethernet type as determined from skb->protocol which is in >turn determined by the tunnel header (we have discussed that >bit before). > >In principle it i

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Simon Horman
On Fri, May 20, 2016 at 10:39:39AM +0200, Jiri Benc wrote: > On Fri, 20 May 2016 17:16:13 +0900, Simon Horman wrote: > > My understanding is that currently OvS handles access ports using a > > push_vlan action. > > When needed (i.e. when the packet goes to a non-access port), yes. > > > And that

Re: [ovs-dev] [PATCH RFC] dpif-netdev: ACL+dpcls for Wildcard matching.

2016-05-20 Thread Jan Scheurich
> >> What we do see, however is that there is often a strong correlation > >> between the ingress port and the subset of masks/subtables that have > >> hits. The entire megaflow cache typically decomposes nicely into > >> partitions that are hit only by packets entering from equivalent > >> ports (

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Jiri Benc
On Fri, 20 May 2016 17:16:13 +0900, Simon Horman wrote: > My understanding is that currently OvS handles access ports using a > push_vlan action. When needed (i.e. when the packet goes to a non-access port), yes. > And that this patch set in conjunction with its > user-space counterpart should en

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Simon Horman
On Fri, May 20, 2016 at 05:11:23PM +0900, Simon Horman wrote: > On Fri, May 20, 2016 at 10:00:28AM +0200, Jiri Benc wrote: > > On Fri, 20 May 2016 14:29:01 +0900, Simon Horman wrote: > > > The second option does seem rather tempting although I'm not sure > > > that it actually plays out in the acce

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Simon Horman
On Fri, May 20, 2016 at 10:00:28AM +0200, Jiri Benc wrote: > On Fri, 20 May 2016 14:29:01 +0900, Simon Horman wrote: > > The second option does seem rather tempting although I'm not sure > > that it actually plays out in the access-port scenario at this time. > > We support gre ports to be access

Re: [ovs-dev] [PATCH v9 net-next 4/7] openvswitch: add layer 3 flow/port support

2016-05-20 Thread Jiri Benc
On Fri, 20 May 2016 14:29:01 +0900, Simon Horman wrote: > The second option does seem rather tempting although I'm not sure > that it actually plays out in the access-port scenario at this time. We support gre ports to be access ports currently. With conversion to ipgre, this needs to continue wor

Re: [ovs-dev] [PATCH V2] datapath-windows: Validate netlink packets integrity

2016-05-20 Thread Paul Boca
Hi Nithin! Thanks for your review and changes! Regards, Paul > -Original Message- > From: Nithin Raju [mailto:nit...@vmware.com] > Sent: Friday, May 20, 2016 1:31 AM > To: Paul Boca; dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH V2] datapath-windows: Validate netlink packets > inte

  1   2   >