On Tue, Nov 25, 2014 at 8:19 PM, YAMAMOTO Takashi
<yamam...@valinux.co.jp> wrote:
>> Since dpif registering for routing table at initialization
>> there is no need to unregister it. Following patch removes
>> support for turning routing table notifications on and off.
>> Due to this change OVS always listens for these
>> notifications.
>
> probably rename route_table_register to route_table_init or such?
>
ok.

>> diff --git a/lib/route-table-bsd.c b/lib/route-table-bsd.c
>> index 9ebfaa3..0d631b4 100644
>> --- a/lib/route-table-bsd.c
>> +++ b/lib/route-table-bsd.c
>> @@ -118,18 +118,7 @@ route_table_get_change_seq(void)
>>  void
>>  route_table_register(void)
>>  {
>> -    if (!register_count)
>> -    {
>> -        pid = getpid();
>> -    }
>> -
>> -    register_count++;
>> -}
>> -
>> -void
>> -route_table_unregister(void)
>> -{
>> -    register_count--;
>> +    pid = getpid();
>>  }
>>
>
> please remove the register_count variable.
>
ok, I send v2.

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

Reply via email to