Re: [ovs-dev] [PATCH v1] OVN: Fix the ovn-controller 100% usage issue with put_mac_bindings

2019-05-16 Thread Ankur Sharma
Hi Numan, Excellent find. I missed the POP while iterating the hmap. Submitted a V2, added relevant tags as well. Please take a look. Regards, Ankur From: Numan Siddique Sent: Thursday, May 16, 2019 11:31 AM To: Ankur Sharma Cc: Han Zhou ; ovs-dev@openvswitch.org Subject: Re: [ovs-dev]

[ovs-dev] [PATCH v2] OVN: Fix the ovn-controller 100% usage issue with put_mac_bindings

2019-05-16 Thread Ankur Sharma
ISSUE: a. As soon as entries get added to put_mac_bindings in pinctrl.c, ovn-controller CPU consumption reached 100%. b. This happens because in wait_put_mac_bindings, if !hmap_is_empty(_mac_bindings) succeeds and calls poll_immediat_wake(). ovn-controller.log:

Re: [ovs-dev] [PATCH] ovn-controller: Fix parsing of OVN tunnel IDs

2019-05-16 Thread venugopal iyer via dev
Hi, Dumitru: On Thursday, May 16, 2019, 2:13:41 AM CDT, Dumitru Ceara wrote: On Wed, May 15, 2019 at 10:30 PM venugopal iyer wrote: > > Thanks for fixing this! a comment below: > > On Monday, May 13, 2019, 6:31:28 AM CDT, Dumitru Ceara > wrote: > > > Add tunnel-id creation and

[ovs-dev] Bien-être au travail

2019-05-16 Thread Augustin
Challenge du personnel, dynamique de groupe : par le sport ! Avec la technologie connectée OuiMoveUp, transformez l'activité en points ! Message commercial destiné aux entreprises et collectivités Si vous ne souhaitez plus recevoir nos messages, merci de répondre à ce mail

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Han Zhou
On Thu, May 16, 2019 at 11:40 AM Numan Siddique wrote: > > > On Thu, May 16, 2019 at 10:20 PM Han Zhou wrote: > >> >> >> On Wed, May 15, 2019 at 11:59 PM Numan Siddique >> wrote: >> >>> >>> >>> On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: >>> On Wed, May 15, 2019 at 11:55 AM

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Numan Siddique
On Thu, May 16, 2019 at 10:20 PM Han Zhou wrote: > > > On Wed, May 15, 2019 at 11:59 PM Numan Siddique > wrote: > >> >> >> On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: >> >>> >>> >>> On Wed, May 15, 2019 at 11:55 AM Numan Siddique >>> wrote: >>> On Thu, May 16, 2019 at 12:10

Re: [ovs-dev] [PATCH v1] OVN: Fix the ovn-controller 100% usage issue with put_mac_bindings

2019-05-16 Thread Numan Siddique
On Thu, May 16, 2019 at 12:26 AM Numan Siddique wrote: > > > On Thu, May 16, 2019 at 12:20 AM Ankur Sharma > wrote: > >> Hi Numan, >> >> Just confirming, anything else pending from my side? >> or given that patch has been Acked, hence someone will apply it to master? >> > > I don't think you

Re: [ovs-dev] [PATCH 2/2] doc: Add CirrusCI status badge to README.

2019-05-16 Thread Aaron Conole
Ilya Maximets writes: > Badge for CirrusCI just like for other CI systems. > > Signed-off-by: Ilya Maximets > --- > README.rst | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/README.rst b/README.rst > index 54d06d04b..e06ddf267 100644 > --- a/README.rst > +++ b/README.rst > @@ -10,6

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Han Zhou
On Wed, May 15, 2019 at 11:59 PM Numan Siddique wrote: > > > On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: > >> >> >> On Wed, May 15, 2019 at 11:55 AM Numan Siddique >> wrote: >> >>> >>> >>> On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote: >>> On Wed, May 15, 2019 at 11:36 AM

[ovs-dev] [RFC 2/3] OVN: introduce send_event() action

2019-05-16 Thread Lorenzo Bianconi
Add send_event() ovn action in order to allow ovs-vswitchd to report CMS related events. This commit introduces a new event, empty_lb_backends. This event is raised if a received packet is destined for a load balancer VIP that has no configured backend destinations. For this event, the event info

[ovs-dev] [RFC 3/3] OVN: use send_event action to report 'empty_lb_rule' events

2019-05-16 Thread Lorenzo Bianconi
Add northd logical flows in order to reports that the controller received an IP packet for LB rule witn no backends. This configuration is used by OpenShift to spin up a idle POD Signed-off-by: Mark Michelson Co-authored-by: Mark Michelson Signed-off-by: Lorenzo Bianconi ---

[ovs-dev] [RFC 1/3] OVN: introduce Controller_Event table

2019-05-16 Thread Lorenzo Bianconi
Add Controller_Event table to OVN SBDB in order to report CMS related event. Introduce event_table hashmap array and controller_event related structures to ovn-controller in order to track pending events forwarded by ovs-vswitchd. Moreover integrate event_table hashmap array with event_table

[ovs-dev] [RFC 0/3] OVN: add Controller Events

2019-05-16 Thread Lorenzo Bianconi
There are situations where arrival of certain types of traffic into OVS does not warrant a "typical" action, such as output to a specific port or dropping. Rather, the decision about what to do needs to be left to a CMS. The series here introduces a new table, Controller_Event, for this purpose.

Re: [ovs-dev] [PATCH 1/2] doc: Fix cropped what-is-ovs page.

2019-05-16 Thread Stephen Finucane
On Thu, 2019-05-16 at 16:40 +0300, Ilya Maximets wrote: > Despite of comments in both files no-one ever adjusted start/end-line > in 'what-is-ovs' document. As a result, current document contains > truncated "tools" section. > > Let's replace start/end-line with start-after/end-before which

[ovs-dev] [PATCH 2/2] doc: Add CirrusCI status badge to README.

2019-05-16 Thread Ilya Maximets
Badge for CirrusCI just like for other CI systems. Signed-off-by: Ilya Maximets --- README.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.rst b/README.rst index 54d06d04b..e06ddf267 100644 --- a/README.rst +++ b/README.rst @@ -10,6 +10,8 @@ Open vSwitch :target:

[ovs-dev] [PATCH 1/2] doc: Fix cropped what-is-ovs page.

2019-05-16 Thread Ilya Maximets
Despite of comments in both files no-one ever adjusted start/end-line in 'what-is-ovs' document. As a result, current document contains truncated "tools" section. Let's replace start/end-line with start-after/end-before which requires less attention. Additionally, 'make docs-check' will fail if

[ovs-dev] [PATCH 0/2] doc: what-is-ovs fix + CirrusCI badge.

2019-05-16 Thread Ilya Maximets
Ilya Maximets (2): doc: Fix cropped what-is-ovs page. doc: Add CirrusCI status badge to README. Documentation/intro/what-is-ovs.rst | 8 README.rst | 7 +-- 2 files changed, 9 insertions(+), 6 deletions(-) -- 2.17.1

[ovs-dev] [PATCH] OVN: run local logical flows first in S_ROUTER_OUT_SNAT table

2019-05-16 Thread Lorenzo Bianconi
Run local logical flows first if the gw router port is scheduled on the local chassis in order to properly manage snat traffic Tested-by: Eran Kuris Signed-off-by: Lorenzo Bianconi --- ovn/northd/ovn-northd.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

Re: [ovs-dev] [PATCH v4 4/4] netdev-offload: Rename offload providers.

2019-05-16 Thread Aaron Conole
Ilya Maximets writes: > On 15.05.2019 23:32, Aaron Conole wrote: >> Aaron Conole writes: >> >>> Ilya Maximets writes: >>> On 15.05.2019 19:13, Ilya Maximets wrote: > Hi Aaron. > > Robot complains about lines not touched by this patch. > This is strange. I

[ovs-dev] Array Networks B2B Contact List

2019-05-16 Thread lilly . martin
Hi, Would you be interested in *Targeted Array Networks B2B Contact List*? We are happy to provide the database across globe, specifically North America, EMEA, APAC and Latin America. *We also have other users like:* Cloud flare, VMware, Incapsula, Fastly, Zscaler, Distil Networks and many

Re: [ovs-dev] [PATCH] cirrus: Disable coredumps on FreeBSD.

2019-05-16 Thread Ilya Maximets
On 15.05.2019 21:15, Ben Pfaff wrote: > On Wed, May 15, 2019 at 11:14:15AM +0300, Ilya Maximets wrote: >> On 15.05.2019 8:51, Ben Pfaff wrote: >>> On Tue, May 14, 2019 at 07:23:59PM +0300, Ilya Maximets wrote: Some tests uses 'kill -SEGV' to simulate segfault of a child process. This

[ovs-dev] Toxicologist Contact Info

2019-05-16 Thread Jody Kohn
Hi, Would you be interested in Toxicologists Contact’s list? Such as: Audiologists, Nurses, Diagnostic Radiologists, Oncology Nurses, Urologist, Internist's, Dental Laboratories, Dermatologist’s, Rheumatologist’, ENT Specialists, Family Practitioners, Pulmonologists, Orthodontists,

Re: [ovs-dev] [RFC V2] netdev-rte-offloads: HW offload virtio-forwarder

2019-05-16 Thread Roni Bar Yanai
>-Original Message- >From: Ilya Maximets >Sent: Wednesday, May 15, 2019 4:37 PM >To: Roni Bar Yanai ; ovs-dev@openvswitch.org; Ian >Stokes ; Kevin Traynor >Cc: Eyal Lavee ; Oz Shlomo ; Eli >Britstein ; Rony Efraim ; Asaf >Penso >Subject: Re: [RFC V2] netdev-rte-offloads: HW offload

Re: [ovs-dev] [PATCHv8] netdev-afxdp: add new netdev type for AF_XDP.

2019-05-16 Thread Ilya Maximets
On 16.05.2019 2:27, William Tu wrote: > Hi Ilya, > > Thanks for your feedback. > > On Mon, May 13, 2019 at 10:48 AM Ilya Maximets wrote: >> >> On 10.05.2019 2:54, William Tu wrote: >>> The patch introduces experimental AF_XDP support for OVS netdev. >>> AF_XDP, Address Family of the eXpress

Re: [ovs-dev] [PATCH] ovn-controller: Fix parsing of OVN tunnel IDs

2019-05-16 Thread Dumitru Ceara
On Wed, May 15, 2019 at 10:30 PM venugopal iyer wrote: > > Thanks for fixing this! a comment below: > > On Monday, May 13, 2019, 6:31:28 AM CDT, Dumitru Ceara > wrote: > > > Add tunnel-id creation and parsing functions in encaps.h. > > Reported-at: https://bugzilla.redhat.com/1708131 >

Re: [ovs-dev] [PATCH v2] ovn: Add a new logical switch port type - 'virtual'

2019-05-16 Thread Numan Siddique
On Thu, May 16, 2019 at 1:12 AM Han Zhou wrote: > > > On Wed, May 15, 2019 at 11:55 AM Numan Siddique > wrote: > >> >> >> On Thu, May 16, 2019 at 12:10 AM Han Zhou wrote: >> >>> >>> >>> On Wed, May 15, 2019 at 11:36 AM wrote: >>> > >>> > From: Numan Siddique >>> > >>> > This new type is

Re: [ovs-dev] [PATCHv8] netdev-afxdp: add new netdev type for AF_XDP.

2019-05-16 Thread Ilya Maximets
On 16.05.2019 2:20, William Tu wrote: >>> + OVS AF_XDP netdev is using the userspace datapath, the same datapath >>> + as used by OVS-DPDK. So it requires --disable-system for ovs-vswitchd >>> + and datapath_type=netdev when adding a new bridge. >> >> I don't think that '--disable-system'

Re: [ovs-dev] [PATCH v4 4/4] netdev-offload: Rename offload providers.

2019-05-16 Thread Ilya Maximets
On 15.05.2019 23:32, Aaron Conole wrote: > Aaron Conole writes: > >> Ilya Maximets writes: >> >>> On 15.05.2019 19:13, Ilya Maximets wrote: Hi Aaron. Robot complains about lines not touched by this patch. This is strange. >>> >>> I suspect that it's because of file renaming.