Codel, fq_codel in ns3

src/internet/model/tcp-socket-base.cc

 /* -*- Mode:C++; c-file-style:"gnu"; indent-tabs-mode:nil; -*- */
 /*
  * Copyright (c) 2007 Georgia Tech Research Corporation
  * Copyright (c) 2010 Adrian Sai-wah Tam
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 as
  * published by the Free Software Foundation;
  *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  *
  * Author: Adrian Sai-wah Tam <adrian.sw.tam@gmail.com>
  */
 
 #define NS_LOG_APPEND_CONTEXT \
   if (m_node) { std::clog << Simulator::Now ().GetSeconds () << " [node " << m_node->GetId () << "] "; }
 
+#include <cstdlib>
 #include "ns3/abort.h"
 #include "ns3/node.h"
 #include "ns3/inet-socket-address.h"
 #include "ns3/inet6-socket-address.h"
 #include "ns3/log.h"
 #include "ns3/ipv4.h"
 #include "ns3/ipv6.h"
 #include "ns3/ipv4-interface-address.h"
 #include "ns3/ipv4-route.h"
 #include "ns3/ipv6-route.h"
@@ skipping 63 matching lines @@
 }
 
 TcpSocketBase::TcpSocketBase (void)
   : m_dupAckCount (0),
     m_delAckCount (0),
     m_endPoint (0),
     m_endPoint6 (0),
     m_node (0),
     m_tcp (0),
     m_rtt (0),
-    m_nextTxSequence (0),
+    m_nextTxSequence (std::rand ()),

[Tom Henderson, 2012/08/17 21:19:31]

We would rather use the ns-3 random numbers here.
Ptr<UniformRandomVariable> m_rand;
m_rand->GetInteger (min, max);

[Tommaso Pecorella, 2012/11/10 22:56:39]

beside Tom's comment, this change is not strictly related to CoDel. My
suggestion is to open a bug and propose a change in main ns-3 code.

[Tom Henderson, 2012/11/12 19:29:34]

Here is the bug/feature request:
https://www.nsnam.org/bugzilla/show_bug.cgi?id=1529

     // Change this for non-zero initial sequence number
     m_highTxMark (0),
     m_rxBuffer (0),
     m_txBuffer (0),
     m_state (CLOSED),
     m_errno (ERROR_NOTERROR),
     m_closeNotified (false),
     m_closeOnEmpty (false),
     m_shutdownSend (false),
     m_shutdownRecv (false),
@@ skipping 2164 matching lines @@
 {
 }
 
 /** Placeholder function for future extension that changes the TCP header */
 void
 TcpSocketBase::AddOptions (TcpHeader&)
 {
 }
 
 } // namespace ns3