Bug 250 – Tcp breaks if you set the delackcount > 2

Bug 250 - Tcp breaks if you set the delackcount > 2

Summary:

Tcp breaks if you set the delackcount > 2

Status:	RESOLVED FIXED

Product:	ns-3
Component:	internet-stack
Version:	ns-3-dev
Platform:	All All

Importance:	P1 normal
Assigned To:	Rajib Bhattacharjea

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-07-09 14:27 EDT by Rajib Bhattacharjea
Modified:	2008-12-06 03:13 EDT (History)

Attachments
simple fix (547 bytes, patch) 2008-12-01 10:42 EDT, Rajib Bhattacharjea	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Rajib Bhattacharjea 2008-07-09 14:27:13 EDT

If you try to tell TCP to ack each third(4th, 5th, etc) packet instead of each
second one, the code has a logical error preventing its correct operation. 
This is TcpSocket attribute DelAckCount.

Offending code is lines 1129-1139 in
http://code.nsnam.org/ns-3-dev/file/266c6a334762/src/internet-stack/tcp-socket-impl.cc

------- Comment #1 From Rajib Bhattacharjea 2008-12-01 10:42:17 EDT -------

Created an attachment (id=321) [details]
simple fix

Test case:
./waf --run="tcp-large-transfer --ns3::TcpSocket::DelAckCount=3"

Appears to work as proposed.

------- Comment #2 From Rajib Bhattacharjea 2008-12-01 10:47:12 EDT -------

Patch looks good, reassigning to release manager.

------- Comment #3 From Rajib Bhattacharjea 2008-12-03 11:20:24 EDT -------

Reassigned to myself.