"dev" <dev-boun...@openvswitch.org> wrote on 29/07/2016 11:46:09 AM:

> From: Darrell Ball <dlu...@gmail.com>
> To: dlu...@gmail.com, d...@openvswitch.com, b...@ovn.org
> Date: 29/07/2016 11:46 AM
> Subject: [ovs-dev] [patch_v1] ovn: Add datapaths of interest filtering 
(RFC).
> Sent by: "dev" <dev-boun...@openvswitch.org>
> 
> This patch adds datapaths of interest support where only datapaths of
> local interest are monitored by the ovn-controller ovsdb client.  The
> idea is to do a flood fill in ovn-controller of datapath associations
> calculated by northd. A new column is added to the SB database
> datapath_binding table - related_datapaths to facilitate this.  This
> allows monitoring to adapt quickly with a single new monitor setting for
> all datapaths of interest locally.
> 
> Presently, the code does not cleanup logical port monitor contexts
> as some refactoring may need to be done given several recent
> changes in this area.
> 
> Signed-off-by: Darrell Ball <dlu...@gmail.com>

First I would mention that this code is based on the patch: "ovn: 
implementation of conditional monitor usage" 
http://openvswitch.org/pipermail/dev/2016-July/076532.html
This new patch introduce an optimization to the base patch by exposing a 
new column in the datapath_binding table called related_datapaths. 
I think that we should agree on applying the basic conditional monitor 
usage to OVN first before we start to add any optimization to it. Then if 
we will see that we need to cut the latency of datapaths and ports in the 
ovn-controller, we can easily add this functionality to the code.

BTW, in this patch I do not see any calls for add_clause_false so this 
code will retrieve the whole SB DB in the first iteration.

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to