Bug 2110 - vanet-routing-compare with no routing still sending RREQ
vanet-routing-compare with no routing still sending RREQ
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: wave module
ns-3-dev
All All
: P5 minor
Assigned To: ns-bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-05-04 08:37 UTC by Scott Carpenter
Modified: 2015-05-05 18:26 UTC (History)
3 users (show)

See Also:


Attachments
patch (1.58 KB, patch)
2015-05-05 04:28 UTC, Tommaso Pecorella
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Scott Carpenter 2015-05-04 08:37:07 UTC
When executing the vanet-routing-compare script with internet routing disabled (i.e., --run "src/wave/examples/vanet-routing-compare --protocol=0", there are still RREQ msgs being sent.

I think this is a bug.  (Or maybe it is a feature).  For IP-routing, the internet stack is installed onto every node.  However, with routing disabled by --protocol=0, no data is then submitted for routing through the network.  However, this doesn't seem to be preventing the initial route request messages from being sent (RREQ).  However, I think they are unanswered (i.e., no RREP).  Nonetheless, this is unnecessary overhead.

Furthermore, the use of IP addresses is used to determine the sender node for every received packet.  An alternative would be to instead use the MAC address to resolve sender / receiver, but this will require some refactoring of the code.
Comment 1 Tom Henderson 2015-05-04 15:04:27 UTC
assigning to aodv component (seems to be more of an aodv issue)
Comment 2 Tommaso Pecorella 2015-05-05 04:28:03 UTC
Created attachment 2036 [details]
patch

The discussion is here:
https://groups.google.com/forum/#!topic/ns-3-users/0CAxbtmfRfo

and it's a bug in the example, not related to AODV.
Comment 3 Scott Carpenter 2015-05-05 08:43:08 UTC
Patch tested locally, and works.  Am suggesting okay to push.
Comment 4 Tom Henderson 2015-05-05 09:19:33 UTC
I will slip this in today.
Comment 5 Tom Henderson 2015-05-05 18:26:09 UTC
pushed in changeset 11369:1eb9df0ecfd5