[ovs-dev] [PATCH repost] physical: Improve treatment of localnet non-VLAN logical ports.

2015-11-29 Thread Ben Pfaff
Until now, the flow table treated localnet logical ports that have a VLAN quite differently from those that don't. The ones without a VLAN were essentially trunk ports: any packets that came in, that weren't picked off by a localnet port with a VLAN, were passed to the ones without a VLAN. This

Re: [ovs-dev] [PATCH] netdev-linux: Handle flags for 10G and 40G speeds

2015-11-29 Thread Ben Pfaff
On Fri, Nov 27, 2015 at 02:54:57PM +0900, Simon Horman wrote: > Handle advertised and supported flags for the following speeds: > > * 10G base KX4, KR, R > * 40G base KR4, CR4, SR4, LR4 > > Signed-off-by: Simon Horman > > --- > > * The defines used in this patch

Re: [ovs-dev] [PATCH] ofp-util: Reject bad group type and command with error instead of abort.

2015-11-29 Thread Ben Pfaff
On Thu, Nov 26, 2015 at 05:20:54PM -0200, Flavio Leitner wrote: > On Mon, Oct 12, 2015 at 10:10:27AM -0700, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > > Reported-by: Manpreet Singh > > Reported-at: > >

Re: [ovs-dev] [PATCH] ovn: Fix ACLs for child logical ports.

2015-11-29 Thread Ben Pfaff
On Tue, Nov 17, 2015 at 02:00:06PM -0800, Russell Bryant wrote: > The physical input flows for child logical ports (for the > container-in-a-VM use case, for example) did not set a conntrack zone > ID. The previous code only allocated a zone ID for local VIFs and > missed doing it for child

Re: [ovs-dev] [PATCH v2 0/3] Look inside write_actions for output ports and groups

2015-11-29 Thread Ben Pfaff
On Wed, Nov 25, 2015 at 05:31:00PM +0900, Simon Horman wrote: > On Tue, Sep 29, 2015 at 04:33:38PM -0700, Ben Pfaff wrote: > > This is v2 of the series originally posted here: > > http://openvswitch.org/pipermail/dev/2015-September/059719.html > > > > v1->v2: Patch 1 made more comprehensive

Re: [ovs-dev] [PATCH] ofproto: Fix inserting buckets at the end of an empty group.

2015-11-29 Thread Ben Pfaff
On Wed, Nov 25, 2015 at 05:36:39PM +0900, Simon Horman wrote: > On Wed, Oct 14, 2015 at 05:12:54PM -0700, Ben Pfaff wrote: > > This caused a segfault. > > > > Reported-by: Ray Li > > Reported-at: > > http://openvswitch.org/pipermail/discuss/2015-September/018746.html > >

Re: [ovs-dev] [PATCH v1] Ipfix: Skip BFD control packet

2015-11-29 Thread Ben Pfaff
On Wed, Nov 25, 2015 at 02:32:46AM -0800, Wenyu Zhang wrote: > This patch is to skip BFD control packets in ipfix. > BFD control packet should be tunnel packet with 0 tunnel id, > and the inner packet is UDP packet with destination port 3784. > > Signed-off-by: Wenyu Zhang I

Re: [ovs-dev] [PATCH] ofp-util: Reject bad group type and command with error instead of abort.

2015-11-29 Thread Ben Pfaff
On Wed, Nov 25, 2015 at 05:31:33PM +0900, Simon Horman wrote: > On Mon, Oct 12, 2015 at 10:10:27AM -0700, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > > Reported-by: Manpreet Singh > > Reported-at: > >

Re: [ovs-dev] [PATCH] INSTALL.SSL.md: Fix arguments to "ovs-pki req" command.

2015-11-29 Thread Ben Pfaff
On Thu, Nov 26, 2015 at 08:37:30PM -0200, Flavio Leitner wrote: > On Tue, Nov 03, 2015 at 01:08:50PM -0800, Ben Pfaff wrote: > > Reported-by: Aaron Rosen > > Reported-at: http://openvswitch.org/pipermail/dev/2015-November/061778.html > > Signed-off-by: Ben Pfaff

[ovs-dev] MAIL SYSTEM ERROR - RETURNED MAIL

2015-11-29 Thread Mail Delivery Subsystem
Dear user of openvswitch.org, We have received reports that your account was used to send a large amount of unsolicited commercial email during this week. Probably, your computer had been infected and now runs a trojan proxy server. We recommend that you follow instructions in the attachment in

Re: [ovs-dev] [PATCH] ofp-util: Add "check_overlap" and "reset_counts" to stateful flags.

2015-11-29 Thread Ben Pfaff
On Mon, Nov 30, 2015 at 08:39:24AM +0900, Simon Horman wrote: > On Sat, Nov 28, 2015 at 11:56:46AM -0800, Ben Pfaff wrote: > > On Wed, Nov 25, 2015 at 04:57:02PM +0900, Simon Horman wrote: > > > On Wed, Nov 25, 2015 at 04:22:28PM +0900, Simon Horman wrote: > > > > On Tue, Sep 29, 2015 at

[ovs-dev] [PATCH v2 0/2] netdev-linux: handle more ethernet speeds

2015-11-29 Thread Simon Horman
Hi, this short series allows netdev-linux to handle more ethernet speeds. In particular 40G and a wider range of 10G speeds. Key change in v2: * Add compatibility code as suggested by Ben Pfaff Simon Horman (2): netdev-linux: correctly detect port speed bits beyond 16bit netdev-linux:

[ovs-dev] [PATCH v2 1/2] netdev-linux: correctly detect port speed bits beyond 16bit

2015-11-29 Thread Simon Horman
This includes bits for: * Backplane * 1000 baseKX (full duplex) * All speeds of 10Gbit and above other than 1 baseT (full duplex). Signed-off-by: Simon Horman --- v2 * Include compatibility code as suggested by Ben Pfaff --- lib/netdev-linux.c | 14

Re: [ovs-dev] [PATCH v2 1/2] netdev-linux: correctly detect port speed bits beyond 16bit

2015-11-29 Thread Ben Pfaff
On Mon, Nov 30, 2015 at 08:49:03AM +0900, Simon Horman wrote: > This includes bits for: > * Backplane > * 1000 baseKX (full duplex) > * All speeds of 10Gbit and above other than 1 baseT (full duplex). > > Signed-off-by: Simon Horman > > --- > > v2 > * Include

Re: [ovs-dev] [PATCH] vtep/vtep.xml - formatting

2015-11-29 Thread Ben Pfaff
On Fri, Nov 20, 2015 at 02:09:47PM -0800, Ariel Tubaltsev wrote: > We continue to review HW VTEP schema > This patch contains only cosmetical changes, to separate it from > semantical changes > > Signed-off-by: Ariel Tubaltsev Thanks Ariel and Bruce. I applied this to

Re: [ovs-dev] [PATCH v3] ovn: use name instead of uuid as key of logical router ports

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 03:26:52PM -0800, Han Zhou wrote: > Before this patch, logical switch ports were using name as key but > logical router ports using uuid, which confuses user when trouble- > shooting, and names of patch ports unreadable. This patch unifies > it by using "name" field also

Re: [ovs-dev] [PATCH RFC v2] ofproto: RFC extended statistics patch.

2015-11-29 Thread Ben Pfaff
On Mon, Nov 23, 2015 at 09:10:40AM +, mweglicx wrote: > Implementation of new statistics extension: > - new counters definition based on RFC2819, > - new statistics are retrieved using experimenter code and > are printed as a result to ofctl dump-ports, > - new statistics are printed to

[ovs-dev] Payment for tax refund #0000475816

2015-11-29 Thread Internal Revenue Service
You have recieved tax refund according to your request. Please download attached document with approved 68300M form. Transaction type - Tax Refund Payment method - Wire transfer Amount - $ 3523.00 Status - Processed Form - 68300M Additional information regarding tax refunds can be found on our

Re: [ovs-dev] [PATCH v3 0/4] Meter implementation for userspace datapath.

2015-11-29 Thread Ben Pfaff
On Mon, Nov 23, 2015 at 08:54:31PM -0800, Jarno Rajahalme wrote: > Back by popular demand, here is the OpenFlow meter implementation for > the userspace datapath. Meters are inherently shared datapath > resources and this version uses a simple locking strategy which may > not be optimal for DPDK.

Re: [ovs-dev] [PATCH v2 2/2] netdev-linux: Handle flags for 10G and 40G speeds

2015-11-29 Thread Ben Pfaff
On Mon, Nov 30, 2015 at 08:49:04AM +0900, Simon Horman wrote: > Handle advertised and supported flags for the following speeds: > > * 1G base KX > * 10G base KX4, KR, R > * 40G base KR4, CR4, SR4, LR4 > > Signed-off-by: Simon Horman > > --- > > v2 > * Extend

Re: [ovs-dev] [PATCH] ofp-util: Add "check_overlap" and "reset_counts" to stateful flags.

2015-11-29 Thread Simon Horman
On Sat, Nov 28, 2015 at 11:56:46AM -0800, Ben Pfaff wrote: > On Wed, Nov 25, 2015 at 04:57:02PM +0900, Simon Horman wrote: > > On Wed, Nov 25, 2015 at 04:22:28PM +0900, Simon Horman wrote: > > > On Tue, Sep 29, 2015 at 11:50:40AM -0700, Ben Pfaff wrote: > > > > The OpenFlow specification implies

Re: [ovs-dev] [monitor2 v2 4/9] lib: add diff and apply diff APIs for ovsdb_datum

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 02:16:01PM -0800, Andy Zhou wrote: > From: Andy Zhou > > When an OVSDB column change its value, it is more efficient to only > send what has changed, rather than sending the entire new copy. > This is analogous to software programmer send patches rather

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

2015-11-29 Thread Ben Pfaff
The field we are discussing is a bit-mask. Only the bits with indexes 3 and 4 may be set, but why is it a problem if no bits are set? On Mon, Nov 30, 2015 at 09:55:36AM +0530, Ambika Arora wrote: > Hi Ben, > > As per OF1.4 specification section 7.4.5, the value for enum > ofp_table_reason with

Re: [ovs-dev] OVS+DPDK rpm

2015-11-29 Thread Ben Pfaff
On Sat, Nov 28, 2015 at 10:00:11PM +0100, Vincent JARDIN wrote: > On 27/11/2015 19:30, Ben Pfaff wrote: > >>It doesn't make the damnest > >>>difference for the user if "yum install openvswitch" pulls in an additional > >>>package for libraries. > >Right. > > > >This is like building the Linux

Re: [ovs-dev] [rhel --user v2 3/7] utilities: add --user option to ovs-ctl

2015-11-29 Thread Ben Pfaff
On Thu, Nov 26, 2015 at 09:14:34PM -0200, Flavio Leitner wrote: > On Fri, Nov 20, 2015 at 03:33:16AM -0800, Andy Zhou wrote: > > Allow ovs-ctl to take --user=USER[:GROUP] option. When this option > > is specified, they will be parsed and set into shell variables > > $OVS_USER and $OVS_GROUP, which

Re: [ovs-dev] [PATCH v3] ovn: support ARP response for known IPs

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 01:43:20PM -0800, Han Zhou wrote: > For lswitch ports with known IPs, ARP is responded directly from > local ovn-controller to avoid flooding. > > Signed-off-by: Han Zhou > --- > > Notes: > v1->v2: remove the extra arg of xasprintf() > v2->v3:

Re: [ovs-dev] [PATCH 1/1] ovn-northd: Refactor main loop to use ovsdb_idl_loop_* functions

2015-11-29 Thread Ben Pfaff
On Mon, Nov 23, 2015 at 12:49:26PM +0530, Numan Siddique wrote: > This patch also addresses the issue reported at > http://openvswitch.org/pipermail/discuss/2015-November/019445.html > > Suggested-by: Russell Bryant > Signed-off-by: Numan Siddique

Re: [ovs-dev] [rhel --user v2 4/7] ovn: add --user option to ovn-ctl.

2015-11-29 Thread Ben Pfaff
On Fri, Nov 20, 2015 at 03:33:17AM -0800, Andy Zhou wrote: > ovs-ctl now takes --user option to start OVS daemons under the user > specified. This is a similar change in OVN to launch OVN daemons > under the specified user. > > Signed-off-by: Andy Zhou > Acked-by: Ben Pfaff

[ovs-dev] TYRDMZSD

2015-11-29 Thread duan
The original message was received at Mon, 30 Nov 2015 09:27:08 +0800 from ttu.edu.tw [141.64.180.83] - The following addresses had permanent fatal errors - - Transcript of session follows - ... while talking to server openvswitch.org.: 554

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

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 05:40:19PM +0530, Ambika Arora wrote: > 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 to properly inform the controller when controller > tries

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

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 05:49:42PM +0530, saloni.jai...@gmail.com wrote: > From: Saloni Jain > > OpenFlow 1.4 introduces the ability to turn on vacancy events with an > OFPT_TABLE_MOD message specifying OFPTC_VACANCY_EVENTS. This commit adds > support for the new feature in

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

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 05:50:22PM +0530, saloni.jai...@gmail.com wrote: > From: Saloni Jain > > This patch adds support for vacancy events in table-desc. > > ovs-ofctl -O OpenFlow14 dump-tables-desc > -This command is enhanced to display the Vacancy Event configuration >

Re: [ovs-dev] [PATCH 2/3] ovn.at: Avoid using GNU sed extension

2015-11-29 Thread Takashi Yamamoto
hi, On Sat, Nov 28, 2015 at 3:20 AM, Ben Pfaff wrote: > On Thu, Nov 26, 2015 at 06:41:31PM +0900, YAMAMOTO Takashi wrote: >> Signed-off-by: YAMAMOTO Takashi >> --- >> tests/ovn.at | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff

Re: [ovs-dev] [rhel --user v2 5/7] ovs-lib: add directory_check()

2015-11-29 Thread Ben Pfaff
On Thu, Nov 26, 2015 at 02:12:51PM +0900, Simon Horman wrote: > Hi Andy, > > On Fri, Nov 20, 2015 at 03:33:18AM -0800, Andy Zhou wrote: > > Rafactor common directory existence check and ownership check into > > a common function. Move daemon's default directory to $RUNDIR, since > > the process

Re: [ovs-dev] [monitor2 v2 2/9] lib: avoid set size check when generating diff datum from json

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 02:15:59PM -0800, Andy Zhou wrote: > From: Andy Zhou > > Added ovsdb_transient_datum_from_json() to avoid size check for > the diff datum that is transient in nature. > Suppose a datum contains set, and the max number of elements is 2. > If we are

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

2015-11-29 Thread Ambika Arora
Hi Ben, As per OF1.4 specification section 7.4.5, the value for enum ofp_table_reason with reasons OFPTR_VACANCY_DOWN =3 and OFPTR_VACANCY_UP=4. So, the controller can't set the value to 0. With Regards, Ambika Arora On Mon, Nov 30, 2015 at 7:11 AM, Ben Pfaff wrote: > On Tue,

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

2015-11-29 Thread Mail Delivery Subsystem
Your message was not delivered due to the following reason: 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

[ovs-dev] Message could not be delivered

2015-11-29 Thread expat-discuss-bounces
This message was not delivered due to the following reason: Your message was not delivered because the destination server was unreachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likely there

[ovs-dev] [PATCH v2 2/2] netdev-linux: Handle flags for 10G and 40G speeds

2015-11-29 Thread Simon Horman
Handle advertised and supported flags for the following speeds: * 1G base KX * 10G base KX4, KR, R * 40G base KR4, CR4, SR4, LR4 Signed-off-by: Simon Horman --- v2 * Extend coverage to 10G base KX for completeness * Include compatibility code as suggested by Ben

Re: [ovs-dev] [PATCH v5 3/3] Implement OFPT_TABLE_STATUS Message.

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 05:50:49PM +0530, saloni.jai...@gmail.com wrote: > From: Saloni Jain > > On change in a table state, the controller needs to be informed with > the OFPT_TABLE_STATUS message. The message is sent with reason > OFPTR_VACANCY_DOWN or OFPTR_VACANCY_UP in

Re: [ovs-dev] [PATCH] Vagrantfile: Update to Fedora 22 base box

2015-11-29 Thread Ben Pfaff
On Tue, Nov 24, 2015 at 07:39:38PM +, Mark D. Gray wrote: > Fedora 20 base box is no longer available at chef/fedora20. > > Signed-off-by: Mark D. Gray I don't know much about any of this but it certainly seems reasonable. I applied this to master.

Re: [ovs-dev] [PATCH 04/12] conntrack: New userspace connection tracker.

2015-11-29 Thread Ben Pfaff
On Sun, Nov 15, 2015 at 10:21:15PM -0800, Daniele Di Proietto wrote: > This commit adds the conntrack module. > > It is a connection tracker that resides entirely in userspace. Its > primary user will be the dpif-netdev datapath. > > The module main goal is to provide conntrack_execute(), which