We have the following comment in ofproto-dpif-xlate.c:
/* Prevent nested translation of OpenFlow groups.
*
* OpenFlow allows this restriction. We enforce this restriction only
* because, with the current architecture, we would otherwise have to
* take a
Justin, Miguel, thank you!
applied to master, branch-2.3, 2.2, and 2.1.
On Tue, Jun 2, 2015 at 5:12 PM, Miguel Ángel Ajo wrote:
> Congratulations :)
>
> Midokura did something very smart ;)
>
> Miguel Ángel Ajo
>
> On Tuesday, 2 de June de 2015 at 8:03, Justin Pettit wrote:
>
> Congratulations o
On Mon, Jun 1, 2015 at 8:13 PM, Jesse Gross wrote:
> On Mon, Jun 1, 2015 at 7:08 PM, Andy Zhou wrote:
>> Covert tabs into spaces. Found by inspection.
>>
>> Signed-off-by: Andy Zhou
>
> Acked-by: Jesse Gross
Thanks, pushed.
___
dev mailing list
dev@o
> On Jun 2, 2015, at 10:59 AM, Ben Pfaff wrote:
>
> On Mon, Jun 01, 2015 at 06:17:48PM -0700, Jarno Rajahalme wrote:
>> Applied to master with the following incremental based on the above,
>
> Great, I'm happy to see this.
I’ll work on a v3 for the rest of the series, no need to review v2 furt
Thanks for the review, really good points, adopt both,
On Tue, Jun 2, 2015 at 12:57 PM, Andy Zhou wrote:
> On Mon, Jun 1, 2015 at 11:24 AM, Alex Wang wrote:
> > This commit adds function that allows the appending of one list
> > content to the other. Also, it adds functions which allows list
>
On Mon, Jun 1, 2015 at 11:24 AM, Alex Wang wrote:
> This commit adds function that allows the appending of one list
> content to the other. Also, it adds functions which allows list
> to be sorted.
>
> Signed-off-by: Alex Wang
> ---
> RFC->RFC V2:
> - Adopt Ben's suggestions.
>
> ---
> lib/list
Thomas Graf writes:
> This is the first series in a greater effort to bring the scalability
> and programmability advantages of OVS to the rest of the network
> stack and to get rid of as much OVS specific code as possible.
>
> This first series focuses on getting rid of OVS tunnel vports and use
>> Perhaps other people working on things like Rust or Go might be
>> interested in adding links there as well. The more the merrier I
>> think.
>
> Sure, I'd be happy to have links to all the bindings that are available
> (I didn't know about Rust bindings!).
I don't know of a Rust binding eith
On Tue, Jun 02, 2015 at 06:11:45PM +, William Adams wrote:
> Thanks a lot! that was pretty darned fast.
When people make it easy by submitting pull requests to the webpage, I
have no excuse to be slow ;-)
> Perhaps other people working on things like Rust or Go might be
> interested in addin
Thanks a lot! that was pretty darned fast.
Perhaps other people working on things like Rust or Go might be interested in
adding links there as well. The more the merrier I think.
-- William
===
- Shaping clay is easier than digging it out of the ground.
---
On Mon, Jun 01, 2015 at 06:17:48PM -0700, Jarno Rajahalme wrote:
> Applied to master with the following incremental based on the above,
Great, I'm happy to see this.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
From: "Thomas F. Herbert"
Signed-off-by: Thomas F Herbert
---
NEWS | 2 ++
utilities/ovs-ofctl.8.in | 3 +--
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index a480607..cbdde3a 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,7 @@
Post-v2.3.0
-
From: "Thomas F. Herbert"
This test tests the user space actions for 802.1q and 802.1ad.
This test was originally contributed by Dave Benson.
Signed-off-by: Thomas F Herbert
---
tests/ofproto-dpif.at | 40
1 file changed, 40 insertions(+)
diff --git a/
From: "Thomas F. Herbert"
Netlink parsing and flow key conversion. Netlink attribute encoding is done
with double encap attributes. Netlink attributes for 802.1ad look like
the following:
eth_type(0x88a8),vlan(vid=100),encap(eth_type(0x8100), vlan(vid=200),
encap(eth_type(0x0800)
From: "Thomas F. Herbert"
Changes to allow the tpid to be specified and all vlan tpid checking to be
generalized.
Signed-off-by: Thomas F Herbert
---
lib/odp-execute.c | 2 +-
lib/packets.c | 8
lib/packets.h | 7 +++
3 files changed, 12 insertions(+), 5 deletions(-)
diff
From: "Thomas F. Herbert"
The flow structure is updated to hold the customer tci.
Flow key extraction is changed to add support for ctci and both TPIDs.
Parsing to support pushing and popping with both single and double
tagged vlans. In response to reviewers comments on V6, all changes
affected b
From: "Thomas F. Herbert"
Version 10: Use doubly nested encap attributes to encode 802.1ad. Rebase
to master. V8 and V9 were skipped and this version was called V10 to
accompany V10 kernel module patch simultaneously to net-next.
This patch will is incopatible with old versions of the kernel mod
On Mon, Jun 01, 2015 at 06:06:14PM -0700, Jarno Rajahalme wrote:
>
> > On Jun 1, 2015, at 2:48 PM, Ben Pfaff wrote:
> >
> > On Mon, Jun 01, 2015 at 01:58:41PM -0700, Jarno Rajahalme wrote:
> >>
> >>> On Jun 1, 2015, at 1:47 PM, Jarno Rajahalme wrote:
> >>>
> >>>
> On May 29, 2015, at 5:
It seems patch 01 didn't make it to ovs dev mailing list,
but it is available on netdev mailing list.
fbl
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Add support for 802.1ad to netlink parsing and flow conversation. Uses
double nested encap attributes to represent double tagged vlan.
Signed-off-by: Thomas F Herbert
---
net/openvswitch/flow_netlink.c | 186 ++---
1 file changed, 157 insertions(+), 29 deletio
Add support for 802.1ad including the ability to push and pop double
tagged vlans.
Signed-off-by: Thomas F Herbert
---
net/openvswitch/flow.c | 82 ++
net/openvswitch/flow.h | 3 ++
2 files changed, 73 insertions(+), 12 deletions(-)
diff --git a/
This patch adds a function to check for vlan ethernet types. There is a
use case in openvswitch and it should be useful elsewhere.
Signed-off-by: Thomas F Herbert
---
include/linux/if_vlan.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/include/linux/if_vlan.h b/include/linux/if_v
openvswitch: Add support for 8021.AD
Change the description of the VLAN tpid field.
Signed-off-by: Thomas F Herbert
---
include/uapi/linux/openvswitch.h | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/include/uapi/linux/openvswitch.h b/include/uapi/linux/op
Add support for 802.1AD to the openvswitch kernel module.
V10: Implement reviewer comments: Consolidate vlan parsing functions.
Splits netlink parsing and flow conversion into a separate patch. Uses
double encap attribute encapsulation for 802.1ad. Netlink attributes
now look like this:
eth_
Works for me! I merged it.
On Tue, Jun 02, 2015 at 05:25:24PM +, William Adams wrote:
> OK. I have made the change in the 'development' area. I hope you find that
> acceptable.
>
> -- William
>
>
>
> ===
> - Shaping clay is easier than digging it out of the
On Fri, May 29, 2015 at 12:42:00PM +0100, Mark D. Gray wrote:
> From: Daniele Di Proietto
>
> Non pmd threads have a core_id == UINT32_MAX, while queue ids used by
> netdevs range from 0 to the number of CPUs. Therefore core ids cannot
> be used directly to select a queue.
>
> This commit intro
OK. I have made the change in the 'development' area. I hope you find that
acceptable.
-- William
===
- Shaping clay is easier than digging it out of the ground.
> Date: Tue, 2 Jun 2015 08:02:00 -0700
> From: b...@nicira.
For UFID, I sent a patch to reuse UUID for UFID:
http://openvswitch.org/pipermail/dev/2015-June/055890.html
I'm currently thinking that for 128-bit maskable bitfields like
conn_label, I'll have something similar to this original patch, which
will represent the value as "conn_label=0xf00bar/0xc
On Tue, Jun 02, 2015 at 11:17:29AM -0400, Russell Bryant wrote:
> On 06/02/2015 09:24 AM, Flavio Leitner wrote:
> > On Fri, May 29, 2015 at 02:51:26PM -0700, Ben Pfaff wrote:
> >> On Fri, May 29, 2015 at 05:07:19PM -0400, Russell Bryant wrote:
> >>> On 05/29/2015 04:50 PM, Ben Pfaff wrote:
> O
On Mon, 1 Jun 2015 15:11:03 -0700, Jesse Gross wrote:
> On Thu, May 14, 2015 at 11:10 AM, Jiri Benc wrote:
> > diff --git a/include/net/vxlan.h b/include/net/vxlan.h
> > index 0082b5d33d7d..a4ed7241c56a 100644
> > --- a/include/net/vxlan.h
> > +++ b/include/net/vxlan.h
> > +int vxlan6_xmit_skb(str
On Wed, May 27, 2015 at 02:11:32PM -0700, Jesse Gross wrote:
> On Wed, May 27, 2015 at 2:07 PM, Hauke Mehrtens wrote:
> > On 05/27/2015 10:14 PM, Jesse Gross wrote:
> >> On Tue, May 26, 2015 at 4:59 PM, Ben Pfaff wrote:
> >>> diff --git a/datapath/linux/compat/include/linux/if_vlan.h
> >>> b/dat
On Mon, 1 Jun 2015 15:09:54 -0700, Jesse Gross wrote:
> On Thu, May 14, 2015 at 11:10 AM, Jiri Benc wrote:
> > Other users of IPv6 tunneling (e.g. openvswitch) may not have a net_device
> > that represents the tunnel.
> >
> > Signed-off-by: Jiri Benc
>
> I think this will probably not make peopl
On Mon, 1 Jun 2015 15:07:11 -0700, Jesse Gross wrote:
> On Thu, May 14, 2015 at 11:10 AM, Jiri Benc wrote:
> > diff --git a/net/openvswitch/vport.h b/net/openvswitch/vport.h
> > index b22d45775eb5..ca9ebb9eb0f7 100644
> > --- a/net/openvswitch/vport.h
> > +++ b/net/openvswitch/vport.h
> > +static
On Mon, 1 Jun 2015 14:40:41 -0700, Jesse Gross wrote:
> On Thu, May 14, 2015 at 11:10 AM, Jiri Benc wrote:
> > diff --git a/include/uapi/linux/openvswitch.h
> > b/include/uapi/linux/openvswitch.h
> > index 4d26da40b01f..ba7ae3b05308 100644
> > --- a/include/uapi/linux/openvswitch.h
> > +++ b/incl
On Mon, 1 Jun 2015 14:22:48 -0700, Jesse Gross wrote:
> On Thu, May 14, 2015 at 11:10 AM, Jiri Benc wrote:
> > diff --git a/net/openvswitch/flow.h b/net/openvswitch/flow.h
> > index 2af6ffbf2f2e..78e96a120120 100644
> > --- a/net/openvswitch/flow.h
> > +++ b/net/openvswitch/flow.h
> > -struct ovs_
On Tue, Jun 02, 2015 at 08:25:14AM -0700, Jesse Gross wrote:
> On Tue, Jun 2, 2015 at 8:03 AM, Ben Pfaff wrote:
> > On Tue, Jun 02, 2015 at 07:58:48AM -0700, Ben Pfaff wrote:
> >> On Mon, Jun 01, 2015 at 03:35:23PM -0700, Jesse Gross wrote:
> >> > On Mon, Jun 1, 2015 at 2:55 PM, Ben Pfaff wrote:
On Tue, Jun 2, 2015 at 8:03 AM, Ben Pfaff wrote:
> On Tue, Jun 02, 2015 at 07:58:48AM -0700, Ben Pfaff wrote:
>> On Mon, Jun 01, 2015 at 03:35:23PM -0700, Jesse Gross wrote:
>> > On Mon, Jun 1, 2015 at 2:55 PM, Ben Pfaff wrote:
>> > > On Mon, Jun 01, 2015 at 01:49:30PM -0700, Jesse Gross wrote:
>
On 06/02/2015 09:24 AM, Flavio Leitner wrote:
> On Fri, May 29, 2015 at 02:51:26PM -0700, Ben Pfaff wrote:
>> On Fri, May 29, 2015 at 05:07:19PM -0400, Russell Bryant wrote:
>>> On 05/29/2015 04:50 PM, Ben Pfaff wrote:
On Fri, Apr 24, 2015 at 01:06:06PM -0400, Russell Bryant wrote:
> This
On Tue, Jun 02, 2015 at 07:58:48AM -0700, Ben Pfaff wrote:
> On Mon, Jun 01, 2015 at 03:35:23PM -0700, Jesse Gross wrote:
> > On Mon, Jun 1, 2015 at 2:55 PM, Ben Pfaff wrote:
> > > On Mon, Jun 01, 2015 at 01:49:30PM -0700, Jesse Gross wrote:
> > >> We have a special flow_metadata structure to repr
On Tue, Jun 02, 2015 at 05:18:09AM +, William Adams wrote:
> I have been working on some luajit bindings/code for openvswitch.
> First of all, is there knowledge of an existing lua based binding
> already? If not, what's the best way of contributing this work?
> Should I just point to my own g
On Mon, Jun 01, 2015 at 03:35:23PM -0700, Jesse Gross wrote:
> On Mon, Jun 1, 2015 at 2:55 PM, Ben Pfaff wrote:
> > On Mon, Jun 01, 2015 at 01:49:30PM -0700, Jesse Gross wrote:
> >> We have a special flow_metadata structure to represent the parts
> >> of a packet that aren't carried in the payload
On Mon, Jun 01, 2015 at 02:22:48PM -0700, Jesse Gross wrote:
> On Thu, May 14, 2015 at 11:10 AM, Jiri Benc wrote:
> > diff --git a/net/openvswitch/flow.h b/net/openvswitch/flow.h
> > index 2af6ffbf2f2e..78e96a120120 100644
> > --- a/net/openvswitch/flow.h
> > +++ b/net/openvswitch/flow.h
> > -stru
On Thu, May 14, 2015 at 08:10:44PM +0200, Jiri Benc wrote:
> The custom alignment of struct ovs_key_ipv4_tunnel was originally introduced
> by commit 1139e241ec436 ("openvswitch: Compact sw_flow_key."). At that time,
> the size of the structure was not a multiply of 64bit. This is not the case
> an
On Thu, May 28, 2015 at 02:51:55PM +0200, Jiri Benc wrote:
> On Wed, 27 May 2015 14:27:59 -0700, Jesse Gross wrote:
> > Isn't this really a bug in the decapsulation code? It seems like this
> > should trigger in other cases as well - what happens on normal
> > forwarding where we compute the checks
On Fri, May 29, 2015 at 02:51:26PM -0700, Ben Pfaff wrote:
> On Fri, May 29, 2015 at 05:07:19PM -0400, Russell Bryant wrote:
> > On 05/29/2015 04:50 PM, Ben Pfaff wrote:
> > > On Fri, Apr 24, 2015 at 01:06:06PM -0400, Russell Bryant wrote:
> > >> This patch creates a new subpackage for OVN, openvsw
Wow the crowd with our captivating array of fine Duplicate watches.
http://ibhsedu.com/0ct48i/64phy.html";>More Details
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Congratulations :)
Midokura did something very smart ;)
Miguel Ángel Ajo
On Tuesday, 2 de June de 2015 at 8:03, Justin Pettit wrote:
> Congratulations on the new job!
>
> Acked-by: Justin Pettit mailto:jpet...@nicira.com)>
>
> --Justin
>
>
> > On Jun 1, 2015, at 10:35 PM, YAMAMOTO Ta
5
ê
Ͳ ¦ÆG¶>9yï's1ò
¢GïórªD^<¤ÐvúprÎ^5á`/ïjhÍËèZ[äu"ZB;/ü,¿}¸K*mî¡5
l:-RØé>'CÆëPQ¯uNõÔô/áñxODÊbZ{G¢²n$óT?T(W_]û)îÀgjÝX3,*?´èÔî3ÔÔfáÎÒðqv"6gÐAu¾åýÓä»é«þ6!¥\A.ßÛ
ÐÐ*¢dJ.ßüþ¨0ûʹf¦µ7óë
W»¹ÉUj,´ì?½Í%ù¥\ésô~
¦#ÎM±Þp¡7snªæ¯r%ÏòVÎtÔÌ_ÐÁ9Ip«ø?*ûÐ.Òd§~¹^ñ
mm4
ÈLsÆþ
On Wed, Apr 8, 2015 at 8:31 AM, Jesse Gross wrote:
> When we parse the text representation of the Geneve action the
> header is not fully initialized. Besides the obvious potential
> to generate an action that the user did not actually specify, this
> also causes intermittent unit test failures wh
On Wed, Apr 8, 2015 at 8:31 AM, Jesse Gross wrote:
> Windows doesn't like that the Geneve header has an array of
> options with each have a zero length member (the variable data).
> Nothing is accessing the data now, so just replace the member with
> a comment - we can use pointer arithmetic when
50 matches
Mail list logo
