[ovs-dev] [RFC PATCH] xlate: Use dp_hash for select groups.

2016-04-18 Thread Jarno Rajahalme
Add a new select group selection method "dp_hash", which uses minimal number of bits from the datapath calculated packet hash to inform the select group bucket selection. This makes the datapath flows more generic resulting in less upcalls to userspace, but adds recirculation prior to group

[ovs-dev] Returned mail: Data format error

2016-04-18 Thread The Post Office
Ž3–B‡ØOo{ˆŽøàӎáâDP¿Wx÷όƒˆ`OC5¶Ç]S‘ªJ$BåW_1ì†ZW)Ðc˜BÚ\üx)µy  ÐMu$LF¡ÌÕ·öŽ®˜/2D)ÛhÀÓ«r}ÒǨŒ¸Hõ0ê¼Xc !¥dOêûÁÖ&¦©¯Êk¶!iëÔǚŸ‡ø:zª¼çAþM’0­ó3ü"êœR[lÒ<‚gmç²×’››_†¼Æça(ٔstcxЧ… Áõi|‰èó8D¤-XݚB÷›CölEõc§g¼ƒ`ߑCðOØ ãÏTimj³_sø‡õ˓{Rö

Re: [ovs-dev] Implementation of Packet-Out in Bundles

2016-04-18 Thread Ben Pfaff
Thanks. I see the pull request. We'd be happy to have support for additional OpenFlow request in bundles. On Mon, Apr 18, 2016 at 09:59:33PM +, André Mantas wrote: > Hi. Sorry for the long delay. Just issued a pull request in github, I hope > thats ok. Please let me know if I did something

[ovs-dev] Olá querido,

2016-04-18 Thread Celyn Abi
Olá querido,   Prazer em conhecê-lo, meu nome é Celyn, eu vim através de seu contato em uma mediana social e tornou-se interessado em saber que você, eu espero que você não se importa se eu ser seu amigo, eu quero que nós conhecemos muito bem há algo importante que eu quero compartilhar com

Re: [ovs-dev] Implementation of Packet-Out in Bundles

2016-04-18 Thread André Mantas
Hi. Sorry for the long delay. Just issued a pull request in github, I hope thats ok. Please let me know if I did something wrong. I'm also interested in adding support for group_mod, meter_mod and table_mod in the future (if that's ok with you). Ben Pfaff escreveu no dia terça,

Re: [ovs-dev] [PATCH RFC] ovn: distributed logical port for VM metadata access

2016-04-18 Thread Aaron Rosen
I like this idea as well. The one question I have with is is how we should determine which ip address to select for the 'distributed' port? In your example above you picked 10.0.0.100. I'm wondering if there is a more flexible way to handle this. It seems to me like this ip needs to be selected a

Re: [ovs-dev] [PATCH] checkpatch: Accept form feeds.

2016-04-18 Thread Daniele Di Proietto
On 18/04/2016 14:26, "Ben Pfaff" wrote: >On Mon, Apr 18, 2016 at 02:02:37PM -0700, Daniele Di Proietto wrote: >> CodingStyle.md says: >> >> "Use form feeds (control+L) to divide long source files into logical >> pieces. A form feed should appear as the only character on a

Re: [ovs-dev] [PATCH] checkpatch: Accept form feeds.

2016-04-18 Thread Ben Pfaff
On Mon, Apr 18, 2016 at 02:02:37PM -0700, Daniele Di Proietto wrote: > CodingStyle.md says: > > "Use form feeds (control+L) to divide long source files into logical > pieces. A form feed should appear as the only character on a line." > > checkpatch.py currently complains about form feed. For

Re: [ovs-dev] [PATCH v7 08/16] dpif-netdev: Add pmd thread local port cache for transmission.

2016-04-18 Thread Daniele Di Proietto
On 18/04/2016 07:50, "Ilya Maximets" wrote: >On 08.04.2016 06:13, Daniele Di Proietto wrote: >> Signed-off-by: Daniele Di Proietto >> --- >> lib/dpif-netdev.c | 243 >>+++--- >> 1 file changed,

Re: [ovs-dev] [PATCH v2 00/15] Userspace (DPDK) connection tracker

2016-04-18 Thread Daniele Di Proietto
Hi Antonio, thanks for the feedback On 18/04/2016 09:56, "Fischetti, Antonio" wrote: >Hi Daniele, >I just started to have a look to your new v2 patch set. >A minor comment - I know this is a bit of nit-picking - but I ran >utilities/checkpatch.py and I got some

[ovs-dev] [PATCH] checkpatch: Accept form feeds.

2016-04-18 Thread Daniele Di Proietto
CodingStyle.md says: "Use form feeds (control+L) to divide long source files into logical pieces. A form feed should appear as the only character on a line." checkpatch.py currently complains about form feed. For example, on commit 2c06d9a927c5("ovstest: Add test-netlink-conntrack command."),

Re: [ovs-dev] [PATCH 4/4] classifier: Avoid inserting duplicates to cmaps.

2016-04-18 Thread Ryan Moats
> --- Original Message --- > Staged lookup indices assumed that cmap is efficient fealing with > duplicates. Duplicates are implemented as linked lists, however, > causing removal of rules to become (O^2) and highly cache-inefficient > with large number of duplicates. > > This was problematic

Re: [ovs-dev] [PATCH 3/4] classifier: Remove rare optimization case.

2016-04-18 Thread Ryan Moats
> --- Original Message --- > This optimization applied when a staged lookup index would narrow down > to a single rule, which happens sometimes is simple test cases, but > presumably less often in more populated flow tables. The result of > this optimization allowed a bit more general megaflows,

[ovs-dev] [PATCH 2/4] classifier: Remove logging.

2016-04-18 Thread Ryan Moats
> --- Original Message --- > The only vlog line was a left over from debugging. > > Signed-off-by: Jarno Rajahalme > --- Acked-by: Ryan Moats ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/4] classifier: Remove redundant index.

2016-04-18 Thread Ryan Moats
> --- Original Message --- > The test for figuring out if the last index had the same fields as the > actual rules map as broken, resulting into keeping an unnecessary > index around. > > Signed-off-by: Jarno Rajahalme > --- Acked-by: Ryan Moats

Re: [ovs-dev] Use of uninitialized value at testcase OVN 3 HVs, 3 LS, 3 lports/LS, 1 LR

2016-04-18 Thread Ryan Moats
Ben, were you going spin a real patch with this change? (I've got an acked-by waiting if you are) Ryan ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [RFC PATCH] create vxlan device using rtnetlink interface

2016-04-18 Thread Jesse Gross
On Mon, Apr 18, 2016 at 3:27 AM, Jiri Benc wrote: > On Fri, 15 Apr 2016 20:36:51 -0700, Jesse Gross wrote: >> I'm not too excited about this. It seems like it would be a regression >> - currently OVSDB allows remote creation of tunnels, so it seems like >> this would break

[ovs-dev] Delivery reports about your e-mail

2016-04-18 Thread Returned mail
Dear user dev@openvswitch.org, Your email account was used to send a huge amount of spam messages during the recent week. Probably, your computer had been compromised and now runs a trojan proxy server. Please follow instructions in order to keep your computer safe. Best regards,

[ovs-dev] [PATCH v2 RFC] ovn: Support native dhcp using 'continuations'

2016-04-18 Thread Numan Siddique
To support native dhcp in ovn - A new column 'dhcp-options' is added in 'Logical_Switch' north db. - A logical flow is added for each logical port to handle dhcp packets if the CMS has defined dhcp options in this column. Eg. action=(dhcp_offer(offerip = 10.0.0.2, router = 10.0.0.1,

[ovs-dev] [PATCH v2 0/1 RFC] ovn: Support native dhcp using 'continuations'

2016-04-18 Thread Numan Siddique
v1 -> v2 changes -- * Added basic documentation * Added support for classless static routes - dhcp option 121 * Added a new egress table (Table 0) to skip the ACL stages for response dhcp packets. This is required until the below reported bug is fixed. Please see -

[ovs-dev] [PATCH V3] add lrouter and lrport related commands to ovn-nbctl

2016-04-18 Thread nghosh
ovn-nbctl provides a shortcut to perform commands related lswitch, lport and such but it doesn't have similar commands related to logical routers and logical router ports. Also, 'ovn-nbctl show' is supposed to show an overview of database contents, which means it should show the routers as well.

[ovs-dev] [PATCH V3] add lrouter and lrport related commands to ovn-nbctl

2016-04-18 Thread nghosh
ovn-nbctl provides a shortcut to perform commands related lswitch, lport and such but it doesn't have similar commands related to logical routers and logical router ports. Also, 'ovn-nbctl show' is supposed to show an overview of database contents, which means it should show the routers as well.

Re: [ovs-dev] [PATCH v2] tunneling: Improving tunneling performance using DPDK Rx checksum offloading feature.

2016-04-18 Thread Jesse Gross
On Mon, Apr 18, 2016 at 2:12 AM, Chandran, Sugesh wrote: >> -Original Message- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Friday, April 15, 2016 5:04 PM >> To: Chandran, Sugesh >> Cc: pravin shelar ; ovs

Re: [ovs-dev] [PATCH v2 00/15] Userspace (DPDK) connection tracker

2016-04-18 Thread Fischetti, Antonio
Hi Daniele, I just started to have a look to your new v2 patch set. A minor comment - I know this is a bit of nit-picking - but I ran utilities/checkpatch.py and I got some output like patch 4/15 W(1692): Line has trailing whitespace +static inline void ct_lock_lock(struct ct_lock *lock) patch

[ovs-dev] [PATCH V2] add lrouter and lrport related commands to ovn-nbctl

2016-04-18 Thread nghosh
ovn-nbctl provides a shortcut to perform commands related lswitch, lport and such but it doesn't have similar commands related to logical router and logical router port. Also, 'nbctl showi' is supposed to show an overview of database contents, which means it should show the routers as well. "show

Re: [ovs-dev] [PATCH 1/4] docs: OVSDB replication design document

2016-04-18 Thread Marcelo E. Magallon
Hi, sorry about the delay in responding. I was actually catching up with emails on the mailing list to try to gauge if we are indeed trying to accomplish the same thing or not. On Mon, Apr 11, 2016 at 03:44:09PM -0700, Ben Pfaff wrote: > On Fri, Apr 01, 2016 at 10:52:26AM -0700, Ben Pfaff

Re: [ovs-dev] [ovs-dev,v2,2/3] Add skeleton for OF1.6 support.

2016-04-18 Thread Ben Pfaff
On Sun, Apr 17, 2016 at 05:21:53PM -0500, Ryan Moats wrote: > > > --- Original Message --- > > Signed-off-by: Ben Pfaff > > --- > > FAQ.md | 33 +-- > > build-aux/extract-ofp-errors | 3 +- > > build-aux/extract-ofp-msgs | 3

Re: [ovs-dev] [PATCH] smap,shash: add numeric and flexible sort

2016-04-18 Thread Worth, Kevin
>> The default implementation of shash_sort and smap_sort provide only >> lexigraphic sorting. This patch allows both a numeric sort and a flexible >> version where the caller provides a comparison function >> ("compar" being the name used in other standard functions e.g. qsort(3)) >> >>

Re: [ovs-dev] Is it possible to create and manage a custom table in openvswitch?

2016-04-18 Thread Ben Pfaff
No. No. On Mon, Apr 18, 2016 at 02:37:56PM +, Pynbiang Hadem wrote: > Thanks Ben,        It seems the "learn" action will manipulate the flow > tables only. Can it be used for creating a new customized table?. Is there > any working sample on this?. Pls share if any.ThanksHadem > > >

Re: [ovs-dev] [PATCH v7 08/16] dpif-netdev: Add pmd thread local port cache for transmission.

2016-04-18 Thread Ilya Maximets
On 08.04.2016 06:13, Daniele Di Proietto wrote: > Signed-off-by: Daniele Di Proietto > --- > lib/dpif-netdev.c | 243 > +++--- > 1 file changed, 175 insertions(+), 68 deletions(-) > > diff --git a/lib/dpif-netdev.c

Re: [ovs-dev] [RFC PATCH] create vxlan device using rtnetlink interface

2016-04-18 Thread Thadeu Lima de Souza Cascardo
On Mon, Apr 18, 2016 at 12:36:31PM +0200, Jiri Benc wrote: > On Mon, 18 Apr 2016 06:57:22 -0300, Thadeu Lima de Souza Cascardo wrote: > > But... wait a minute. We don't support adding devices name as vxlan_sys*. > > Such > > names are reserved. I think that means we could probably rely on the

[ovs-dev] Returned mail: Data format error

2016-04-18 Thread Post Office
The original message was received at Mon, 18 Apr 2016 16:55:10 +0530 from 26.65.250.5 - The following addresses had permanent fatal errors - dev@openvswitch.org ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-18 Thread Xiao Liang
On Mon, Apr 18, 2016 at 5:46 PM, Takashi YAMAMOTO wrote: > for some reasons you want to change of_name without re-creating a port? > why? (just curious) > I don't have special use cases, just for convenience. > > On Mon, Apr 18, 2016 at 4:19 PM, Xiao Liang

Re: [ovs-dev] [RFC PATCH] create vxlan device using rtnetlink interface

2016-04-18 Thread Jiri Benc
On Mon, 18 Apr 2016 06:57:22 -0300, Thadeu Lima de Souza Cascardo wrote: > I don't see how this address the case when the user adds a vxlan interface > created by the system. Like: > > ip link add vxlan_sys_4789 type vxlan id 10 remote 192.168.123.1 dstport 4789 > ovs-vsctl add-port br0

Re: [ovs-dev] [RFC PATCH] create vxlan device using rtnetlink interface

2016-04-18 Thread Jiri Benc
On Fri, 15 Apr 2016 20:36:51 -0700, Jesse Gross wrote: > What about using the driver name exposed through ethtool? I believe > that all of the tunnel and internal devices expose this in a > consistent way - i.e. a VXLAN device can be queried and get back the > string "vxlan". Any driver other than

Re: [ovs-dev] [RFC PATCH] create vxlan device using rtnetlink interface

2016-04-18 Thread Jiri Benc
On Fri, 15 Apr 2016 18:30:59 -0300, Thadeu Lima de Souza Cascardo wrote: > Jiri has suggested that we require users to create the interfaces themselves, > by > using whatever method their OS has, and add them as netdev devices. That would > still require fetching some of the configuration from

Re: [ovs-dev] [RFC PATCH] create vxlan device using rtnetlink interface

2016-04-18 Thread Thadeu Lima de Souza Cascardo
On Fri, Apr 15, 2016 at 08:36:51PM -0700, Jesse Gross wrote: > On Fri, Apr 15, 2016 at 2:30 PM, Thadeu Lima de Souza Cascardo > wrote: > > Hi, this is an RFC patch (could probably be 2 patches - more below), that > > creates VXLAN devices in userspace and adds them as netdev

[ovs-dev] [PATCH V3, 2/3] datapath-windows: Removed double initialization on local variables

2016-04-18 Thread Paul Boca
Signed-off-by: Paul-Daniel Boca Acked-by: Sorin Vinturis --- v3: fixed a minor compilation issue. --- datapath-windows/ovsext/Actions.c | 10 +- datapath-windows/ovsext/Flow.c| 2 +-

Re: [ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-18 Thread Takashi YAMAMOTO
for some reasons you want to change of_name without re-creating a port? why? (just curious) On Mon, Apr 18, 2016 at 4:19 PM, Xiao Liang wrote: > By introducing of_name, ovs_name serves as a key of ports which is > shared by ofproto and netdev. It's easier to find and

Re: [ovs-dev] [PATCH v2] tunneling: Improving tunneling performance using DPDK Rx checksum offloading feature.

2016-04-18 Thread Chandran, Sugesh
Regards _Sugesh > -Original Message- > From: Jesse Gross [mailto:je...@kernel.org] > Sent: Friday, April 15, 2016 5:04 PM > To: Chandran, Sugesh > Cc: pravin shelar ; ovs dev > Subject: Re: [ovs-dev] [PATCH v2]

Re: [ovs-dev] [PATCH V4] ovn-controller: reload configured SB probe timer

2016-04-18 Thread Lei Huang
On 4/14/16, 3:24 PM, "dev on behalf of ngh...@us.ibm.com" < dev-boun...@openvswitch.org on behalf of ngh...@us.ibm.com> wrote: There are four sessions established from ovn-controller to the following: OVN Southbound — JSONRPC based Local ovsdb — JSONRPC based Local vswitchd — openflow based from

Re: [ovs-dev] [PATCH V4] ovn-controller: reload configured SB probe timer

2016-04-18 Thread Huang, Lei
On 4/14/16, 3:24 PM, "dev on behalf of ngh...@us.ibm.com" wrote: >There are four sessions established from ovn-controller to the following: >OVN Southbound — JSONRPC based >Local ovsdb — JSONRPC based >Local vswitchd — openflow

Re: [ovs-dev] [PATCH v7 16/16] netdev-dpdk: Use ->reconfigure() call to change rx/tx queues.

2016-04-18 Thread Kavanagh, Mark B
> > > >On 14/04/2016 05:37, "Kavanagh, Mark B" wrote: > >>Hi Daniele, >> >>One comment inline. >> >>Thanks, >>Mark >> >>> >>>This introduces in dpif-netdev and netdev-dpdk the first use for the >>>newly introduce reconfigure netdev call. >>> >>>When a request to change

Re: [ovs-dev] [PATCH v7 09/16] hmap: Use struct for hmap_at_position().

2016-04-18 Thread Kavanagh, Mark B
> >Hi Mark, > >On 14/04/2016 05:36, "Kavanagh, Mark B" wrote: > >>Hi Daniele, >> >>One minor comment inline. >> >>Cheers, >>Mark >> >>> >>>The interface will be more similar to the cmap. >>> >>>Signed-off-by: Daniele Di Proietto >>>--- >>>

[ovs-dev] [PATCH v4 1/1] ovn: Add column enabled to table Logical_Router

2016-04-18 Thread JunoZhu
This patch add column "enabled" to table Logical_Router for setting router administrative state. The type of "enabled" is bool. If the administrative state is false, delete all the flows relevant to the logical router from table Logical_Flow. Signed-off-by: Na Zhu ---

[ovs-dev] [PATCH V2, 3/3] datapath-windows: Removed always true condition in VXLAN

2016-04-18 Thread Paul Boca
Instance ID flag must be set to 1 in case of valid VXLAN id Signed-off-by: Paul-Daniel Boca Acked-by: Sorin Vinturis --- datapath-windows/ovsext/Vxlan.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

[ovs-dev] [PATCH V2, 2/3] datapath-windows: Removed double initialization on local variables

2016-04-18 Thread Paul Boca
Signed-off-by: Paul-Daniel Boca Acked-by: Sorin Vinturis --- datapath-windows/ovsext/Actions.c | 10 +- datapath-windows/ovsext/Flow.c| 2 +- datapath-windows/ovsext/IpHelper.c| 10 +-

[ovs-dev] [PATCH V2, 1/3] datapath-windows: Avoid using uninitialized gOvsExtDriverHandle

2016-04-18 Thread Paul Boca
Ensure gOvsExtDriverHandle is not used if initialization fails Added PAGED_CODE() where needed Signed-off-by: Paul-Daniel Boca Acked-by: Sorin Vinturis --- datapath-windows/ovsext/Datapath.c | 4 +++-

[ovs-dev] [PATCH] datapath-windows: Fixed buffer overflow in OvsInitVportWithNicParam

2016-04-18 Thread Paul Boca
nicParam->PermanentMacAddress is 32 bytes and vport->permMacAddress is 6 bytes Signed-off-by: Paul-Daniel Boca --- datapath-windows/ovsext/DpInternal.h | 6 +++--- datapath-windows/ovsext/Vport.h | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-)

Re: [ovs-dev] [PATCH 3/3] datapath-windows: Removed always true condition in VXLAN

2016-04-18 Thread Sorin Vinturis
Acked-by: Sorin Vinturis -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Paul Boca Sent: Friday, 15 April, 2016 18:05 To: dev@openvswitch.org Subject: [ovs-dev] [PATCH 3/3] datapath-windows: Removed always true condition

Re: [ovs-dev] [PATCH 2/3] datapath-windows: Removed double initialization on local variables

2016-04-18 Thread Sorin Vinturis
I had a few minor comments inline, but the patch looks good. Acked-by: Sorin Vinturis -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Paul Boca Sent: Friday, 15 April, 2016 18:05 To: dev@openvswitch.org Subject: [ovs-dev]

Re: [ovs-dev] [PATCH 1/3] datapath-windows: Avoid using uninitialized gOvsExtDriverHandle

2016-04-18 Thread Sorin Vinturis
Acked-by: Sorin Vinturis -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Paul Boca Sent: Saturday, 16 April, 2016 00:59 To: dev@openvswitch.org Subject: [ovs-dev] [PATCH 1/3] datapath-windows: Avoid using uninitialized

Re: [ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-18 Thread Xiao Liang
By introducing of_name, ovs_name serves as a key of ports which is shared by ofproto and netdev. It's easier to find and convert ports back and forth. of_name and kernel_name could be configured (if supported) independently of each other. On Mon, Apr 18, 2016 at 11:43 AM, Takashi YAMAMOTO

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

2016-04-18 Thread Automatic Email Delivery Software
Dear user of openvswitch.org, We have detected that your account was used to send a large amount of spam messages during the recent week. Most likely your computer had been infected and now runs a trojaned proxy server. We recommend that you follow the instructions in order to keep your