[ovs-dev] Extensible Flow Entry Statistics' EXT-334 in OpenFlow 1.5.

Hi All I am Working For Tata Consultancy Services Ltd., Its my great pleasure to participate in the OpenVSwtich Community. I have gone through the code, mail threads and existing implementation OVS for OpenFlow 1.5 - Extensible Flow Entry Statistics' EXT-334' . But I didn't found any implementat

Re: [ovs-dev] please help review patch "allow DHCPv6 respond multiple IA Address Options"

The patch will get reviewed, there's no need to ask again. On Tue, Aug 30, 2016 at 11:09:23AM +0800, Zong Kai Li wrote: > Hi, Numan and Justin. > I wish you can help review patch "allow DHCPv6 respond multiple IA > Address Options". > http://patchwork.ozlabs.org/patch/663493/ > > For current DHCP

[ovs-dev] [PATCH RFC v2] ovn-northd: add default_dhcpvx_options for Logical_Switch

This patch adds default_dhcpv4_options and default_dhcpv6_options columns for Logical_Switch, which should help CMS not to calculate and set dhcpv4_options and dhcpv6_options columns for lswitch ports on lswitchs one by one, when most of lswitch ports on the same lswitch are using the DHCPv4_Option

[ovs-dev] [PATCH v2] ovn-nbctl: Add LB commands.

Add a name column for the load balancer. This must be unique among all load balancers. This name has no special meaning or purpose other than to provide convenience for human interaction with the ovn-nb database. This patch provides the command line to create load-balancer and add the unit tests

[ovs-dev] please help review patch "allow DHCPv6 respond multiple IA Address Options"

Hi, Numan and Justin. I wish you can help review patch "allow DHCPv6 respond multiple IA Address Options". http://patchwork.ozlabs.org/patch/663493/ For current DHCPv6 implementation, it will respond one IPv6 address for a lswitch port in a DHCPv6 Reply message. The "one IPv6 address", I mean, one

[ovs-dev] DIRTY Mildrid Meneses needs to have some FUN, Dev

Hi anal punisher . Would you mind to finding a young and nice girl? :-) My name is Mildrid. I am from Ukraine =) Have you ever heard that the loveliest girls in the world live in my country? Don't even doubt!My page is - http://nebdfmzl.datingts。ru I am waiting for you-"Mildrid1998" src="http:/

Re: [ovs-dev] [PATCH v2 13/15] coverage: Rename init functions to avoid symbol collisions.

On Mon, Aug 22, 2016 at 04:31:39PM -0700, Jarno Rajahalme wrote: > ofproto now uses various *_init() functions, so use something else for > coverage constructors. > > Signed-off-by: Jarno Rajahalme __ is pretty common too, I'd suggest adding "coverage" or "counter" somewhere in the name. ___

Re: [ovs-dev] [PATCH v2 12/15] ofproto-dpif-xlate: Allow translating without side-effects.

On Mon, Aug 22, 2016 at 04:31:38PM -0700, Jarno Rajahalme wrote: > Extend 'may_learn' attribute to also control the treatment of > FIN_TIMEOUT action and asynchronous messages (packet ins, > continuations), so that when 'may_learn' is 'false' and > 'resubmit_stats' is 'NULL', no OpenFlow-visible si

Re: [ovs-dev] [PATCH v2 02/15] lib: Retire packet buffering feature.

On Mon, Aug 29, 2016 at 02:07:40PM -0700, Ben Pfaff wrote: > On Mon, Aug 22, 2016 at 04:31:28PM -0700, Jarno Rajahalme wrote: > > OVS implementation of buffering packets that are sent to the > > controller is not compliant with the OpenFlow specifications after > > OpenFlow 1.0, which is possibly t

Re: [ovs-dev] [PATCH v2 11/15] ofproto: Use ofproto_flow_mod for learn execution from xlate cache.

On Mon, Aug 22, 2016 at 04:31:37PM -0700, Jarno Rajahalme wrote: > Use ofproto_flow_mod with a reference to an existing or new rule > instead of ofputil_flow_mod for learn action execution from xlate > cache > > Typically we would find that when a learn xlate cache entry is > created, a preceding

Re: [ovs-dev] [PATCH] ipsec: Do not allow ipsec_gre tunnel traffic to exit unencrypted

On Mon, Aug 29, 2016 at 11:57 AM, Ansis Atteka wrote: > If ipsec_gre tunnel configuration is changed in OVSDB, > then GRE packets may sometimes exit unencrypted until > per-tunnel IPsec policies are installed by ovs-monitor-ipsec > daemon. > > This patch fixes this issue by installing single, low

Re: [ovs-dev] [PATCH] datapath-windows: Fix connection state tracking in Conntrack

Acked-by: Sairam Venugopal On 8/29/16, 12:12 PM, "Paul Boca" wrote: >A connection can be both a reply and related. > >Signed-off-by: Paul-Daniel Boca >--- > datapath-windows/ovsext/Conntrack.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/datapath-windows/ovsext/Connt

[ovs-dev] [PATCH] Flush freshly-polled sFlow counters promptly.

This patch changes the order of the steps that are followed every second in the sFlow agent. By moving the receiver_tick() step to the end, we ensure that any counters that were polled during the poller_tick() step are flushed immediately to the sFlow collector. This eliminates what was a variab

Re: [ovs-dev] [PATCH v2 09/15] ofproto-dpif-xlate: Expose xlate cache.

On Mon, Aug 22, 2016 at 04:31:35PM -0700, Jarno Rajahalme wrote: > Later patches will need to create xlate cache entries from different > modules. This patch refactors the xlate cache code in preparation > without any functional changes, so that the changes are clearly > visible in the following p

Re: [ovs-dev] [PATCH v2 07/15] object-collection: Remove access to stub.

On Mon, Aug 22, 2016 at 04:31:33PM -0700, Jarno Rajahalme wrote: > Better not use access to the *_collection_stub(), as it is an internal > implementation detail. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff ___ dev mailing list dev@openvswi

Re: [ovs-dev] [PATCH v2 10/15] ofproto-dpif-xlate: Add xlate cache type XC_TABLE.

On Mon, Aug 22, 2016 at 04:31:36PM -0700, Jarno Rajahalme wrote: > Xlate cache entry type XC_TABLE is required for the table stats > (number of misses and matches) to be correctly attributed. > > It appears that table stats have been off ever since xlate cache was > introduced. This was now revea

Re: [ovs-dev] [PATCH v2 09/15] ofproto-dpif-xlate: Expose xlate cache.

On Mon, Aug 22, 2016 at 04:31:35PM -0700, Jarno Rajahalme wrote: > Later patches will need to create xlate cache entries from different > modules. This patch refactors the xlate cache code in preparation > without any functional changes, so that the changes are clearly > visible in the following p

Re: [ovs-dev] [PATCH v2 08/15] connmgr: Make connmgr_wants_packet_in_on_miss() lock-free.

On Mon, Aug 22, 2016 at 04:31:34PM -0700, Jarno Rajahalme wrote: > Make connmgr_wants_packet_in_on_miss() use an atomic int instead of a > list traversal taking the 'ofproto_mutex'. This allows > connmgr_wants_packet_in_on_miss() to be called also when > 'ofproto_mutex' is already held, and makes

[ovs-dev] [PATCH] ovs-bugtool: Add interfaces file contents.

It is useful to know the contents of interfaces file for debugging in debian based systems. Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bu

Re: [ovs-dev] [PATCH v2 06/15] ofproto: Remove stale comment.

On Mon, Aug 22, 2016 at 04:31:32PM -0700, Jarno Rajahalme wrote: > The previous line tells that this comment is now stale. Remove it. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.or

Re: [ovs-dev] [PATCH v2 05/15] ofproto: Change rule's 'removed' member to a tri-state 'state'.

On Mon, Aug 22, 2016 at 04:31:31PM -0700, Jarno Rajahalme wrote: > As a rule may not be re-inserted to ofproto data structures, it is > cleaner to have three states for the rule, rather than just two. This > will be useful for managing learned flows in later patches. > > Signed-off-by: Jarno Raja

Re: [ovs-dev] [PATCH v2 04/15] ofproto-dpif: Remove prototype for an undefined function.

On Mon, Aug 22, 2016 at 04:31:30PM -0700, Jarno Rajahalme wrote: > We do not have a 'ofproto_dpif_refresh_rule()' function. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/

Re: [ovs-dev] [PATCH v2 03/15] types.h: Move mirror_mask_t here.

On Mon, Aug 22, 2016 at 04:31:29PM -0700, Jarno Rajahalme wrote: > Move mirror_mask_t from ofproto/ofproto-dpif-mirror.h to > openvswitch/types.h to avoid including function definitions with > colliding names (e.g., mirror_destroy(), which is also the name of a > static function in bridge.c). > >

Re: [ovs-dev] [PATCH v2 02/15] lib: Retire packet buffering feature.

On Mon, Aug 22, 2016 at 04:31:28PM -0700, Jarno Rajahalme wrote: > OVS implementation of buffering packets that are sent to the > controller is not compliant with the OpenFlow specifications after > OpenFlow 1.0, which is possibly true since OpenFlow 1.0 is not really > specifying the packet buffer

[ovs-dev] [PATCH v2] upcall: reduce log level for "no reference to recirc flow" message

Reduce log level from "warn" to "debug" for "upcall: no reference to recirc flow" log message. Suggested-by: Jarno Rajahalme Signed-off-by: Lance Richardson --- v2: Instead of ignoring log message in system-traffic.at, reduce level so it no longer appears in the log. ofproto/ofproto-dpif-

Re: [ovs-dev] [PATCH] system-traffic: Ignore "no reference for recirc flow" in ct controller test

> From: "Jarno Rajahalme" > To: "Lance Richardson" > Cc: dev@openvswitch.org > Sent: Monday, August 29, 2016 4:41:28 PM > Subject: Re: [ovs-dev] [PATCH] system-traffic: Ignore "no reference for > recirc flow" in ct controller test > > Lance, > > We discussed this off-line last week and the pro

Re: [ovs-dev] [PATCH] system-traffic: Ignore "no reference for recirc flow" in ct controller test

Lance, We discussed this off-line last week and the proper fix would be to change the log level for that message to to debug. Could you post a patch to do that? Jarno > On Aug 29, 2016, at 1:28 PM, Lance Richardson wrote: > > The conntrack - controller test case currently fails for the kern

[ovs-dev] [PATCH] system-traffic: Ignore "no reference for recirc flow" in ct controller test

The conntrack - controller test case currently fails for the kernel data path due to occurrences of the log message "upcall: no reference for recirc flow". This began with commit b4c632526b68 ("upcall: Log failure to flow_put for dpif-netlink."), which introduced this message for the kernel data pa

Re: [ovs-dev] [replication SMv2 7/7] ovsdb: Replication usability improvements

On Mon, Aug 29, 2016 at 3:14 AM, Numan Siddique wrote: > > > On Sat, Aug 27, 2016 at 4:45 AM, Andy Zhou wrote: > >> Added the '--no-sync' option base on feedbacks of current >> implementation. >> >> Added appctl command "ovsdb-server/sync-status" based on feedbacks >> of current implementation.

Re: [ovs-dev] [PATCH 0/3] Add LB commands.

Thanks for your review, I will submit the v2 patches soon. > On Aug 30, 2016, at 1:14 AM, Guru Shetty wrote: > > > On 28 August 2016 at 02:05, nickcooper-zhangtonghao > > wrote: > This patches provide the command line to create load-balancer and

[ovs-dev] [PATCH] datapath-windows: Fix connection state tracking in Conntrack

A connection can be both a reply and related. Signed-off-by: Paul-Daniel Boca --- datapath-windows/ovsext/Conntrack.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath-windows/ovsext/Conntrack.c b/datapath-windows/ovsext/Conntrack.c index af3d966..74fb38c 100644 --- a

[ovs-dev] [PATCH] datapath-windows: Fixed tcp flags byte order in conntrack

In user mode the flags are interpreted as little endian. This fix makes the kernel mode compatible with user mode. Signed-off-by: Paul-Daniel Boca --- datapath-windows/ovsext/Conntrack-tcp.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/datapath-windows/ovsex

[ovs-dev] [PATCH] ipsec: Do not allow ipsec_gre tunnel traffic to exit unencrypted

If ipsec_gre tunnel configuration is changed in OVSDB, then GRE packets may sometimes exit unencrypted until per-tunnel IPsec policies are installed by ovs-monitor-ipsec daemon. This patch fixes this issue by installing single, low priority IPsec block policy that drops all GRE packets coming out

[ovs-dev] [PATCH 2/2] ovs-ofctl: Extract tunnel metadata correctly when sorting flows.

When flow fields are sorted before dumping in ovs-ofctl, each significant field is extracted for sorting. However, in the case of tunnel metadata a mapping table is necessary to know where each field begins and ends. This information is current stripped off before fetching the field data and return

[ovs-dev] [PATCH 1/2] ovs-ofctl: Fix crash with replace-flows and diff-flows with tunnel metadata.

When flows are read by ovs-ofctl (either from a switch or a file), tunnel metadata space is dynamically allocated since there isn't a preset table. This works well for single flows but doesn't handle groups of flows that must be compared to each other. In this case, each flow will have its own inde

Re: [ovs-dev] [PATCH v2 01/15] vswitchd: Deprecate packet buffering in OVS 2.7.

> On Aug 29, 2016, at 10:53 AM, Ben Pfaff wrote: > > On Mon, Aug 22, 2016 at 04:31:27PM -0700, Jarno Rajahalme wrote: >> OVS implementation of buffering packets that are sent to the >> controller is not compliant with the OpenFlow specifications after >> OpenFlow 1.0. OVS implementation execute

Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output

From: Guru Shetty [mailto:g...@ovn.org] Sent: Monday, August 29, 2016 9:20 PM To: Paul Boca Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output So you are saying that running $srcdir/test-unixctl.py on Windows produces output in stderr. Why? Is there

Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output

> > > > So you are saying that running $srcdir/test-unixctl.py on Windows > produces output in stderr. Why? Is there a bug with Windows port? > > *[Paul Boca] *This is the way logging library works on both Windows and > Linux (https://docs.python.org/2/howto/logging.html - > > “If you call the func

Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output

From: Guru Shetty [mailto:g...@ovn.org] Sent: Monday, August 29, 2016 8:44 PM To: Paul Boca Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output On 29 August 2016 at 10:36, Paul Boca mailto:pb...@cloudbasesolutions.com>> wrote: Hi Guru, please see m

Re: [ovs-dev] [PATCH v2 01/15] vswitchd: Deprecate packet buffering in OVS 2.7.

On Mon, Aug 22, 2016 at 04:31:27PM -0700, Jarno Rajahalme wrote: > OVS implementation of buffering packets that are sent to the > controller is not compliant with the OpenFlow specifications after > OpenFlow 1.0. OVS implementation executes the buffered packet against > the actions of the modified

Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output

On 29 August 2016 at 10:36, Paul Boca wrote: > Hi Guru, > > > > please see my comments inline. > > > > Thanks, > > Paul > > > > *From:* Guru Shetty [mailto:g...@ovn.org] > *Sent:* Friday, August 26, 2016 9:04 PM > *To:* Paul Boca > *Cc:* dev@openvswitch.org > *Subject:* Re: [ovs-dev] [PATCH 08/12

Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output

Hi Guru, please see my comments inline. Thanks, Paul From: Guru Shetty [mailto:g...@ovn.org] Sent: Friday, August 26, 2016 9:04 PM To: Paul Boca Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 08/12] python tests: Ignore stderr output On 26 August 2016 at 07:40, Paul Boca mailto:pb...@

Re: [ovs-dev] [PATCH] Documentation: Newbie guide for OVN

On Mon, Aug 22, 2016 at 10:50:03AM -0400, Marcin Mirecki wrote: > From 448df4a4a57ab4648c99e867596b359aa8484252 Mon Sep 17 00:00:00 2001 > From: mirecki > Date: Mon, 22 Aug 2016 16:33:32 +0200 > Subject: [PATCH] Documentation: Newbie guide for OVN > > Document describing the basic steps needed to

Re: [ovs-dev] [PATCH 0/3] Add LB commands.

On 28 August 2016 at 02:05, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > This patches provide the command line to create load-balancer and add > the unit tests. > Thank you for working on this. I have a few higher level comments. 1. I do not see the 2nd patch in the

[ovs-dev] [PATCH] Avoid segfault in sFlow when datapath input port for tunnel sample is unknown.

This patch avoids a segfault that was found and documented here: http://openvswitch.org/pipermail/discuss/2016-August/022513.html I raised a pull-request on github. Signed-off-by: Neil McKee --- ofproto/ofproto-dpif-sflow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ofp

Re: [ovs-dev] [PATCH] ovn-northd: Avoid excessive work to find routerports.

On Thu, Aug 25, 2016 at 10:37:37AM -0500, Ryan Moats wrote: > > > "dev" wrote on 08/17/2016 03:58:13 PM: > > > From: Ben Pfaff > > To: dev@openvswitch.org > > Cc: Ben Pfaff > > Date: 08/17/2016 03:58 PM > > Subject: [ovs-dev] [PATCH] ovn-northd: Avoid excessive work to find > > router ports.

[ovs-dev] [PATCH V4 1/1] ovs-vtep: vtep-ctl and ovs-vtep support of adding explicit tunnel key

From: itamarofek This patch adds support for handeling a per-tunnel tunnel key in the ovs-vtep and vtep-ctl. The aim of this patch is to support the usage of hardware vtep switch as an inter-cloud gateway, which is used to connect two separated l2 broadcast domains Requested-by: "Ofer Ben-Yacov

Re: [ovs-dev] [PATCH 01/12] python tests: Skip python tests specific to Linux

Hi Guru, please see my comment inline. Thanks, Paul From: Guru Shetty [mailto:g...@ovn.org] Sent: Friday, August 26, 2016 8:39 PM To: Paul Boca Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 01/12] python tests: Skip python tests specific to Linux On 26 August 2016 at 07:40, Paul Boc

[ovs-dev] [PATCH v2] json: Use reference counting in json objects

After profiling OVSDB insert performance it was found that some significant portion of its time OVSDB is calling the function json_clone. Also, the current usages of json_clone never modify the json, just keeps it to prevent it to be freed. With that in mind the struct json, json_create, json_clo

[ovs-dev] Mail System Error - Returned Mail

`%ÃԞ2œš ôRäù¢q)‡i ѹ4Å IÏÏ5ñ×ß*_mî°Ê\їáÒíΏ†0¢X–·#Ã!£}L~êL²Ô#†û2hl[õn¿¯Yý7üz–jÄu«E0 ?.œ´ œØP8ðõ‘NÚ&´A±in¶75Á¢®åOôc ¤Š¿ýV{X‡dÐÜñ;Z¨ŽËß—kÊâ'ÀdׄÐ\?¯jqr‹‹³‘} íÙ9ÜIЇò<êküª‚1š1™«Ü_‚ºú |¬ƒµxٚ±×ªú‡ŽÖ".§¶°b’{$Ô)öoe<ÄNYŸßÖb´hŠÖÄE‚„´©¶5ib`ÌCŸ¼]ы”Ä÷± K{¿ˆ„ýl‹ìdÔ`\÷ý [8é7ßR®M—?GlãÓÞ_åÎ8¸±:ߨ_ó7硌¢

[ovs-dev] Returned mail: Data format error

The original message was received at Mon, 29 Aug 2016 18:54:53 +0530 from openvswitch.org [214.155.226.94] - The following addresses had permanent fatal errors - dev@openvswitch.org ___ dev mailing list dev@openvswitch.org http://openvswitch.

[ovs-dev] (no subject)

This 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 RFC v1] ovn-northd: add default_dhcpvx_options for Logical_Switch

From: Zongkai LI This patch adds default_dhcpv4_options and default_dhcpv6_options columns for Logical_Switch, which should help CMS not to calculate and set dhcpv4_options and dhcpv6_options columns for lswitch ports on lswitchs one by one, when most of lswitch ports on the same lswitch are usin

[ovs-dev] [patch v10 1/2] Check and allocate free qdisc queue id for ports with qos parameters

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 free queue for the chassis that this port belongs. The queue_id information is stored in the options field of Port_binding table.

[ovs-dev] [patch v10 2/2] DSCP marking on packets egressing VIF interface

ovn-northd sets 'ip.dscp' to the DSCP value Signed-off-by: Babu Shanmugam --- ovn/lib/logical-fields.c| 2 +- ovn/northd/ovn-northd.8.xml | 45 ovn/northd/ovn-northd.c | 72 +++- ovn/ovn-nb.xml | 6

[ovs-dev] [patch v10 0/2] QOS updates with DSCP support

v9 -> v10 - Rebased on top of master. - DSCP marking is done at a new logical ingress stage. - Added relevant documentation for the DSCP flow. Babu Shanmugam (2): Check and allocate free qdisc queue id for ports with qos parameters DSCP marking on packets egressing VIF interface include/o

Re: [ovs-dev] [PATCH v9 2/2] DSCP marking on packets egressing VIF interface

I am sending out v10 shortly with the agreed changes. Thank you for the review, Mickey. On Saturday 27 August 2016 01:00 AM, Mickey Spiegel wrote: On Wed, Aug 17, 2016 at 6:39 AM, > wrote: From: Babu Shanmugam mailto:bscha...@redhat.com>> ovn-northd sets

Re: [ovs-dev] [replication SMv2 7/7] ovsdb: Replication usability improvements

On Sat, Aug 27, 2016 at 4:45 AM, Andy Zhou wrote: > Added the '--no-sync' option base on feedbacks of current > implementation. > > Added appctl command "ovsdb-server/sync-status" based on feedbacks > of current implementation. > > Added a test to simulate the integration of HA manager with OVSDB

Re: [ovs-dev] [PATCH 06/12] python tests: Fixed flake8 errors in daemon_unix

Hi Guru, I retested without this patch and indeed there is no flake error. For some reason it gave me some errors when I made the changes. I will get this changes out. Thanks, Paul From: Guru Shetty [mailto:g...@ovn.org] Sent: Friday, August 26, 2016 9:01 PM To: Paul Boca Cc: dev@openvswitch.or

Re: [ovs-dev] [PATCH 03/12] python tests: Ported UNIX sockets to Windows

Hi Guru, Thanks for suggestion. Paul From: Guru Shetty [mailto:g...@ovn.org] Sent: Friday, August 26, 2016 8:58 PM To: Paul Boca Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH 03/12] python tests: Ported UNIX sockets to Windows On 26 August 2016 at 07:40, Paul Boca mailto:pb...@cloud