Bug 77 - Packet uid increments on shallow copies
Packet uid increments on shallow copies
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: core
pre-release
All All
: P3 minor
Assigned To: ns-bugs
: bug
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-09-17 01:45 UTC by Tom Henderson
Modified: 2007-10-13 18:15 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Henderson 2007-09-17 01:45:22 UTC
the packet uid increments when a Packet constructor is called, leading to "missing" packets when observing packet flow in the ascii traces.  It seems that this uid incrementing should be tied to deep and not shallow copies.

(reported also by George two weeks ago; can be observed in any of the ascii trace files)
Comment 1 Mathieu Lacage 2007-09-21 01:37:17 UTC
It is not tied to shallow copies, it is tied to new packet construction and this is what is expected. There is clearly more going on here than what you are reporting.
Comment 2 Tom Henderson 2007-09-21 12:07:35 UTC
At the very least, we need to better specify what the semantics are of Uid.  We may also want to consider whether or how we include it in the ascii traces, to avoid confusion.  There is some history to deal with, in that these uids have been used in ns-2 and other simulators to track notional packets.  A user may not understand from the current doxygen that the constructor may also be called internally in the stack for various reasons.

The problem is that code segments may use new packet construction as a mechanism to do things within a model, but the black box behavior is such that new "missing" packets are absent from the traces.  e.g. point-to-point-net-device.cc:209

However, it is not trivial to try to make this UID correspond to notions of actual packet counts, however.  What should the packet uids be when packet is sent broadcast over shared media, or when fragmentation occurs?

Perhaps the best way to handle this for now is to better document somewhere the fact that pkt-uid is an internal uid and cannot be counted on to provide an accurate counter of how many "simulated packets" of a particular protocol are in the system, and that if a user wants that, he or she may choose to look at IP id or other counters at other layers. 
Comment 3 Tom Henderson 2007-10-13 18:15:09 UTC
clarified in doxygen with changeset 1782.  Will add something to the manual when the packet chapter is revised.