Bugzilla – Bug 2654
Rraa wifi manager cannot be used with 802.11b
Last modified: 2017-03-01 16:21:01 UTC
Testing 802.11b with Rraa ... assert failed. cond="false", msg="Thresholds for an unknown mode are asked (DsssRate11Mbps)", file=../src/wifi/model/rraa-wifi-manager.cc, line=501 libc++abi.dylib: terminating Either this manager should be extended for 802.11b, either we should trigger immediately an assert that rraa does not support 802.11b, similarly to what we have for HT/VHT/HE. I guess same issue is also encountered in 802.11g.
Info needed from Matias and/or Tom obout the preferred option.
(In reply to sebastien.deronne from comment #1) > Info needed from Matias and/or Tom obout the preferred option. I can extend it. I'll do it for next week.
(In reply to Matías Richart from comment #2) > (In reply to sebastien.deronne from comment #1) > > Info needed from Matias and/or Tom obout the preferred option. > > I can extend it. I'll do it for next week. Thanks, I will assign this to you.
FYI, it fails for 802.11b, 802.11-10MHz and 802.11-5MHz.
Created attachment 2785 [details] Patch to fix Current implementation of RRAA uses fixed values for ORI and MTL parameters. The values are taken from the table provided in the paper: http://svn.camrdale.org/school/papers/802.11lans.pdf which only considers 802.11a. My patch proposes to calculate those values at initialization, following the guidelines proposed in the same paper. There is a slight difference between the values of the table and the calculated values. This is because the paper does not clarify the exact values for the different parameters that are used in the calculations (e.g. frame length). I also improve naming of variables, comments and logging.
Thanks Matias. Is the initial author still involved in ns-3? If yes, I suggest he reviews your patch.
(In reply to sebastien.deronne from comment #6) > Thanks Matias. Is the initial author still involved in ns-3? If yes, I > suggest he reviews your patch. No, he isn't.
Then if it works fine I suggest to deliver.
I have some merge conflicts, could you have a look? I cannot handle this error: ./ns3/rrpaa-wifi-manager.h:54:8: error: redefinition of 'Thresholds' struct Thresholds
(In reply to sebastien.deronne from comment #9) > I have some merge conflicts, could you have a look? > I cannot handle this error: > ./ns3/rrpaa-wifi-manager.h:54:8: error: redefinition of 'Thresholds' > struct Thresholds Matias, while you are at it, can you please rename Thresholds to something more specific such as RraaThresholds or WifiRraaThresholds, or else move it into the class, because Thresholds is a somewhat generic name to put into namespace ns3?
Matias, any update on this? I think the patch is almost ready.
(In reply to sebastien.deronne from comment #11) > Matias, any update on this? > I think the patch is almost ready. Sorry for that, I didn't have RRPAA merged when testing the patch. I uploaded the updated patch. I also propose to change the name of Thresholds in Rrpaa to avoid any troubles in the future.
Created attachment 2787 [details] Patch to fix
Created attachment 2788 [details] Change name of struct Thresholds in RrpaaWifiManager to avoid future conflicts
Thanks Matias, it is working fine. I will deliver your changes.
Fixed in changeset 12724:74b99de24416