Bugzilla – Bug 2306
wifi Dcf busy if CCA is busy
Last modified: 2016-09-28 00:20:34 EDT
DcfManager has a method that checks whether the medium is busy for reasons of RXing, TXing, or NAV, but not for CCA busy for some reason. The attached patch will raise DcfManager::IsBusy() when the phy state is in CCA busy. It causes pcap-trace based regression traces to change, and one DcfManagerTest test to fail (change included in the patch) but other tests pass with this change.
Created attachment 2290 [details] patch to fix
Looks ok for me at first sight. Does anyone has some doubts on the proposed change?
I'm surprised of this bug and makes me wonder if it was done this way for some reason. But in my opinion the solution looks good and needs to be applied. For the case of power adaptation this is an important change and makes some results of experiments I've made in the past to change a lot.
It looks like this patch makes routing-aodv-regression test fail. I assume we should simply rescan pcap traces, to be checked though.
(In reply to sebastien.deronne from comment #4) > It looks like this patch makes routing-aodv-regression test fail. > I assume we should simply rescan pcap traces, to be checked though. Tom will have a look.
pushed in changeset 12342:2e7e4f29be88; all tests were reviewed and updated as needed