Re: [ovs-dev] [PATCH v6] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

2016-12-14 Thread Chandran, Sugesh
Regards _Sugesh > -Original Message- > From: Ben Pfaff [mailto:b...@ovn.org] > Sent: Tuesday, December 13, 2016 4:29 PM > To: Chandran, Sugesh > Cc: d...@openvswitch.org; je...@kernel.org > Subject: Re: [ovs-dev] [PATCH v6] netdev-dpdk: Enable Rx checksum >

[ovs-dev] [PATCH v2 1/4] doc: Split dpdk, dpdk-advanced into multiple docs

2016-12-14 Thread Stephen Finucane
Combined, the dpdk and dpdk-advanced installation documents provide a lot of useful information, but most of this information is unrelated to installation. Rework these documents, completely breaking up the dpdk-advanced document into multiple smaller documents in other sections and moving

[ovs-dev] [PATCH v2 3/4] doc: Document Patchwork instance

2016-12-14 Thread Stephen Finucane
I know more than a little bit about this :) Signed-off-by: Stephen Finucane --- Documentation/automake.mk | 1 + Documentation/index.rst | 1 + Documentation/internals/index.rst | 1 + Documentation/internals/patchwork.rst | 61

[ovs-dev] [PATCH v2 0/4] doc: Rework DPDK guide

2016-12-14 Thread Stephen Finucane
As promised, this series reworks the largest of the installation guides to suit the new structure of the documentation. A couple of related patches are included to build upon this. Changes since v2: - Rebase onto master Stephen Finucane (4): doc: Split dpdk, dpdk-advanced into multiple docs

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Stephen Finucane
On Tue, 2016-12-13 at 11:41 -0800, Ben Pfaff wrote: > On Tue, Dec 13, 2016 at 05:57:22PM +, Stephen Finucane wrote: > > This will make upcoming changes easier to review. > > > > Signed-off-by: Stephen Finucane > > "git am" tells me: > > fatal: corrupt patch at line

Re: [ovs-dev] [PATCH] docs: Add link to man pages.

2016-12-14 Thread Stephen Finucane
On Tue, 2016-12-13 at 15:09 -0500, Russell Bryant wrote: > > > On Tue, Dec 13, 2016 at 1:30 PM, Stephen Finucane > wrote: > > On Tue, 2016-12-13 at 18:17 +, Stephen Finucane wrote: > > > On Tue, 2016-12-13 at 10:57 -0500, Russell Bryant wrote: > > > > Our new docs site

[ovs-dev] [PATCH v2 2/4] doc: Move testing to testing section

2016-12-14 Thread Stephen Finucane
This makes more sense here, seeing as it's not exactly installation related. Signed-off-by: Stephen Finucane --- Documentation/intro/install/general.rst | 348 +-- Documentation/topics/testing.rst| 359 +++- 2

[ovs-dev] [PATCH v2 4/4] gitignore: Ignore venv

2016-12-14 Thread Stephen Finucane
This is the traditional name for Python virtualenv directories. I use this when building docs to avoid installating system libraries. Signed-off-by: Stephen Finucane --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index

Re: [ovs-dev] [PATCH net-next] net: remove abuse of VLAN DEI/CFI bit

2016-12-14 Thread Michał Mirosław
On Tue, Dec 13, 2016 at 05:21:18PM -0800, Stephen Hemminger wrote: > On Sat, 3 Dec 2016 10:22:28 +0100 (CET) > Michał Mirosław wrote: > > > This All-in-one patch removes abuse of VLAN CFI bit, so it can be passed > > intact through linux networking stack. > > > >

Re: [ovs-dev] [PATCH 3/3] tests: Ignore options order in dhcpv4 ovn test.

2016-12-14 Thread Balazs Nemeth
>On 12/12/2016 13:14, "Ben Pfaff" > wrote: > >>On Thu, Dec 08, 2016 at 06:50:32PM -0800, Daniele Di Proietto wrote: >>> The order of the options in the packet generated by ovs-controller >>> depends on the hash function. I believe that murmur hash (our default)

[ovs-dev] [PATCH v2 1/3] netdev-dpdk: add hotplug support

2016-12-14 Thread Ciara Loftus
In order to use dpdk ports in ovs they have to be bound to a DPDK compatible driver before ovs is started. This patch adds the possibility to hotplug (or hot-unplug) a device after ovs has been started. The implementation adds two appctl commands: netdev-dpdk/attach and netdev-dpdk/detach After

[ovs-dev] [PATCH v2 3/3] netdev-dpdk: Add support for virtual DPDK PMDs (vdevs)

2016-12-14 Thread Ciara Loftus
Prior to this commit, the 'dpdk' port type could only be used for physical DPDK devices. Now, virtual devices (or 'vdevs') are supported. 'vdev' devices are those which use virtual DPDK Poll Mode Drivers eg. null, pcap. To add a DPDK vdev, a valid 'dpdk-devargs' must be set for the given dpdk

[ovs-dev] Cómo Defendese del Fisco - En Línea

2016-12-14 Thread Principales medios de defensa
En línea y en Vivo / Para todo su Equipo con una sola Conexión CÓMO DEFENDERSE DEL FISCO 28 de diciembre - Online en Vivo - 10:00 a 13:00 Hrs En un juicio ante el SAT, las probabilidades de que la autoridad gane son mayores. Pero hay mecanismos que le ayudan al contribuyente a

[ovs-dev] [PATCH] ovn-northd: fix monitor process naming

2016-12-14 Thread Lance Richardson
Currently the ovn-northd monitor process and the ovn-northd process have the same name, e.g. ps -ef | grep northd shows (edited for space): ... ovn-northd --detach --monitor --log-file=ovn-northd.log --pidfile ... ovn-northd --detach --monitor --log-file=ovn-northd.log --pidfile With the

Re: [ovs-dev] [PATCH] ovsdb-data: Add support for integer ranges in database commands

2016-12-14 Thread Łukasz Rząsik
Hi Ben, Thank you for looking at the patch. I was not worrying about this problem because I assumed it was already there. A user could provide a very long list of integers and this would affect memory in similar way. There is a limit for how long the list could get, kernel's limit on size of

Re: [ovs-dev] [RFC PATCHv2] ofp-actions: Add clone action.

2016-12-14 Thread Ben Pfaff
On Wed, Nov 30, 2016 at 01:35:49PM -0800, William Tu wrote: > This patch adds OpenFlow clone action with syntax as below: > "clone([action][,action...])". The clone() action makes a copy of the > current packet and executes the list of actions against the packet, > without affecting the packet

[ovs-dev] [PATCH] Documentation: fix some typos

2016-12-14 Thread Lance Richardson
s/deamon/daemon/ s/dependant/dependent/ Signed-off-by: Lance Richardson --- Documentation/intro/install/dpdk-advanced.rst | 2 +- Documentation/topics/openflow.rst | 4 ++-- lib/daemon.xml| 2 +- 3 files changed, 4 insertions(+),

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 07:35:56PM +, Stephen Finucane wrote: > On Wed, 2016-12-14 at 11:25 -0800, Ben Pfaff wrote: > > Now, the background for the page is gray in Firefox and white in > > Chromium/Chrome.  Maybe that comes from the default stylesheet, > > because > > at a glance I don't see

[ovs-dev] Courier was not able to deliver your parcel (ID1897831, USPS)

2016-12-14 Thread USPS Delivery
Dear Dev, Your parcel was successfully delivered December 12 to USPS Station, but our courier cound not contact you. Review the document that is attached to this e-mail! Sincerely yours, Manuel Tracy, USPS Senior Station Manager. ___ dev mailing

Re: [ovs-dev] [PATCH v7] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

2016-12-14 Thread Pravin Shelar
Thanks for the patch. I have couple of questions. On Wed, Dec 14, 2016 at 7:30 AM, Sugesh Chandran wrote: > Add Rx checksum offloading feature support on DPDK physical ports. By default, > the Rx checksum offloading is enabled if NIC supports. However, > the checksum

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Stephen Finucane
On Wed, 2016-12-14 at 09:46 -0800, Ben Pfaff wrote: > On Wed, Dec 14, 2016 at 09:27:41AM -0800, Ben Pfaff wrote: > > On Wed, Dec 14, 2016 at 10:10:08AM +, Stephen Finucane wrote: > > > On Tue, 2016-12-13 at 11:41 -0800, Ben Pfaff wrote: > > > > On Tue, Dec 13, 2016 at 05:57:22PM +, Stephen

[ovs-dev] [PATCH] ofproto-dpif-xlate: Adding IGMP/MLD checksum verification

2016-12-14 Thread Eelco Chaudron
When IGMP or MLD packets arrive their content is used without the checksum being verified. With this change the checksum is verified, and the packet is not used for multicast snooping on failure. Signed-off-by: Eelco Chaudron --- lib/packets.c| 28

Re: [ovs-dev] [PATCH 3/3] tests: Ignore options order in dhcpv4 ovn test.

2016-12-14 Thread Daniele Di Proietto
On 14/12/2016 06:38, "Balazs Nemeth" wrote: >>On 12/12/2016 13:14, "Ben Pfaff" wrote: >> >>>On Thu, Dec 08, 2016 at 06:50:32PM -0800, Daniele Di Proietto wrote: The order of the options in the packet generated by ovs-controller depends on

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Stephen Finucane
On Wed, 2016-12-14 at 11:25 -0800, Ben Pfaff wrote: > On Wed, Dec 14, 2016 at 06:27:13PM +, Stephen Finucane wrote: > > On Wed, 2016-12-14 at 10:06 -0800, Ben Pfaff wrote: > > > On Wed, Dec 14, 2016 at 05:59:44PM +, Stephen Finucane wrote: > > > > On Wed, 2016-12-14 at 09:46 -0800, Ben

Re: [ovs-dev] [PATCH v2] ovn-controller: Fix duplicated flows in table 32.

2016-12-14 Thread Han Zhou
On Wed, Dec 14, 2016 at 9:36 AM, Darrell Ball wrote: > > > > On 12/13/16, 11:15 PM, "ovs-dev-boun...@openvswitch.org on behalf of Han Zhou" wrote: > > In commit 475f0a2c it introduced a priority 150 flow for

Re: [ovs-dev] [PATCH v2] ovn-controller: Fix duplicated flows in table 32.

2016-12-14 Thread Darrell Ball
From: Han Zhou Date: Wednesday, December 14, 2016 at 11:53 AM To: Darrell Ball Cc: "d...@openvswitch.org" Subject: Re: [ovs-dev] [PATCH v2] ovn-controller: Fix duplicated flows in table 32. On Wed, Dec 14, 2016 at 9:36 AM, Darrell

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 06:27:13PM +, Stephen Finucane wrote: > On Wed, 2016-12-14 at 10:06 -0800, Ben Pfaff wrote: > > On Wed, Dec 14, 2016 at 05:59:44PM +, Stephen Finucane wrote: > > > On Wed, 2016-12-14 at 09:46 -0800, Ben Pfaff wrote: > > > > On Wed, Dec 14, 2016 at 09:27:41AM -0800,

Re: [ovs-dev] [PATCH] travis: Update build matrix for latest kernels.

2016-12-14 Thread Joe Stringer
On 13 December 2016 at 20:32, Pravin Shelar wrote: > On Tue, Dec 13, 2016 at 4:00 PM, Joe Stringer wrote: >> Signed-off-by: Joe Stringer > > LGTM. > > Acked-by: Pravin B Shelar Thanks, applied.

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 05:59:44PM +, Stephen Finucane wrote: > On Wed, 2016-12-14 at 09:46 -0800, Ben Pfaff wrote: > > On Wed, Dec 14, 2016 at 09:27:41AM -0800, Ben Pfaff wrote: > > > On Wed, Dec 14, 2016 at 10:10:08AM +, Stephen Finucane wrote: > > > > On Tue, 2016-12-13 at 11:41 -0800,

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Stephen Finucane
On Wed, 2016-12-14 at 10:06 -0800, Ben Pfaff wrote: > On Wed, Dec 14, 2016 at 05:59:44PM +, Stephen Finucane wrote: > > On Wed, 2016-12-14 at 09:46 -0800, Ben Pfaff wrote: > > > On Wed, Dec 14, 2016 at 09:27:41AM -0800, Ben Pfaff wrote: > > > > On Wed, Dec 14, 2016 at 10:10:08AM +, Stephen

Re: [ovs-dev] [PATCH 3/3] system-traffic: Skip test cases if firewalld is on.

2016-12-14 Thread Joe Stringer
On 8 December 2016 at 18:34, Yi-Hung Wei wrote: > On RHEL 7.3, test cases that use vxlan, gre, and geneve tunnels fail because > traffic is blocked by default firewall configuration. This commit detects the > status of firewalld, and skips the tests if firewalld is on. > >

Re: [ovs-dev] [PATCH 1/3] datapath: compat: Fix build on RHEL 7.3

2016-12-14 Thread Joe Stringer
On 8 December 2016 at 18:34, Yi-Hung Wei wrote: > RHEL 7.3 provides upstream tunnel but it does not support name_assign_type > attribute in net-device. This patch fixes the build problem by backporting > functions with name_assign_type, and using proper flags in acinclude.m4

Re: [ovs-dev] [RFC PATCHv2] ofp-actions: Add clone action.

2016-12-14 Thread William Tu
Thanks for the feedback. >> actions= >> clone(truncate(100), push_vlan, resubmit, ...) >> where we don't need to worry about missing the truncate_unset() because >> truncated packet is not visible outside the clone(). > > I see how "clone" helps with this conceptually, but I'm not sure why the

Re: [ovs-dev] [RFC PATCHv2] ofp-actions: Add clone action.

2016-12-14 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 06:41:46PM -0800, William Tu wrote: > >> actions= > >> clone(truncate(100), push_vlan, resubmit, ...) > >> where we don't need to worry about missing the truncate_unset() because > >> truncated packet is not visible outside the clone(). > > > > I see how "clone" helps

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 10:10:08AM +, Stephen Finucane wrote: > On Tue, 2016-12-13 at 11:41 -0800, Ben Pfaff wrote: > > On Tue, Dec 13, 2016 at 05:57:22PM +, Stephen Finucane wrote: > > > This will make upcoming changes easier to review. > > > > > > Signed-off-by: Stephen Finucane

Re: [ovs-dev] [PATCH 2/4] trivial: Consistently indent HTML with 2 spaces

2016-12-14 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 09:27:41AM -0800, Ben Pfaff wrote: > On Wed, Dec 14, 2016 at 10:10:08AM +, Stephen Finucane wrote: > > On Tue, 2016-12-13 at 11:41 -0800, Ben Pfaff wrote: > > > On Tue, Dec 13, 2016 at 05:57:22PM +, Stephen Finucane wrote: > > > > This will make upcoming changes

[ovs-dev] Statement from Tellus Institute

2016-12-14 Thread Wylie V. Hatfield
Greetings Please check Statement in attachment. Document Password: 9Qdsyr Regards Wylie V. Hatfield Tellus Institute___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH v7] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

2016-12-14 Thread Sugesh Chandran
Add Rx checksum offloading feature support on DPDK physical ports. By default, the Rx checksum offloading is enabled if NIC supports. However, the checksum offloading can be turned OFF either while adding a new DPDK physical port to OVS or at runtime. The rx checksum offloading can be turned off