Commit 808118cb authored by Jason Young's avatar Jason Young Committed by John W. Linville
Browse files

mac80211: do not enable ps if 802.1x controlled port is unblocked 



If dynamic_ps is disabled, enabling power save before the 4-way
handshake completes may delay the station from being authorized to
send/receive traffic, i.e. increase roaming times. It also may result in
a failed 4-way handshake depending on the AP's timing requirements and
beacon interval, and the station's listen interval.

To fix this, prevent power save from being enabled while the station
isn't authorized and recalculate power save whenever the station's
authorized state changes.

Signed-off-by: default avatarJason Young <a.young.jason@gmail.com>
Acked-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent d89dba7a

net/mac80211/cfg.c

+4 −0
Original line number Diff line number Diff line
@@ -834,6 +834,10 @@ static int ieee80211_change_station(struct wiphy *wiphy, 


	rcu_read_unlock();



	if (sdata->vif.type == NL80211_IFTYPE_STATION &&

	    params->sta_flags_mask & BIT(NL80211_STA_FLAG_AUTHORIZED))

		ieee80211_recalc_ps(local, -1);



	return 0;

}

net/mac80211/mlme.c

+32 −5
Original line number Diff line number Diff line
@@ -613,6 +613,37 @@ static void ieee80211_change_ps(struct ieee80211_local *local) 
	}

}



static bool ieee80211_powersave_allowed(struct ieee80211_sub_if_data *sdata)

{

	struct ieee80211_if_managed *mgd = &sdata->u.mgd;

	struct sta_info *sta = NULL;

	u32 sta_flags = 0;



	if (!mgd->powersave)

		return false;



	if (!mgd->associated)

		return false;



	if (!mgd->associated->beacon_ies)

		return false;



	if (mgd->flags & (IEEE80211_STA_BEACON_POLL |

			  IEEE80211_STA_CONNECTION_POLL))

		return false;



	rcu_read_lock();

	sta = sta_info_get(sdata, mgd->bssid);

	if (sta)

		sta_flags = get_sta_flags(sta);

	rcu_read_unlock();



	if (!(sta_flags & WLAN_STA_AUTHORIZED))

		return false;



	return true;

}



/* need to hold RTNL or interface lock */

void ieee80211_recalc_ps(struct ieee80211_local *local, s32 latency)

{
@@ -647,11 +678,7 @@ void ieee80211_recalc_ps(struct ieee80211_local *local, s32 latency) 
		count++;

	}



	if (count == 1 && found->u.mgd.powersave &&

	    found->u.mgd.associated &&

	    found->u.mgd.associated->beacon_ies &&

	    !(found->u.mgd.flags & (IEEE80211_STA_BEACON_POLL |

				    IEEE80211_STA_CONNECTION_POLL))) {

	if (count == 1 && ieee80211_powersave_allowed(found)) {

		struct ieee80211_conf *conf = &local->hw.conf;

		s32 beaconint_us;

CRA Git | Maintained and supported by SUSTech CRA and CCSE