Bug 2406 - Poor 802.11g performance in ad-hoc mode
Poor 802.11g performance in ad-hoc mode
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: wifi
ns-3.25
All All
: P5 normal
Assigned To: sebastien.deronne
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-05-08 11:49 UTC by sebastien.deronne
Modified: 2016-05-13 05:32 UTC (History)
2 users (show)

See Also:


Attachments
output of ideal-wifi-manager-example (22.18 KB, application/postscript)
2016-05-08 11:49 UTC, sebastien.deronne
Details
output of ideal-wifi-manager-example (22.18 KB, application/postscript)
2016-05-08 11:51 UTC, sebastien.deronne
Details
proposed patch to fix (3.74 KB, patch)
2016-05-08 11:56 UTC, sebastien.deronne
Details | Diff
output of ideal-wifi-manager-example with the fix (22.18 KB, application/postscript)
2016-05-08 11:58 UTC, sebastien.deronne
Details

Note You need to log in before you can comment on or make changes to this bug.
Description sebastien.deronne 2016-05-08 11:49:27 UTC
Created attachment 2418 [details]
output of ideal-wifi-manager-example

It has been noticed that 802.11g performance is rather poor in our latest release.
Investigation show this only occur in ad-hoc mode. 

Running ideal-wifi-manager-example with 802.11g standard shows we barely reach 10 Mbit/s for higher SNRs. I provide output graph in attachment.

Commands to reproduce:
./waf --run "ideal-wifi-manager-example --standard=802.11g"
gnuplot ideal-wifi-manager-example-802.11g.plt
Comment 1 sebastien.deronne 2016-05-08 11:51:26 UTC
Created attachment 2419 [details]
output of ideal-wifi-manager-example

wrong attachment provided
Comment 2 sebastien.deronne 2016-05-08 11:56:24 UTC
Created attachment 2420 [details]
proposed patch to fix

The problem is that we forgot to add the Basic modes when calling AddAllSupportedModes.
Comment 3 sebastien.deronne 2016-05-08 11:58:03 UTC
Created attachment 2421 [details]
output of ideal-wifi-manager-example with the fix

When applying the patch, we can now reach around 20 Mbit/s at the higher SNRs.
Comment 4 Tom Henderson 2016-05-09 12:33:24 UTC
+1; thanks for fixing
Comment 5 sebastien.deronne 2016-05-13 05:32:54 UTC
pushed in changeset 12116:09f5230a2fe3