Bug 2400 - code-review: SpectrumWifiPhy
code-review: SpectrumWifiPhy
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: wifi
pre-release
PC Linux
: P5 enhancement
Assigned To: sebastien.deronne
:
: 1067 (view as bug list)
Depends on: 2396 2397 2398 2399 2412
Blocks:
  Show dependency treegraph
 
Reported: 2016-05-03 19:23 EDT by Tom Henderson
Modified: 2016-07-23 00:54 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Henderson 2016-05-03 19:23:33 EDT
This is to track the review of SpectrumWifiPhy, which is proposed to allow Wi-Fi devices to connect to Spectrum channels.  This will allow SpectrumWifiPhy to receive signals that are not Wi-Fi, such as in Wi-Fi/LTE coexistence, or interference scenarios.

This version of SpectrumWifiPhy is very basic, in that it is for the most part a clone of YansWifiPhy, with some adaptor code at the very bottom of the Phy where the channel interconnection occurs.  Instead of YansWifiChannel, a SpectrumChannel type must be used (see src/spectrum).

The main difference is that when a foreign (non-Wi-Fi) signal is received, it is simply added to the InterferenceHelper and evaluated for CCA BUSY status, without further receive processing.  The performance of SpectrumWifiPhy and YansWifiPhy should be identical.

Three example programs are provided, and one test suite.  The examples allow users to substitute SpectrumWifiPhy and YansWifiPhy under different conditions (signal loss, interference, saturating traffic) and compare results.

In the long run, SpectrumWifiPhy will use different interference trackers and error models more suited for the spectrum framework, but for the next release, it is proposed to simply reuse the existing Yans-compatible model and interference helper.

This version of SpectrumWifiPhy is known to lack two final things necessary for merging to ns-3-dev, IMO:
1) update the Wi-Fi .rst documentation accordingly
2) it is presently hardwired to the lower U-NII bands (5.180 GHz) and needs to be generalized (mainly, this involves making the spectrum model code more configurable)

However, I wanted to get the code review started while the above two are finished off.

This is a large patch and users can experiment with it by cloning the spectrum-wifi branch from github: https://github.com/tomhenderson/ns-3-dev-git/commit/7b6d894fcd8f82fb017654c9c6a95f58cabcfe02

However, I will also post a Rietveld issue to collect comments on the code.
Comment 1 Tom Henderson 2016-05-03 19:29:31 EDT
*** Bug 1067 has been marked as a duplicate of this bug. ***
Comment 2 Tom Henderson 2016-05-03 20:12:10 EDT
Please leave detailed comments on this Rietveld issue:

https://codereview.appspot.com/293380043/
Comment 3 sebastien.deronne 2016-06-04 06:53:37 EDT
Patchset 2 to be posted by Tom
Comment 4 Tom Henderson 2016-07-04 13:03:36 EDT
Update posted today; asking for final review comments:
https://codereview.appspot.com/293380043/

Reviewers may also check out the 'spectrum-wifi' branch of:
https://github.com/tomhenderson/ns-3-dev-git
Comment 5 Tom Henderson 2016-07-23 00:54:09 EDT
added in changeset 12219:a7e5825d00b6