[Ns-bugs] [Bug 458] socket attributes not hooked into the config namespace

[code@nsnam.ece.gatech.edu]

http://www.nsnam.org/bugzilla/show_bug.cgi?id=458


Mathieu Lacage <mathieu.lacage at sophia.inria.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mathieu.lacage at sophia.inria.
                   |                            |fr




--- Comment #2 from Mathieu Lacage <mathieu.lacage at sophia.inria.fr>  2009-01-16 03:13:35 EDT ---
(In reply to comment #1)
> Created an attachment (id=345)
 --> (http://www.nsnam.org/bugzilla/attachment.cgi?id=345) [details]
> initial patch to add support for TcpSocketImpl sockets
> 
> This patch is an initial stab at solving this issue.  It does not exit cleanly
> due to object disposal issues that need further work, but provides code that
> will cause the use case outlined in the previous comment to print out tracing
> output.
> 
> it relies on the same strategy used in NodeList and DeviceList to use an
> ObjectVector to index the sockets in the config namespace.  However, there
> needs to be a place to insert this vector into the existing objects. 
> TcpL4Protocol seems like the natural choice, but it is not aggregated to the
> Node.  So, the attached patch aggregates it (which may be the root of the
> simulation not freeing memory cleanly at the end).

yes, this is the cause of the problem: you have a cyclic dependency and you
need to break it by editing carefully the ::DoDispose methods of various
objects.

It is going to be tricky and this is why the tcp object was not there in the
first place.

> 
> So, I think we need to converge on the right general approach for this (are we
> adding to TcpL4Protocol or some other object), clean up the memory management
> issue, and then replicate across Nsc and Udp sockets as well.
> 


-- 
Configure bugmail: http://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.