https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-conges... File src/internet/model/tcp-congestion-ops.h (right): https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-conges... src/internet/model/tcp-congestion-ops.h:140: * \brief Trigger events/calculations on occurance cwnd event s/occurance cwnd event/occurrence of a congestion window event https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... File src/internet/model/tcp-socket-base.cc (right): https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... src/internet/model/tcp-socket-base.cc:1564: m_congestionControl->CwndEvent (m_tcb, TcpSocketState::CA_EVENT_LOSS); should UpdateCongState() also be called for each of these transition? I don't see it being called in the current codebase. https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... File src/internet/model/tcp-socket-base.h (right): https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... src/internet/model/tcp-socket-base.h:143: CA_EVENT_FRTO, /* fast recovery timeout (not used currently)*/ why not used?
Sir, I had made the previous patch based on an older Linux, I have modified the same. The two event based on ECN can be added right after ECN phase 2. Also, the event based on congestion window restart is not added, as I was unable to locate any associated event (timer based on idle time) in ns-3 currently. Thanks, Ankit https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-conges... File src/internet/model/tcp-congestion-ops.h (right): https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-conges... src/internet/model/tcp-congestion-ops.h:140: * \brief Trigger events/calculations on occurance cwnd event On 2017/03/28 17:55:08, Tom Henderson wrote: > s/occurance cwnd event/occurrence of a congestion window event Done. https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... File src/internet/model/tcp-socket-base.cc (right): https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... src/internet/model/tcp-socket-base.cc:1564: m_congestionControl->CwndEvent (m_tcb, TcpSocketState::CA_EVENT_LOSS); The Linux codebase has cong state changes associated with some of the congestion event https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... File src/internet/model/tcp-socket-base.h (right): https://codereview.appspot.com/314670043/diff/1/src/internet/model/tcp-socket... src/internet/model/tcp-socket-base.h:143: CA_EVENT_FRTO, /* fast recovery timeout (not used currently)*/ Removed based on Linux 4.11