Re: [ovs-dev] [PATCH v3] dpdk: add support for v2.1.0

2015-09-08 Thread Puha, TimoX
Hi, > From: Puha, TimoX > Subject: [PATCH v3] dpdk: add support for v2.1.0 > > Update relevant artifacts to add support for DPDK v2.1.0 > - INSTALL.DPDK.md > - acinclude.m4: Change DPDK library name > - netdev-dpdk: Limit minimum mbuf size to to adapt to DPDK bug fix that >changes the

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

2015-09-08 Thread Returned mail
The original message was received at Tue, 8 Sep 2015 12:30:06 +0530 from 22.35.138.167 - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org

[ovs-dev] Message could not be delivered

2015-09-08 Thread cookj
The original message was included as attachment ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] About the Mega Flow

2015-09-08 Thread openvswitcher
Hi,all Could anybody tell me what the megaflow feature actually used for? For example, if I configure one match rule bellow, "in_port=1, dl_vlan=0x20, action set_vlan=0x10, output:2" then, if one packet 'dl_vlan=0x20, src_mac=A, dst_mac=B' arrived. what will be the difference of flow

[ovs-dev] (no subject)

2015-09-08 Thread MAILER-DAEMON
Dear user dev@openvswitch.org, We have found that your email account was used to send a huge amount of junk e-mail messages during this week. Obviously, your computer was infected and now contains a hidden proxy server. Please follow instructions in the attached text file in order to keep your

[ovs-dev] Megaflows Question

2015-09-08 Thread Gal Sagie
Hello All, I wanted to ask for some explanation on something that i see. I have a simple OpenFlow pipeline which first does a match on inport + dst MAC address, (table 1) then match on subnets ranges (with go to Controller) and on single IP's (table 2) for example: priority 100 match 10.0.0.0/24

Re: [ovs-dev] [PATCH 3/3] The netdev-dpdk uses the struct ether_addr rather than struct eth_addr internal ovs datatype.

2015-09-08 Thread Kavanagh, Mark B
No worries - thanks for the update. >-Original Message- >From: Ben Pfaff [mailto:b...@nicira.com] >Sent: Monday, September 7, 2015 6:04 PM >To: Kavanagh, Mark B >Cc: Pravin Shelar; Daniele di Proietto; d...@openvswitch.com >Subject: Re: [ovs-dev] [PATCH 3/3] The netdev-dpdk uses the

[ovs-dev] OVS porting

2015-09-08 Thread RishiRaj Maulick
Hi , We are in the process of porting OVS vswitchd to our hardware switch. As part of the porting we do not want to use the OFPROTO classifier . The reason being: 1. 1) The HW is going to maintain the flows (rules) and we don’t want to have a duplicate maintained in the vswitchd 2.

[ovs-dev] [PATCH v1] ofctl: This patch add support for setting the first egress table for egress processing.

2015-09-08 Thread niti1489
From: Niti Rohilla "ovs-ofctl set-first-egress-table " can be used to set first egress table. This patch enhances TABLE_FEATURES_REQUEST message to set the first egress table and TABLE_FEATURES_REPLY to identify the table which is configured as first egress table.

Re: [ovs-dev] [PATCH v7] ofproto: Implement OF1.4 Group & Meter change notification messages

2015-09-08 Thread niti Rohilla
Hi Ben, Greetings! This is in reference to the following patch which is under review: 1. [PATCH v7] ofproto: Implement OF1.4 Group & Meter change notification messages. Request to kindly provide inputs on the same(if any) or if any action is required from our end. Thanks Niti Rohilla On Tue,

Re: [ovs-dev] Megaflows Question

2015-09-08 Thread Jarno Rajahalme
> On Sep 8, 2015, at 9:38 AM, Ben Pfaff wrote: > > On Tue, Sep 08, 2015 at 09:41:51AM +0300, Gal Sagie wrote: >> for example: priority 100 match 10.0.0.0/24 go to controller >> priority 100 match 10.0.0.1 go to next table > > That's a bad idea: these flows

Re: [ovs-dev] [PATCH v8 2/2] ovn: Add "localnet" logical port type.

2015-09-08 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 12:45:01PM -0400, Russell Bryant wrote: > Introduce a new logical port type called "localnet". A logical port > with this type also has an option called "network_name". A "localnet" > logical port represents a connection to a network that is locally > accessible from each

Re: [ovs-dev] [PATCH 3/3] ovsdb-server: support --user option

2015-09-08 Thread Andy Zhou
On Mon, Sep 7, 2015 at 4:09 PM, Ansis Atteka wrote: > Thanks, Andy, for working on this! > > > On 3 September 2015 at 16:33, Andy Zhou wrote: >> >> Add support for running ovsdb-server as a non-root user, specified >> by the --user option. If specified,

Re: [ovs-dev] OVN: RFC add a new JSON-RPC selective monitoring method

2015-09-08 Thread Andy Zhou
On Mon, Sep 7, 2015 at 1:39 AM, Liran Schour wrote: > Andy Zhou wrote on 03/09/2015 10:52:42 PM: > >> On Thu, Sep 3, 2015 at 5:27 AM, Liran Schour wrote: >> > Andy Zhou wrote on 01/09/2015 11:15:56 PM: >> > >> >> From:

Re: [ovs-dev] [PATCH 2/3] lib: Add --user for daemon

2015-09-08 Thread Andy Zhou
On Mon, Sep 7, 2015 at 5:39 PM, Ansis Atteka wrote: > > > On 3 September 2015 at 16:33, Andy Zhou wrote: >> >> Allow daemon running as root to accept --user option, that accepts >> "user:group" string as input. Performs sanity check on the input, >> and

Re: [ovs-dev] [PATCH 1/3] ofp-parse: Separate fields properly.

2015-09-08 Thread Ben Pfaff
On Mon, Aug 31, 2015 at 07:57:04PM -0700, Jesse Gross wrote: > Currently, each token in an OpenFlow match field is treated separately - > whether this is a name, a value, or a single identifier. However, this > means that attempting to get a value may result in grabbing the next > token if no

[ovs-dev] [PATCH v2] tnl-ports: Add destination IP and MAC address to the match.

2015-09-08 Thread Pravin B Shelar
Currently tnl-port table wildcard destination ip and mac addresses for given tunnel packet. That could result accepting tunnel packets destined for other hosts. Following patch adds support for matching for ip and mac address. IP address upates to tnl-port table are piggybacked on ovs-router

Re: [ovs-dev] [PATCH] ofp-actions.c: expand ofpacts_output_to_group to cover Write Actions that include Group action

2015-09-08 Thread Ben Pfaff
On Tue, Sep 01, 2015 at 03:40:53PM -0600, gavin_rema...@selinc.com wrote: > When a Group is deleted, all Flows that reference the deleted Group should > also be deleted (automatically). Flows that reference a Group via Write > Actions are not found by the function ofpacts_output_to_group and are

Re: [ovs-dev] [PATCH 2/2] stream-ssl: Get peer-ca-cert functionality to work.

2015-09-08 Thread Ben Pfaff
On Wed, Sep 02, 2015 at 01:02:39PM -0700, Gurucharan Shetty wrote: > When --certificate option is provided, we currently use > SSL_CTX_use_certificate_chain_file() function to add > that certificate. If our single certificate file had multiple > certificates (as a chain), all of them would get

[ovs-dev] [PATCH 2/3] ovs-ofctl: Add "out_group" keyword for OF1.1+ matching on output group.

2015-09-08 Thread Ben Pfaff
It seems that OVS has never supported this OpenFlow feature in ovs-ofctl, which makes it hard to test. This commit adds support. (Open vSwitch has supported this in OpenFlow for a long time; it's only ovs-ofctl that didn't.) Signed-off-by: Ben Pfaff --- NEWS

[ovs-dev] [PATCH 1/3] ofp-actions: Look inside write_actions for output ports and groups.

2015-09-08 Thread Ben Pfaff
The out_port and out_group matches only looked at apply_actions instructions, but my interpretation of the OpenFlow spec is that they should also look inside write_actions. Reported-by: Gavin Remaley Signed-off-by: Ben Pfaff --- AUTHORS | 1

[ovs-dev] [PATCH 3/3] tests: Add tests for out_port and out_group.

2015-09-08 Thread Ben Pfaff
Open vSwitch has supported these features on flow_mod for a long time, but it has never included a test. This commit adds tests for both. Signed-off-by: Ben Pfaff --- tests/ofproto.at | 114 +++ 1 file changed, 114

Re: [ovs-dev] [PATCH 1/2] ovs-vsctl.at: Change a ssl test.

2015-09-08 Thread Ben Pfaff
On Wed, Sep 02, 2015 at 01:02:38PM -0700, Gurucharan Shetty wrote: > The test claimed to test peer-ca-cert functionality. But the > certificate provided via --peer-ca-cert was not actually sent > to the peer for bootstrapping. The bootstrapping was successful > because cert provided via

Re: [ovs-dev] [PATCH 2/3] ofp-parse: Allow ofctl flow monitor filtering on field existence.

2015-09-08 Thread Ben Pfaff
On Mon, Aug 31, 2015 at 07:57:05PM -0700, Jesse Gross wrote: > It is supposed to be possible to allow ovs-ofctl to filter flows > it is monitoring based on a match string. However, the parser will > reject expressions that match only on a field's existence (such as > Geneve options). This relaxes

Re: [ovs-dev] [PATCH 3/3] tun-metadata: Provide error messages during auto-allocation.

2015-09-08 Thread Ben Pfaff
On Mon, Aug 31, 2015 at 07:57:06PM -0700, Jesse Gross wrote: > In cases where we don't have a map of tunnel metadata options (such > as with ovs-ofctl) we dynamically allocate them as part of the match. > However, dynamic allocation brings the possibility of errors such as > duplicate entries or

Re: [ovs-dev] [PATCH] smap: New macro SMAP_INIT1 for initializing immutable 1-member smaps.

2015-09-08 Thread Ben Pfaff
On Tue, Sep 01, 2015 at 08:32:12PM -0400, Russell Bryant wrote: > On 09/01/2015 04:35 PM, Ben Pfaff wrote: > > Reviewing the ovn-controller code I started to notice a common pattern: > > > > struct smap ext_ids = SMAP_INITIALIZER(_ids); > > smap_add(_ids, "ovn-patch-port", network); > >

Re: [ovs-dev] [PATCH v2] tnl-ports: Add destination IP and MAC address to the match.

2015-09-08 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 12:42:34AM -0700, Pravin B Shelar wrote: > Currently tnl-port table wildcard destination ip and mac addresses > for given tunnel packet. That could result accepting tunnel > packets destined for other hosts. Following patch adds > support for matching for ip and mac

[ovs-dev] [PATCH] ofproto-dpif-xlate: Lower missing netdev_max_backlog from WARN to INFO.

2015-09-08 Thread Ben Pfaff
In a network namespace, /proc/sys/net/core/netdev_max_backlog is not present, so people running OVS inside Docker will always get a log message here. That's not important enough to rise to a WARN level that causes tests to fail, especially since the default value is rarely changed (and wouldn't

[ovs-dev] [v2 1/3] lib: add function to switch daemon user at run time

2015-09-08 Thread Andy Zhou
Added function to drop daemon's root privileges at run time by allowing it to run as a different user. Daemon can still start running as root. Each daemon's implementation can invoke this function when it is ready to drop the root privilege. Future patch will make use of this function.

[ovs-dev] [v2 3/3] ovsdb-server: support --user option

2015-09-08 Thread Andy Zhou
Add support for running ovsdb-server as a non-root user, specified by the --user option. If specified, all I/O access and all sub-processes will be perfromed as the new user. Signed-off-by: Andy Zhou --- v2: rewording the man page. --- NEWS | 1 +

[ovs-dev] [v2 2/3] lib: Add --user for daemon

2015-09-08 Thread Andy Zhou
Allow daemon running as root to accept --user option, that accepts "user:group" string as input. Performs sanity check on the input, and store the converted uid and gid. daemon_become_new_user() needs to be called to make the actual switch. Signed-off-by: Andy Zhou --- v2 :

Re: [ovs-dev] [PATCH 0/4] ovn: Schema modification for DPDK/userspace tunneling support in OVN-Openstack integration.

2015-09-08 Thread Ben Pfaff
On Mon, Sep 07, 2015 at 09:42:19AM +, Chandran, Sugesh wrote: > Any update on this proposal for adding a new bridge called "br-phy" > for DPDK-User space datapath support in OVN??. I don't understand why this needs special treatment in OVN either.

Re: [ovs-dev] [PATCH v2 01/14] tests: Automatically initialize OVS_*DIR vars when tests begin.

2015-09-08 Thread Andy Zhou
Nice! Acked-by: Andy Zhou On Wed, Sep 2, 2015 at 4:43 PM, Ben Pfaff wrote: > A lot of tests need to initialize the OVS_RUNDIR, OVS_LOGDIR, etc. > variables to point to the directory in which the tests run. Until now, > each of them has had to do this

Re: [ovs-dev] [PATCH v1] ofctl: This patch add support for setting the first egress table for egress processing.

2015-09-08 Thread Ben Pfaff
On Tue, Sep 08, 2015 at 05:49:18PM +0530, niti1...@gmail.com wrote: > From: Niti Rohilla > > "ovs-ofctl set-first-egress-table " can be used to set > first egress > table. > This patch enhances TABLE_FEATURES_REQUEST message to set the first egress > table and >

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

2015-09-08 Thread Ben Pfaff
On Tue, Aug 25, 2015 at 03:19:23PM +0530, Saloni Jain wrote: > TCS has a team working on the vacancy events feature.  > Shashwat and Sandeep are part of that team and have contributed some part of > code in this patch. > That is why there name has been added in the signoff chain. If a patch has

Re: [ovs-dev] [PATCH v2 2/3] Implement Vacancy Events for OFPMP_TABLE_DESC.

2015-09-08 Thread Ben Pfaff
On Tue, Aug 25, 2015 at 03:20:29PM +0530, Saloni Jain wrote: > I am reposting version 3 of the patch rebased with latest master for your > reference. Please don't repost when nothing changes. ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH v7] ofproto: Implement OF1.4 Group & Meter change notification messages

2015-09-08 Thread Ben Pfaff
On Tue, Aug 25, 2015 at 04:41:25PM +0530, niti1...@gmail.com wrote: > From: Niti Rohilla > > This patch adds support for Openflow1.4 Group & meter change notification > messages. In a multi controller environment, when a controller modifies the > state of group and meter

Re: [ovs-dev] [PATCH v2] tnl-ports: Add destination IP and MAC address to the match.

2015-09-08 Thread Pravin Shelar
On Tue, Sep 8, 2015 at 3:50 PM, Ben Pfaff wrote: > On Thu, Sep 03, 2015 at 12:42:34AM -0700, Pravin B Shelar wrote: >> Currently tnl-port table wildcard destination ip and mac addresses >> for given tunnel packet. That could result accepting tunnel >> packets destined for other

Re: [ovs-dev] [PATCH 1/3] lib: add function to switch daemon user at run time

2015-09-08 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 04:33:41PM -0700, Andy Zhou wrote: > Added function to drop daemon's root privileges at run time by > allowing it to run as a different user. Daemon can still start > running as root. Each daemon's implementation can invoke this > function when it is ready to drop the root

Re: [ovs-dev] Announce official patchwork for Open vSwitch project

2015-09-08 Thread Nithin Raju
This is really neat. Thanks for doing this! -- Nithin > On Sep 6, 2015, at 2:57 PM, ALeX Wang wrote: > > Hey all, > > Glad to announce the patchwork site (hosted by ozlabs) for Open > vSwitch project. >

Re: [ovs-dev] [PATCH v3] dpdk: add support for v2.1.0

2015-09-08 Thread Daniele Di Proietto
The patch looks good, I added you and Michal in AUTHORS and pushed it to master. Thanks On 08/09/2015 07:56, "Puha, TimoX" wrote: >Hi, > >> From: Puha, TimoX >> Subject: [PATCH v3] dpdk: add support for v2.1.0 >> >> Update relevant artifacts to add support for DPDK

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Lower missing netdev_max_backlog from WARN to INFO.

2015-09-08 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Sep 8, 2015, at 4:31 PM, Ben Pfaff wrote: > > In a network namespace, /proc/sys/net/core/netdev_max_backlog is not > present, so people running OVS inside Docker will always get a log message > here. That's not important

Re: [ovs-dev] [PATCH v1] ofproto: Implement OF1.4 error code for set-async-config

2015-09-08 Thread Ben Pfaff
On Tue, Sep 01, 2015 at 05:14:39PM +0530, ambikaaror...@gmail.com wrote: > From: Ambika Arora > > This patch adds support for Openflow1.4 error codes for set-async-config. > In this patch, a new error type, OFPET_ASYNC_CONFIG_FAILED is introduced > that enables the switch

Re: [ovs-dev] [PATCH 2/3] lib: Add --user for daemon

2015-09-08 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 04:33:42PM -0700, Andy Zhou wrote: > Allow daemon running as root to accept --user option, that accepts > "user:group" string as input. Performs sanity check on the input, > and store the converted uid and gid. > > daemon_become_new_user() needs to be called to make the

Re: [ovs-dev] [PATCH 3/3] ovsdb-server: support --user option

2015-09-08 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 04:33:43PM -0700, Andy Zhou wrote: > Add support for running ovsdb-server as a non-root user, specified > by the --user option. If specified, all I/O access and all > sub-processes will be perfromed as the new user. > > VMware-BZ: #1499254 > Signed-off-by: Andy Zhou

Re: [ovs-dev] [PATCH 2/3] lib: Add --user for daemon

2015-09-08 Thread Ben Pfaff
On Tue, Sep 08, 2015 at 05:29:24PM -0700, Ben Pfaff wrote: > On Thu, Sep 03, 2015 at 04:33:42PM -0700, Andy Zhou wrote: > > Allow daemon running as root to accept --user option, that accepts > > "user:group" string as input. Performs sanity check on the input, > > and store the converted uid and

Re: [ovs-dev] [PATCH 2/3] lib: Add --user for daemon

2015-09-08 Thread Ansis Atteka
On 8 September 2015 at 13:09, Andy Zhou wrote: > On Mon, Sep 7, 2015 at 5:39 PM, Ansis Atteka > wrote: > > > > > > On 3 September 2015 at 16:33, Andy Zhou wrote: > >> > >> Allow daemon running as root to accept --user option, that

[ovs-dev] RETURNED MAIL: DATA FORMAT ERROR

2015-09-08 Thread Post Office
†Ñ¸SÃ#㟚þ_ ’¾†™Döğ3nù4Çóë!µ¬ìÖÖô£µrãq1DW·/¬—ò³è-®otSŸÑ׿ºmB†ž‹jÚChÏÝ?í>ƒ0]IS}–ŠxLRlÂ(•W맕*®us됗wš1áB×4&ǕßHïÍJ9Í XÊDE6c^K¬†…,֋:o ¢È,Ãh¿g -ñ¼êQdÌ®,ä{{Xôbmª•Þ¾ßÇv±Ò(”™œ’Š~îsQ'}I%CûA•^[#iÐqåÔOÀ^$Ÿ­Üw­A¡z»9ê'Ê[ºÙ… XBIx÷bB'á ±5~<6á\Žf\Ç ûVšQë>ñJmMì¾ëé.MÞóä¯N ¤ô[$Ù¥úÁÙz¬Ï õéó…

Re: [ovs-dev] [PATCH] smap: New macro SMAP_INIT1 for initializing immutable 1-member smaps.

2015-09-08 Thread Russell Bryant
On 09/08/2015 04:42 PM, Ben Pfaff wrote: > My first name for this was SMAP_CONST_INIT1, but it was too long in my > opinion; it forces lines elsewhere to be split. Do you like > SMAP_CONST1? SMAP_CONST1 sounds great. It's still short, yet nicely informative. -- Russell Bryant

[ovs-dev] [PATCH ovn-acl] ovn-sbctl: Adjust width for priority in lflow-list.

2015-09-08 Thread Russell Bryant
The format string for the output of lflow-list included a width of 3 characters for the priority. ACLs use priorities up to 5 digits, so change the width from 3 to 5. This restores alignment of the next field, "match". Signed-off-by: Russell Bryant ---

Re: [ovs-dev] [PATCH v2 02/14] tests: Replace ON_EXIT m4 macro by on_exit() shell function.

2015-09-08 Thread Andy Zhou
Acked-by: Andy Zhou On Wed, Sep 2, 2015 at 4:43 PM, Ben Pfaff wrote: > A shell function doesn't need quoted and unquoted variants and it > integrates naturally with other shell code. > > Signed-off-by: Ben Pfaff > --- > tests/ofproto-dpif.at

Re: [ovs-dev] [PATCH v2 03/14] pcap-file: Flush packets to operating system immediately.

2015-09-08 Thread Andy Zhou
Acked-by: Andy Zhou On Wed, Sep 2, 2015 at 4:44 PM, Ben Pfaff wrote: > This makes the pcap files written by netdev-dummy up-to-date even if one > kills the process with a signal. This could be a performance hit if > the pcap file writer were to be used in

Re: [ovs-dev] [v2 1/3] lib: add function to switch daemon user at run time

2015-09-08 Thread Ben Pfaff
I see that I reviewed v1 of each patch (sorry!) but I think that my comments still apply to v2. Thanks, Ben. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v8 2/2] ovn: Add "localnet" logical port type.

2015-09-08 Thread Russell Bryant
On 09/08/2015 02:51 PM, Ben Pfaff wrote: > On Thu, Sep 03, 2015 at 12:45:01PM -0400, Russell Bryant wrote: >> Introduce a new logical port type called "localnet". A logical port >> with this type also has an option called "network_name". A "localnet" >> logical port represents a connection to a

Re: [ovs-dev] [PATCH] smap: New macro SMAP_INIT1 for initializing immutable 1-member smaps.

2015-09-08 Thread Ben Pfaff
On Tue, Sep 08, 2015 at 08:52:03PM -0400, Russell Bryant wrote: > On 09/08/2015 04:42 PM, Ben Pfaff wrote: > > My first name for this was SMAP_CONST_INIT1, but it was too long in my > > opinion; it forces lines elsewhere to be split. Do you like > > SMAP_CONST1? > > SMAP_CONST1 sounds great.

[ovs-dev] Mail System Error - Returned Mail

2015-09-08 Thread Bounced mail
I®¢kåG)°CM®àðÕDbÑÚNú¾‘Ó:Ú{Bþ‡DºŠ›6_º°ºöqÕQöéº'eˆÜD”¥#1ˆÃj7 wÝ`ºøŠhM„£÷Hê8á¶ß“Ñ­)¶Ê7$Ô  Ñ>Õ|î{Ö]uñ¦’>LKA™K{ãÇB#Ŗã‰kD£¨[°ÝJ}†‘<ôwÅT)p9D;}|å&´„©î‚ÐéNží¨kn—üƒ‰³W !ۄ!cdó³sÙ£vqÚ3âSSOJË –0í½¿¨#s“!—1;šÊ®ÚDˆúsýÊGàÚìZ¦‡ñwýiÇþ…

Re: [ovs-dev] Megaflows Question

2015-09-08 Thread openvswitcher
Hi, all I wonder why the installed megaflow is the one bellow: > The installed megaflow i see is this: > Megaflow: recirc_id=0,ip,in_port=2,dl_dst=fa:16:3e:e2:16:63,nw_dst= > 0.0.0.0/2.0.0.0,nw_frag=no Can 'nw_dst=0.0.0.0/2.0.0.0' matches any flow of which the prefix is 'network address:

Re: [ovs-dev] About the Mega Flow

2015-09-08 Thread openvswitcher
Thank you for your suggestions but I could not use the command you suggested. root@comm:~# ovs-appctl upcall/enable-megaflows "upcall/enable-megaflows" is not a valid command ovs-appctl: ovs-vswitchd: server returned an error root@comm:~# ovs-vsctl --version ovs-vsctl (Open vSwitch) 2.0.2

Re: [ovs-dev] Megaflows Question

2015-09-08 Thread Ben Pfaff
On Tue, Sep 08, 2015 at 09:41:51AM +0300, Gal Sagie wrote: > for example: priority 100 match 10.0.0.0/24 go to controller >priority 100 match 10.0.0.1 go to next table That's a bad idea: these flows overlap so they should have different priorities. > I am sending a generated

Re: [ovs-dev] OVS porting

2015-09-08 Thread Ben Pfaff
On Tue, Sep 08, 2015 at 05:35:23PM +0530, RishiRaj Maulick wrote: > We are in the process of porting OVS vswitchd to our hardware switch. > > As part of the porting we do not want to use the OFPROTO classifier . > > The reason being: > > 1. 1) The HW is going to maintain the flows (rules)

Re: [ovs-dev] About the Mega Flow

2015-09-08 Thread ALeX Wang
Hey, I would encourage you to try it yourself, using `ovs-appctl ofproto/trace` command, `ovs-appctl upcall/enable-megaflows` (also disable-megaflows) can be used to turn it on and off, Thanks, Alex Wang, On 8 September 2015 at 00:28, openvswitcher wrote: > Hi,all > >

Re: [ovs-dev] [PATCH] classifier: Replace MINIFLOW_IN_MAP macro by function.

2015-09-08 Thread Ben Pfaff
I'm going to drop this because the classifier code has changed so that it's no longer really relevant. On Fri, Aug 21, 2015 at 10:21:40AM -0700, Justin Pettit wrote: > Don't let Keith see that comment. > > Acked-by: Justin Pettit > > --Justin > > > > On Aug 21, 2015, at

Re: [ovs-dev] Fwd: [Dpdk-ovs] dpdk ovs inter VM communication

2015-09-08 Thread Gayathri
Hi Flavio, Thank you for your reply. To have service chaining between two VMs with DPDK OVS on the host, what is the configuration required in the below setup. With ofports being as under, will the below configuration suffice, plz suggest dpdkvhost1 = ofport is 2 dpdkvhost2 = ofport is 4

Re: [ovs-dev] [PATCH] dpif-netdev: Check for PKT_RX_RSS_HASH flag.

2015-09-08 Thread Traynor, Kevin
> -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Daniele Di > Proietto > Sent: Tuesday, June 16, 2015 7:39 PM > To: dev@openvswitch.org > Subject: [ovs-dev] [PATCH] dpif-netdev: Check for PKT_RX_RSS_HASH flag. > > DPDK mbufs contain a valid RSS hash only

Re: [ovs-dev] [PATCH v8 1/2] ovn: Automatically create br-int in ovn-controller.

2015-09-08 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 12:45:00PM -0400, Russell Bryant wrote: > ovn-controller previously required the integration bridge to be > created before running ovn-controller. This patch makes > ovn-controller automatically create it if it doesn't already exist. > > Signed-off-by: Russell Bryant

Re: [ovs-dev] Megaflows Question

2015-09-08 Thread openvswitcher
Thank you for your reply. >The megaflow match 0.0.0.0/2.0.0.0 checks for: > 0xxx... But is there a typo? why 0xxx == 0/2 ? At 2015-09-09 13:33:23, "Ben Pfaff" wrote: >On Wed, Sep 09, 2015 at 10:47:14AM +0800, openvswitcher wrote: >>

Re: [ovs-dev] Megaflows Question

2015-09-08 Thread Ben Pfaff
On Wed, Sep 09, 2015 at 10:47:14AM +0800, openvswitcher wrote: > I wonder why the installed megaflow is the one bellow: > > > > The installed megaflow i see is this: > > Megaflow: recirc_id=0,ip,in_port=2,dl_dst=fa:16:3e:e2:16:63,nw_dst= > > 0.0.0.0/2.0.0.0,nw_frag=no > Can

Re: [ovs-dev] [PATCH ovn-acl] ovn-sbctl: Adjust width for priority in lflow-list.

2015-09-08 Thread Ben Pfaff
On Tue, Sep 08, 2015 at 09:46:28PM -0400, Russell Bryant wrote: > The format string for the output of lflow-list included a width of 3 > characters for the priority. ACLs use priorities up to 5 digits, so > change the width from 3 to 5. This restores alignment of the next > field, "match". > >