Bug 889 - Minstrel implementation should be high latency
Minstrel implementation should be high latency
Status: RESOLVED INVALID
Product: ns-3
Classification: Unclassified
Component: wifi
ns-3-dev
All All
: P5 enhancement
Assigned To: Matías Richart
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-04-21 10:48 EDT by Nicola Baldo
Modified: 2020-04-18 09:23 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 Nicola Baldo 2010-04-21 10:48:12 EDT
(with reference to the definition of "low latency" and "high latency" given in this paper: "IEEE 802.11 Rate Adaptation: A Practical Approach", by M. Lacage, M.H. Manshaei, and T. Turletti)

The original minstrel implementation in madwifi is high latency. 
However, the minstrel implementation in ns-3 is low latency: statistics are updated everytime DoReportDataOk and DoReportDataFailed are called, and these updated stats are then used for selecting the rate for the next transmission attempt.
Comment 1 Matías Richart 2016-04-01 06:32:42 EDT
Although it is true that in Minstrel "statistics are updated everytime DoReportDataOk and DoReportDataFailed are called", the ns3 implementation emulates the high-latency behaviour in the manager.
Specifically, it implements the MRR-chain in the manager, and select the next rate from this chain until the frame is correctly received or dropped, without using the stats.
It is done this way because ns3 does not implement the idea of MRR chain.

Then, I think we can consider ns3 Minstrel as high-latency, although it does not use the ns3 API for high-latency algorithms. 

In my opinion the bug isn't completely correct and should be closed. Nevertheless, it is true that a clear implementation of Minstrel is possible, but this would need to change ns3 rx/tx path to implement the MRR chain idea.
Comment 2 sebastien.deronne 2016-10-15 11:21:52 EDT
Matias, I assigned you to this bug, I let you further check whether this (old) bug should be rejected or not.
Comment 3 sebastien.deronne 2020-04-18 09:23:22 EDT
Incorrect according to Matias, so closing this bug