[ovs-dev] Message could not be delivered

2016-06-06 Thread Mail Delivery Subsystem
Dear user dev@openvswitch.org, administration of openvswitch.org would like to let you know that, We have received reports that your e-mail account was used to send a large amount of unsolicited commercial email messages during this week. Most likely your computer was infected and now contains a

Re: [ovs-dev] [PATCH] doc: Fix an error in FAQ.

2016-06-06 Thread Han Zhou
Sorry, I think I just realized that the "not" here is optional in English. But it did take me sometime to figure out :( Han On Mon, Jun 6, 2016 at 10:56 PM, Han Zhou wrote: > Signed-off-by: Han Zhou > --- > FAQ.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/FAQ.md

[ovs-dev] [PATCH] doc: Fix an error in FAQ.

2016-06-06 Thread Han Zhou
Signed-off-by: Han Zhou --- FAQ.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FAQ.md b/FAQ.md index df6f225..cc4fdf6 100644 --- a/FAQ.md +++ b/FAQ.md @@ -1056,7 +1056,7 @@ A: Yes. For traffic that egresses from a switch, OVS supports traffic Keep in mind that ingr

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

2016-06-06 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 v10 1/2] ovn-controller: Add 'put_dhcp_opts' action in ovn-controller

2016-06-06 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

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

2016-06-06 Thread Numan Siddique
v9 > v10 changes --- * Resolved the patch conflicts in tests/ovn.at * The dhcp test case "put_dhcp_opts"was failing because of the commit "ovn-controller: Assign conntrack zones for gateway router". Fixed it. v8 > v9 changes --- * Resolved the patch conflicts v7

Re: [ovs-dev] [PATCH monitor_cond V6 01/11] ovsdb: create column index mapping between ovsdb row to monitor row

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 06/06/2016 06:13:44 PM: > On Mon, Jun 06, 2016 at 01:11:44PM +0300, Liran Schour wrote: > > Ben Pfaff wrote on 01/06/2016 09:18:46 PM: > > > This new columns_index_map means that it's no longer necessary to work > > > so hard to find duplicates--rather, we can find a duplicat

Re: [ovs-dev] [PATCH] ofproto-dpif-mirror: add snaplen/truncate support.

2016-06-06 Thread William
Hi Ben, Thank you, I will fix it and resubmit together with the truncate patch. Regards, William > On Jun 6, 2016, at 9:50 PM, Ben Pfaff wrote: > >> On Fri, May 27, 2016 at 11:22:10AM -0700, William Tu wrote: >> This patch adds a 'snaplen' config for mirroring table. A mirrored packet >> with

Re: [ovs-dev] [patch_v4] ovn: Add local router support (RFC)

2016-06-06 Thread Ben Pfaff
On Tue, May 31, 2016 at 09:37:10AM -0700, Darrell Ball wrote: > This patch adds local router support. The CMS provides a hint for > a NB distributed logical router as to which logical switches the > logical router services; the CMS always knows this information. A > new external ID tentatively na

Re: [ovs-dev] [PATCH v2 3/3] datapath-windows: Add GRE checksum

2016-06-06 Thread Alin Serdean
> > >+curNbl = *newNbl; > > >+curNb = NET_BUFFER_LIST_FIRST_NB(curNbl); > > > > The reason we were doing the partial copy earlier is to make sure that > > ETH > > + IP + GRE headers are all in the first MDL. The code access these > > + headers > > assuming they are contiguous in memory. Is

Re: [ovs-dev] [PATCH] ofproto-dpif-mirror: add snaplen/truncate support.

2016-06-06 Thread Ben Pfaff
On Fri, May 27, 2016 at 11:22:10AM -0700, William Tu wrote: > This patch adds a 'snaplen' config for mirroring table. A mirrored packet > with size larger than snaplen bytes will be truncated in datapath before > sending to the mirror output port. A snaplen of 0 equals 65535, which means > no tru

Re: [ovs-dev] [PATCH v2] INSTALL.md: Note use of "hacking" flake8 plugin.

2016-06-06 Thread Ben Pfaff
On Thu, Jun 02, 2016 at 03:53:46PM -0400, Russell Bryant wrote: > The automatic flake8 check that runs against Python code has some > warnings enabled that come from the "hacking" flake8 plugin. If it's > not installed, the warnings just won't occur until it's run on a system > with "hacking" inst

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

2016-06-06 Thread Ben Pfaff
On Fri, May 27, 2016 at 11:53:35AM -0400, Flavio Fernandes wrote: > When responding to icmp echo requests (aka ping) packets, the logical > router should not restrict responses based on the inport. Thanks for the patch (and especially for writing a test)! I applied this to master. ___

[ovs-dev] Facing some problems while Using the DPDK with ovs-vswitchd

2016-06-06 Thread THEEYAGU S
Hii all, I am trying to configure and build ovs-vswitchd in userplane with the help of DPDK. I am following INSTALL.DPDK.md , but i am getting the following error root@osboxes:/usr/src/ovs# ovs-vsctl add-bond br0 dpdkbond dpdk0 dpdk1 -- set Interface dpdk0 type=

Re: [ovs-dev] [PATCH v2 3/3] datapath-windows: Add GRE checksum

2016-06-06 Thread Ben Pfaff
On Tue, May 31, 2016 at 06:09:05PM +, Nithin Raju wrote: > -Original Message- > From: dev on behalf of Alin Serdean > > Date: Tuesday, May 24, 2016 at 9:14 AM > To: "dev@openvswitch.org" > Subject: [ovs-dev] [PATCH v2 3/3] datapath-windows: Add GRE checksum > > >@@ -369,11 +383,25 @

Re: [ovs-dev] [PATCH] tests: Skip vlog close and set if no python is installed

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 08:21:26PM +, Alin Serdean wrote: > > -Mesaj original- > > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > > Trimis: Friday, June 3, 2016 4:07 PM > > Către: dev@openvswitch.org > > Subiect: [ovs-dev] [PATCH] tests: Skip vlog close and set if

Re: [ovs-dev] [PATCH V3 1/4] tests: Fixed access denied on ovs-vswitchd.log

2016-06-06 Thread Ben Pfaff
This test wasn't aimed specifically to test what happens if the log file isn't closed before it's overwritten, so it seemed reasonable to me to just close it everywhere. I prefer to have only a single code path where we can. On Mon, Jun 06, 2016 at 07:09:58PM +, Alin Serdean wrote: > From win

Re: [ovs-dev] [PATCH V3 1/4] tests: Fixed access denied on ovs-vswitchd.log

2016-06-06 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 01:05:54PM +, Paul Boca wrote: > On Windows trying to overwrite the opened ovs-vswitchd.log > fails with access denied. Closing it before trying to overwrite it > solves the problem > > Signed-off-by: Paul-Daniel Boca > --- > V2: Use same command both for Windows and L

Re: [ovs-dev] [ovs-dev, v17, 5/5] Add incremental proessing to lflow_run and physical_run

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 02:04:37PM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 06/06/2016 10:51:57 AM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 06/06/2016 10:52 AM > > Subject: Re: [ovs-dev, v17, 5/5] Add incremental proessing to > > lflow_

Re: [ovs-dev] [PATCH] tests: Skip vlog close and set if no python is installed

2016-06-06 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 01:06:51PM +, Paul Boca wrote: > AT_CAPTURE_FILE([log.old]) and AT_CAPTURE_FILE([log]) will fail in case no > python > is installed on the system > > Signed-off-by: Paul-Daniel Boca I don't understand this. AT_CAPTURE_FILE doesn't rely on Python. Also, AT_CAPTURE_F

[ovs-dev] look for a companion

2016-06-06 Thread dev
Hello, Just wanted to know where I should send you the profits you'll be making? My team is all setup and ready to welcome you on board, so that we can start you on one of the most amazing pesos making journeys you'll ever take. >> CLICK HERE TO START YOUR JOURNEY Let's not waste another m

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

2016-06-06 Thread Babu Shanmugam
On Monday 06 June 2016 08:30 PM, Ben Pfaff wrote: On Mon, Jun 06, 2016 at 02:31:03PM +0530, Babu Shanmugam wrote: On Saturday 04 June 2016 01:17 AM, Ben Pfaff wrote: On Wed, May 25, 2016 at 03:35:44PM +0530, bscha...@redhat.com wrote: I'm a little worried about the way that this will re-set a

Re: [ovs-dev] [PATCH net-next v10 2/5] openvswitch: set skb protocol and mac_len when receiving on internal device

2016-06-06 Thread Simon Horman
On Thu, Jun 02, 2016 at 03:01:47PM -0700, pravin shelar wrote: > On Wed, Jun 1, 2016 at 11:24 PM, Simon Horman > wrote: > > * Set skb protocol based on contents of packet. I have observed this is > > necessary to get actual protocol of a packet when it is injected into an > > internal device e

Re: [ovs-dev] [PATCH net-next v10 3/5] openvswitch: add support to push and pop mpls for layer3 packets

2016-06-06 Thread Simon Horman
On Thu, Jun 02, 2016 at 03:02:00PM -0700, pravin shelar wrote: > On Wed, Jun 1, 2016 at 11:24 PM, Simon Horman > wrote: > > Allow push and pop mpls actions to act on layer 3 packets by teaching > > them not to access non-existent L2 headers of such packets. > > > > Signed-off-by: Simon Horman > >

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

2016-06-06 Thread Simon Horman
On Thu, Jun 02, 2016 at 03:02:18PM -0700, pravin shelar wrote: > On Wed, Jun 1, 2016 at 11:24 PM, Simon Horman > wrote: [...] > > diff --git a/net/openvswitch/actions.c b/net/openvswitch/actions.c > > index 15f130e4c22b..5567529904fa 100644 > > --- a/net/openvswitch/actions.c > > +++ b/net/openv

Re: [ovs-dev] [PATCH v2 0/7] PMD Testsuite.

2016-06-06 Thread Daniele Di Proietto
Hi Ilya, thanks for the series, I really appreciate the effort to test the pmd threads. I reviewed the whole series and it looks good to me, but I found some problems with travis and non linux systems. 1 On FreeBSD (and I assume also on windows), the pmd tests fail, because ovs_numa_get_n_core

[ovs-dev] [PATCH 4/5] vswitchd: Add --dummy-numa command line option.

2016-06-06 Thread Daniele Di Proietto
This option is used to initialize the ovs_numa module with a fake configuration and to avoid pthread_setaffinity_np() calls. It will be useful to test dpif-netdev with pmd threads. Since it is only used for testing it is not documented in the man pages. Signed-off-by: Daniele Di Proietto --- l

[ovs-dev] [PATCH 5/5] tests: Allow extra cmd line args to OVS_VSWITCHD_START.

2016-06-06 Thread Daniele Di Proietto
This will be used by a following commit, to add dummy-numa options. Signed-off-by: Daniele Di Proietto --- tests/ofproto-macros.at | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/ofproto-macros.at b/tests/ofproto-macros.at index 632f747..105d9ca 100644 --- a/tes

[ovs-dev] [PATCH 1/5] ovs-numa: Remove non-linux stubs.

2016-06-06 Thread Daniele Di Proietto
Instead of having static inline stubs for non linux platform we can use the implementations in ovs-numa.c. With one small change to ovs_numa_dump_cores_on_numa(), they will behave exactly like the stubs for the non-linux case, because 'found_numa_and_core' will be false and the socket and cpu hmap

[ovs-dev] [PATCH 2/5] ovs-numa: Introduce function to set current thread affinity.

2016-06-06 Thread Daniele Di Proietto
This commit moves the code that sets the pmd threads affinity from netdev-dpdk to ovs-numa. There's one small part left in netdev-dpdk, to set the lcore_id. Now dpif-netdev will call both modules (ovs-numa and netdev-dpdk) when starting a pmd thread. This change will allow having a dummy impleme

[ovs-dev] [PATCH 3/5] netdev-dummy: Introduce sched_yield() in rxq_recv() for pmd devices.

2016-06-06 Thread Daniele Di Proietto
Signed-off-by: Daniele Di Proietto --- lib/netdev-dummy.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/lib/netdev-dummy.c b/lib/netdev-dummy.c index 54a1152..aa244b6 100644 --- a/lib/netdev-dummy.c +++ b/lib/netdev-dummy.c @@ -19,6 +19,7 @@ #include "dummy.h" #include

[ovs-dev] Await your response

2016-06-06 Thread Eoin Power
I av a business proposal for you, reply for details. Regards. Zero Harm Tallaght Hospital: Know Your Medicines Tallaght Hospital is a tobacco free campus The information transmitted is intended only for the addressee and may contain confidential and/or privileged information. Any review, retran

Re: [ovs-dev] [PATCH] tests: fix issue in use of OVS_APP_EXIT_AND_WAIT

2016-06-06 Thread Alin Serdean
Hi Lance, Thanks a lot for the patch just one small nit on the windows side, please see inlined comment. Thanks, Alin. > diff --git a/tests/ovs-macros.at b/tests/ovs-macros.at index > e5710a0..fadd2be 100644 > --- a/tests/ovs-macros.at > +++ b/tests/ovs-macros.at > @@ -132,20 +132,22 @@ m4_defi

Re: [ovs-dev] [PATCH net-next] NSH(Network Service Header) implementation

2016-06-06 Thread pravin shelar
On Mon, Jun 6, 2016 at 2:34 AM, Yi Yang wrote: > IETF defined NSH(Network Service Header) for Service > Function Chaining, this is an IETF draft > > https://tools.ietf.org/html/draft-ietf-sfc-nsh-05 > > It will be a IETF standard shortly, this patch implemented > NSH for Open vSwitch. > > Signed-o

[ovs-dev] (no subject)

2016-06-06 Thread Returned mail
Your message was not delivered due to the following reason(s): Your message could not be delivered because the destination server was not reachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likel

Re: [ovs-dev] [PATCH V2 2/4] ovs-ofctl: Fixed PID file naming on windows

2016-06-06 Thread Alin Serdean
From the commit message it is hard to understand what are you trying to fix or why. Are you trying to get the absolute path if "name" is missing ":" ? Please state what are you trying to achieve and why, in the commit message. Thanks, Alin. > -Mesaj original- > De la: dev [mailto:dev-b

Re: [ovs-dev] [PATCH] VxLAN-gpe implementation

2016-06-06 Thread Jesse Gross
On Sat, Jun 4, 2016 at 6:39 AM, Yi Yang wrote: [...] > datapath/vport-netdev.c | 3 +- > datapath/vport-vxlan.c| 17 ++- These changes aren't upstream yet. Please do that before backporting them here. However, the changes to vport-vxlan.c

Re: [ovs-dev] [PATCH] WIP OVN ND for Logical_Port

2016-06-06 Thread Ben Pfaff
On Fri, May 27, 2016 at 10:32:56AM +0800, Zong Kai LI wrote: > From: lzklibj > > This patch tries to implement ND for Logical_Port in OVN. > > Signed-off-by: lzklibj Thanks for working on this. I'm CCing Justin, who is working on related features, to bring this patch to his attention. "spars

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

2016-06-06 Thread Andy Zhou
On Fri, Jun 3, 2016 at 7:07 PM, Joe Stringer wrote: > On 23 May 2016 at 16:30, Andy Zhou wrote: > > > > > > On Fri, May 20, 2016 at 11:49 AM, Joe Stringer wrote: > >> > >> The default vagrant box for fedora23 uses the images provided by Chef's > >> "bento" project, which has support for 3 backe

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

2016-06-06 Thread Andy Zhou
On Fri, Jun 3, 2016 at 6:54 PM, Joe Stringer wrote: > On 23 May 2016 at 18:33, Joe Stringer wrote: > > On 23 May 2016 at 16:25, Andy Zhou wrote: > >> > >> > >> On Fri, May 20, 2016 at 11:49 AM, Joe Stringer wrote: > >>> > >>> Signed-off-by: Joe Stringer > >>> --- > >>> Vagrantfile | 2 ++ > >

[ovs-dev] [add ovsdb lock tests 3/3] tests: add ovsdb lock tests

2016-06-06 Thread Andy Zhou
Add more ovsdb lock tests. Signed-off-by: Andy Zhou --- tests/automake.mk | 1 + tests/ovsdb-lock.at | 77 + tests/ovsdb.at | 1 + 3 files changed, 79 insertions(+) create mode 100644 tests/ovsdb-lock.at diff --git a/tests/automake.m

[ovs-dev] [add ovsdb lock tests 2/3] ovsdb: Add ovsdb-client options for testing lock

2016-06-06 Thread Andy Zhou
RFC 7047 lock operation has been fully implemented in ovsdb-server for a while, but it is not well covered in unit testing. This patch adds options for the ovsdb-client tool to issue lock operations. The next patch will make use those options. Please see ovsdb-client(1) changes for more details.

[ovs-dev] [add ovsdb lock tests 1/3] ovsdb: fix ovsdb-client --help output format

2016-06-06 Thread Andy Zhou
Signed-off-by: Andy Zhou --- This patch should be backported. --- ovsdb/ovsdb-client.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ovsdb/ovsdb-client.c b/ovsdb/ovsdb-client.c index 80819a8..5fc7ad9 100644 --- a/ovsdb/ovsdb-client.c +++ b/ovsdb/ovsdb-client.c @@ -261,

Re: [ovs-dev] [PATCH] tests: Skip vlog close and set if no python is installed

2016-06-06 Thread Alin Serdean
That will disable the test if it is under > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Friday, June 3, 2016 4:07 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH] tests: Skip vlog close and set if no python is > installed >

Re: [ovs-dev] [PATCH] Add *.c to datapath/linux/.gitignore

2016-06-06 Thread Jesse Gross
On Mon, Jun 6, 2016 at 11:41 AM, Aaron Rosen wrote: > This should prevent any additional *.c files from sneaking in here. > > Signed-off-by: Aaron Rosen Thanks, applied to master. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/

Re: [ovs-dev] deadline extended as per your request

2016-06-06 Thread dev
Hello, Just wanted to know where I should send you the profits you'll be making? My team is all setup and ready to welcome you on board, so that we can start you on one of the most amazing pesos making journeys you'll ever take. CLICK HERE TO START YOUR JOURNEY Let's not waste another minu

Re: [ovs-dev] [PATCH] tests: fix issue in use of OVS_APP_EXIT_AND_WAIT

2016-06-06 Thread Lance Richardson
- Original Message - > From: "Lance Richardson" > To: dev@openvswitch.org > Sent: Friday, June 3, 2016 6:09:26 PM > Subject: [ovs-dev] [PATCH] tests: fix issue in use of OVS_APP_EXIT_AND_WAIT > > Commit f9b11f2a09b4 introduced a loop to wait for process exit > in OVS_APP_EXIT_AND_WAIT aft

Re: [ovs-dev] [PATCH V3 1/4] tests: Fixed access denied on ovs-vswitchd.log

2016-06-06 Thread Alin Serdean
From windows point of view, it looks good, because we need to close file (handle) before we try to move it. In my opinion we should not change anything on the Linux side, so either ifdef it or create a new macro for this particular case (closing a log file) which can be used later on. Maybe th

Re: [ovs-dev] [ovs-dev, v17, 5/5] Add incremental proessing to lflow_run and physical_run

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/06/2016 10:51:57 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/06/2016 10:52 AM > Subject: Re: [ovs-dev, v17, 5/5] Add incremental proessing to > lflow_run and physical_run > > On Mon, Jun 06, 2016 at 10:26:46AM -0500, Ryan Moat

Re: [ovs-dev] [PATCH] Add *.c *.h to datapath/linux/.gitignore

2016-06-06 Thread Aaron Rosen
Done, see, "Subject: [PATCH] Add *.c to datapath/linux/.gitignore" On Mon, Jun 6, 2016 at 11:06 AM, Jesse Gross wrote: > On Thu, Jun 2, 2016 at 6:02 PM, Aaron Rosen wrote: > > This should prevent any additional files from sneaking in here. > > > > Signed-off-by: Aaron Rosen > > I agree that

[ovs-dev] [PATCH] Add *.c to datapath/linux/.gitignore

2016-06-06 Thread Aaron Rosen
This should prevent any additional *.c files from sneaking in here. Signed-off-by: Aaron Rosen --- datapath/linux/.gitignore | 52 +-- 1 file changed, 1 insertion(+), 51 deletions(-) diff --git a/datapath/linux/.gitignore b/datapath/linux/.gitignore i

Re: [ovs-dev] [PATCH] tests: Skip "daemon --service" test on Windows from non-admin console

2016-06-06 Thread Alin Serdean
Small nit: maybe rename the macro from OVS_SKIP_NON_ADMIN to OVS_SKIP_NON_ADMIN_WIN Acked-by: Alin Gabriel Serdean Tested-by: Alin Gabriel Serdean > +dnl OVS_SKIP_NON_ADMIN() > +dnl > +dnl Checks if we have enough rights to create a service > +m4_define([OVS_SKIP_NON_ADMIN], > + [ > + AT_SK

Re: [ovs-dev] [PATCH] Add *.c *.h to datapath/linux/.gitignore

2016-06-06 Thread Jesse Gross
On Thu, Jun 2, 2016 at 6:02 PM, Aaron Rosen wrote: > This should prevent any additional files from sneaking in here. > > Signed-off-by: Aaron Rosen I agree that this is better since these files are just autogenerated and not very interesting. I think that I would leave kcompat.h in the list tho

[ovs-dev] [PATCH] tests: make ovn logical router test case more reliable

2016-06-06 Thread Lance Richardson
The "ovn -- 1 HVs, 2 LSs, 1 lport/LS, 1 LR" test case creates a configuration including a logical router, then: 1) Sends a packet that is expected to be forwarded by the logical router. 2) Disables the logical router. 3) Sends another packet, identical to the one sent in (1), tha

Re: [ovs-dev] [PATCH V2 4/4] datapath-windows: Add ECN support on STT decapsulation

2016-06-06 Thread Sairam Venugopal
Thanks for incorporating the changes. Acked-by: Sairam Venugopal On 6/6/16, 9:45 AM, "Paul Boca" wrote: >Signed-off-by: Paul-Daniel Boca >--- >V2: Removed previously added OvsGetEthHdr and use OveExtractLayers instead >on STT decapsulation. Extracted layers will be used in SetOffloads

[ovs-dev] [PATCH 0/8] Make ovs-bugtool flake8 compatible.

2016-06-06 Thread Gurucharan Shetty
*** BLURB HERE *** Gurucharan Shetty (8): ovs-bugtool.in: Remove unused imports. ovs-bugtool.in: Fix errors around spaces and line length. ovs-bugtool.in: Remove unused variables. ovs-bugtool.in: Remove usage of 'has_key'. ovs-bugtool.in: Test for membership should be 'not in'. ovs-bug

[ovs-dev] [PATCH 4/8] ovs-bugtool.in: Remove usage of 'has_key'.

2016-06-06 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index cd1924d..95ea482 100755 --- a/utilities/bugtool/ovs-bugtool.

[ovs-dev] [PATCH 3/8] ovs-bugtool.in: Remove unused variables.

2016-06-06 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index 605a6bf..cd1924d 100755 --- a/utilities/bugtool/ovs-bugtool.in +++ b/utilities/b

[ovs-dev] [PATCH 2/8] ovs-bugtool.in: Fix errors around spaces and line length.

2016-06-06 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 235 ++- 1 file changed, 135 insertions(+), 100 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index 9e1f535..605a6bf 100755 --- a/utilities/

[ovs-dev] [PATCH 1/8] ovs-bugtool.in: Remove unused imports.

2016-06-06 Thread Gurucharan Shetty
Also take care of a 'import not at top of file' warning from flake8. Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index aa

Re: [ovs-dev] [PATCH V2 1/4] datapath-windows: Improved offloading on STT tunnel

2016-06-06 Thread Sairam Venugopal
Thanks for making the changes. Acked-by: Sairam Venugopal On 6/6/16, 9:45 AM, "Paul Boca" wrote: >*Added OvsExtractLayers - populates only the layers field without >unnecessary >memory operations for flow part >*If in STT header the flags are 0 then force packets checksums calculation >on rec

Re: [ovs-dev] [PATCH V2 2/4] datapath-windows: Add VLAN support to STT

2016-06-06 Thread Sairam Venugopal
Just in case it¹s required for V2: Acked-by: Sairam Venugopal On 6/6/16, 9:45 AM, "Paul Boca" wrote: >Add VLAN to STT header and on receive applyit to encapsulated packet > >Signed-off-by: Paul-Daniel Boca >Acked-by: Sairam Venugopal >--- >V2: No changes >--- > datapath-windows/ovsext/Stt.c

Re: [ovs-dev] [PATCH V2 3/4] datapath-windows: STT reassemble small fix

2016-06-06 Thread Sairam Venugopal
Just in case it¹s required for V2: Acked-by: Sairam Venugopal On 6/6/16, 9:45 AM, "Paul Boca" wrote: >Fixed possible deadlock in case NdisGetDataBuffer fails >Validate the segment length and offset on reassemble to avoid buffer >overflow > >Signed-off-by: Paul-Daniel Boca >Acked-by: Sairam

[ovs-dev] [PATCH V2 4/4] datapath-windows: Add ECN support on STT decapsulation

2016-06-06 Thread Paul Boca
Signed-off-by: Paul-Daniel Boca --- V2: Removed previously added OvsGetEthHdr and use OveExtractLayers instead on STT decapsulation. Extracted layers will be used in SetOffloads if needed. --- datapath-windows/ovsext/NetProto.h | 10 - datapath-windows/ovsext/Stt.c | 85

[ovs-dev] [PATCH V2 3/4] datapath-windows: STT reassemble small fix

2016-06-06 Thread Paul Boca
Fixed possible deadlock in case NdisGetDataBuffer fails Validate the segment length and offset on reassemble to avoid buffer overflow Signed-off-by: Paul-Daniel Boca Acked-by: Sairam Venugopal --- V2: No changes --- datapath-windows/ovsext/Stt.c | 16 +--- datapath-windows/ovsext/St

[ovs-dev] [PATCH V2 2/4] datapath-windows: Add VLAN support to STT

2016-06-06 Thread Paul Boca
Add VLAN to STT header and on receive applyit to encapsulated packet Signed-off-by: Paul-Daniel Boca Acked-by: Sairam Venugopal --- V2: No changes --- datapath-windows/ovsext/Stt.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/datapath-windows/ovsext

[ovs-dev] [PATCH V2 1/4] datapath-windows: Improved offloading on STT tunnel

2016-06-06 Thread Paul Boca
*Added OvsExtractLayers - populates only the layers field without unnecessary memory operations for flow part *If in STT header the flags are 0 then force packets checksums calculation on receive. *Ensure correct pseudo checksum is set for LSO both on send and receive. Linux includes the segment le

[ovs-dev] Powered by Google.

2016-06-06 Thread Google Incorporation�
Dear Google User, You have been selected as a Google Ambassador for using Google services. Find attached letter for more details and Processing of your claims. Best Regards, Matt Brittin Chairman of the Board and Managing Director, Google United Kingdom _

Re: [ovs-dev] [ovs-dev, v17, 5/5] Add incremental proessing to lflow_run and physical_run

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/06/2016 10:51:57 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/06/2016 10:52 AM > Subject: Re: [ovs-dev, v17, 5/5] Add incremental proessing to > lflow_run and physical_run > > On Mon, Jun 06, 2016 at 10:26:46AM -0500, Ryan Mo

Re: [ovs-dev] [ovs-dev,v17,4/5] Persist ovn flow tables.

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/06/2016 10:50:43 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/06/2016 10:50 AM > Subject: Re: [ovs-dev,v17,4/5] Persist ovn flow tables. > > On Mon, Jun 06, 2016 at 10:25:32AM -0500, Ryan Moats wrote: > > Ben Pfaff wrote on

Re: [ovs-dev] [ovs-dev, v17, 5/5] Add incremental proessing to lflow_run and physical_run

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 10:26:46AM -0500, Ryan Moats wrote: > > > Ben Pfaff wrote on 06/03/2016 10:55:53 AM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 06/03/2016 10:56 AM > > Subject: Re: [ovs-dev, v17, 5/5] Add incremental proessing to > >

Re: [ovs-dev] [ovs-dev,v17,4/5] Persist ovn flow tables.

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 10:25:32AM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 06/03/2016 10:50:45 AM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 06/03/2016 10:50 AM > > Subject: Re: [ovs-dev,v17,4/5] Persist ovn flow tables. > > > > On Sun,

Re: [ovs-dev] [PATCH monitor_cond V6 11/11] RFC OVN: Quick implementation of conditional monitoring

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 06/06/2016 06:24:09 PM: > On Mon, Jun 06, 2016 at 01:05:19PM +0300, Liran Schour wrote: > > Ben Pfaff wrote on 02/06/2016 08:38:10 PM: > > > Looking at how all this fits together, it seems a little awkward to have > > > condition objects separate from the IDL and to have them

Re: [ovs-dev] [ovs-dev, v17, 5/5] Add incremental proessing to lflow_run and physical_run

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/03/2016 10:55:53 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/03/2016 10:56 AM > Subject: Re: [ovs-dev, v17, 5/5] Add incremental proessing to > lflow_run and physical_run > > On Sun, May 22, 2016 at 04:36:22PM -0500, Ryan Mo

Re: [ovs-dev] [ovs-dev, v17, 2/5] Convert binding_run to incremental processing.

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 10:16:09AM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 06/02/2016 07:02:29 PM: > > On Sun, May 22, 2016 at 04:36:19PM -0500, Ryan Moats wrote: > > The following changes weren't immediately obvious to me, can you hint > > about why they are needed? > > > > > diff --git a/o

Re: [ovs-dev] [ovs-dev,v17,4/5] Persist ovn flow tables.

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/03/2016 10:50:45 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/03/2016 10:50 AM > Subject: Re: [ovs-dev,v17,4/5] Persist ovn flow tables. > > On Sun, May 22, 2016 at 04:36:21PM -0500, Ryan Moats wrote: > > Ensure that ovn flow

Re: [ovs-dev] [PATCH monitor_cond V6 11/11] RFC OVN: Quick implementation of conditional monitoring

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 01:05:19PM +0300, Liran Schour wrote: > Ben Pfaff wrote on 02/06/2016 08:38:10 PM: > > Looking at how all this fits together, it seems a little awkward to have > > condition objects separate from the IDL and to have them require a > > separate "update" call. It would be ea

Re: [ovs-dev] [ovs-dev, v17, 2/5] Convert binding_run to incremental processing.

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/02/2016 07:02:29 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/02/2016 07:02 PM > Subject: Re: [ovs-dev,v17,2/5] Convert binding_run to incremental processing. > > On Sun, May 22, 2016 at 04:36:19PM -0500, Ryan Moats wrote: >

Re: [ovs-dev] [PATCH monitor_cond V6 01/11] ovsdb: create column index mapping between ovsdb row to monitor row

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 01:11:44PM +0300, Liran Schour wrote: > Ben Pfaff wrote on 01/06/2016 09:18:46 PM: > > This new columns_index_map means that it's no longer necessary to work > > so hard to find duplicates--rather, we can find a duplicate at the time > > we try to add a column: a new column

Re: [ovs-dev] [ovs-dev, v17, 1/5] Change encaps_run to work incrementally

2016-06-06 Thread Ryan Moats
Ben Pfaff wrote on 06/02/2016 06:35:25 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 06/02/2016 06:35 PM > Subject: Re: [ovs-dev,v17,1/5] Change encaps_run to work incrementally > > On Sun, May 22, 2016 at 04:36:18PM -0500, Ryan Moats wrote: > > As a si

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

2016-06-06 Thread Ben Pfaff
On Mon, Jun 06, 2016 at 02:31:03PM +0530, Babu Shanmugam wrote: > On Saturday 04 June 2016 01:17 AM, Ben Pfaff wrote: > >On Wed, May 25, 2016 at 03:35:44PM +0530, bscha...@redhat.com wrote: > >I'm a little worried about the way that this will re-set all of the > >queues on every iteration through t

[ovs-dev] Fwd: Delivery Status Notification (Failure)

2016-06-06 Thread THEEYAGU S
-- Forwarded message -- From: Mail Delivery Subsystem Date: 6 June 2016 at 18:27 Subject: Delivery Status Notification (Failure) To: thee...@gmail.com Delivery to the following recipient failed permanently: dev@openvswitch.org Technical details of permanent failure: Messag

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

2016-06-06 Thread Mail Delivery Subsystem
Dear user dev@openvswitch.org, Your account was used to send a huge amount of unsolicited e-mail messages during this week. Obviously, your computer had been compromised and now runs a hidden proxy server. Please follow our instruction in order to keep your computer safe. Best regards, The ope

Re: [ovs-dev] [PATCH 0/5] Fixes for big-endian architectures

2016-06-06 Thread Gerhard Stenzel
On 06/06/2016 01:33 AM, Ben Pfaff wrote: On Fri, Jun 03, 2016 at 01:29:26PM -0700, Ben Pfaff wrote: ... Hi, unless I overlooked something those patches are not in the git yet. Any plans to commit them to master and also 2.5? Now applied to master, backport to 2.5 coming up... Now backport

Re: [ovs-dev] [PATCH 0/5] Fixes for big-endian architectures

2016-06-06 Thread Gerhard Stenzel
On 06/03/2016 10:29 PM, Ben Pfaff wrote: On Fri, Jun 03, 2016 at 04:36:23PM +0200, Gerhard Stenzel wrote: ... Hi, unless I overlooked something those patches are not in the git yet. Any plans to commit them to master and also 2.5? Now applied to master, backport to 2.5 coming up... teste

Re: [ovs-dev] [PATCH monitor_cond V6 10/11] tests: add testing for idl conditional monitoring

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 02/06/2016 08:08:14 PM: > On Tue, May 17, 2016 at 05:27:08PM +0300, Liran Schour wrote: > > Add tests for conditional monitoring to IDL. > > > > Signed-off-by: Liran Schour > > Thanks for adding tests. > > Usually we squash tests for new features with the patches that imple

Re: [ovs-dev] [PATCH monitor_cond V6 08/11] lib: add monitor_cond_update API to C IDL lib

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 02/06/2016 08:04:39 PM: > On Tue, May 17, 2016 at 05:27:06PM +0300, Liran Schour wrote: > > Add API that allows the user to create condition iteratively > > and send condition_update to the server. > > > > Signed-off-by: Liran Schour > > There seems to be a significant amoun

Re: [ovs-dev] [PATCH monitor_cond V6 07/11] lib: add to ovsdb-idl monitor_id

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 02/06/2016 07:47:20 PM: Seems like I review for 06/11 patch is missing... > On Tue, May 17, 2016 at 05:27:05PM +0300, Liran Schour wrote: > > IDL uses now a uuid to specify a monitoring session that is being > > sent to the server on "monitor_cond" request. > > This uuid will

Re: [ovs-dev] [PATCH monitor_cond V6 05/11] ovsdb-client: support monitor-cond method

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 02/06/2016 03:15:57 AM: > On Tue, May 17, 2016 at 05:27:03PM +0300, Liran Schour wrote: > > Add monitor_cond method to ovsdb-client. Add unit tests. > > See ovsdb-client(1) man page for details. > > Replace monitor2 with monitor_cond. > > > > Signed-off-by: Liran Schour > >

Re: [ovs-dev] [PATCH monitor_cond V6 04/11] ovsdb: generate update notifications for monitor_cond session

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 02/06/2016 03:07:08 AM: > On Tue, May 17, 2016 at 05:27:02PM +0300, Liran Schour wrote: > > Hold session's conditions in ovsdb_monitor_session_condition. Pass it > > to ovsdb_monitor for generating "update2" notifications. > > Add functions that can generate "update2" notificat

Re: [ovs-dev] [PATCH monitor_cond V6 11/11] RFC OVN: Quick implementation of conditional monitoring

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 02/06/2016 08:38:10 PM: > On Tue, May 17, 2016 at 05:27:09PM +0300, Liran Schour wrote: > > Conditional monitor of: Port_Binding, Logical_Flow, Multicast_Group > > MAC_Binding tables. As a result ovn-controller will be notified only about > > records belongs to a datapath that

Re: [ovs-dev] [PATCH monitor_cond V6 02/11] ovsdb: add conditions utilities to support monitor_cond

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 01/06/2016 09:36:37 PM: > On Tue, May 17, 2016 at 05:27:00PM +0300, Liran Schour wrote: > > Change ovsdb_condition to be a 3-element json array or a boolean value. > > Conditions utilities will be used later for conditional monitoring. > > > > Signed-off-by: Liran Schour >

Re: [ovs-dev] [PATCH monitor_cond V6 01/11] ovsdb: create column index mapping between ovsdb row to monitor row

2016-06-06 Thread Liran Schour
Ben Pfaff wrote on 01/06/2016 09:18:46 PM: > On Tue, May 17, 2016 at 05:26:59PM +0300, Liran Schour wrote: > > Columns indexing is different in ovsdb_row then in ovsdb_monitor_row. > > We need mapping between the 2 for condition evaluation. > > > > signed-off-by: Liran Schour > > In ovsdb_moni

Re: [ovs-dev] [PATCH] tests: fix issue in use of OVS_APP_EXIT_AND_WAIT

2016-06-06 Thread Paul Boca
Hi Lance! Thanks for fixing this. Tested on Windows and the fix works. Acked-by: Paul-Daniel Boca > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Lance > Richardson > Sent: Saturday, June 4, 2016 1:09 AM > To: dev@openvswitch.org > Subject: [ovs-dev] [

[ovs-dev] [PATCH net-next] NSH(Network Service Header) implementation

2016-06-06 Thread Yi Yang
IETF defined NSH(Network Service Header) for Service Function Chaining, this is an IETF draft https://tools.ietf.org/html/draft-ietf-sfc-nsh-05 It will be a IETF standard shortly, this patch implemented NSH for Open vSwitch. Signed-off-by: Johnson Li Signed-off-by: Yi Yang --- drivers/net/vxl

Re: [ovs-dev] [PATCH] OVN: datalog man page

2016-06-06 Thread Yusheng Wang
Hi Flaviof?, The datalog version that is going to be used in OVN is a simplified version of datalog and will not be as powerful as pyDatalog. We could think of it as a shorthand of SQL join. Test-interactive will be part of the patch of the engine, to be appear soon. It would be easier to und

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

2016-06-06 Thread Babu Shanmugam
On Saturday 04 June 2016 01:17 AM, Ben Pfaff wrote: On Wed, May 25, 2016 at 03:35:44PM +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 fre

[ovs-dev] STATUS

2016-06-06 Thread Automatic Email Delivery Software
This Message was undeliverable due to the following reason: Your message was not delivered because the destination computer was not reachable 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 ther

Re: [ovs-dev] deadline extended as per your request

2016-06-06 Thread dev
Hello, Just wanted to know where I should send you the profits you'll be making? My team is all setup and ready to welcome you on board, so that we can start you on one of the most amazing pesos making journeys you'll ever take. >> CLICK HERE TO START YOUR JOURNEY Let's not waste another m

[ovs-dev] [RFC PATCH] ovn: Support native DHCPv6

2016-06-06 Thread Numan Siddique
In order for the VMs to use this feature, ovn-controller should send RA with M/O flags set. DHCPv6 options - Server id (2), DNS recursive server (23) and Domain search list (24) are supported in this patch. This patch is on top of native DHCPv4 patches and it also requires the upstream fix "b4f70

  1   2   >