[Ns-bugs] [Bug 958] WiFi uses wrong default values for MTU, FragmentationThreshold and RtsCtsThreshold

[code@nsnam.ece.gatech.edu]

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


Hendrik vom Lehn <vomlehn at cs.rwth-aachen.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #948 is|0                           |1
           obsolete|                            |




--- Comment #6 from Hendrik vom Lehn <vomlehn at cs.rwth-aachen.de>  2010-07-26 07:13:00 EDT ---
Created an attachment (id=952)
 --> (http://www.nsnam.org/bugzilla/attachment.cgi?id=952)
Revised patch dealing with all issues

All in all your patch looks good, but I would still suggest two modifications:

a1) Increase the default values for rts/cts and fragmentation threshold to at
least 1542 (1500 byte data + 8 byte LLC + 30 byte header + 4 byte FCS).
Otherwise 1500 byte Ethernet-frames will get fragmented when they are sent over
a WiFi link.
a2) A reasonable alternative would be to increase them to 2346 byte, because
this is the default value of many wireless network card and access point 
implementations.

b) Change the UIntegerChecker and SetMtu() for the Mtu as well, so that it is
not possible to set larger values than the default value.

Attached is a patch with a2+b.
Thanks for your help with this issue!

-- 
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.