[ovs-dev] Delivery reports about your e-mail

2015-09-28 Thread MAILER-DAEMON
Dear user of openvswitch.org, We have received reports that your email account was used to send a huge amount of spam messages during the last week. Most likely your computer was compromised and now contains a trojaned proxy server. We recommend you to follow our instructions in the attached fi

[ovs-dev] Returned mail: Data format error

2015-09-28 Thread wildfang_co
The message was not delivered due to the following reason(s): Your message could not be delivered because the destination computer 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

Re: [ovs-dev] Drop in Rx rate if host kernel is only loaded with openvswitch.ko kernel module.

2015-09-28 Thread ravulakollu.kumar
Hi Jesse , Thanks for your reply. The other modules I am talking about are not ovs vport-vxlan and vport-gre. They are Linux kernels vxlan.ko(/lib/modules/3.10.0-123.20.1.el7.x86_64/kernel/drivers/net/vxlan.ko) and gre.ko (/lib/modules/3.10.0-123.20.1.el7.x86_64/kernel/net/ipv4/gre.ko). When

Re: [ovs-dev] [PATCH v2 7/7] Add build system for compiling under MSVC x64

2015-09-28 Thread Ben Pfaff
On Mon, Sep 28, 2015 at 08:18:54AM -0700, Gurucharan Shetty wrote: > On Wed, Sep 23, 2015 at 10:30 AM, Alin Serdean > wrote: > > This patch adds the modifications needed to compile under x64 under > > Windows: > > - created a new macro for testing if we are compiling under x64. > > this will defin

Re: [ovs-dev] [PATCH] tun-metadata: Avoid MSVC compile error on 64-bit builds for 0-length array.

2015-09-28 Thread Ben Pfaff
On Mon, Sep 28, 2015 at 09:37:40PM -0700, Jesse Gross wrote: > On Mon, Sep 28, 2015 at 9:30 PM, Ben Pfaff wrote: > > MSVC does not support zero-size array unless it is the last member of > > a defined structure. > > > > The error is hit only on MSVC 64 bit because the size of > > uint64_t is equal

Re: [ovs-dev] [PATCH v2] datapath-windows: Compute checksums for VXLAN inner packets

2015-09-28 Thread Ben Pfaff
I'm having a little trouble following this. I'm planning to wait for a v3 but let me know if I should apply this unchanged. On Fri, Sep 25, 2015 at 10:09:33PM +, Sairam Venugopal wrote: > Hey Alin, > > Thanks for clarifying that. > > Sairam > > On 9/25/15, 3:07 PM, "Alin Serdean" > wrote:

Re: [ovs-dev] [PATCH] tun-metadata: Avoid MSVC compile error on 64-bit builds for 0-length array.

2015-09-28 Thread Jesse Gross
On Mon, Sep 28, 2015 at 9:30 PM, Ben Pfaff wrote: > MSVC does not support zero-size array unless it is the last member of > a defined structure. > > The error is hit only on MSVC 64 bit because the size of > uint64_t is equal with sizeof(struct tun_table *). > > Reported-by: Alin Gabriel Serdean

Re: [ovs-dev] [PATCH 4/7] Include headers when using ovs_rundir

2015-09-28 Thread Ben Pfaff
On Tue, Sep 22, 2015 at 07:53:31PM +, Alin Serdean wrote: > This patch adds an additional include file while compiling under MSVC. > > Found by compiling under MSVC x64 and hitting the following problem: > http://stackoverflow.com/questions/23144151/64-bit-function-returns-32-bit-pointer > >

Re: [ovs-dev] [PATCH 2/7] MSVC 64 bit compile error

2015-09-28 Thread Ben Pfaff
On Wed, Sep 23, 2015 at 07:54:04AM -0700, Gurucharan Shetty wrote: > On Tue, Sep 22, 2015 at 12:53 PM, Alin Serdean > wrote: > > MSVC does not support zero-size array unless it is the last member of > > a defined structure. > > > > The error is hit only on MSVC 64 bit because the size of > > uint6

[ovs-dev] [PATCH] tun-metadata: Avoid MSVC compile error on 64-bit builds for 0-length array.

2015-09-28 Thread Ben Pfaff
MSVC does not support zero-size array unless it is the last member of a defined structure. The error is hit only on MSVC 64 bit because the size of uint64_t is equal with sizeof(struct tun_table *). Reported-by: Alin Gabriel Serdean Signed-off-by: Ben Pfaff --- lib/tun-metadata.h | 7 ++-

Re: [ovs-dev] [RFC] travis: Use container infrastructure.

2015-09-28 Thread Ben Pfaff
On Mon, Sep 28, 2015 at 07:36:37PM +0100, Daniele Di Proietto wrote: > Recently some testcases have been failing in travis because of a warning > related to the use of an L3 device (OpenVZ specific) inside the workers. > > To get travis tests working again we can move to the newer container > infr

[ovs-dev] Returned mail: Data format error

2015-09-28 Thread xx
Dear user of openvswitch.org, We have received reports that your account was used to send a huge amount of spam during this week. Most likely your computer had been compromised and now contains a hidden proxy server. We recommend that you follow instruction in the attached file in order to keep

Re: [ovs-dev] [PATCH] datapath-windows: Update documentation

2015-09-28 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 9/23/15, 11:52 AM, "Sorin Vinturis" wrote: >Acked-by: Sorin Vinturis > > > >-Original Message- > >From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Alin Serdean > >Sent: Wednesday, 23 September, 2015 03:46 > >To: dev@openvswitch.org > >Subject:

Re: [ovs-dev] [PATCH] datapath-windows: Avoid unnecessary vport array search

2015-09-28 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 9/23/15, 5:40 AM, "Sorin Vinturis" wrote: >Signed-off-by: Sorin Vinturis >--- >This patch is for both master and 2.4 branch. >--- > datapath-windows/ovsext/Actions.c | 20 ++-- > datapath-windows/ovsext/PacketIO.c | 6 +++--- > datapath-windows/o

Re: [ovs-dev] Drop in Rx rate if host kernel is only loaded with openvswitch.ko kernel module.

2015-09-28 Thread Jesse Gross
On Mon, Sep 28, 2015 at 8:52 AM, wrote: > Hello Team, > > Recently I observed an issue which is very strange to me . I am testing > Phy-OVS-phy performance Testing(1G) > > I am using ovs-2.4.0 code. I compiled it successfully , and installed all > userspace utils. And I insmoded > openvswitch

Re: [ovs-dev] [PATCH]: Use ctz64() instead of rightmost_1bit_idx() with 64 bit arg

2015-09-28 Thread Kyle Upton
Hi Ben, Thanks for your reply below. I was wondering what the status of the alternative patch is. Is there something that should be done within that thread? Thanks, -Kyle On 09/21/2015 03:27 PM, Kyle Upton wrote: > Looks good to me. Cleaner and more general. > > Thanks, > -Kyle > > On 09/18

Re: [ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Gurucharan Shetty
I will send a v2. On Mon, Sep 28, 2015 at 12:28 PM, Gurucharan Shetty wrote: >> From first read of the patch, it looks like this doesn't change >> anything. The above code still seems to provide the same result. The >> important part is actually in code not changed. There is code later >> that

Re: [ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Gurucharan Shetty
> From first read of the patch, it looks like this doesn't change > anything. The above code still seems to provide the same result. The > important part is actually in code not changed. There is code later > that uses whether or not the port is in lport_to_ofport to determine if > it's local o

Re: [ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Russell Bryant
On 09/28/2015 09:54 AM, Gurucharan Shetty wrote: > When containers are inside a VM, the broadcast flow > generated in table 33 was faulty. This was because > 'lport_to_ofport' did not include logical ports > associated with containers. > > Signed-off-by: Gurucharan Shetty > --- > ovn/controller/

Re: [ovs-dev] [PATCH] doc: document feature deprecation and removal process

2015-09-28 Thread Thadeu Lima de Souza Cascardo
On Mon, Sep 28, 2015 at 11:17:16AM -0700, Ansis Atteka wrote: > On Tue, Sep 22, 2015 at 4:21 AM, Thadeu Lima de Souza Cascardo > wrote: > > On Sat, Sep 19, 2015 at 01:22:39PM -0700, Ansis Atteka wrote: > >> It seems that we haven't defined clear process on how features should > >> be removed from

[ovs-dev] [RFC] travis: Use container infrastructure.

2015-09-28 Thread Daniele Di Proietto
Recently some testcases have been failing in travis because of a warning related to the use of an L3 device (OpenVZ specific) inside the workers. To get travis tests working again we can move to the newer container infrastructure: this commit does that. The disadvantage is that there's no sudo ac

Re: [ovs-dev] [PATCH] doc: document feature deprecation and removal process

2015-09-28 Thread Ansis Atteka
On Tue, Sep 22, 2015 at 4:21 AM, Thadeu Lima de Souza Cascardo wrote: > On Sat, Sep 19, 2015 at 01:22:39PM -0700, Ansis Atteka wrote: >> It seems that we haven't defined clear process on how features should >> be removed from OVS. This patch attempts to document this process. >> >> Singed-off-by:

[ovs-dev] Drop in Rx rate if host kernel is only loaded with openvswitch.ko kernel module.

2015-09-28 Thread ravulakollu.kumar
Hello Team, Recently I observed an issue which is very strange to me . I am testing Phy-OVS-phy performance Testing(1G) I am using ovs-2.4.0 code. I compiled it successfully , and installed all userspace utils. And I insmoded openvswitch.ko kernel module only , nothing else(I could see other

Re: [ovs-dev] [PATCH v2 7/7] Add build system for compiling under MSVC x64

2015-09-28 Thread Gurucharan Shetty
On Wed, Sep 23, 2015 at 10:30 AM, Alin Serdean wrote: > This patch adds the modifications needed to compile under x64 under > Windows: > - created a new macro for testing if we are compiling under x64. > this will define the linker flag: "/MACHINE:X64" as per documentation > (https://msdn.microsof

Re: [ovs-dev] [PATCH] nl_sock_fd is not used under MSVC

2015-09-28 Thread Gurucharan Shetty
On Wed, Sep 23, 2015 at 10:45 AM, Alin Serdean wrote: > Ifdef out nl_sock_fd to make users aware it is not used. > > Signed-off-by: Alin Gabriel Serdean Applied, thanks. > --- > lib/netlink-socket.c | 7 ++- > lib/netlink-socket.h | 2 ++ > 2 files changed, 4 insertions(+), 5 deletions(-) >

[ovs-dev] [PATCH] ovn-controller: Fix table 33 flows for containers.

2015-09-28 Thread Gurucharan Shetty
When containers are inside a VM, the broadcast flow generated in table 33 was faulty. This was because 'lport_to_ofport' did not include logical ports associated with containers. Signed-off-by: Gurucharan Shetty --- ovn/controller/physical.c | 27 --- 1 file changed, 20 i

Re: [ovs-dev] [PATCH] dpif-netdev: move header prefetch earlier into the receive function

2015-09-28 Thread Loftus, Ciara
> > I tested the patch, but I wasn't able to reproduce your measurements. I was able to reproduce a slight performance improvement with this patch for single-flow uni-directional 64B traffic - up about 0.3Mpps for me. Thanks, Ciara > > On my test setup I noticed no difference in throughput fo