[ovs-dev] Openflow1.4 Set/Get Async and Group and Meter change notification

2015-06-12 Thread niti Rohilla
Hi, I want to implement and contribute following Openflow 1.4 features to OpenVSwitch: - Set and Get Asynchronous messages - Group and Meter change notification If anybody has already started working on them then please let us know. Thanks & Regards Niti Rohilla Tata Consultancy Services

Re: [ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

2015-06-12 Thread Sorin Vinturis
Nithin, The previous patch you are referring was one of a series I had pushed when working at the "Multiple NBLs" issue. At that time I had some trouble when generating packets large enough for the a NBL with multiple NBs to be generated. Because Eitan and Ankur, that also worked on this issue,

Re: [ovs-dev] conntrack: nfqueue action

2015-06-12 Thread Franck Baudin
Hi Ansis, On 06/09/15 22:59, Ansis Atteka wrote: Hi Franck On 8 June 2015 at 09:34, Franck BAUDIN > wrote: Hello, Conntrack looks in very good progress on https://github.com/justinpettit/ovs.git However, I didn't find any code related to "n

Re: [ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

2015-06-12 Thread Eitan Eliahu
No need to apologize Sorin. We will get everything addressed. Sorry if you felt like we pushed you. Thanks for the patch, Eitan -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sorin Vinturis Sent: Friday, June 12, 2015 5:27 AM To: Nithin Raju Cc: dev@openvsw

Re: [ovs-dev] Openflow1.4 Set/Get Async and Group and Meter change notification

2015-06-12 Thread Ben Pfaff
On Fri, Jun 12, 2015 at 05:22:19PM +0530, niti Rohilla wrote: > I want to implement and contribute following Openflow 1.4 features to > OpenVSwitch: > >- Set and Get Asynchronous messages >- Group and Meter change notification > > If anybody has already started working on them then please

[ovs-dev] [PATCH ovn 1/2 v5] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-06-12 Thread Russell Bryant
This patch introduces ovn-ctl, which is similar to ovs-ctl. I opted for a new script as everything in OVN so far is nicely isolated, so a new script seemed to make the most sense. If you'd like to run ovn-controller on a host already running ovs: # ovn-ctl start_controller If you'd like to

[ovs-dev] (no subject)

2015-06-12 Thread Russell Bryant
From Russell Bryant # This line is ignored. From: Russell Bryant Subject: [PATCH ovn 0/2 v5] Update Fedora packaging for OVN. In-Reply-To: <1432700278-10112-1-git-send-email-rbry...@redhat.com> v1->v2: - Change systemd units to run ovn-northd and ovn-controller directly instead of using ovn-

[ovs-dev] [PATCH ovn 2/2 v5] fedora.spec: Create openvswitch-ovn package.

2015-06-12 Thread Russell Bryant
This patch creates a new subpackage for OVN, openvswitch-ovn. It also installs systemd unit files for ovn-controller and ovn-northd. If you want to run ovn-controller: # systemctl start ovn-controller If you want to run ovn-northd: # systemctl start ovn-northd Both systemd units are c

Re: [ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

2015-06-12 Thread Nithin Raju
> On Jun 12, 2015, at 5:27 AM, Sorin Vinturis > wrote: > > Nithin, > > The previous patch you are referring was one of a series I had pushed when > working at the "Multiple NBLs" issue. At that time I had some trouble when > generating packets large enough for the a NBL with multiple NBs to b

Re: [ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

2015-06-12 Thread Gurucharan Shetty
I haven't done any research here. A quick google says that Hyper-V does support vagrant. So a way to test kernel module would be to get a vagrant infrastructure for Hyper-V too. We currently have such a infrastructure for Linux inside OVS code base. So a developer can do a 'vagrant up' on Hyper-V a

Re: [ovs-dev] [PATCH] classifier: Simplify versioning.

2015-06-12 Thread Gurucharan Shetty
On Thu, Jun 11, 2015 at 6:20 PM, Jarno Rajahalme wrote: > >> On Jun 11, 2015, at 5:58 PM, Jarno Rajahalme wrote: >> >> >>> On Jun 11, 2015, at 5:51 PM, Gurucharan Shetty wrote: >>> >>> On Thu, Jun 11, 2015 at 5:07 PM, Jarno Rajahalme >>> wrote: > On Jun 11, 2015, at 16:53, Ben Pf

Re: [ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

2015-06-12 Thread Nithin Raju
> On Jun 12, 2015, at 8:56 AM, Gurucharan Shetty wrote: > > I haven't done any research here. A quick google says that Hyper-V > does support vagrant. So a way to test kernel module would be to get a > vagrant infrastructure for Hyper-V too. We currently have such a > infrastructure for Linux ins

[ovs-dev] [PATCH ovn 1/2 v5] Add ovn-ctl to assist with OVN daemon lifecycle.

2015-06-12 Thread Russell Bryant
This patch introduces ovn-ctl, which is similar to ovs-ctl. I opted for a new script as everything in OVN so far is nicely isolated, so a new script seemed to make the most sense. If you'd like to run ovn-controller on a host already running ovs: # ovn-ctl start_controller If you'd like to

[ovs-dev] [PATCH ovn 0/2 v5] Update Fedora packaging for OVN.

2015-06-12 Thread Russell Bryant
(Re-sending, this time without breaking the headers, I hope ...) v1->v2: - Change systemd units to run ovn-northd and ovn-controller directly instead of using ovn-ctl. I kept ovn-ctl around anyway as it seemed useful to provide as an option for integrating with systems without systemd,

[ovs-dev] [PATCH ovn 2/2 v5] fedora.spec: Create openvswitch-ovn package.

2015-06-12 Thread Russell Bryant
This patch creates a new subpackage for OVN, openvswitch-ovn. It also installs systemd unit files for ovn-controller and ovn-northd. If you want to run ovn-controller: # systemctl start ovn-controller If you want to run ovn-northd: # systemctl start ovn-northd Both systemd units are c

[ovs-dev] [PATCH] ovs-rcu: Comment fixes.

2015-06-12 Thread Ben Pfaff
A comment referred to a "Usage" section but the section was named "Use". This fixes the problem (also a grammar error). Signed-off-by: Ben Pfaff --- lib/ovs-rcu.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/ovs-rcu.h b/lib/ovs-rcu.h index c1e3d60..5cd4152 1006

Re: [ovs-dev] [PATCH] ovs-rcu: Comment fixes.

2015-06-12 Thread Alex Wang
Acked-by: Alex Wang On Fri, Jun 12, 2015 at 10:58 AM, Ben Pfaff wrote: > A comment referred to a "Usage" section but the section was named "Use". > This fixes the problem (also a grammar error). > > Signed-off-by: Ben Pfaff > --- > lib/ovs-rcu.h | 8 > 1 file changed, 4 insertions(+)

Re: [ovs-dev] first OVN scale testing thoughts

2015-06-12 Thread Russell Bryant
On 06/11/2015 03:14 AM, Ben Pfaff wrote: > On Wed, Jun 10, 2015 at 11:53:47PM -0700, Ben Pfaff wrote: >> I want to report progress I've made tonight on an extended form of the >> OVS sandbox that I expect to be useful for testing OVN. The main point >> of the extension is to make it easy to start

Re: [ovs-dev] conntrack: nfqueue action

2015-06-12 Thread Ansis Atteka
On Fri, Jun 12, 2015 at 5:50 AM, Franck Baudin wrote: > Hi Ansis, > > > On 06/09/15 22:59, Ansis Atteka wrote: >> >> Hi Franck >> On 8 June 2015 at 09:34, Franck BAUDIN > > wrote: >> >> Hello, >> >> Conntrack looks in very good progress on >> https://gi

[ovs-dev] OVS with NSH (format of the packet towards VM from OVS)

2015-06-12 Thread Anil Gunturu
Hi, With OVS patch for NSH, what is the format of the packets going to the VM? Do they have NSH header or the applications inside the VM completely unaware of the NSH header? Thanks in advance, -Anil ___ dev mailing list dev@openvswitch.org http://open

Re: [ovs-dev] [PATCH] datapath-windows: BSOD when disabling the extension

2015-06-12 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean -Mesaj original- De la: dev [mailto:dev-boun...@openvswitch.org] În numele Sorin Vinturis Trimis: Thursday, May 28, 2015 11:31 PM Către: dev@openvswitch.org Subiect: [ovs-dev] [PATCH] datapath-windows: BSOD when disabling the extension When the filter det

Re: [ovs-dev] [PATCH v4 11/12] ofproto: Use minimatch for making bundles smaller.

2015-06-12 Thread Jarno Rajahalme
> On Jun 10, 2015, at 5:53 PM, Ben Pfaff wrote: > > On Wed, Jun 10, 2015 at 05:52:21PM -0700, Jarno Rajahalme wrote: >> >>> On Jun 10, 2015, at 5:46 PM, Ben Pfaff wrote: >>> >>> On Tue, Jun 09, 2015 at 05:24:18PM -0700, Jarno Rajahalme wrote: struct match in ofputil_flow_mod uses a lot o

Re: [ovs-dev] [PATCH v4 10/12] ofproto: Postpone sending flow removed messages.

2015-06-12 Thread Jarno Rajahalme
> On Jun 10, 2015, at 5:57 PM, Jarno Rajahalme wrote: > > >> On Jun 10, 2015, at 5:43 PM, Ben Pfaff wrote: >> >> On Tue, Jun 09, 2015 at 05:24:17PM -0700, Jarno Rajahalme wrote: >>> The final flow stats are available only after there are no references >>> to the rule. Postpone sending the fl

[ovs-dev] [PATCH v5 2/4] ofproto: Postpone sending flow removed messages.

2015-06-12 Thread Jarno Rajahalme
The final flow stats are available only after there are no references to the rule. Postpone sending the flow removed message until the final stats are available. Signed-off-by: Jarno Rajahalme --- include/openflow/openflow-common.h |2 ++ lib/ofp-print.c|1 + ofproto

[ovs-dev] [PATCH v5 1/4] classifier: Simplify versioning.

2015-06-12 Thread Jarno Rajahalme
After all, there are some cases in which both the insertion version and removal version of a rule need to be considered. This makes the cls_match a bit bigger, but makes classifier versioning much simpler to understand. Also, avoid using type larger than int in an enum, as it is not portable C.

[ovs-dev] [PATCH v5 0/4] Bundles refinements.

2015-06-12 Thread Jarno Rajahalme
These patches Finish up some loose ends with OF 1.4 bundles. Only patch 03 is new, and patch 04 adds new documentation. Jarno Rajahalme (4): classifier: Simplify versioning. ofproto: Postpone sending flow removed messages. ofproto: Generalize deferred work execution. ofproto: Support port

[ovs-dev] [PATCH v5 3/4] ofproto: Generalize deferred work execution.

2015-06-12 Thread Jarno Rajahalme
Previous patch adds more work being executed from the ovs-rcu thread. Executing big chunks of work from ovs-rcu thread should be avoided, however, as the rcu callback facility is shared resource used by all threads. This patch generalizes the exising "rule_executes" list to be useful for also othe

[ovs-dev] [PATCH v5 4/4] ofproto: Support port mods in bundles.

2015-06-12 Thread Jarno Rajahalme
Add support for port mods in an OpenFlow 1.4 bundle, as required for the minimum support level by the OpenFlow 1.4 specification. If the bundle includes port mods, it may not specify the OFPBF_ATOMIC flag. Port mods and flow mods in a bundle are always applied in order and the consecutive flow mod

Re: [ovs-dev] [PATCH v5 1/4] classifier: Simplify versioning.

2015-06-12 Thread Ben Pfaff
On Fri, Jun 12, 2015 at 02:36:21PM -0700, Jarno Rajahalme wrote: > After all, there are some cases in which both the insertion version > and removal version of a rule need to be considered. This makes the > cls_match a bit bigger, but makes classifier versioning much simpler > to understand. > >

Re: [ovs-dev] [PATCH v5 2/4] ofproto: Postpone sending flow removed messages.

2015-06-12 Thread Ben Pfaff
On Fri, Jun 12, 2015 at 02:36:22PM -0700, Jarno Rajahalme wrote: > The final flow stats are available only after there are no references > to the rule. Postpone sending the flow removed message until the > final stats are available. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff __

Re: [ovs-dev] [PATCH v5 3/4] ofproto: Generalize deferred work execution.

2015-06-12 Thread Ben Pfaff
On Fri, Jun 12, 2015 at 02:36:23PM -0700, Jarno Rajahalme wrote: > Previous patch adds more work being executed from the ovs-rcu thread. > Executing big chunks of work from ovs-rcu thread should be avoided, > however, as the rcu callback facility is shared resource used by all > threads. > > This

Re: [ovs-dev] [PATCH v5 4/4] ofproto: Support port mods in bundles.

2015-06-12 Thread Ben Pfaff
On Fri, Jun 12, 2015 at 02:36:24PM -0700, Jarno Rajahalme wrote: > Add support for port mods in an OpenFlow 1.4 bundle, as required for > the minimum support level by the OpenFlow 1.4 specification. If the > bundle includes port mods, it may not specify the OFPBF_ATOMIC flag. > Port mods and flow

Re: [ovs-dev] [PATCH] ovs-rcu: Comment fixes.

2015-06-12 Thread Ben Pfaff
Thanks! Applied. On Fri, Jun 12, 2015 at 11:00:10AM -0700, Alex Wang wrote: > Acked-by: Alex Wang > > On Fri, Jun 12, 2015 at 10:58 AM, Ben Pfaff wrote: > > > A comment referred to a "Usage" section but the section was named "Use". > > This fixes the problem (also a grammar error). > > > > Si

Re: [ovs-dev] [PATCH v5 3/4] ofproto: Generalize deferred work execution.

2015-06-12 Thread Jarno Rajahalme
> On Jun 12, 2015, at 3:37 PM, Ben Pfaff wrote: > > On Fri, Jun 12, 2015 at 02:36:23PM -0700, Jarno Rajahalme wrote: >> Previous patch adds more work being executed from the ovs-rcu thread. >> Executing big chunks of work from ovs-rcu thread should be avoided, >> however, as the rcu callback fac

Re: [ovs-dev] [PATCH v5 1/4] classifier: Simplify versioning.

2015-06-12 Thread Jarno Rajahalme
> On Jun 12, 2015, at 3:21 PM, Ben Pfaff wrote: > > On Fri, Jun 12, 2015 at 02:36:21PM -0700, Jarno Rajahalme wrote: >> After all, there are some cases in which both the insertion version >> and removal version of a rule need to be considered. This makes the >> cls_match a bit bigger, but makes

Re: [ovs-dev] [PATCH v5 2/4] ofproto: Postpone sending flow removed messages.

2015-06-12 Thread Jarno Rajahalme
> On Jun 12, 2015, at 3:24 PM, Ben Pfaff wrote: > > On Fri, Jun 12, 2015 at 02:36:22PM -0700, Jarno Rajahalme wrote: >> The final flow stats are available only after there are no references >> to the rule. Postpone sending the flow removed message until the >> final stats are available. >> >>

Re: [ovs-dev] [PATCH v5 4/4] ofproto: Support port mods in bundles.

2015-06-12 Thread Jarno Rajahalme
> On Jun 12, 2015, at 3:41 PM, Ben Pfaff wrote: > > On Fri, Jun 12, 2015 at 02:36:24PM -0700, Jarno Rajahalme wrote: >> Add support for port mods in an OpenFlow 1.4 bundle, as required for >> the minimum support level by the OpenFlow 1.4 specification. If the >> bundle includes port mods, it ma

Re: [ovs-dev] [PATCH v2] flow: Make assertions about offsets within struct flow easier to follow.

2015-06-12 Thread Jarno Rajahalme
Found this from junk mail... Acked-by: Jarno Rajahalme Jarno > On Jun 9, 2015, at 11:32, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > --- > v1->v2: Found and converted a few more assertions. > > lib/flow.c | 60 +--- > lib/util.h

[ovs-dev] [PATCH ovn] Update location for Neutron plugin.

2015-06-12 Thread Russell Bryant
The git repository for the neutron plugin has been renamed to reflect that it is now officially part of the OpenStack Neutron project. The repo now lives in the "openstack" namespace. Also remove the link to the todo file as those are now just tracked in the networking-ovn bug tracker (launchpad

[ovs-dev] Header match after skippng header offset or length

2015-06-12 Thread Junguk Cho
Hi. Does OVS supports header match after skipping some header fields and use match rules with the rest of header? For example, I have tunneled packet. L2-IPV4(1)-UDP-GRE-IPV4(2)-TCP. In this case, does ovs check IPV4(2) *without virtual port mechanism?* I read ovs-ofctl.8.pdf, but it does not su