Commit 413ad50a authored by Johannes Berg's avatar Johannes Berg Committed by John W. Linville
Browse files

mac80211: properly track HT operation_mode 



When we disassociate, we set the channel to non-HT which
obviously invalidates any ht_operation_mode setting. But
when we then associate with the next AP again, we might
still have the ht_operation_mode from the previous AP
cached and fail to configure the hardware with the new
(but unchanged) operation mode. This patch fixes it by
separately tracking whether our cache is valid.

Signed-off-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 9ed6bcce

net/mac80211/ieee80211_i.h

+6 −0
Original line number Diff line number Diff line
@@ -427,6 +427,12 @@ struct ieee80211_sub_if_data { 


	int drop_unencrypted;



	/*

	 * keep track of whether the HT opmode (stored in

	 * vif.bss_info.ht_operation_mode) is valid.

	 */

	bool ht_opmode_valid;



	/* Fragment table for host-based reassembly */

	struct ieee80211_fragment_entry	fragments[IEEE80211_FRAGMENT_MAX];

	unsigned int fragment_next;

net/mac80211/mlme.c

+6 −1
Original line number Diff line number Diff line
@@ -155,9 +155,11 @@ static u32 ieee80211_enable_ht(struct ieee80211_sub_if_data *sdata, 
	ht_opmode = le16_to_cpu(hti->operation_mode);



	/* if bss configuration changed store the new one */

	if (sdata->vif.bss_conf.ht_operation_mode != ht_opmode) {

	if (!sdata->ht_opmode_valid ||

	    sdata->vif.bss_conf.ht_operation_mode != ht_opmode) {

		changed |= BSS_CHANGED_HT;

		sdata->vif.bss_conf.ht_operation_mode = ht_opmode;

		sdata->ht_opmode_valid = true;

	}



	return changed;
@@ -1047,6 +1049,9 @@ static void ieee80211_set_disassoc(struct ieee80211_sub_if_data *sdata, 
	/* channel(_type) changes are handled by ieee80211_hw_config */

	local->oper_channel_type = NL80211_CHAN_NO_HT;



	/* on the next assoc, re-program HT parameters */

	sdata->ht_opmode_valid = false;



	local->power_constr_level = 0;



	del_timer_sync(&local->dynamic_ps_timer);

CRA Git | Maintained and supported by SUSTech CRA and CCSE