[Olsr-dev] [PATCH 2/4] smart gateway: do not stop choosing a new (better) gateway

[Ferry Huberts]

On 12/08/2011 08:20 AM, Henning Rogge wrote:
> On 12/06/2011 10:14 AM, Ferry Huberts wrote:
>> From: Ferry Huberts<(spam-protected)>
>>
>> Signed-off-by: Ferry Huberts<(spam-protected)>
>> ---
>>   src/gateway_default_handler.c |    6 ------
>>   1 files changed, 0 insertions(+), 6 deletions(-)
>>
>> diff --git a/src/gateway_default_handler.c
>> b/src/gateway_default_handler.c
>> index a18dba8..50986b3 100644
>> --- a/src/gateway_default_handler.c
>> +++ b/src/gateway_default_handler.c
>> @@ -76,12 +76,6 @@ static void gw_default_choose_gateway(void) {
>>     if (inet_ipv6&&  !dual) {
>>       olsr_set_inet_gateway(&inet_ipv6->originator, false, true, false);
>>     }
>> -
>> -  /* finished ? */
>> -  if (gw_def_finished_ipv4&&  gw_def_finished_ipv6) {
>> -    olsr_stop_timer(gw_def_timer);
>> -    gw_def_timer = NULL;
>> -  }
>>   }
>>
>>   /* timer for lazy gateway selection */
> 
> Have to look at this more closely. The idea behind the "default" gateway
> selector is to select a gateway and stick with it until its gone. Maybe
> "gone" is a bit too late, but if we start choosing another gateway too
> early, we might get gateway-flapping again.
> 
> Henning
> 
> 
> 


Henning,
do you have comments about patches 3 and 4? if not, I'll push them.

I'm working on respinning patch 2 to use a configuration setting
(instead of the define you suggested). It's default will be to keep
current behaviour (do not continue looking for better gateway)

-- 
Ferry Huberts