"dev" <[email protected]> wrote on 07/08/2016 02:38:06 AM:
> From: Gurucharan Shetty <[email protected]> > To: [email protected] > Date: 07/08/2016 12:36 PM > Subject: [ovs-dev] [PATCH 2/2] ovn-controller: Change strategy for > gateway conntrack zone allocation. > Sent by: "dev" <[email protected]> > > Commit 263064aeaa31e7 (Convert binding_run to incremental processing.) > changed the way patched_datapaths were handled. Previously we would > destroy the datastructure in every run and re-create it fresh. The new > way causes problems with the way conntrack zones are allocated as now > we can have stale port_binding entries causing segmentation faults. > > With this commit, we simply don't depend on port_binding records in > conntrack zone allocation and instead store the UUID as a string in > the patch_datapath datastructure. > > Fixes: 263064aeaa31e7 ("Convert binding_run to incremental processing.") > Signed-off-by: Gurucharan Shetty <[email protected]> > --- I like what this is doing, but I'd like it more if it had a test case that would fail without this patch, but pass with it, so that we don't regress... Ryan _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
