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

Aaron Kaplan (spam-protected)
Sat Nov 10 01:01:46 CET 2007


On Nov 2, 2007, at 9:43 PM, Hannes Gredler wrote:

> hi aaron,
>
> sorry i do not think that those are related ...

  I wanted to reply to that earlies, but then wanted to research a  
bit more, but anyway...
I think I can explain why this urban legend came to be...

When I was in Berlin at the CCC I discussed that with Elektra and I  
believe even Thomas.
But seems like everybody had a wrong picture or the code changed over  
time or something like that.
I simply remember re-asking a few times if the
>
   winsize* helloInt < validity time

formula is correct. But, alas! at least we know now that the code  
says something else :))

thanks for clarifying this.

a.

> 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
> |
>

---
there's no place like 127.0.0.1







More information about the Olsr-dev mailing list