Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Fix truncate and native tunnnel

2017-10-20 Thread Guru Shetty
On 13 October 2017 at 21:33, William Tu wrote: > Previous commit a67b337dc281 breaks the truncate and native > tunnel testcase by removing the truncate flag. The patch fixes > it by putting it back. Reproduce the error by: > > make check-system-userspace TESTSUITEFLAGS='17'

Re: [ovs-dev] [PATCH v2] rhel.rst: Add python-sphinx as a dependency.

2017-10-20 Thread Guru Shetty
Thanks you Yi-Hung and Aaron. I applied this with your suggestions. On 20 October 2017 at 13:12, Yi-Hung Wei wrote: > Thanks for the patch. LGTM. > > Acked-by: Yi-Hung Wei > > BTW, I think we can also update the docs in fedora.rst. as the following.

Re: [ovs-dev] [PATCH v2] rhel.rst: Add python-sphinx as a dependency.

2017-10-20 Thread Yi-Hung Wei
Thanks for the patch. LGTM. Acked-by: Yi-Hung Wei BTW, I think we can also update the docs in fedora.rst. as the following. --- a/Documentation/intro/install/fedora.rst +++ b/Documentation/intro/install/fedora.rst @@ -43,7 +43,7 @@ in the :doc:`general`. Specific packages

[ovs-dev] [PATCH v2] rhel.rst: Add python-sphinx as a dependency.

2017-10-20 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- Documentation/intro/install/rhel.rst | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Documentation/intro/install/rhel.rst b/Documentation/intro/install/rhel.rst index 86c5cf3..b68cb7d 100644 ---

Re: [ovs-dev] [PATCH] ovs-save: Handle different 'ip addr show' output.

2017-10-20 Thread Guru Shetty
On 20 October 2017 at 11:25, Yi-Hung Wei wrote: > Acked-by: Yi-Hung Wei > Thanks. I applied this to master and 2.8 > > On Wed, Oct 18, 2017 at 10:45 PM, Gurucharan Shetty wrote: > > On RHEL 7.4 (with iproute-3.10.0-87), a DHCP

[ovs-dev] Bug#878249: Please package >= 2.7.0

2017-10-20 Thread Thomas Goirand
Hi Ben, I don't mind becoming a co-maintainer, though there's a few things that need to change to facilitate packaging. First of all, I'd like to switch to a Git packaging workflow, using git-buildpackage. The debian folder upstream here should be removed:

Re: [ovs-dev] [PATCH] ovs-save: Handle different 'ip addr show' output.

2017-10-20 Thread Yi-Hung Wei
Acked-by: Yi-Hung Wei On Wed, Oct 18, 2017 at 10:45 PM, Gurucharan Shetty wrote: > On RHEL 7.4 (with iproute-3.10.0-87), a DHCP provided > ipv4 address has the "dynamic" keyword set. For e.g > "ip addr show breth0 | grep inet" shows: > > inet

[ovs-dev] [PATCH 2/3] system-common-macros: Check for ct_clear action in datapath

2017-10-20 Thread Eric Garver
New macro OVS_CHECK_CT_CLEAR() to check if ct_clear action is supported by the datapath. Signed-off-by: Eric Garver --- tests/system-common-macros.at | 4 1 file changed, 4 insertions(+) diff --git a/tests/system-common-macros.at b/tests/system-common-macros.at index

[ovs-dev] [PATCH 3/3] system-traffic: Add conntrack floating IP test

2017-10-20 Thread Eric Garver
This test cases uses floating IP (FIP) addresses for each endpoint. If the destination is a FIP, the packet will undergo a transformation of the form (dst=FIP, src=non-FIP) --> (dst=non-FIP, src=FIP) before egress. Otherwise the packet is untouched. This exercises the ct_clear action in the

[ovs-dev] [PATCH 0/3] Add dpif support for ct_clear action

2017-10-20 Thread Eric Garver
This series adds dpif support for OVS_ACTION_ATTR_CT_CLEAR. Previously the ct_clear action was a userspace only operation, but it has use cases in the dpif as well. Namely changing a packet's tuple after a ct() lookup has occurred. Kernel support has already be accepted upstream: b8226962b1c4

[ovs-dev] [PATCH] rhel.rst: Add python-sphinx as a dependency.

2017-10-20 Thread Gurucharan Shetty
Signed-off-by: Gurucharan Shetty --- Documentation/intro/install/rhel.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/intro/install/rhel.rst b/Documentation/intro/install/rhel.rst index 86c5cf3..aff6ccf 100644 ---

Re: [ovs-dev] [PATCH v2] datapath-windows: Update OvsIPv4TunnelKey flags in geneve decap.

2017-10-20 Thread Alin Serdean
Thanks Anand and Sai. I applied this on master and on branch-2.8. Should we apply this on branch-2.7 also? Thanks, Alin. > -Original Message- > From: Sairam Venugopal [mailto:vsai...@vmware.com] > Sent: Friday, October 20, 2017 12:15 AM > To: Anand Kumar ;

Re: [ovs-dev] [PATCH v2] datapath-windows: Update OvsIPv4TunnelKey flags in geneve decap.

2017-10-20 Thread aserdean
Acked-by: Alin Gabriel Serdean > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Anand Kumar > Sent: Thursday, October 19, 2017 11:26 PM > To: d...@openvswitch.org > Subject: [ovs-dev] [PATCH v2]

[ovs-dev] [PATCH v2] Documentation: Add document describing RBAC

2017-10-20 Thread Mark Michelson
Role based access control is a relatively new addition to OVS/OVN, and aside from the database documentation in ovn-sb(5), there is not much explaining what RBAC is, how to use it, and the available roles. This document remedies that situation. It is hopeful that any new roles added will be added

[ovs-dev] [PATCH] netdev-dpdk: replace uint8_t with dpdk_port_t

2017-10-20 Thread Mark Kavanagh
netdev_dpdk_detach() declares a 'port_id' variable, of type uint8_t. This variable should instead be of type dpdk_port_t. Fixes: bb37956ac ("netdev-dpdk: Use uint8_t for port_id.") CC: Ilya Maximets Signed-off-by: Mark Kavanagh ---

[ovs-dev] [PATCH] netdev-dpdk: replace uint8_t with dpdk_port_t

2017-10-20 Thread Mark Kavanagh
netdev_dpdk_detach() declares a 'port_id' variable, of type uint8_t. This variable should instead be of type dpdk_port_t. Fixes: bb37956ac ("netdev-dpdk: Use uint8_t for port_id.") CC: Ilya Maximets Signed-off-by: Mark Kavanagh ---

Re: [ovs-dev] [PATCH net-next v12] openvswitch: enable NSH support

2017-10-20 Thread Yang, Yi
On Fri, Oct 20, 2017 at 04:05:35PM +0800, Jiri Benc wrote: > On Fri, 20 Oct 2017 05:53:12 +0800, Yang, Yi wrote: > > For push_nsh, my typical use scinario is push_nsh then set then output > > to vxlangpe port. > > Okay. Then I just need to do the below change against v12. diff --git

Re: [ovs-dev] [PATCH 3/3] dpif-netdev: Calculate rxq cycles prior to rxq_cycle_sort calls.

2017-10-20 Thread Kevin Traynor
Ping again. This was requested to make the sort function behave in a more standard manner. I don't think there's anything controversial in it. thanks, Kevin. On 09/22/2017 08:21 PM, Darrell Ball wrote: > Are there any other comments for this patch? > > > On 8/30/17, 11:01 AM, "Darrell Ball"

Re: [ovs-dev] [PATCH 1/3] dpif-netdev: Rename rxq_interval.

2017-10-20 Thread Kevin Traynor
Ping again. This is a simple variable rename that was requested. On 09/22/2017 08:22 PM, Darrell Ball wrote: > Are there any other comments? > > > > On 8/30/17, 10:49 AM, "Darrell Ball" wrote: > > Thanks Kevin > > Naming is hard. > The name looks a bit more

Re: [ovs-dev] OVS-DPDK public meeting

2017-10-20 Thread Kevin Traynor
18th October 2017 Attendees: Aaron, Ian, Ciara, Sugesh, Billy, Thomas, Olga, Johann, Simon, Mark, Zoltan, Kevin (may have missed some) === GENERAL === - OVS-DPDK branch - Ian will be collating the DPDK patches on his branch here: https://github.com/istokes/ovs/tree/dpdk_merge - Ian will

Re: [ovs-dev] [PATCH v8 1/6] netdev-dpdk: fix management of pre-existing mempools.

2017-10-20 Thread Kavanagh, Mark B
>From: Fischetti, Antonio >Sent: Thursday, October 19, 2017 5:54 PM >To: d...@openvswitch.org >Cc: Kavanagh, Mark B ; Aaron Conole >; Darrell Ball ; Fischetti, Antonio > >Subject: [PATCH v8 1/6]

Re: [ovs-dev] [PATCH v8 6/6] netdev-dpdk: Rename dpdk_mp_put as dpdk_mp_free.

2017-10-20 Thread Fischetti, Antonio
> -Original Message- > From: Kavanagh, Mark B > Sent: Friday, October 20, 2017 9:44 AM > To: Fischetti, Antonio ; d...@openvswitch.org > Cc: Darrell Ball ; Loftus, Ciara ; > Kevin Traynor ; Aaron

Re: [ovs-dev] [PATCH v8 4/6] netdev-dpdk: manage failure in mempool name creation.

2017-10-20 Thread Kavanagh, Mark B
>From: Fischetti, Antonio >Sent: Friday, October 20, 2017 10:38 AM >To: Kavanagh, Mark B ; d...@openvswitch.org >Cc: Darrell Ball ; Loftus, Ciara ; >Kevin Traynor ; Aaron Conole >Subject:

Re: [ovs-dev] [PATCH v8 4/6] netdev-dpdk: manage failure in mempool name creation.

2017-10-20 Thread Fischetti, Antonio
> -Original Message- > From: Kavanagh, Mark B > Sent: Friday, October 20, 2017 10:28 AM > To: Fischetti, Antonio ; d...@openvswitch.org > Cc: Darrell Ball ; Loftus, Ciara ; > Kevin Traynor ; Aaron

Re: [ovs-dev] [PATCH v8 4/6] netdev-dpdk: manage failure in mempool name creation.

2017-10-20 Thread Kavanagh, Mark B
>From: Fischetti, Antonio >Sent: Thursday, October 19, 2017 5:54 PM >To: d...@openvswitch.org >Cc: Kavanagh, Mark B ; Darrell Ball >; Loftus, Ciara ; Kevin Traynor >; Aaron Conole ;

Re: [ovs-dev] [PATCH RFC v2] netdev-dpdk: Allow specification of index for PCI devices

2017-10-20 Thread Kevin Traynor
On 10/19/2017 07:27 PM, Loftus, Ciara wrote: >> >> On 10/17/2017 11:48 AM, Ciara Loftus wrote: >>> Some NICs have only one PCI address associated with multiple ports. This >>> patch extends the dpdk-devargs option's format to cater for such >>> devices. Whereas before only one of N ports

Re: [ovs-dev] [PATCH v8 2/6] Fix mempool names to reflect socket id.

2017-10-20 Thread Kavanagh, Mark B
>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] >On Behalf Of Kavanagh, Mark B >Sent: Friday, October 20, 2017 9:40 AM >To: Fischetti, Antonio ; d...@openvswitch.org >Subject: Re: [ovs-dev] [PATCH v8 2/6] Fix mempool names to reflect

Re: [ovs-dev] [PATCH v8 0/6] netdev-dpdk: Fix mempool management and other cleanup.

2017-10-20 Thread Kevin Traynor
On 10/19/2017 07:00 PM, Fischetti, Antonio wrote: > > >> -Original Message- >> From: Stokes, Ian >> Sent: Thursday, October 19, 2017 6:22 PM >> To: Fischetti, Antonio ; d...@openvswitch.org >> Subject: RE: [ovs-dev] [PATCH v8 0/6] netdev-dpdk: Fix mempool

Re: [ovs-dev] [PATCH v8 6/6] netdev-dpdk: Rename dpdk_mp_put as dpdk_mp_free.

2017-10-20 Thread Kevin Traynor
On 10/19/2017 05:54 PM, antonio.fische...@intel.com wrote: > For readability purposes dpdk_mp_put is renamed as dpdk_mp_free. > > CC: Mark B Kavanagh > CC: Darrell Ball > CC: Ciara Loftus > CC: Kevin Traynor

Re: [ovs-dev] [PATCH v8 4/6] netdev-dpdk: manage failure in mempool name creation.

2017-10-20 Thread Kevin Traynor
On 10/19/2017 05:54 PM, antonio.fische...@intel.com wrote: > In case a mempool name could not be generated log a message > and return a null mempool pointer to the caller. > > CC: Mark B Kavanagh > CC: Darrell Ball > CC: Ciara Loftus

Re: [ovs-dev] [PATCH v8 6/6] netdev-dpdk: Rename dpdk_mp_put as dpdk_mp_free.

2017-10-20 Thread Kavanagh, Mark B
>From: Fischetti, Antonio >Sent: Thursday, October 19, 2017 5:54 PM >To: d...@openvswitch.org >Cc: Kavanagh, Mark B ; Darrell Ball >; Loftus, Ciara ; Kevin Traynor >; Aaron Conole ;

Re: [ovs-dev] [PATCH v8 2/6] Fix mempool names to reflect socket id.

2017-10-20 Thread Kavanagh, Mark B
>From: Fischetti, Antonio >Sent: Thursday, October 19, 2017 5:54 PM >To: d...@openvswitch.org >Cc: Kavanagh, Mark B ; Aaron Conole >; Fischetti, Antonio >Subject: [PATCH v8 2/6] Fix mempool names to reflect socket id. >

[ovs-dev] Adding new action

2017-10-20 Thread Prabhu
Hello all, I want to implement a custom action which should handle the below process defined as FLOW actions: Buffer size : 1000 Table:0 Rule 1 ->  IN_PORT  match MAC Address1 and ACTION:send_to_buffer_0, go_to_table1 (buffer should log its backlog) Rule 2 ->  IN_PORT match MAC Address2 and

Re: [ovs-dev] [PATCH net-next v12] openvswitch: enable NSH support

2017-10-20 Thread Jiri Benc
On Fri, 20 Oct 2017 05:53:12 +0800, Yang, Yi wrote: > For push_nsh, my typical use scinario is push_nsh then set then output > to vxlangpe port. Okay. Jiri ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev