[ovs-dev] [PATCH v2] doc: v2: fix bad link to dpdk advance installation guide

2016-10-20 Thread Mauricio Vasquez B
Previous fix was also wrong. Fixes: 167703d ("doc: Convert INSTALL.DPDK to rST") Signed-off-by: Mauricio Vasquez B --- INSTALL.DPDK.rst | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/INSTALL.DPDK.rst b/INSTALL.DPDK.rst index 5780909..4ca9b27 100644 --- a/INSTALL.

[ovs-dev] [PATCH] doc: fix duplicated colons in rst documents

2016-10-20 Thread Mauricio Vasquez B
Signed-off-by: Mauricio Vasquez B --- INSTALL.DPDK.rst | 148 +- INSTALL.Debian.rst| 20 +-- INSTALL.Docker.rst| 76 +++--- INSTALL.KVM.rst | 24 ++-- INSTALL.Windows.rst | 136

Re: [ovs-dev] [PATCH net-next v12 8/9] openvswitch: allow L3 netdev ports

2016-10-20 Thread Pravin Shelar
On Mon, Oct 17, 2016 at 6:02 AM, Jiri Benc wrote: > Allow ARPHRD_NONE interfaces to be added to ovs bridge. > > Based on previous versions by Lorand Jakab and Simon Horman. > > Signed-off-by: Lorand Jakab > Signed-off-by: Simon Horman > Signed-off-by: Jiri Benc Acked-by: Pravin B Shelar _

Re: [ovs-dev] [PATCH net-next v12 7/9] openvswitch: add Ethernet push and pop actions

2016-10-20 Thread Pravin Shelar
On Mon, Oct 17, 2016 at 6:02 AM, Jiri Benc wrote: > It's not allowed to push Ethernet header in front of another Ethernet > header. > > It's not allowed to pop Ethernet header if there's a vlan tag. This > preserves the invariant that L3 packet never has a vlan tag. > > Based on previous versions

Re: [ovs-dev] [PATCH net-next v12 6/9] openvswitch: netlink: support L3 packets

2016-10-20 Thread Pravin Shelar
On Mon, Oct 17, 2016 at 6:02 AM, Jiri Benc wrote: > Extend the ovs flow netlink protocol to support L3 packets. Packets without > OVS_KEY_ATTR_ETHERNET attribute specify L3 packets; for those, the > OVS_KEY_ATTR_ETHERTYPE attribute is mandatory. > > Push/pop vlan actions are only supported for Eth

Re: [ovs-dev] [PATCH net-next v12 5/9] openvswitch: add processing of L3 packets

2016-10-20 Thread Pravin Shelar
On Mon, Oct 17, 2016 at 6:02 AM, Jiri Benc wrote: > Support receiving, extracting flow key and sending of L3 packets (packets > without an Ethernet header). > > Note that even after this patch, non-Ethernet interfaces are still not > allowed to be added to bridges. Similarly, netlink interface for

Re: [ovs-dev] [PATCH] datapath: compat: tunnel: Check if device is UP.

2016-10-20 Thread Pravin Shelar
On Thu, Oct 20, 2016 at 6:49 PM, Joe Stringer wrote: > On 19 October 2016 at 11:27, Pravin B Shelar wrote: >> On upstream kernel datapath OVS make use of networking devices >> where networking stack does check if device is UP. following >> patch adds same check in case of compat tunneling impleme

Re: [ovs-dev] [PATCH] datapath: compat: tunnel: Check if device is UP.

2016-10-20 Thread Joe Stringer
On 19 October 2016 at 11:27, Pravin B Shelar wrote: > On upstream kernel datapath OVS make use of networking devices > where networking stack does check if device is UP. following > patch adds same check in case of compat tunneling implementation. > This check also fixes kernel crash in case vxlan

[ovs-dev] [PATCH V16] Function tracer to trace all function calls

2016-10-20 Thread nghosh
From: Nirapada Ghosh In some circumstances, we might need to figure out where in code, the CPU time is being spent most, so as to pinpoint the bottleneck and thereby resolve it with proper changes. Using '-finstrument-functions' flag, that can be achieved, and this patch exactly does that. There

Re: [ovs-dev] [PATCH 2/2] configure: Support compiling with Linux 4.8.

2016-10-20 Thread Jarno Rajahalme
> On Oct 20, 2016, at 2:55 PM, Pravin Shelar wrote: > > On Tue, Oct 18, 2016 at 5:01 PM, Jarno Rajahalme wrote: >> Datapath should now compile and work with Linux 4.8. >> >> Signed-off-by: Jarno Rajahalme > Acked-by: Pravin B Shelar Thanks for the review, pushed to master and branch-2.6.

Re: [ovs-dev] [PATCH 1/2] datapath: Support a fixed size of 128 distinct labels.

2016-10-20 Thread Jarno Rajahalme
> On Oct 20, 2016, at 2:55 PM, Pravin Shelar wrote: > > On Tue, Oct 18, 2016 at 5:01 PM, Jarno Rajahalme > wrote: >> Port upstream change in conntrack labels extension. Add a new >> configure macro HAVE_NF_CONN_LABELS_WITH_WORDS to detect the old >> definition. Unfortuna

Re: [ovs-dev] [PATCH 1/2] datapath: Support a fixed size of 128 distinct labels.

2016-10-20 Thread Pravin Shelar
On Tue, Oct 18, 2016 at 5:01 PM, Jarno Rajahalme wrote: > Port upstream change in conntrack labels extension. Add a new > configure macro HAVE_NF_CONN_LABELS_WITH_WORDS to detect the old > definition. Unfortunately there is no conntrack API to hide the > difference, so the this makes conntrack.c

Re: [ovs-dev] [PATCH 2/2] configure: Support compiling with Linux 4.8.

2016-10-20 Thread Pravin Shelar
On Tue, Oct 18, 2016 at 5:01 PM, Jarno Rajahalme wrote: > Datapath should now compile and work with Linux 4.8. > > Signed-off-by: Jarno Rajahalme Acked-by: Pravin B Shelar ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo

Re: [ovs-dev] ovn: Improving southbound database security

2016-10-20 Thread Han Zhou
On Thu, Oct 20, 2016 at 11:51 AM, Russell Bryant wrote: > > On Thu, Oct 20, 2016 at 1:47 PM, Ben Pfaff wrote: > > > On Thu, Oct 13, 2016 at 07:32:53PM +0530, Numan Siddique wrote: > > > > > 5) Remove support from ovn-controller updating the 'Chassis.hv_cfg' > > > column and handle the side effect

[ovs-dev] [PATCH v5] dpcls: Use 32 packet batches for lookups.

2016-10-20 Thread Bhanuprakash Bodireddy
This patch increases the number of packets processed in a batch during a lookup from 16 to 32. Processing batches of 32 packets improves performance and also one of the internal loops can be avoided here. Signed-off-by: Bhanuprakash Bodireddy Co-authored-by: Antonio Fischetti Signed-off-by: Anto

Re: [ovs-dev] [PATCH] ovn-test: Fix 'test-ovn composition' crash

2016-10-20 Thread Andy Zhou
On Wed, Oct 19, 2016 at 12:58 PM, Ben Pfaff wrote: > On Tue, Oct 11, 2016 at 12:33:22PM -0700, Andy Zhou wrote: > > Without this fix, the added test will core dump. > > > > Signed-off-by: Andy Zhou > > Thanks! > > Acked-by: Ben Pfaff > Thanks for the review. Pushed to master and branch-2.6. __

Re: [ovs-dev] ovn: Improving southbound database security

2016-10-20 Thread Russell Bryant
On Thu, Oct 20, 2016 at 1:47 PM, Ben Pfaff wrote: > On Thu, Oct 13, 2016 at 07:32:53PM +0530, Numan Siddique wrote: > > > ​5) Remove support from ovn-controller updating the 'Chassis.hv_cfg' > > column​ and handle the side effect in "--wait=hv" in ovn-nbctl. > > The ability to wait for hypervisor

Re: [ovs-dev] [PATCH net-next v12 5/9] openvswitch: add processing of L3 packets

2016-10-20 Thread Pravin Shelar
On Wed, Oct 19, 2016 at 9:52 AM, Jiri Benc wrote: > On Tue, 18 Oct 2016 22:13:45 -0700, Pravin Shelar wrote: >> On Mon, Oct 17, 2016 at 6:02 AM, Jiri Benc wrote: >> > - skb_reset_network_header(skb); >> > + skb->protocol = parse_ethertype(skb); >> >> I am not sure about changi

Re: [ovs-dev] [PATCH 03/15] doc: Convert MAINTAINERS to rST

2016-10-20 Thread Russell Bryant
On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane wrote: > Signed-off-by: Stephen Finucane > --- > MAINTAINERS.md | 29 -- > MAINTAINERS.rst | 75 ++ > +++ > Makefile.am | 2 +- > 3 files changed, 76 insertions(+), 30

Re: [ovs-dev] ovn: Improving southbound database security

2016-10-20 Thread Ben Pfaff
On Thu, Oct 13, 2016 at 07:32:53PM +0530, Numan Siddique wrote: > We may have to add one more item in the task breakdown list. Please see > below > > > On Wed, Oct 12, 2016 at 11:21 PM, Russell Bryant wrote: > > > Hello, I'm back to looking at southbound database security concerns in > > OVN.

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

2016-10-20 Thread Flavio Leitner
On Tue, Jun 14, 2016 at 10:53:36AM +0800, Xiao Liang wrote: > On Mon, Jun 6, 2016 at 9:35 AM, Ben Pfaff wrote: > > On Sun, Jun 05, 2016 at 12:57:02AM +0800, Xiao Liang wrote: > >> On Sat, Jun 4, 2016 at 1:15 AM, Ben Pfaff wrote: > >> > On Tue, May 24, 2016 at 03:07:23PM +0800, Xiao Liang wrote: >

Re: [ovs-dev] [PATCH v2] ovsdb: implement read-only remote connection type

2016-10-20 Thread Ben Pfaff
On Tue, Oct 18, 2016 at 05:16:39PM -0400, Lance Richardson wrote: > > From: "Ben Pfaff" > > To: "Lance Richardson" > > Cc: dev@openvswitch.org, az...@ovn.org, nusid...@redhat.com, > > bscha...@redhat.com, jpet...@ovn.org, russ...@ovn.org > > Sent: Monday, October 17, 2016 2:35:22 PM > > Subject:

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-20 Thread Ben Pfaff
On Thu, Oct 20, 2016 at 07:53:37AM -0400, Russell Bryant wrote: > Thanks for taking a look and for the "git am -3" tip. I'll proceed with > what you've already posted for now. "git am -3" is like magic. I don't know why it isn't the default. ___ dev ma

Re: [ovs-dev] [RFC PATCH] datapath: allow tunnels to be created with rtnetlink

2016-10-20 Thread Pravin Shelar
On Tue, Sep 20, 2016 at 7:01 AM, Thadeu Lima de Souza Cascardo wrote: > In order to use rtnetlink to create new tunnel vports, the backported > tunnels require some code that was removed from their upstream version, > mainly the necessary code for newlink and for start_xmit. > > This patch adds ba

Re: [ovs-dev] [PATCH 02/15] doc: Convert IntegrationGuide to rST

2016-10-20 Thread Russell Bryant
On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane wrote: > Signed-off-by: Stephen Finucane > --- > IntegrationGuide.md | 169 -- > IntegrationGuide.rst| 195 ++ > ++ > Makefile.am | 2 +

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-20 Thread Russell Bryant
On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane wrote: > Signed-off-by: Stephen Finucane > --- > CONTRIBUTING.md | 4 +- > CodingStyle.md | 578 - > CodingStyle.rst | 642 ++ > + > Makefi

Re: [ovs-dev] [PATCH] fedora: do not restart the service on a pkg upgrade

2016-10-20 Thread Russell Bryant
On Thu, Oct 20, 2016 at 8:48 AM, Flavio Leitner wrote: > On Wed, Oct 19, 2016 at 03:26:10PM -0400, Russell Bryant wrote: > > On Tue, Oct 18, 2016 at 1:04 PM, Flavio Leitner wrote: > > > > > There is no reliable way to restore the previous networking > > > state after a service restart. Many thin

Re: [ovs-dev] [PATCH] DocumentationStyle: Clarify what documentation is

2016-10-20 Thread Russell Bryant
On Thu, Oct 20, 2016 at 3:19 AM, Stephen Finucane wrote: > We're going to convert all docs to reStructuredText, so the "legacy > documents" warning is going to become outdated rather quickly. Simply > remove it. > > Signed-off-by: Stephen Finucane > --- > DocumentationStyle.rst | 10 --

Re: [ovs-dev] [PATCH RFC 2/2] netdev-dpdk: Add vHost User PMD

2016-10-20 Thread Loftus, Ciara
> > Sorry, CC mail-list > > On 20.10.2016 17:20, Ilya Maximets wrote: > > Not a complete review. > > Few comments inline. Thanks for the review. Comments inline. Thanks, Ciara > > > > Best regards, Ilya Maximets. > > > > On 14.10.2016 17:08, Ciara Loftus wrote: > >> The vHost PMD allows vHost

[ovs-dev] [PATCH RFC v3 0/2] Add DPDK 16.11 & vHost PMD Support

2016-10-20 Thread Ciara Loftus
Previous: http://openvswitch.org/pipermail/dev/2016-October/080667.html This RFC patchset adds support for DPDK 16.11 (rc1) and the vHost PMD. Thanks to a lack of API changes that affect OVS, the DPDK 16.11 support patch requires just one minimal code change and is mostly documentation updates.

[ovs-dev] [PATCH RFC v3 1/2] netdev-dpdk: Add support for DPDK 16.11

2016-10-20 Thread Ciara Loftus
This commit announces support for DPDK 16.11. Compaitibilty with DPDK v16.07 is not broken yet thanks to only minor code changes being needed for the upgrade. This may change soon however with the addition of subsequent commits that integrate 16.11-only features. Signed-off-by: Ciara Loftus ---

[ovs-dev] [PATCH RFC v3 2/2] netdev-dpdk: Add vHost User PMD

2016-10-20 Thread Ciara Loftus
The vHost PMD allows vHost User ports to be controlled by the librte_ether API, like physical 'dpdk' ports and IVSHM 'dpdkr' ports. This commit integrates this PMD into OVS and removes direct calls to the librte_vhost DPDK library. This commit requires DPDK v16.11 functionality that isn't availabl

Re: [ovs-dev] [PATCH RFC 2/2] netdev-dpdk: Add vHost User PMD

2016-10-20 Thread Ilya Maximets
Sorry, CC mail-list On 20.10.2016 17:20, Ilya Maximets wrote: > Not a complete review. > Few comments inline. > > Best regards, Ilya Maximets. > > On 14.10.2016 17:08, Ciara Loftus wrote: >> The vHost PMD allows vHost User ports to be controlled by the >> librte_ether API, like physical 'dpdk' p

Re: [ovs-dev] packets getting dropped in vhostuser port

2016-10-20 Thread Kapil Adhikesavalu
Hi I was moving to latest version of ovs + dpdk for jumbo frame support. Will i be able to get jumbo frame support in ovs 2.5.9 + dpdk 16.04 ? i dont want a way to configure mtu as in 2.6, just a hard coded jumbo frame size should do. i see testpmd has a way to configure mtu on dpdk(in 16.04), So

Re: [ovs-dev] [PATCH] fedora: do not restart the service on a pkg upgrade

2016-10-20 Thread Flavio Leitner
On Wed, Oct 19, 2016 at 03:26:10PM -0400, Russell Bryant wrote: > On Tue, Oct 18, 2016 at 1:04 PM, Flavio Leitner wrote: > > > There is no reliable way to restore the previous networking > > state after a service restart. Many things like firewall > > configuration, traffic shaping, stacked devic

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-20 Thread Russell Bryant
On Thu, Oct 20, 2016 at 3:07 AM, Stephen Finucane wrote: > On 2016-10-20 01:25, Russell Bryant wrote: > >> On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane >> wrote: >> >> Signed-off-by: Stephen Finucane >>> --- >>> CONTRIBUTING.md | 4 +- >>> CodingStyle.md | 578 >>> -

[ovs-dev] [PATCH] DocumentationStyle: Clarify what documentation is

2016-10-20 Thread Stephen Finucane
We're going to convert all docs to reStructuredText, so the "legacy documents" warning is going to become outdated rather quickly. Simply remove it. Signed-off-by: Stephen Finucane --- DocumentationStyle.rst | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/Documenta

[ovs-dev] (no subject)

2016-10-20 Thread Returned mail
The original message was received at Thu, 20 Oct 2016 12:43:34 +0530 from [14.249.198.248] - The following addresses had permanent fatal errors - dev@openvswitch.org ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/list

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-20 Thread Stephen Finucane
On 2016-10-20 01:25, Russell Bryant wrote: On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane wrote: Signed-off-by: Stephen Finucane --- CONTRIBUTING.md | 4 +- CodingStyle.md | 578 - CodingStyle.rst | 642 +++