[ovs-dev] Mail System Error - Returned Mail

2015-11-03 Thread Mail Delivery Subsystem
Dear user dev@openvswitch.org, Your e-mail account was used to send a large amount of junk email during the last week. Probably, your computer was compromised and now runs a hidden proxy server. We recommend that you follow instructions in order to keep your computer safe. Best wishes, openvswi

[ovs-dev] Delivery failed

2015-11-03 Thread Returned mail
This message was undeliverable due to the following reason: Your message was not 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 likely there

[ovs-dev] how ovs sends port-shut message to controller ?

2015-11-03 Thread tech_kals Kals
Hi experts, Just want to understand how port shut message sent by OVS to the controller when any one of the port admin state is set to down through CLI command.. As per openflow specification, when a port state is changed to down state, the switch should send OFPT_PORT_STATUS message to noti

[ovs-dev] how to print message on OVS

2015-11-03 Thread tech_kals Kals
Hi, I have added few helping comments in OVS code using "printf" and they are not getting printed on my console though those functions are hit... anyone has tried it out ? thanks, Kals ___ dev mailing list dev@openvswitch.org http://openvswitch.org/

[ovs-dev] VMs are in different VLAN (but on the same host) ping each other...!

2015-11-03 Thread tech_kals Kals
Hi experts, i have the following setup. VM1 - OVS ---VM2 *eth0 eth1* (vlan100,200) (vlan400,500) Have created two VMs

Re: [ovs-dev] VMs are in different VLAN (but on the same host) ping each other...!

2015-11-03 Thread tech_kals Kals
am still confusing why it works ? On Tue, Nov 3, 2015 at 4:38 PM, tech_kals Kals wrote: > Hi experts, > > i have the following setup. > > VM1 - OVS > ---VM2 > *eth0

[ovs-dev] Transaction PNF51MY6

2015-11-03 Thread Kermit Walker
Transaction Amount: 29491.64 US Dollars Date: 11-02-2015 Transaction Status: Please see enclosed Statement. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] How to use vlog in 'datapath' module?

2015-11-03 Thread wenhao...@gmail.com
hi all, I'm trying to add something into datapath module and need to use vlog for debugging. I tried to put #include "vlog.h" at the biginning of datapath.c as well as VLOG_DEFINE_THIS_MODULE(datapath); I still got nothing in /var/log/openvswitch/ovs-vswitchd.log from the function VLOG_INFO("

Re: [ovs-dev] How to use vlog in 'datapath' module?

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 11:46:14PM +0800, wenhao...@gmail.com wrote: > I'm trying to add something into datapath module and need to use vlog > for debugging. > > I tried to put > #include "vlog.h" > at the biginning of datapath.c as well as > VLOG_DEFINE_THIS_MODULE(datapath); > I still got not

Re: [ovs-dev] VMs are in different VLAN (but on the same host) ping each other...!

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 04:38:00PM +0530, tech_kals Kals wrote: > i have the following setup. > > VM1 - OVS > ---VM2 > *eth0 eth1* >

Re: [ovs-dev] how ovs sends port-shut message to controller ?

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 04:02:59PM +0530, tech_kals Kals wrote: > Just want to understand how port shut message sent by OVS to the controller > when any one of the port admin state is set to down through CLI > command.. > > As per openflow specification, when a port state is changed to down st

Re: [ovs-dev] [mointor2 7/9] ovsdb-client: support monitor2

2015-11-03 Thread Andy Zhou
On Mon, Nov 2, 2015 at 11:50 PM, Liran Schour wrote: > "dev" wrote on 22/10/2015 07:45:29 AM: >> >> Add monito2 option to ovsdb-client. Sed ovsdb-client(1) manpage patch >> for details. >> >> Signed-off-by: Andy Zhou >> --- > ... > >> static void >> +monitor2_print_row(struct json *row, const c

Re: [ovs-dev] how to print message on OVS

2015-11-03 Thread Justin Pettit
> On Nov 3, 2015, at 2:47 AM, tech_kals Kals wrote: > > Hi, > > I have added few helping comments in OVS code using "printf" and they are > not getting printed on my console though those functions are hit... > > > anyone has tried it out ? Make sure that you're not running OVS in the backgro

Re: [ovs-dev] How to use vlog in 'datapath' module?

2015-11-03 Thread Ben Pfaff
Please don't drop the list. You need to use the usual Linux kernel tools, such as printk, to debug the kernel module. On Wed, Nov 04, 2015 at 01:35:56AM +0800, wenhao...@gmail.com wrote: > Then what should i do to debug in datapath? Is there any other way to get > information from this module?

Re: [ovs-dev] [PATCH v2] datapath-windows: Updating an External Adapter causes flow lookup failure

2015-11-03 Thread Ben Pfaff
On Mon, Nov 02, 2015 at 05:17:07PM -0800, Sairam Venugopal wrote: > This patch fixes an issue with updating the propeties of an external > adapter in Windows. The issue causes flow lookups to fail until the > kernel is reinstalled. > > Associated bug - https://github.com/openvswitch/ovs-issues/iss

Re: [ovs-dev] vlan_tci flow key attribute becoming zero

2015-11-03 Thread Ben Pfaff
On Mon, Nov 02, 2015 at 02:27:21PM +, ravulakollu.ku...@wipro.com wrote: > I am using openvswitch 2.4 version , compiled under centos 7 (kernel version > 3.18.22) . I am sending double tagged traffic to one trunk port , the same > and coming out from the other trunk port (vswitchd is configu

Re: [ovs-dev] [PATCH] bfd: improve ovs-vswitchd.conf.db(5) manpage

2015-11-03 Thread Justin Pettit
> On Oct 22, 2015, at 8:43 PM, Andy Zhou wrote: > > Use the wording from RFC 5880 to describe the "diagnostic" and > "remote_diagnostic" fields. > > Reported-by: Justin Pettit > Signed-off-by: Andy Zhou > --- > vswitchd/vswitch.xml | 13 +++-- > 1 file changed, 7 insertions(+), 6 delet

Re: [ovs-dev] [PATCH] datapath-windows: Report correctly when trying to add tunnel types

2015-11-03 Thread Ben Pfaff
On Thu, Oct 29, 2015 at 06:15:44AM +, Alin Serdean wrote: > Report invalid parameter to the userspace if the user tries to add a vport > tunnel type which is not supported by the kernel extension. > > Signed-off-by: Alin Gabriel Serdean Applied to master, thanks!

Re: [ovs-dev] [PATCH] bfd: improve ovs-vswitchd.conf.db(5) manpage

2015-11-03 Thread Andy Zhou
On Tue, Nov 3, 2015 at 10:53 AM, Justin Pettit wrote: > >> On Oct 22, 2015, at 8:43 PM, Andy Zhou wrote: >> >> Use the wording from RFC 5880 to describe the "diagnostic" and >> "remote_diagnostic" fields. >> >> Reported-by: Justin Pettit >> Signed-off-by: Andy Zhou >> --- >> vswitchd/vswitch.xm

Re: [ovs-dev] PassiveStream class in IDL library doesn't support AF_INET socket

2015-11-03 Thread Ben Pfaff
On Wed, Oct 28, 2015 at 07:39:42AM +, D M, Vikas wrote: > We thought of using ovs IDL library for implementing the L2gateway in > openstack with ovsdb hardwatre vtep schema. > (https://wiki.openstack.org/wiki/Neutron/L2-GW ). > > ovsdb server initiates the connection to l2gateway agent with t

Re: [ovs-dev] [PATCH v4 1/3] Implement Openflow 1.4 Vacancy Events for OFPT_TABLE_MOD.

2015-11-03 Thread Ben Pfaff
On Tue, Oct 27, 2015 at 10:42:41AM +0530, Saloni Jain wrote: > >In ovs-ofctl.c, I don't think the table-mod code handles the case where > >OF1.4 or OF1.5 is enabled but the switch does not support it. > > This means that in ovs-ofctl.c, for table-mod, it can happen that OF1.4 and > OF1.5 are supp

Re: [ovs-dev] [PATCH 1/3] ovn: Fix check on existing encap row.

2015-11-03 Thread Ben Pfaff
On Tue, Oct 27, 2015 at 06:05:33PM +0900, Russell Bryant wrote: > This code does some checking to validate the existing encaps for a > chassis to see if they need to be updated. This typo resulted in > ovn-controller re-creating its encap(s) every time this code ran, making > ovn-controller and ov

Re: [ovs-dev] [PATCH 2/3] ovn: Remove duplicate versions from schemas.

2015-11-03 Thread Ben Pfaff
On Tue, Oct 27, 2015 at 06:05:34PM +0900, Russell Bryant wrote: > Since commit 5935835968c9d36ffe306863f0c8079d3b670e2a, the OVN nb and sb > schema definitions have included duplicate version entries. In the nb > case, the version has since been updated to 2.0.0, but only in one > place. Remove t

[ovs-dev] [PATCH] Fix command typo in INSTALL.SSL.md

2015-11-03 Thread YourName
From: Aaron Rosen Signed-off-by: Aaron Rosen --- INSTALL.SSL.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/INSTALL.SSL.md b/INSTALL.SSL.md index 06be303..f294a27 100644 --- a/INSTALL.SSL.md +++ b/INSTALL.SSL.md @@ -202,7 +202,7 @@ more secure. 1. Run the following co

[ovs-dev] [PATCH] Fix command typo in INSTALL.SSL.md

2015-11-03 Thread Aaron Rosen
Signed-off-by: Aaron Rosen --- INSTALL.SSL.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/INSTALL.SSL.md b/INSTALL.SSL.md index 06be303..f294a27 100644 --- a/INSTALL.SSL.md +++ b/INSTALL.SSL.md @@ -202,7 +202,7 @@ more secure. 1. Run the following command on the Open vS

Re: [ovs-dev] [PATCH 2/3] ovn: Remove duplicate versions from schemas.

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 12:26:42PM -0800, Ben Pfaff wrote: > I'm surprised that the OVS JSON parser doesn't reject this. I might > have to have another look at it. The OVS JSON parser always uses the last value when there is a duplicate key within an object. It doesn't complain in any way. I wo

Re: [ovs-dev] [PATCH 3/3] ovn-northd: Fix table ID in a comment.

2015-11-03 Thread Ben Pfaff
On Tue, Oct 27, 2015 at 06:05:35PM +0900, Russell Bryant wrote: > This changed from 2 to 3 when ACLs got implemented, as it turned out > ACLs needed two tables (1 and 2). > > While we're at it, do a bit of OCD formatting cleanup by fixing the > alignment of '\' at the end of each line in the logic

Re: [ovs-dev] [PATCH] datapath-windows: STT - Offload inner checksum calculation

2015-11-03 Thread Ben Pfaff
On Tue, Oct 27, 2015 at 02:36:03PM -0700, Sairam Venugopal wrote: > Offload the inner checksum computation to NDIS in OvsDecapStt function. > > Signed-off-by: Sairam Venugopal Applied, thanks! I'd ordinarily wait for a Cloudbase review but this feature is so new that this patch can't really cau

Re: [ovs-dev] [PATCH] Fix command typo in INSTALL.SSL.md

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 12:29:21PM -0800, Aaron Rosen wrote: > Signed-off-by: Aaron Rosen > --- > INSTALL.SSL.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/INSTALL.SSL.md b/INSTALL.SSL.md > index 06be303..f294a27 100644 > --- a/INSTALL.SSL.md > +++ b/INSTALL.SSL.md

Re: [ovs-dev] [PATCH] datapath-windows: STT - Offload inner checksum calculation

2015-11-03 Thread Sairam Venugopal
Ben, Thanks for applying this one. Sairam On 11/3/15, 12:38 PM, "Ben Pfaff" wrote: >On Tue, Oct 27, 2015 at 02:36:03PM -0700, Sairam Venugopal wrote: >> Offload the inner checksum computation to NDIS in OvsDecapStt function. >> >> Signed-off-by: Sairam Venugopal > >Applied, thanks! > >I'd or

Re: [ovs-dev] [PATCH] Fix command typo in INSTALL.SSL.md

2015-11-03 Thread Aaron Rosen
Whoops I see: req NAME Create new private key and certificate request named NAME-privkey.pem and NAME-req.pem, resp. $ ovs-pki req sc switch /usr/local/bin/ovs-pki: req must have exactly one argument; use --help for help Maybe the typo then is the option reg

[ovs-dev] [PATCH] INSTALL.SSL.md: Fix arguments to "ovs-pki req" command.

2015-11-03 Thread Ben Pfaff
Reported-by: Aaron Rosen Reported-at: http://openvswitch.org/pipermail/dev/2015-November/061778.html Signed-off-by: Ben Pfaff --- INSTALL.SSL.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/INSTALL.SSL.md b/INSTALL.SSL.md index 06be303..714c6ec 100644 --- a/INSTALL.SSL.md

Re: [ovs-dev] [PATCH] Fix command typo in INSTALL.SSL.md

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 12:49:15PM -0800, Aaron Rosen wrote: > Whoops I see: > > req NAME Create new private key and certificate request >named NAME-privkey.pem and NAME-req.pem, resp. > > > $ ovs-pki req sc switch > /usr/local/bin/ovs-pki: req must have exa

Re: [ovs-dev] [PATCH 3/3] ovn-northd: Fix table ID in a comment.

2015-11-03 Thread Russell Bryant
On 11/03/2015 03:34 PM, Ben Pfaff wrote: > On Tue, Oct 27, 2015 at 06:05:35PM +0900, Russell Bryant wrote: >> This changed from 2 to 3 when ACLs got implemented, as it turned out >> ACLs needed two tables (1 and 2). >> >> While we're at it, do a bit of OCD formatting cleanup by fixing the >> alignm

Re: [ovs-dev] [PATCH] ovn-architecture: Start describing how to connect to physical networks.

2015-11-03 Thread Ben Pfaff
On Thu, Oct 22, 2015 at 09:28:51AM -0700, Ben Pfaff wrote: > Signed-off-by: Ben Pfaff I'm abandoning this patch. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Analysis of failures seen when adding eth0 and eth0.300 vlan to OVS bridges

2015-11-03 Thread Karol Mroz
Hello, I have a colleague who is creating an openvswitch configuration consisting of an eth0, a vlan eth0.300 and two OVS bridges: br-ex and br-fixed. The system uses openvswitch-2.1.2 and linux kernel version 3.12.X Initially, eth0 and eth0.300 are configured with the following addresses: eth0

Re: [ovs-dev] [PATCH] ovn-architecture: Start describing how to connect to physical networks.

2015-11-03 Thread Han Zhou
Hi Ben, Could you share the reason of abandoning? Looking forward to a new patch regarding this topic. Han On Tue, Nov 3, 2015 at 1:42 PM, Ben Pfaff wrote: > On Thu, Oct 22, 2015 at 09:28:51AM -0700, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > > I'm abandoning this patch. > __

[ovs-dev] [ICMP v2 1/7] ovn-northd.8: Correct description of sending out inport.

2015-11-03 Thread Justin Pettit
Signed-off-by: Justin Pettit --- ovn/northd/ovn-northd.8.xml |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml index c5760a5..4bd1990 100644 --- a/ovn/northd/ovn-northd.8.xml +++ b/ovn/northd/ovn-northd.8.xml @@ -37

[ovs-dev] [ICMP v2 2/7] ovn-sb: Fix "ip.ttl--" lower limit description.

2015-11-03 Thread Justin Pettit
To decrement the IP TTL, the existing TTL can't be less than two. The field is not bit-maskable, though, so "ip.ttl < 2" will not work. Signed-off-by: Justin Pettit --- ovn/ovn-sb.xml |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml inde

[ovs-dev] [ICMP v2 0/7] Logical Router Pings

2015-11-03 Thread Justin Pettit
This patch series makes it possible to modify ICMPv4 packets and ping logical router ports in OVN. It also has a few minor cleanups related to OVN. v1->v2: Resend without patches that have been already merged and rebased on latest master. Justin Pettit (7): ovn-northd.8: Correct descri

[ovs-dev] [ICMP v2 6/7] vswitchd: Allow modifying ICMP type and code.

2015-11-03 Thread Justin Pettit
Signed-off-by: Justin Pettit --- NEWS |1 + lib/flow.h|4 ++-- lib/meta-flow.h |8 lib/odp-execute.c | 12 ++-- lib/odp-util.c| 44 +--- lib/ofp-actions.c |4 tests/ofproto.at |4 ++--

[ovs-dev] [ICMP v2 4/7] ovn: Change printed stage names.

2015-11-03 Thread Justin Pettit
The stage names were getting long and throwing off the formatting when dumping the logical flows. Signed-off-by: Justin Pettit --- ovn/northd/ovn-northd.c | 40 ovn/utilities/ovn-sbctl.c |2 +- 2 files changed, 21 insertions(+), 21 deletions(-) d

[ovs-dev] [ICMP v2 7/7] ovn-northd: Support pinging logical router ports.

2015-11-03 Thread Justin Pettit
Signed-off-by: Justin Pettit --- ovn/northd/ovn-northd.8.xml |5 + ovn/northd/ovn-northd.c | 23 --- 2 files changed, 21 insertions(+), 7 deletions(-) diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml index 6f0a420..e7dec72 100644 --- a/ovn/

[ovs-dev] [ICMP v2 3/7] ovn: Use "ip.ttl--" instead of "ip4.ttl--".

2015-11-03 Thread Justin Pettit
Signed-off-by: Justin Pettit --- ovn/lib/actions.c |4 ++-- ovn/northd/ovn-northd.8.xml | 14 +++--- ovn/northd/ovn-northd.c |2 +- tests/ovn.at|5 +++-- 4 files changed, 13 insertions(+), 12 deletions(-) diff --git a/ovn/lib/actions.c b/ovn/li

[ovs-dev] [ICMP v2 5/7] packets: Add support for modifying ICMP type and code.

2015-11-03 Thread Justin Pettit
Signed-off-by: Justin Pettit --- lib/packets.c | 18 ++ lib/packets.h |1 + 2 files changed, 19 insertions(+), 0 deletions(-) diff --git a/lib/packets.c b/lib/packets.c index 342d8b7..701a5ec 100644 --- a/lib/packets.c +++ b/lib/packets.c @@ -905,6 +905,24 @@ packet_set_sct

[ovs-dev] [PATCH] tests: add documentation for OVS_WAIT_UNTIL and OVS_WAIT_WHILE macros

2015-11-03 Thread Ansis Atteka
It is very easy to misuse these macros, because when the COMMAND returns exit code "0" it is actually considered as if condition evaluated to "true" and not "false" as some might think. This patch ensures that this is clearly reflected in documentation. Signed-off-by: Ansis Atteka --- tests/ovs

Re: [ovs-dev] [PATCH] tests: add documentation for OVS_WAIT_UNTIL and OVS_WAIT_WHILE macros

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 04:04:33PM -0800, Ansis Atteka wrote: > It is very easy to misuse these macros, because when the COMMAND > returns exit code "0" it is actually considered as if condition > evaluated to "true" and not "false" as some might think. > > This patch ensures that this is clearly

Re: [ovs-dev] [PATCH] tests: add documentation for OVS_WAIT_UNTIL and OVS_WAIT_WHILE macros

2015-11-03 Thread Ansis Atteka
On Tue, Nov 3, 2015 at 4:07 PM, Ben Pfaff wrote: > On Tue, Nov 03, 2015 at 04:04:33PM -0800, Ansis Atteka wrote: >> It is very easy to misuse these macros, because when the COMMAND >> returns exit code "0" it is actually considered as if condition >> evaluated to "true" and not "false" as some mig

[ovs-dev] [PATCHv2] tests: add documentation for OVS_WAIT_UNTIL and OVS_WAIT_WHILE macros

2015-11-03 Thread Ansis Atteka
It is very easy to misuse these macros, because when the COMMAND returns exit code "0" it is actually considered as if condition evaluated to "true" and not "false" as some might think. This patch ensures that this is clearly reflected in documentation. Signed-off-by: Ansis Atteka --- tests/ovs

Re: [ovs-dev] [PATCHv2] tests: add documentation for OVS_WAIT_UNTIL and OVS_WAIT_WHILE macros

2015-11-03 Thread Ben Pfaff
On Tue, Nov 03, 2015 at 04:19:16PM -0800, Ansis Atteka wrote: > It is very easy to misuse these macros, because when the COMMAND > returns exit code "0" it is actually considered as if condition > evaluated to "true" and not "false" as some might think. > > This patch ensures that this is clearly

Re: [ovs-dev] [PATCHv2] tests: add documentation for OVS_WAIT_UNTIL and OVS_WAIT_WHILE macros

2015-11-03 Thread Ansis Atteka
On Tue, Nov 3, 2015 at 4:22 PM, Ben Pfaff wrote: > On Tue, Nov 03, 2015 at 04:19:16PM -0800, Ansis Atteka wrote: >> It is very easy to misuse these macros, because when the COMMAND >> returns exit code "0" it is actually considered as if condition >> evaluated to "true" and not "false" as some mig

[ovs-dev] [PATCH 03/11] ct-dpif: Add ct_dpif_dump_{start, next, done}().

2015-11-03 Thread Daniele Di Proietto
These function can be used to dump conntrack entries from a datapath. They simply call a function pointer in the dpif_class. No dpif currently implements the interface. The next commits will provide an implementation in dpif-netlink. Signed-off-by: Daniele Di Proietto --- lib/ct-dpif.c |

[ovs-dev] [PATCH 11/11] system-kmod-macros: Do not require the 'conntrack' tool.

2015-11-03 Thread Daniele Di Proietto
We can use 'test-ovs test-netlink-conntrack' instead. Now that it is not required anymore, we can remove the HAVE_CONNTRACK macro in the build system. Signed-off-by: Daniele Di Proietto --- tests/atlocal.in| 7 --- tests/system-kmod-macros.at | 5 ++--- 2 files changed, 2 insert

[ovs-dev] [PATCH 07/11] dpif-netlink: Implement ct_flush.

2015-11-03 Thread Daniele Di Proietto
This member function is used by the ct-dpif module to provide its services. It's implemented using the netlink-conntrack module. N.B. The Linux kernel datapaths share the connection tracker among them and with the rest of the system. Therefore the operations are not really dpif specific. Signed

[ovs-dev] [PATCH 00/11] Conntrack debugging appctl/dpctl

2015-11-03 Thread Daniele Di Proietto
The goal of this series is to introduce two dpctl command to interact with the Linux kernel connection tracker. The same infrastructure will be used by the userspace connection tracker. First, it defines some structures and some formatting routines (ct-dpif). Then, it adds some code to transform

[ovs-dev] [PATCH 08/11] dpctl: Add new 'flush-conntrack' command.

2015-11-03 Thread Daniele Di Proietto
Signed-off-by: Daniele Di Proietto --- lib/dpctl.c | 30 ++ lib/dpctl.man | 6 ++ 2 files changed, 36 insertions(+) diff --git a/lib/dpctl.c b/lib/dpctl.c index aadfd68..a87e71e 100644 --- a/lib/dpctl.c +++ b/lib/dpctl.c @@ -1285,6 +1285,35 @@ dpctl_dump_conntr

[ovs-dev] [PATCH 09/11] ovs-test: Add test-netlink-conntrack command.

2015-11-03 Thread Daniele Di Proietto
Add a new test module to help debug Linux kernel conntrack development unsing the netlink-conntrack module. The tool has three uses: * `ovstest test-netlink-conntrack dump [zone=zone]` shows a list of the connection table * `ovstest test-netlink-conntrack monitor` displays the updates on t

[ovs-dev] [PATCH 10/11] system-traffic: use `dpctl/*conntrack` instead of `conntrack` tool.

2015-11-03 Thread Daniele Di Proietto
Often in the tests we inspect the conntrack tables with the 'conntrack' command line utility. Since this may not always be available, and since these tests are supposed to run with the upcoming userspace connection tracker, it is better to use the newly implemented dpctl command. Due to the tcp s

[ovs-dev] [PATCH 05/11] dpif-netlink: Implement ct_dump_{start, next, done}.

2015-11-03 Thread Daniele Di Proietto
These member functions are used by the ct-dpif module to provide its services. They're implemented using the netlink-conntrack module. N.B. The Linux kernel datapaths share the connection tracker among them and with the rest of the system. Therefore the operations are not really dpif specific.

[ovs-dev] [PATCH 01/11] ct-dpif: New module.

2015-11-03 Thread Daniele Di Proietto
This defines some structures (and their related formatting functions) to manipulate entries in connection tracking tables. It will be used by next commits. Based on original work by Jarno Rajahalme Signed-off-by: Jarno Rajahalme Signed-off-by: Daniele Di Proietto --- lib/automake.mk | 2 +

[ovs-dev] [PATCH 06/11] dpctl: Add 'conntrack-dump' command.

2015-11-03 Thread Daniele Di Proietto
It can be used to inspect the connection tracking entries in the datapath. Signed-off-by: Daniele Di Proietto --- lib/dpctl.c | 45 + lib/dpctl.man | 24 2 files changed, 69 insertions(+) diff --git a/lib/dpctl.c b/lib/dpctl

[ovs-dev] [PATCH 04/11] ct-dpif: Add ct_dpif_flush().

2015-11-03 Thread Daniele Di Proietto
This function will flush the connection tracking tables of a specific datapath. It simply calls a function pointer in the dpif_class. No dpif currently implements the required interface. The next commits will provide an implementation in dpif-netlink. Signed-off-by: Daniele Di Proietto --- lib

[ovs-dev] [PATCH 02/11] netlink-conntrack: New module.

2015-11-03 Thread Daniele Di Proietto
This module uses the netlink interface provide by the Linux kernel connection tracker to provide some visibility into the conntrack tables. The module provides functions to: * Convert a netlink representation of a connection into a struct 'ct_dpif_entry'. * Dump all the connections. * Flush a

[ovs-dev] [PATCH 2/2] ofproto: Implement OFPT_QUEUE_GET_CONFIG_REQUEST for OFPP_ANY in OF1.1+.

2015-11-03 Thread Ben Pfaff
I was not previously aware that this feature was missing. Reported-by: Minoru TAKAHASHI Reported-at: http://openvswitch.org/pipermail/discuss/2015-October/019229.html Signed-off-by: Ben Pfaff --- NEWS | 1 + lib/ofp-util.c | 22 - lib/ofp-util.

[ovs-dev] [PATCH 1/2] ovs-ofctl: Generalize code for finding ports into general-purpose iterator.

2015-11-03 Thread Ben Pfaff
The port_iterator will acquire another user in an upcoming commit. Signed-off-by: Ben Pfaff --- utilities/ovs-ofctl.c | 233 ++ 1 file changed, 123 insertions(+), 110 deletions(-) diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c index ee

Re: [ovs-dev] [PATCH] ovn-architecture: Start describing how to connect to physical networks.

2015-11-03 Thread Ben Pfaff
More accurately, I'm abandoning the iptables method. There are better ways that avoid using it. I'll be back in a while with better ways to connect to physical networks. On Tue, Nov 03, 2015 at 02:48:15PM -0800, Han Zhou wrote: > Hi Ben, > > Could you share the reason of abandoning? Looking for

Re: [ovs-dev] [PATCH] ovn-tutorial: Add a section on ACLs.

2015-11-03 Thread Ben Pfaff
On Thu, Oct 22, 2015 at 11:06:53AM -0400, Russell Bryant wrote: > On 10/21/2015 04:17 PM, Russell Bryant wrote: > > Add a section that gives a quick introduction to applying ACLs. > > This section doesn't include ovs-appctl commands to run, as I haven't > > figured out a useful series of commands t

Re: [ovs-dev] [PATCH] ovn-tutorial: Add a section on ACLs.

2015-11-03 Thread Russell Bryant
> On Nov 3, 2015, at 8:23 PM, Ben Pfaff wrote: > >> On Thu, Oct 22, 2015 at 11:06:53AM -0400, Russell Bryant wrote: >>> On 10/21/2015 04:17 PM, Russell Bryant wrote: >>> Add a section that gives a quick introduction to applying ACLs. >>> This section doesn't include ovs-appctl commands to run, a

[ovs-dev] 代-开>>税-票

2015-11-03 Thread gfji5eorfgh
您好! 我公司有行业税--票可对外代--开; 如:(1)商品销售税--票; (2)广告业、各类服务业等等税--票。 贵公司如有需要,欢迎您来电与我联系: 负责人:钱先生(0)13537741120 ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] dpctl: Fix jump through wild pointer in "dpctl/help".

2015-11-03 Thread Ben Pfaff
On Tue, Oct 20, 2015 at 10:55:37PM +, Daniele Di Proietto wrote: > > On 17/10/2015 14:24, "Ben Pfaff" wrote: > > >dpctl_unixctl_handler() didn't fully initialize the dpctl_params structure > >it passed to the handler, which meant that dpctl_help() could see a > >nonnull > >(indeterminate) 'u

[ovs-dev] Mail System Error - Returned Mail

2015-11-03 Thread Post Office
Your message could not be delivered ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ofp-parse: Fix parsing, formatting of multiple fields in NTR extension.

2015-11-03 Thread Ben Pfaff
On Mon, Oct 19, 2015 at 11:17:16AM +0900, Simon Horman wrote: > On Fri, Oct 16, 2015 at 08:21:06AM -0700, Ben Pfaff wrote: > > Until now, the only way to specify multiple fields in the "fields" > > parameter for the Netronome groups extension, was to specify "fields" > > more than once, e.g. fields

Re: [ovs-dev] [PATCH v2] netdev_dpdk.c: Add QoS functionality.

2015-11-03 Thread Ben Pfaff
On Thu, Oct 22, 2015 at 09:29:37AM +, Stokes, Ian wrote: > Just wondering if anyone has input on v2 of the netdev-dpdk QoS patch? > Any feedback would be much appreciated. I don't object but I'm the wrong person to affirmatively review it. ___ dev ma

Re: [ovs-dev] ovs-atomic: loss some patches for v2.3.2

2015-11-03 Thread Ben Pfaff
On Tue, Oct 13, 2015 at 11:39:10AM +0800, linhaifeng wrote: > 于 2015/10/12 23:24, Ben Pfaff 写道: > >Please propose which ones you want to cherry-pick, and what problems > >they fix. > we apply cmap patches from v2.4 for v2.3.2 found that dead loop in > cmap_find__. > because read_even_counter read t

Re: [ovs-dev] [PATCH] bugfix of ovsdb-client connecting error when updating ca_crt.pem file many times

2015-11-03 Thread Ben Pfaff
On Tue, Oct 13, 2015 at 12:41:14PM +, Liuyongqiang (A) wrote: > From 786c6d16ab18197a750f832e4eed1ccfa1183d04 Mon Sep 17 00:00:00 2001 > From: YongQiangLiu > Date: Tue, 13 Oct 2015 19:37:32 +0800 > Subject: [PATCH] bugfix of ovsdb-client connecting error when updating > ca_crt.pem file many t

[ovs-dev] Returned mail: Data format error

2015-11-03 Thread Automatic Email Delivery Software
Your message was undeliverable due to the following reason(s): Your message could not be 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 lik

[ovs-dev] [PATCH] Better abstract OFPT_SET_CONFIG and OFPT_GET_CONFIG_REPLY, make stricter.

2015-11-03 Thread Ben Pfaff
The OFPT_SET_CONFIG and OFPT_GET_CONFIG_REPLY messages, which have the same format, have a 'flags' field in which OpenFlow defines some bits, which change somewhat from one version to another, and does not define others. Until now, Open vSwitch has not abstracted these messages at all and has igno

Re: [ovs-dev] [PATCH] ofp-parse: Fix parsing, formatting of multiple fields in NTR extension.

2015-11-03 Thread Simon Horman
On Tue, Nov 03, 2015 at 07:53:06PM -0800, Ben Pfaff wrote: > On Mon, Oct 19, 2015 at 11:17:16AM +0900, Simon Horman wrote: > > On Fri, Oct 16, 2015 at 08:21:06AM -0700, Ben Pfaff wrote: > > > Until now, the only way to specify multiple fields in the "fields" > > > parameter for the Netronome groups

Re: [ovs-dev] PassiveStream class in IDL library doesn't support AF_INET socket

2015-11-03 Thread D M, Vikas
Hi Ben, In my earlier email, I wanted to enquire whether the feature to receive socket open requests from an OVSDB server is available in the IDL class (per the configuration in the Hardware_Vtep Manager table). Currently, I do not have a bandwidth to contribute in the IDL library due to oth

[ovs-dev] problems debugging datapath module

2015-11-03 Thread wenhao...@gmail.com
hi all, I tried to put a "printk" line into the biginning of function "ovs_dp_process_packet", since i assumed that every packet appeared in the links to the switch will trigger that function. but when i updated ovs and start mininet with the modified ovs, i still cannot see the output of "pr