Johannes Berg wrote:
On Fri, 2009-08-21 at 11:03 -0700, Rafael Laufer wrote:
+ * @IEEE80211_TX_CTL_RATE_RADIOTAP: completely internal to mac80211,
+ * used to indicate that the rate was defined in the received radiotap
+ * header and therefore the rate control algorithm should not change it.
This should be an internal flag, the driver doesn't care.
right, and where are those set?
+ /* In monitor mode, if the IEEE80211_RADIOTAP_RATE option is set in
+ * the received radiotap header, do not call the rate control algorithm.
+ */
coding style
+ /* Get the rate parameter from the radiotap header,
+ * allowing rate selection on a per-packet basis
+ */
coding style
I am a newbie, I am gonna look into the coding style, but I assume you
are talking about the missing blank line in the beginning
+ case IEEE80211_RADIOTAP_RATE:
+ bitrate = (*iterator.this_arg) * 5;
+ for (i = 0; i < sband->n_bitrates; i++) {
+ if (sband->bitrates[i].bitrate == bitrate)
+ break;
+ }
+ if (i != sband->n_bitrates) {
+ info->control.rates[0].idx = i;
+ info->flags |= IEEE80211_TX_CTL_RATE_RADIOTAP;
+ }
You never set the counter, or any other fields.
This is a good point and now I see that Gábor pointed this out as well.
There are other fields in the radiotap header that define the RTS and
DATA retries. However, if those fields are not set, there must be a
default value in this case. Are there any?
/*
* Please update the file
* Documentation/networking/mac80211-injection.txt
And you even quote the instructions.
come on...
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
