[Ns-bugs] [Bug 1274] New: Patches to TCP

[code@nsnam.ece.gatech.edu]

https://www.nsnam.org/bugzilla/show_bug.cgi?id=1274

           Summary: Patches to TCP
           Product: ns-3
           Version: pre-release
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P5
         Component: tcp
        AssignedTo: adrian.sw.tam at gmail.com
        ReportedBy: adrian.sw.tam at gmail.com
                CC: ns-bugs at isi.edu
   Estimated Hours: 0.0


Created attachment 1250
  --> https://www.nsnam.org/bugzilla/attachment.cgi?id=1250
Patch to tcp-socket-base.cc and tcp-socket-base.h

The current TCP implementation (TcpSocketBase) has some minor problems, which
causing several bugs filed (e.g. bug 1227). This patch closes several problems
found, listed below:

1. Make connection count (m_cnCount) a separate variable than the number of
connection retries (m_cnRetries), so that the number of connection retries can
be cloned without affecting the number of connections to be made in the cloned
sockets

2. Instead of ignoring the out-of-order packets (i.e. seq number < expected),
now sending an ACK for each received to catch the case that the peer may have
lost the ACK. This closes bug 1227.

3. There was a case that the m_highTxMark was mistaken as m_nextTxSequence

4. The endpoint allocation/deallocation is rewritten so that the endpoint now
remembers the correct peer's address. So the RecvFrom() call can use the data
in endpoint to return the peer's address

5. The socket was not closed nicely as the endpoint is not released upon close.
Now fixed.

6. TCP can now move from TIME_WAIT state to CLOSED state after 2*MSL, i.e. 4
minutes. Because of this, a new event is created upon moving to TIME_WAIT
state.

7. Added Raj's email to the preamble, as TcpSocketBase was modified from
TcpSocketImpl.

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