Re: [ovs-dev] DPDK: How to send packets from one OVS bridge to another OVS bridge while running in the same user space ?

2018-02-13 Thread Kapil A
Hi Jan, Thanks for the clarification. I also came across patch port as the best option between ovs Bridges. But, is it possible to use patch ports to an external OpenFlow bridge? Like one bridge is OVS, other is Lagopus bridge or LINC bridge? For my use case, I want to send packets from OVS

[ovs-dev] Detecte Fraudes Oportunamente

2018-02-13 Thread No mas Fugas de Recursos
En línea y en Vivo / Para todo su Equipo con una sola Conexión Sistemas Eficaces de Control a travéz de Auditorías Administrativas 22 de Febrero - Online en Vivo - Horario de 10:00 a 13:00 y de 15:00 a 18:00 ¡No más fugas de recursos! Detecte y prevenga fraudes de manera oportuna,

Re: [ovs-dev] [PATCH] ofp-flow: Fix return value for ofputil_decode_flow_stats_reply().

2018-02-13 Thread Ben Pfaff
Thanks for the review. I applied this to master. On Tue, Feb 13, 2018 at 03:14:56PM -0800, Yifeng Sun wrote: > Looks good to me. Thanks for the correction. > > Reviewed-by: Yifeng Sun > > On Tue, Feb 13, 2018 at 10:49 AM, Ben Pfaff wrote: > > > This

[ovs-dev] Felicidad aplicada a la productividad

2018-02-13 Thread Logre una vida más plena
Cómo ser feliz en mi trabajo Febrero 21 - webinar Interactivo Que ganarás con este webinar: Renovar tu pasión y vocación para sentirte orgulloso de tu trabajo, tu empresa, tu jefe y colaboradores. Combatir positivamente situaciones estresantes para no dejarte llevar por la desmotivación.

Re: [ovs-dev] [PATCH] ofp-flow: Fix return value for ofputil_decode_flow_stats_reply().

2018-02-13 Thread Yifeng Sun
Looks good to me. Thanks for the correction. Reviewed-by: Yifeng Sun On Tue, Feb 13, 2018 at 10:49 AM, Ben Pfaff wrote: > This function returned errno values for some errors and OFPERR_* values > for others. The callers all expected OFPERR_* values.

Re: [ovs-dev] [PATCH v2 00/11] Add upstream Linux 4.14 kernel support

2018-02-13 Thread Gregory Rose
On 2/7/2018 7:49 AM, Greg Rose wrote: This patch set includes new features from upstream Linux kernel 4.14 as well as some additional bug fixes specific to the new feature. It also updates the travis build kernel list and makes a few other small changes so OVS can compile on Linux 4.14 kernels.

Re: [ovs-dev] [PATCH 0/3] add acl reject rule support introducing icmp4 action

2018-02-13 Thread Mark Michelson
Hi Lorenzo, I had a look through the patch series and it looks good to me. However, I think there should be a test added to the testsuite to ensure that the ACL reject action works as expected. On 02/13/2018 08:43 AM, Lorenzo Bianconi wrote: Changes since RFC: - update ovn-sb.xml - squash

[ovs-dev] [PATCH] rhel: don't drop capabilities when running as root

2018-02-13 Thread Aaron Conole
Currently, regardless of which user is being set as the running user, Open vSwitch daemons on RHEL systems drop capabilities. This means the very powerful CAP_SYS_ADMIN is dropped, even when the user is 'root'. For the majority of use cases this behavior works, as the user can enable or disable

Re: [ovs-dev] [PATCH 4/4] openflow: Update list of features.

2018-02-13 Thread Ben Pfaff
On Tue, Feb 13, 2018 at 11:12:45AM -0800, William Tu wrote: > On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff wrote: > > Bundles are implemented for both OF1.3 and OF1.4+, so no need to keep it > > in the list. Packet type aware pipeline is now implemented too. > > > > Signed-off-by: Ben

[ovs-dev] HII

2018-02-13 Thread Smith
-- Hello My Dear, is nice to meet you, my names are Mary Smith, how are you doing,i hope all is well with you (mariasmitha...@gmail.com),i look forward to reading from you soon, Yours Friend, Mary Smith. ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH 1/4] Implement OF1.3 extension for OF1.4 role status feature.

2018-02-13 Thread William Tu
On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff wrote: > ONF extension pack 1 for OpenFlow 1.3 defines how to implement the OpenFlow > 1.4 "role status" message in OpenFlow 1.3. This commit implements that > feature. > > ONF-JIRA: EXT-191 > Signed-off-by: Ben Pfaff > ---

Re: [ovs-dev] [PATCH 2/4] extract-ofp-errors: Minor improvements.

2018-02-13 Thread William Tu
On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff wrote: > This removes the requirement of exactly two spaces before the error > description (now one or more is fine). It also makes an error message > clearer. > > Signed-off-by: Ben Pfaff > --- Looks good to me. Acked-by:

Re: [ovs-dev] [PATCH 3/4] ofp-errors: Add remaining OF1.4 and OF1.5 errors.

2018-02-13 Thread William Tu
On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff wrote: > Also implements the backports of relevant errors to OF1.3 as specified in > ONF extension pack 1 for OF1.3. > > ONF-JIRA: EXT-237 > ONF-JIRA: EXT-230 > ONF-JIRA: EXT-264 > Signed-off-by: Ben Pfaff > --- Looks good to

Re: [ovs-dev] [PATCH 4/4] openflow: Update list of features.

2018-02-13 Thread William Tu
On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff wrote: > Bundles are implemented for both OF1.3 and OF1.4+, so no need to keep it > in the list. Packet type aware pipeline is now implemented too. > > Signed-off-by: Ben Pfaff Looks good to me. Acked-by: William Tu

[ovs-dev] [PATCH] ofp-flow: Fix return value for ofputil_decode_flow_stats_reply().

2018-02-13 Thread Ben Pfaff
This function returned errno values for some errors and OFPERR_* values for others. The callers all expected OFPERR_* values. This fixes the problem. Signed-off-by: Ben Pfaff --- lib/ofp-flow.c | 36 1 file changed, 20 insertions(+), 16

Re: [ovs-dev] [PATCH 1/2] ofp-util: Remove prototypes for unimplemented functions.

2018-02-13 Thread Yifeng Sun
Thanks for the clean up. Looks good to me. Reviewed-by: Yifeng Sun On Fri, Feb 9, 2018 at 2:57 PM, Ben Pfaff wrote: > Signed-off-by: Ben Pfaff > --- > include/openvswitch/ofp-util.h | 12 > 1 file changed, 12 deletions(-) > >

Re: [ovs-dev] OVS DPDK: dpdk_merge pull request for master

2018-02-13 Thread Ben Pfaff
Thanks for all the pull requests. I merged all of these. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v8 0/3] dpif-netdev: Detailed PMD performance metrics and supervision

2018-02-13 Thread Jan Scheurich
Gentle reminder to review this series which unfortunately missed the 2.9 deadline. I checked and the patches still apply on today's master. So far I have received one comment from Billy https://mail.openvswitch.org/pipermail/ovs-dev/2018-January/343808.html Thanks, Jan > -Original

Re: [ovs-dev] [PATCH v1 branch-2.9] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Jan Scheurich
Thanks, Ian! This will give us time to come up with a proper solution for 2.10. Let's work on that now. /Jan > -Original Message- > From: ovs-dev-boun...@openvswitch.org > [mailto:ovs-dev-boun...@openvswitch.org] On Behalf Of Stokes, Ian > Sent: Tuesday, 13 February, 2018 16:52 > To:

Re: [ovs-dev] DPDK: How to send packets from one OVS bridge to another OVS bridge while running in the same user space ?

2018-02-13 Thread Jan Scheurich
Hi Kapil, This is what patch ports are for. They are only traversed for the first packet of a flow in the slow path. The resulting datapath flow entry collapses the processing in both bridges into a single megaflow. So there is no performance overhead compared to having a single bridge.

Re: [ovs-dev] [PATCH v1 branch-2.9] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Stokes, Ian
> > > > On 02/13/2018 10:59 AM, Ian Stokes wrote: > > > This commit manually reverts the current per port mempool model to > > > the previous shared mempool model for DPDK ports. > > > > > > OVS previously used a shared mempool model for ports with the same > > > MTU configuration. This was

[ovs-dev] DPDK: How to send packets from one OVS bridge to another OVS bridge while running in the same user space ?

2018-02-13 Thread Kapil A
Hello, In my setup, i have two DPDK ports, where dpdk0 is part of br0 and and dpdk1 is part of br1. How can i send packets from br0 to br1 in an efficient way within userspace with good performance? I came across, veth pair as one option, but i couldn't find if it will provide good performance in

[ovs-dev] OVS DPDK: dpdk_merge pull request for branch-2.6

2018-02-13 Thread Stokes, Ian
Hi Ben, The following changes since commit d893a3a47501e236bf514efabdd1d29575aa5766: gre: strip gre-tso offload flags (2018-01-26 09:43:50 -0800) are available in the git repository at: https://github.com/istokes/ovs dpdk_merge_2_6 for you to fetch changes up to

[ovs-dev] OVS DPDK: dpdk_merge pull request for branch-2.7

2018-02-13 Thread Stokes, Ian
Hi Ben, The following changes since commit eee9eec34c6131988af1ea3816fadd13af026c9d: netdev-dpdk: Fix requested MTU size validation. (2018-01-26 20:43:18 +) are available in the git repository at: https://github.com/istokes/ovs dpdk_merge_2_7 for you to fetch changes up to

[ovs-dev] OVS DPDK: dpdk_merge pull request for branch-2.8

2018-02-13 Thread Stokes, Ian
Hi Ben, The following changes since commit 995933f224d3f682541ef060a949986620df5f05: ovn: Allow DNS lookups over IPv6 (2018-02-09 09:59:50 -0800) are available in the git repository at: https://github.com/istokes/ovs dpdk_merge_2_8 for you to fetch changes up to

[ovs-dev] OVS DPDK: dpdk_merge pull request for master

2018-02-13 Thread Stokes, Ian
Hi Ben, The following changes since commit 6feddcd5417d8b57a342f4378776f3b3751ea341: poc: Introduce Proof of Concepts (Package building) (2018-02-12 10:47:03 -0800) are available in the git repository at: https://github.com/istokes/ovs dpdk_merge for you to fetch changes up to

Re: [ovs-dev] [RFC] ovn-northd: Store computed logical flow hash in SBDB.

2018-02-13 Thread Mark Michelson
Nice! Acked-by: Mark Michelson On 02/13/2018 07:57 AM, Jakub Sitnicki wrote: Each time ovn-northd processes the NBDB contents it has to compute the hash of every logical flow stored in the SBDB in order to identify ones that need adding to or deleting from SBDB

[ovs-dev] [PATCH 3/3] OVN: add acl reject support using icmp4 action

2018-02-13 Thread Lorenzo Bianconi
Whenever the acl reject rule is hit send back an ICMPv4 destination unreachable packet and do not handle reject rule as drop one. Treat TCP connections as DROP for the moment since tcp_reset{} action has not been implemented yet. Signed-off-by: Lorenzo Bianconi ---

[ovs-dev] [PATCH 2/3] OVN: add icmp4{} action support

2018-02-13 Thread Lorenzo Bianconi
icmp4 action is used to replace the IPv4 packet been processed with an ICMPv4 packet initialized based on incoming IPv4 one. Ethernet and IPv4 fields not listed are not changed: - ip.proto = 1 (ICMPv4) - ip.frag = 0 (not a fragment) - icmp4.type = 3 (destination unreachable) - icmp4.code = 1 (host

[ovs-dev] [PATCH 1/3] OVN: rename encode_nested_neighbor_actions in encode_nested_actions

2018-02-13 Thread Lorenzo Bianconi
Rename encode_nested_neighbor_actions routine in encode_nested_actions in order to have a more general name for nested actions encoder. Signed-off-by: Lorenzo Bianconi --- ovn/lib/actions.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff

[ovs-dev] [PATCH 0/3] add acl reject rule support introducing icmp4 action

2018-02-13 Thread Lorenzo Bianconi
Changes since RFC: - update ovn-sb.xml - squash patch1 and patch2 - added Jakub comments - added build_reject_acl_rules routine to add reject entries in logical flow hashmap - treat TCP connections as DROP for the moment since tcp_reset{} action has not been implemented yet - renamed

[ovs-dev] [PATCH v3] tests: Add system-dpdk-testsuite

2018-02-13 Thread Marcin Rybka
New OVS-DPDK testsuite, which can be launched via `make check-dpdk`, tests OVS using a DPDK datapath. The testsuite contains already initial tests: 1. EAL init 2. Add standard DPDK PHY port 3. Add vhost-user-client port Signed-off-by: Marcin Rybka --- Changed

Re: [ovs-dev] [PATCH 1/8] doc: Add an overview of the 'dpdk' port

2018-02-13 Thread Aaron Conole
Hi Stephen, Stephen Finucane writes: > These ports are used to allow ingress/egress from the host and are > therefore _reasonably_ important. However, there is no clear overview of > what these ports actually are or why things are done the way they are. > Start closing this

[ovs-dev] 2018 lottery

2018-02-13 Thread fnfgeewgwefg bgwergegrw
___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [RFC] ovn-northd: Store computed logical flow hash in SBDB.

2018-02-13 Thread Jakub Sitnicki
Each time ovn-northd processes the NBDB contents it has to compute the hash of every logical flow stored in the SBDB in order to identify ones that need adding to or deleting from SBDB (build_lflows()). Avoid it by storing the logical flow hash together with its fields the moment we first add it

Re: [ovs-dev] [PATCH v1 branch-2.9] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Stokes, Ian
> -Original Message- > From: Kevin Traynor [mailto:ktray...@redhat.com] > Sent: Tuesday, February 13, 2018 11:36 AM > To: Stokes, Ian ; d...@openvswitch.org > Cc: Antonio Fischetti ; Ilya Maximets > ; Jan Scheurich

Re: [ovs-dev] [PATCH v1 branch-2.9] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Kevin Traynor
On 02/13/2018 10:59 AM, Ian Stokes wrote: > This commit manually reverts the current per port mempool model to the > previous shared mempool model for DPDK ports. > > OVS previously used a shared mempool model for ports with the same MTU > configuration. This was replaced by a per port mempool

Re: [ovs-dev] [RFC Patch v1] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Stokes, Ian
> -Original Message- > From: Venkatesan Pradeep [mailto:venkatesan.prad...@ericsson.com] > Sent: Tuesday, February 13, 2018 11:01 AM > To: Kevin Traynor ; Stokes, Ian > ; d...@openvswitch.org > Cc: Ilya Maximets ;

Re: [ovs-dev] [RFC Patch v1] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Stokes, Ian
> -Original Message- > From: Kevin Traynor [mailto:ktray...@redhat.com] > Sent: Tuesday, February 13, 2018 10:35 AM > To: Stokes, Ian ; d...@openvswitch.org > Cc: antonio.fische...@gmail.com; Ilya Maximets ; > Jan Scheurich

Re: [ovs-dev] [RFC Patch v1] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Venkatesan Pradeep
Hi, I've also reviewed the code and done tests to exercise the mempool code. The behavior seems to be same as with previous versions that had the shared mempool code. Thanks, Pradeep > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- >

[ovs-dev] [PATCH v1 branch-2.9] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Ian Stokes
This commit manually reverts the current per port mempool model to the previous shared mempool model for DPDK ports. OVS previously used a shared mempool model for ports with the same MTU configuration. This was replaced by a per port mempool model to address issues flagged by users such as:

Re: [ovs-dev] [RFC Patch v1] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Kevin Traynor
On 02/13/2018 09:07 AM, Stokes, Ian wrote: > Hi all, > > Just pinging for feedback for this patch. If we are to include this for > branch 2.9 as a bug fix I think we need feedback quite soon. > > Have people had time to validate and review? > Hi Ian, As per the other thread I have reviewed

Re: [ovs-dev] [RFC Patch v1] netdev-dpdk: Reintroduce shared mempools.

2018-02-13 Thread Stokes, Ian
Hi all, Just pinging for feedback for this patch. If we are to include this for branch 2.9 as a bug fix I think we need feedback quite soon. Have people had time to validate and review? Thanks Ian > -Original Message- > From: Stokes, Ian > Sent: Thursday, February 8, 2018 11:39 AM >