Re: [ovs-dev] [PATCH] ovn-controller: Remove local_datapaths_by_uuid.

2016-07-18 Thread Russell Bryant
On Mon, Jul 18, 2016 at 4:22 PM, Ryan Moats wrote: > Russell Bryant wrote on 07/18/2016 03:01:41 PM: > > > From: Russell Bryant > > To: dev@openvswitch.org > > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > > Date: 07/18/2016 03:01 PM > > Subject: [PATCH] ovn-controller: Remove local_datapat

Re: [ovs-dev] [PATCH] ovn-controller: Remove local_datapaths_by_uuid.

2016-07-18 Thread Ryan Moats
Russell Bryant wrote on 07/18/2016 03:01:41 PM: > From: Russell Bryant > To: dev@openvswitch.org > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant > Date: 07/18/2016 03:01 PM > Subject: [PATCH] ovn-controller: Remove local_datapaths_by_uuid. > > binding.c included a static local_datapaths_by_uui

[ovs-dev] [PATCH] ovn-controller: Remove local_datapaths_by_uuid.

2016-07-18 Thread Russell Bryant
binding.c included a static local_datapaths_by_uuid but it was not used for anything. In fact, the hash node used when inserting into this hmap is overwritten in another code path for a different hmap. Fixes: 263064aeaa31 ("Convert binding_run to incremental processing.") Signed-off-by: Russell B