On Fri, May 29, 2015 at 11:34 AM, Ben Pfaff <[email protected]> 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 <[email protected]>
>>
>> ---
>> 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 <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
