On Fri, May 29, 2015 at 11:34 AM, Ben Pfaff <b...@nicira.com> wrote:
> 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 <az...@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. */
Fixed.
>
> Acked-by: Ben Pfaff <b...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
