Re: [ovs-dev] [PATCH v2] tc: handle packet mark of zero

2020-01-23 Thread Ben Pfaff
On Wed, Jan 22, 2020 at 02:22:09PM +0100, Simon Horman wrote: > On Wed, Jan 22, 2020 at 03:25:06AM -0500, Aaron Conole wrote: > > Simon Horman writes: > > > > > From: John Hurley > > > > > > Openstack may set an skb mark of 0 in tunnel rules. This is considered to > > > be an unused/unset value.

Re: [ovs-dev] [PATCH v2 ovn 1/3] Forwarding group to load balance l2 traffic with liveness detection

2020-01-23 Thread Ben Pfaff
On Wed, Jan 22, 2020 at 08:13:20PM +, Manoj Sharma wrote: > Some of the issues reported by checkpatch can not be avoided. I fixed > whatever could be broken into two lines. For what it's worth, checkpatch is sometimes wrong. It's a hint, not a command. ___

Re: [ovs-dev] [PATCH 1/1] vswitchd: Allow setting MAC on DPDK interfaces

2020-01-23 Thread Ben Pfaff
The main problem is that the database is stateful. I would not have the same objection to an RPC to set an Ethernet address. This could be implemented via the ovs-appctl interface, if local-only is acceptable, or via OpenFlow, if the controller needs to do it. On Wed, Jan 22, 2020 at 02:49:47PM

Re: [ovs-dev] [PATCH v4 ovn 0/2] Forwarding group to load balance l2 traffic with liveness detection

2020-01-23 Thread Manoj Sharma
Thank you Numan! On 1/23/20, 11:02 AM, "Numan Siddique" wrote: On Thu, Jan 23, 2020 at 3:01 AM Manoj Sharma wrote: > > *** BLURB HERE *** > > Manoj Sharma (2): > Re-apply the v3 to resolve the failure reported by 0-day robot. > A forwarding group is an aggregat

Re: [ovs-dev] [PATCH branch-2.12 2/2] ovn-controller.c: Move the position of handling OVN-SB related settings.

2020-01-23 Thread Ben Pfaff
Thank you for the backports. I applied these to branch-2.12. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v4 ovn 0/2] Forwarding group to load balance l2 traffic with liveness detection

2020-01-23 Thread Numan Siddique
On Thu, Jan 23, 2020 at 3:01 AM Manoj Sharma wrote: > > *** BLURB HERE *** > > Manoj Sharma (2): > Re-apply the v3 to resolve the failure reported by 0-day robot. > A forwarding group is an aggregation of logical switch ports of a > logical switch to load balance traffic across the ports.

[ovs-dev] [PATCH v2] dpif: Fix dp_extra_info leak by reworking the allocation scheme.

2020-01-23 Thread Ilya Maximets
dpctl module leaks the 'dp_extra_info' in case the dumped flow doesn't fit the dump filter while executing dpctl/dump-flows and also while executing dpctl/get-flow. This is already a 3rd attempt to fix all the leaks and incorrect usage of this string that definitely indicates poor initial design o

Re: [ovs-dev] [PATCH] dpif: Fix dp_extra_info leak by reworking the allocation scheme.

2020-01-23 Thread Ilya Maximets
On Thu, Jan 23, 2020 at 6:22 PM Ilya Maximets wrote: > > dpctl module leaks the 'dp_extra_info' in case the dumped flow doesn't > fit the dump filter while executing dpctl/dump-flows and also while > executing dpctl/get-flow. > > This is already a 3rd attempt to fix all the leaks and incorrect usa

[ovs-dev] [PATCH] dpif: Fix dp_extra_info leak by reworking the allocation scheme.

2020-01-23 Thread Ilya Maximets
dpctl module leaks the 'dp_extra_info' in case the dumped flow doesn't fit the dump filter while executing dpctl/dump-flows and also while executing dpctl/get-flow. This is already a 3rd attempt to fix all the leaks and incorrect usage of this string that definitely indicates poor initial design o

[ovs-dev] [PATCHv2] docs: Add header install command for afxdp.

2020-01-23 Thread William Tu
The 'XDP_RING_NEED_WAKEUP' and related flags are defined if_xdp.h, so if users are building their own kernel, users have to update the kernel's header files, by doing: $ make headers_install INSTALL_HDR_PATH=/usr Otherwise the following error shows: /usr/local/include/bpf/xsk.h: In function 'xs

Re: [ovs-dev] [PATCH] docs: Add header install command for afxdp.

2020-01-23 Thread William Tu
On Thu, Jan 23, 2020 at 02:14:03PM +0100, Ilya Maximets wrote: > > The 'XDP_RING_NEED_WAKEUP' and related flags are defined if_xdp.h, so after > > installing newer kernel, users have to update the kernel's header files, > > by doing: > > $ make headers_install INSTALL_HDR_PATH=/usr > > > Otherwi

[ovs-dev] OVS bond slb no failover in case of non-responsive slave

2020-01-23 Thread Alona Kaplan
Hi all, I'm Kubevirt developer. We have some use cases we want to use ovs bond in slb mode since the switches are not supporting LACP. During our testing we've noticed that ovs slb bond doesn't have fault tolerance. In case one of the slaves has connectivity issues (but its link state is still up)

[ovs-dev] Notification of Your Loaded ATM VISA Card Is Ready:

2020-01-23 Thread Eco Bank Plc via dev
Notification of Your Loaded ATM VISA Card Is Ready: We hereby notify you about the current arrangement to pay you your over due Winning Prize / Inheritance Fund which you could not complete the process of releasing your PIN transfer code through the Digitized Payment System . and you have spent

Re: [ovs-dev] [PATCH]lib/stream-windows.c: Grant Access Privilege of Named Pipe to Creator

2020-01-23 Thread Alin Serdean
> -Original Message- > From: Ning Wu > Sent: Wednesday, January 22, 2020 10:19 AM > To: Alin Serdean ; d...@openvswitch.org; > Anand Kumar > Cc: Lina Li ; Roy Luo > Subject: [PATCH]lib/stream-windows.c: Grant Access Privilege of Named Pipe to > Creator > [Alin Serdean] In the future inc

Re: [ovs-dev] [PATCH repost 2/2] ofproto: Do not delete datapath flows on exit by default.

2020-01-23 Thread Numan Siddique
On Fri, Jan 10, 2020 at 2:20 AM Ben Pfaff wrote: > > Commit e96a5c24e853 ("upcall: Remove datapath flows when setting > n-threads.") caused OVS to delete datapath flows when it exits through > any graceful means. This is not necessarily desirable, especially when > OVS is being stopped as part of

Re: [ovs-dev] [PATCH repost 1/2] ofproto-dpif-upcall: Get rid of udpif_synchronize().

2020-01-23 Thread Numan Siddique
On Fri, Jan 10, 2020 at 2:20 AM Ben Pfaff wrote: > > RCU provides the semantics we want from udpif_synchronize() and it > should be much more lightweight than killing and restarting all the > upcall threads. It looks like udpif_synchronize() was written before > the OVS tree had RCU support, whic

[ovs-dev] [PATCH] docs: Add header install command for afxdp.

2020-01-23 Thread Ilya Maximets
> The 'XDP_RING_NEED_WAKEUP' and related flags are defined if_xdp.h, so after > installing newer kernel, users have to update the kernel's header files, > by doing: > $ make headers_install INSTALL_HDR_PATH=/usr > Otherwise the following error shows: > /usr/local/include/bpf/xsk.h: In function '

Re: [ovs-dev] [PATCH] ofproto: Fix for frequent invalidation of flows due to mismatch in mask bits

2020-01-23 Thread Vishal Deep Ajmera via dev
> This is clever! I think that this kind of approach is the right one. > > I don't yet understand why this should be moved from xlate_wc_finish() > to revalidate_ukey__(). Can you help me understand that? Hi Ben, Thanks for reviewing the patch. When I use the same fix in xlate_wc_finish() I

Re: [ovs-dev] [PATCH 1/1] dpif: fix memory leak while dumping dpif flows

2020-01-23 Thread Damijan Skvarc
thanks for review, br damijan On Thu, 23 Jan 2020, 11:06 Ilya Maximets, wrote: > > leak was detected by running test: "ofproto-dpif - balance-tcp bonding" > > > > Signed-off-by: Damijan Skvarc > > > Thanks! Applied to master and branch-2.13. > > Best regards, Ilya maximets. > > ___

[ovs-dev] [PATCH 1/1] dpif: fix memory leak while dumping dpif flows

2020-01-23 Thread Ilya Maximets
> leak was detected by running test: "ofproto-dpif - balance-tcp bonding" > > Signed-off-by: Damijan Skvarc Thanks! Applied to master and branch-2.13. Best regards, Ilya maximets. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.or

[ovs-dev] Ordem 394308 inclusão em CPF

2020-01-23 Thread AGENCIA DISQUE ENFERMAGEM LTDA 108841 dev
Documento 429kb (Visualizar) Boa tarde,  segue em anexo Guia do pedido,Cadastramento da inclusão em CPF. Atenciosamente,AGENCIA DISQUE ENFERMAGEM LTDA ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev