From: Karl Beldan <karl.beldan@xxxxxxxxxxxxxxxx> ATM minstrel_ht does not check whether the sampling rate is supported. Unsupported rates attempts can trigger when there are holes between supported MCSes belonging to the same group (e.g many devices are capable of MCS32 without being capable of MCS33->MCS39). This change replaces an unsupported sample index with the fls of the bitfield of supported indexes. This is not a problem in minstrel which fills a per STA sample table with only supported rates (though only at init). Signed-off-by: Karl Beldan <karl.beldan@xxxxxxxxxxxxxxxx> --- net/mac80211/rc80211_minstrel_ht.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/mac80211/rc80211_minstrel_ht.c b/net/mac80211/rc80211_minstrel_ht.c index aeec401..1b835ad 100644 --- a/net/mac80211/rc80211_minstrel_ht.c +++ b/net/mac80211/rc80211_minstrel_ht.c @@ -703,6 +703,8 @@ minstrel_get_sample_rate(struct minstrel_priv *mp, struct minstrel_ht_sta *mi) mg = &mi->groups[mi->sample_group]; sample_idx = sample_table[mg->column][mg->index]; + if (!(mg->supported & BIT(sample_idx))) + sample_idx = fls(sample_idx) - 1; mr = &mg->rates[sample_idx]; sample_group = mi->sample_group; sample_idx += sample_group * MCS_GROUP_RATES; -- 1.8.2 -- 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
