Bug 739 - OLSR: Strange HTime value in HELLO messages
: OLSR: Strange HTime value in HELLO messages
Status: RESOLVED FIXED
: ns-3
routing
: ns-3-dev
: All All
: P3 trivial
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2009-11-12 07:45 EDT by
Modified: 2009-11-25 10:04 EDT (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2009-11-12 07:45:27 EDT
I see (in wireshark) that HTime field in OLSR hello messages is 3 times larger
than actual hello emission interval. I see this in the code as well:
src/routing/olsr/olsr-routing-protocol.cc:1451

  hello.SetHTime (Scalar (3) * m_helloInterval);

Meanwhile RFC 3626 Page 27 says:

      HTime

         This field specifies the HELLO emission interval used by the
         node on this particular interface, i.e., the time before the
         transmission of the next HELLO (this information may be used in
         advanced link sensing, see section 14).  The HELLO emission
         interval is represented by its mantissa (four highest bits of
         Htime field) and by its exponent (four lowest bits of Htime
         field).  In other words:

              HELLO emission interval=C*(1+a/16)*2^b  [in seconds]

Please explain (in the comment inplace) or fix.
------- Comment #1 From 2009-11-24 06:49:51 EDT -------
changeset:   5771:a8245e39c546
------- Comment #2 From 2009-11-25 03:16:57 EDT -------
(In reply to comment #1)
> changeset:   5771:a8245e39c546

You should update reference traces for PCAP based regression tests:

$ ./test.py -s routing-olsr-regression
FAIL: TestSuite routing-olsr-regression
0 of 1 tests passed (0 passed, 0 skipped, 1 failed, 0 crashed, 0 valgrind
errors)

To do so you have to:

$ cd src/routing/olsr/test/
$ grep "WRITE_VECTORS = false" *.cc
hello-regression-test.cc:const bool WRITE_VECTORS = false;
tc-regression-test.cc:const bool WRITE_VECTORS = false;

Then change these "= false" to "= true" and rerun test suite. Reference PCAP
traces will be updated. Revert changes in *.cc files and commit *.pcap ones.
------- Comment #3 From 2009-11-25 05:33:53 EDT -------
I thought I did commit new pcap files?...
------- Comment #4 From 2009-11-25 10:04:29 EDT -------
(In reply to comment #3)
> I thought I did commit new pcap files?...

 that's true ;)