Bug 2606

Summary: Ideal rate manager does not work correctly when a HT/VHT station is transmitting to a legacy access point
Product: ns-3 Reporter: sebastien.deronne
Component: wifiAssignee: Matías Richart <matis18>
Status: RESOLVED FIXED    
Severity: normal CC: matis18, ns-bugs, tomh
Priority: P3    
Version: ns-3.26   
Hardware: All   
OS: All   
Bug Depends on: 2605    
Bug Blocks:    
Attachments: Patch to fix

Description sebastien.deronne 2017-01-02 17:41:16 UTC 
Even with the fix for bug 2605, the obtained throughput with Ideal RAA is very low  when a HT/VHT station is transmitting to a legacy access point.
From traces, all frames are sent using the lowest legacy rate.
This can be reproduced by using test script and fix attached for bug 2605 and set RAA parameters to Ideal.
Comment 1 sebastien.deronne 2017-01-02 17:41:45 UTC 
Is this something Tom or Matias could investigate?
Comment 2 Tom Henderson 2017-01-07 12:16:27 UTC 
How can I reproduce?

I patched with the patch to bug 2605, then I tried:
./waf --run 'test-backward-compatibility --apVersion=80211a --staVersion=80211ac --apRaa=Ideal --staRaa=Ideal'

I get:
assert failed. cond="m_qosSupported", file=../src/wifi/model/regular-wifi-mac.cc, line=782
Comment 3 sebastien.deronne 2017-01-07 15:08:38 UTC 
(In reply to Tom Henderson from comment #2)
> How can I reproduce?
> 
> I patched with the patch to bug 2605, then I tried:
> ./waf --run 'test-backward-compatibility --apVersion=80211a
> --staVersion=80211ac --apRaa=Ideal --staRaa=Ideal'
> 
> I get:
> assert failed. cond="m_qosSupported",
> file=../src/wifi/model/regular-wifi-mac.cc, line=782

Sorry, I updated the patch for bug 2605, VHT was not handled properly.
Comment 4 sebastien.deronne 2017-01-24 14:00:47 UTC 
Tom, any updates on this one?
Comment 5 sebastien.deronne 2017-01-30 15:15:23 UTC 
I would still like to have this one fixed for ns-3.27.
Could Tom or Matias investigate this week?
Comment 6 Matías Richart 2017-02-01 12:49:40 UTC 
Created attachment 2770 [details]
Patch to fix

It is missing some control to see if the remote station is ht compatible when searching for modes.
I attach a patch.
Comment 7 sebastien.deronne 2017-02-01 15:17:42 UTC 
(In reply to Matías Richart from comment #6)
> Created attachment 2770 [details]
> Patch to fix
> 
> It is missing some control to see if the remote station is ht compatible
> when searching for modes.
> I attach a patch.

Thanks, it works fine :-)
I know Tom was also about to investigate. I guess this is ok for him as well.
Comment 8 sebastien.deronne 2017-02-02 14:51:50 UTC 
Fixed in changeset 12615:89edf33adb50