Re: [ovs-dev] [PATCH] ofp-util: Fix length field of bundle messages

2015-04-08 Thread YAMAMOTO Takashi
hi, thank you for review. you are right. i didn't notice the code in ofconn_send. thank you. i tend to think it's nicer to have ofputil_encode_xxx generate the complete packet, though. background: my code [1] uses libofproto to generate on-wire packet and write it out to a file. thus ofconn_s

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

2015-04-08 Thread Mail Administrator
QÐÍú‡¸0jBy“Œúˆ0s…òšZG#r˘âH1¿‹~b  ´Öb®R?ɬ•Ä¬ËÚõP&ú¥3‘¼M ßº–wðݺÌy„ÝefFîíy͊¸¿u ~×ÕòQäWTSšP)v–#LÆi°©ó:íy'ˆàÑy]Úõ·‡$vc[t¬/X¿ë•,wyqâ ÃHåzȞ_Ó^ˆ„‘èÜÐ~Ò-½cýêÝ ç×Þ é÷Q²úã÷;27èøª]L¤æö ^òq ®œ¨||ëüÛ4–”9·Óµëìc \íþ™´3Ùaպ硅H0sJ¥Šd̏x„s:`¸Sà57ÚU¼™JkœÁXÚúfT\-â °jЙöŠG7´ )uÔbn~eùÖ2K-ñžLC)wÔðŽG;óüQ7Ï´

[ovs-dev] [PATCH v2] INSTALL.Windows: update documentation

2015-04-08 Thread Nithin Raju
The documentation to run the executables has fallen behind quite a bit. In this patch, we make a few updates. There's more to come. The patch assumes the user has not run 'make install' or does not have access to an installer. Signed-off-by: Nithin Raju --- v2: updated the steps based on the off

[ovs-dev] Always in fit with best ED meds!

2015-04-08 Thread Sophie Hoggarth
According to some forecasts within the next 25 years the number of men with impotence may double. http://x.co/8npXq ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH RFC 0/1] Fix travis errors

2015-04-08 Thread Mark Kavanagh
I observe two errors when running the travis scripts: - prepare.sh: environmental variables are not passed to 'sudo', such that the apt-get commands time out due to lack of a specified proxy server env var (which is specified for my user, but not root); - build.sh: two string co

[ovs-dev] [PATCH RFC 1/1] travis: fix errors

2015-04-08 Thread Mark Kavanagh
Fix two issues observed in travis scripts: - prepare.sh: add '-E' flag to ensure user environmental variables (for example, URL of the proxy server) are passed to 'sudo' - build.sh: add quotes around compiler variable to fix 'unary operator expected' error Signed-off-by: Mark Kavanagh --- .t

[ovs-dev] I found it hard to drop belt sizes, until I tried for myself!

2015-04-08 Thread Letha
This weight loss medication is suitable both for adults and for children! I am sure you are willing to try anything to get thin again. Everything you demand easy to buy on our site! http://x.co/8nvC3 ___ dev mailing list dev@openvswitch.org http://op

Re: [ovs-dev] [PATCH] testsuite: work around patch bug

2015-04-08 Thread Gurucharan Shetty
On Wed, Apr 8, 2015 at 9:35 AM, Flavio Leitner wrote: > Due to a CVE bug, the patch was modified to not allow following > symlinks out of the workdir. However, this causes a bug when > there is a regular file out of the workdir which breaks the current > testsuite build. > > Steps to reproduce: >

Re: [ovs-dev] [PATCH] ovn-nbd: Rename "ovn_txn" variable to "ovnsb_txn".

2015-04-08 Thread Justin Pettit
> On Apr 8, 2015, at 2:34 PM, Russell Bryant wrote: > > On 04/08/2015 05:31 PM, Justin Pettit wrote: >> Commit ec78987 (ovn: Rename "ovn" database to "ovn-sb".) changed most of >> the instances of "ovn" to some variation of "ovn-sb", but missed one in >> ovn-nbd. >> >> Suggested-by: Russell Bry

Re: [ovs-dev] [PATCH] ovn-nbd: Rename "ovn_txn" variable to "ovnsb_txn".

2015-04-08 Thread Russell Bryant
On 04/08/2015 05:31 PM, Justin Pettit wrote: > Commit ec78987 (ovn: Rename "ovn" database to "ovn-sb".) changed most of > the instances of "ovn" to some variation of "ovn-sb", but missed one in > ovn-nbd. > > Suggested-by: Russell Bryant > Signed-off-by: Justin Pettit Acked-by: Russell Bryant

Re: [ovs-dev] [PATCH] ovn: Rename "ovn" database to "ovn-sb".

2015-04-08 Thread Justin Pettit
> On Apr 8, 2015, at 12:04 PM, Russell Bryant wrote: > > On 04/08/2015 12:48 PM, Russell Bryant wrote: >> On 04/08/2015 12:19 PM, Justin Pettit wrote: >>> On Apr 8, 2015, at 8:53 AM, Russell Bryant wrote: On 04/07/2015 08:56 PM, Justin Pettit wrote: > It was confusing havin

[ovs-dev] [PATCH] ovn-nbd: Rename "ovn_txn" variable to "ovnsb_txn".

2015-04-08 Thread Justin Pettit
Commit ec78987 (ovn: Rename "ovn" database to "ovn-sb".) changed most of the instances of "ovn" to some variation of "ovn-sb", but missed one in ovn-nbd. Suggested-by: Russell Bryant Signed-off-by: Justin Pettit --- ovn/ovn-nbd.c | 22 +++--- 1 files changed, 11 insertions(+),

Re: [ovs-dev] [PATCH ovn v3 2/5] util: Add more bitwise operations.

2015-04-08 Thread Russell Bryant
On 04/01/2015 12:52 AM, Ben Pfaff wrote: > To be used in upcoming commits. > > Signed-off-by: Ben Pfaff I left a couple of comments, but they're not critical, so: Acked-by: Russell Bryant > --- > lib/util.c | 139 > + > lib/util.h

Re: [ovs-dev] [PATCH] DPDK: add support for v2.0.0

2015-04-08 Thread Kavanagh, Mark B
>Thanks for the patch, I’m looking forward to use new >DPDK 2.0 features! > Hi Daniele - sure, no problem :) >I have some doubts about the required -mssse3 option. >The main point is: does DPDK always requires a CPU >with SSSE3? > >I believe the answer is yes (Please correct me if I’m wrong). >If

Re: [ovs-dev] [PATCH ovn v3 1/5] lex: New lexical analyzer module for use in OVN.

2015-04-08 Thread Russell Bryant
On 04/01/2015 12:52 AM, Ben Pfaff wrote: > I'm determined not to let the terrible style of pseudo-parsing we have in > OVS leak into OVN. Here's the first step. > > Signed-off-by: Ben Pfaff Did you consider flex and bison for this patch series? It's mostly a weakly informed question out of cur

[ovs-dev] Don't be a prisoner of your age! Have fun with our powerful meds.

2015-04-08 Thread Les Cramer
For many men of your age it is impossible to maintain erection sufficient for having normal sex. http://zzb.bz/BZ5RJ ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Can you see yourself thin again

2015-04-08 Thread Gabriel Simpson
Good to meet you There is no risks at all to your health. http://zzb.bz/KzjSc ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 2/2] tunneling: Invalid packets should be cleared.

2015-04-08 Thread Jesse Gross
If we receive a packet with an invalid tunnel header, we should drop the packet without further processing. Currently we do this by removing any parsed tunnel metadata. However, this is not sufficient to stop processing - this only results in the packet getting dropped by chance when something usua

[ovs-dev] [PATCH 1/2] tunneling: Convert tunnel push/pop functions to act on single packets.

2015-04-08 Thread Jesse Gross
The userspace tunneling API for pushing and popping tunnel headers is currently based on processing batches of packets. However, there is no obvious way to take advantage of batching for these operations and so each tunnel operation has a pair of loops to process the batch. This changes the API to

Re: [ovs-dev] [PATCH] testsuite: work around patch bug

2015-04-08 Thread Gurucharan Shetty
On Wed, Apr 8, 2015 at 9:35 AM, Flavio Leitner wrote: > Due to a CVE bug, the patch was modified to not allow following > symlinks out of the workdir. However, this causes a bug when > there is a regular file out of the workdir which breaks the current > testsuite build. > > Steps to reproduce: >

[ovs-dev] [PATCH ovn 0/2 v3] Updates for container integration.

2015-04-08 Thread Russell Bryant
v1->v2: - Tweaks to ovn-nbctl as suggested by Juston - Update ovn-nbd to handle changes to parent and tag on a logical port after it is first created v2->v3: - After more discussion, change lport-list back to not including the parent_name and tag in its output. - Rebase on top of commit t

[ovs-dev] [PATCH ovn 2/2 v3] ovn-nbd: Update Bindings for container integration.

2015-04-08 Thread Russell Bryant
ovn-nbd updates the Bindings table of the OVN database when there are updates to the OVN_Northbound database. This update makes ovn-nbd copy over the container integration related fields from OVN_Northbound into the Bindings table. Signed-off-by: Russell Bryant --- ovn/ovn-nbd.c | 44 ++

[ovs-dev] [PATCH ovn 1/2 v3] ovn-nbctl: Updates for container integration.

2015-04-08 Thread Russell Bryant
Add support for specifying a parent port name and tag when creating logical ports. Also add commands for getting the parent_name or tag set on a logical port. These are necessary for dealing with container interfaces that sit behind normal interfaces. Signed-off-by: Russell Bryant --- ovn/ovn-

[ovs-dev] [PATCH ovn 2/3 v3] ovn-nbd: Don't shadow hash_node variable.

2015-04-08 Thread Russell Bryant
This variable is already defined at the top of the function, so just reuse that instead of defining it again. Signed-off-by: Russell Bryant --- ovn/ovn-nbd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ovn/ovn-nbd.c b/ovn/ovn-nbd.c index 0e75957..c9d6ce4 100644 --- a/ov

[ovs-dev] [PATCH ovn 3/3 v3] ovn-nbd: Fix unsafe HMAP_FOR_EACH_WITH_HASH usage.

2015-04-08 Thread Russell Bryant
The previous code assumed that hash_node would be NULL when the loop terminated without a match. That's not the case, so track the match a little differently. Signed-off-by: Russell Bryant --- ovn/ovn-nbd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ovn/ovn-nbd.c b

[ovs-dev] [PATCH ovn 1/3 v3] ovn-nbd: Match bindings on lport name.

2015-04-08 Thread Russell Bryant
When ovn-nbd is looking through bindings to figure out if any logical ports in the northbound DB need to have their 'up' state changed, it should be using the logical port's name instead of UUID for matching up to rows in the OVN db. The switch to using the name instead of UUID means we don't have

[ovs-dev] [PATCH ovn 0/3 v3]

2015-04-08 Thread Russell Bryant
v1->v2: - Drop first patch that was merged. - Update 1/3 to fix unsafe HMAP_FOR_EACH_WITH_HASH usage. - Fix issues in pre-existing code to address the same unsafe HMAP_FOR_EACH_WITH_HASH usage. v2->v3: - rebase on top of patch that renamed 'ovn' to 'ovn-sb' [PATCH 1/3] ovn-nbd: Match bindin

Re: [ovs-dev] [PATCH] ovn: Rename "ovn" database to "ovn-sb".

2015-04-08 Thread Russell Bryant
On 04/08/2015 12:48 PM, Russell Bryant wrote: > On 04/08/2015 12:19 PM, Justin Pettit wrote: >> >>> On Apr 8, 2015, at 8:53 AM, Russell Bryant wrote: >>> >>> On 04/07/2015 08:56 PM, Justin Pettit wrote: It was confusing having a database called "ovn", since it's also the name of the proj

[ovs-dev] [PATCH] testsuite: ofproto-dpif: fix test names

2015-04-08 Thread Flavio Leitner
Some tests were not included when running the make TESTSUITEFLAGS="-k ofproto-dpif" check because the test name was out of the expected pattern. Signed-off-by: Flavio Leitner --- tests/ofproto-dpif.at | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/ofproto-dp

[ovs-dev] Don't slow your life down! Be a healthy man.

2015-04-08 Thread Lucille Daniels
There is a horrible story about a man who had no sex at all http://zzb.bz/oStOl ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] DPDK: add support for v2.0.0

2015-04-08 Thread Daniele Di Proietto
Thanks for the patch, I’m looking forward to use new DPDK 2.0 features! I have some doubts about the required -mssse3 option. The main point is: does DPDK always requires a CPU with SSSE3? I believe the answer is yes (Please correct me if I’m wrong). If CONFIG_RTE_MACHINE is set to “default", MAC

Re: [ovs-dev] [PATCH ovn 2/3] ovn-nbctl: Updates for container integration.

2015-04-08 Thread Justin Pettit
> On Apr 8, 2015, at 8:34 AM, Russell Bryant wrote: > > On 04/08/2015 01:58 AM, Justin Pettit wrote: > >> By the way, would you be okay with our changing ovn-nbctl commands to >> more closely mimic ovs-vsctl? The main differences I see are >> changing the names of the following commands: >> >

Re: [ovs-dev] [PATCH] ovn: Rename "ovn" database to "ovn-sb".

2015-04-08 Thread Russell Bryant
On 04/08/2015 12:19 PM, Justin Pettit wrote: > >> On Apr 8, 2015, at 8:53 AM, Russell Bryant wrote: >> >> On 04/07/2015 08:56 PM, Justin Pettit wrote: >>> It was confusing having a database called "ovn", since it's also the >>> name of the project. Since we have an "ovn-nb", rename "ovn" to >>>

[ovs-dev] [PATCH] testsuite: work around patch bug

2015-04-08 Thread Flavio Leitner
Due to a CVE bug, the patch was modified to not allow following symlinks out of the workdir. However, this causes a bug when there is a regular file out of the workdir which breaks the current testsuite build. Steps to reproduce: ./boot.sh mkdir _gcc && cd _gcc ../configure make [...]

Re: [ovs-dev] [PATCH] ovn: Rename "ovn" database to "ovn-sb".

2015-04-08 Thread Justin Pettit
> On Apr 8, 2015, at 8:53 AM, Russell Bryant wrote: > > On 04/07/2015 08:56 PM, Justin Pettit wrote: >> It was confusing having a database called "ovn", since it's also the >> name of the project. Since we have an "ovn-nb", rename "ovn" to >> "ovn-sb". >> >> Signed-off-by: Justin Pettit > >

Re: [ovs-dev] Can openvswitch vxlan tunnel support vtep ip in different subnet?

2015-04-08 Thread Jesse Gross
I would try checking for correctness at each point in the path. Look for iptables rules, ovs-appctl dpif/dump-flows on the tunnel bridge to see what traffic is doing if it makes it there, check ovs-vswitchd logs, dmesg, etc. I would also removing local_ip from the tunnels as well since it makes th

Re: [ovs-dev] [PATCH] ovn: Rename "ovn" database to "ovn-sb".

2015-04-08 Thread Russell Bryant
On 04/07/2015 08:56 PM, Justin Pettit wrote: > It was confusing having a database called "ovn", since it's also the > name of the project. Since we have an "ovn-nb", rename "ovn" to > "ovn-sb". > > Signed-off-by: Justin Pettit Fine with me. I agree that it would make it more clear. One minor

[ovs-dev] [PATCH] DPDK: add support for v2.0.0

2015-04-08 Thread Mark Kavanagh
Update relevant artifacts to add support for DPDK v2.0.0 - INSTALL.DPDK.md - travis build script - netdev-dpdk: fix build with unified offload types in DPDK v2.0.0 Note that this breaks compatibility with DPDK v1.8.0 Signed-off-by: Mark Kavanagh Signed-off-by: Panu Matilainen --- .travis/bu

[ovs-dev] [PATCH] netdev-dpdk: Change eth rx burst size.

2015-04-08 Thread Kevin Traynor
Change eth rx burst size from 192 to 32. This significantly improves performance for packets that will be forwarded through dpdkvhost ports, as the max dpdkvhost tx burst size (32) will not be exceeded. There are negligible effects in other scenarios. Signed-off-by: Kevin Traynor --- lib/netdev-

Re: [ovs-dev] [PATCH ovn 2/3] ovn-nbctl: Updates for container integration.

2015-04-08 Thread Russell Bryant
On 04/08/2015 11:34 AM, Russell Bryant wrote: > On 04/08/2015 01:58 AM, Justin Pettit wrote: >> >>> On Apr 7, 2015, at 8:03 AM, Russell Bryant wrote: >>> >>> On 04/06/2015 06:53 PM, Justin Pettit wrote: >>> It might be nice to print the parent and tag in "lport-list". >>> >>> Any thoughts on

Re: [ovs-dev] [PATCH ovn 2/3] ovn-nbctl: Updates for container integration.

2015-04-08 Thread Russell Bryant
On 04/08/2015 01:58 AM, Justin Pettit wrote: > >> On Apr 7, 2015, at 8:03 AM, Russell Bryant wrote: >> >> On 04/06/2015 06:53 PM, Justin Pettit wrote: >> >>> It might be nice to print the parent and tag in "lport-list". >> >> Any thoughts on formatting? I don't care much. I was trying to keep i

[ovs-dev] [PATCH INFO 1/2] packet: Avoid array of struct with zero length member.

2015-04-08 Thread Jesse Gross
Windows doesn't like that the Geneve header has an array of options with each have a zero length member (the variable data). Nothing is accessing the data now, so just replace the member with a comment - we can use pointer arithmetic when necessary. Reported-by: Gurucharan Shetty Signed-off-by: J

[ovs-dev] [PATCH INFO 2/2] geneve: Zero header before parsing userspace tunneling action.

2015-04-08 Thread Jesse Gross
When we parse the text representation of the Geneve action the header is not fully initialized. Besides the obvious potential to generate an action that the user did not actually specify, this also causes intermittent unit test failures when an action is read in and printed out and the result is di

[ovs-dev] [PATCH INFO 0/2] Userspace tunneling build fixes

2015-04-08 Thread Jesse Gross
These two patches I have already applied to master to fix build issues that cropped up after I pushed the Geneve userspace tunneling series. I'm just sending them to the list as informational, there's nothing that needs to be done unless there is a problem. There appears to be another issue that w

Re: [ovs-dev] Everyone wants a firm backside

2015-04-08 Thread Yolanda Kelly
I just wanted to let you know how simple it is to get thin If it is not for you, then send it back. http://x.co/8jbsq ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Delivery failed

2015-04-08 Thread The Post Office
Dear user of openvswitch.org, administration of openvswitch.org would like to let you know the following: We have found that your e-mail account was used to send a huge amount of unsolicited commercial email messages during this week. Most likely your computer was compromised and now contains a