On Tue, Dec 8, 2015 at 9:28 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Tue, 2015-12-08 at 21:25 +0530, Krishna Chaitanya wrote: >> On Tue, Dec 8, 2015 at 9:23 PM, Johannes Berg <johannes@sipsolutions. >> net> wrote: >> > On Tue, 2015-12-08 at 21:17 +0530, Krishna Chaitanya wrote: >> > > >> > > > If the rate control algorithm doesn't look at sta->sta.rx_nss >> > > > then >> > > > that's their bug. >> > > >> > > Yes, it looks like it. Only BW is handled, not the NSS change, >> > > and >> > > without this patch OP MODE IE in beacon updates neither NSS nor >> > > BW. >> > > >> > > For Action frame OP MODE IE, NSS will be updated. >> Typo BW will be updated. > > Ok. > >> > I don't see how that would happen - they end up in exactly the same >> > code path. >> >> As nss_only is false, BW will be update in case of action frame. > > Well, this patch removes that since the whole concept of nss_only was > wrong. > > So basically you're just saying that minstrel should be fixed to look > at rx_nss (and the relevant change bit, perhaps)... > Exactly :-). -- 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
