On Sat, Jul 16, 2016 at 10:31 PM, Ryan Moats <[email protected]> wrote:
> "dev" <[email protected]> wrote on 07/14/2016 02:47:52 PM: > > > From: Russell Bryant <[email protected]> > > To: [email protected] > > Date: 07/14/2016 02:48 PM > > Subject: [ovs-dev] [PATCH v2] ovn-controller: Clean up bindings handling. > > Sent by: "dev" <[email protected]> > > > > Remove the global set of logical port IDs called 'all_lports'. This is > > no longer used for anything after conntrack ID assignment was moved out > > of binding.c. > > > > Remove the global smap of logical port IDs to ovsrec_interface records. > > We can't persist references to these records, as we may be holding > > references to freed memory. Instead, replace it with a new global sset > > of logical port IDs called 'local_ids'. This is used to track when > > interfaces have been added or removed. We also build a temporary > > shash of logical port IDs to ovs interfaces used for fast lookup > > of the right interface as needed. > > > > Found by inspection. > > > > Signed-off-by: Russell Bryant <[email protected]> > > Though I see Darrell's points, I'm comfortable enough with this to say... > > Acked-by: Ryan Moats <[email protected]> > > and we can rename things in a followon patch. > Thanks, Ryan. I applied this to master. I'm happy to take a follow-up renaming things. -- Russell Bryant _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
