[ovs-dev] [ovsdb speedup v3 12/19] ovsdb-monitor: stores jsonrpc-monitor in a linked-list

Ben Pfaff blp at nicira.com
Fri May 29 18:34:40 UTC 2015


On Thu, Apr 09, 2015 at 06:40:21PM -0700, Andy Zhou wrote:
> Currently, each ovsdb-monitor points to a single jsonrpc_monitor object.
> This means there is 1:1 relationship between them.
> 
> In case multiple jsonrpc-monitors need to monitor the same tables and
> the columns within them, then can share a single ovsdb-monitor, so the
> updates only needs to be maintained once.
> 
> This patch, with a few following patches,  will allow for N:1 mapping
> between jsonrpc-monitor and ovsdb-monitor.
> 
> Maintaining jsonrpc-monitor pointers in a linked-list is essential
> in allowing N:1 mapping. The ovsdb-monitor life cycle
> is now reference counted. An empty list means zero references.
> 
> Signed-off-by: Andy Zhou <azhou at nicira.com>
> 
> ---
> v1->v2: style fixes
> v2->v3: no change

In ovsdb_monitor_remove_jsonrpc_monitor(), s/Destory/Destroy/ here:
+            /* Destory ovsdb monitor if this is the last user.  */

Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list