[Olsr-dev] osx compiles + question re: [Olsr-cvs] olsrd-current CHANGELOG, 1.104, 1.105

[Hannes Gredler]

hi aaron,

sorry i do not think that those are related ...

hello validity time is just a dead timer that
removes the link advertisment once expired.

the winsize is just the length of a loss vector
that determines resolution of the neighbor
loss advertisments for link sensing.
the winsize is unrelated to timing, it just
operates on good/missing hellos irrespective
of timers. check link_set.c for implementation
details.

/hannes

On Fri, Nov 02, 2007 at 12:07:06PM +0100, Aaron Kaplan wrote:
| hi!
| 
| compiles on os x and runs! no regression issues.
| but I was wondering if the default lq config file had a bug:
| 
| HelloInterval    5.0
| HelloValidityTime 200.0
| 
| LinkQualityWinSize    100
| 
| winsize * interval = 500 but validity time is 200. that means if I  
| did not make a mistake now that after 200 units the hello packet is  
| not valid anymore but we check for 500 units... -> the LQ gets rated  
| worse than it acutally is.
| Imho we could add a check here or calculate the params automatically.
| 
| hm? any mistake in above's logic?
| 
| a.
| (seems like a deja vue for me)
| 
| Begin forwarded message:
| 
| > From: Bernd Petrovitsch <(spam-protected)>
| > Date: November 2, 2007 11:57:32 AM GMT+01:00
| > To: (spam-protected)
| > Subject: [Olsr-cvs] olsrd-current CHANGELOG,1.104,1.105
| >
| > Update of /cvsroot/olsrd/olsrd-current
| > In directory sc8-pr-cvs3.sourceforge.net:/tmp/cvs-serv10077
| >
| > Modified Files:
| > 	CHANGELOG
| > Log Message:
| > * completed changelog message
| >
| > Index: CHANGELOG
| > ===================================================================
| > RCS file: /cvsroot/olsrd/olsrd-current/CHANGELOG,v
| > retrieving revision 1.104
| > retrieving revision 1.105
| > diff -C2 -d -r1.104 -r1.105
| > *** CHANGELOG	2 Nov 2007 10:16:39 -0000	1.104
| > --- CHANGELOG	2 Nov 2007 10:57:30 -0000	1.105
| > ***************
| > *** 17,20 ****
| > --- 17,22 ----
| >       So we fall back to the old 's/re/sub/' and quote all '/' and  
| > '.' correctly
| >       (which was the main reason to use '|').
| > +     And while I'm at it: Remove the useless $(CCACHE) from the $ 
| > (CC) for
| > +     dependency generation and beautify (read: "$(strip)") $ 
| > (CPPFLAGS).
| >     * OpenBSD needs an explicit "#include <sys/types.h>" before
| >       "#include <netinet/in.h>".
| >
| >
| > -- 
| > Olsr-cvs mailing list
| > (spam-protected)
| > http://lists.olsr.org/mailman/listinfo/olsr-cvs
| >
| 
| ---
| there's no place like 127.0.0.1
| 
| 
| 
| 
| -- 
| Olsr-dev mailing list
| (spam-protected)
| http://lists.olsr.org/mailman/listinfo/olsr-dev
|