Commit 1ea6f9c0 authored by Johannes Berg's avatar Johannes Berg
Browse files

mac80211: handle TX power per virtual interface 



Even before channel contexts/multi-channel, having a
single global TX power limit was already problematic,
in particular if two managed interfaces connected to
two APs with different power constraints. The channel
context introduction completely broke this though and
in fact I had disabled TX power configuration there
for drivers using channel contexts.

Change everything to track TX power per interface so
that different user settings and different channel
maxima are treated correctly. Also continue tracking
the global TX power though for compatibility with
applications that attempt to configure the wiphy's
TX power globally.

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent c8442118

include/net/mac80211.h

+6 −1
Original line number Diff line number Diff line
@@ -207,6 +207,7 @@ struct ieee80211_chanctx_conf { 
 * @BSS_CHANGED_SSID: SSID changed for this BSS (AP mode)

 * @BSS_CHANGED_AP_PROBE_RESP: Probe Response changed for this BSS (AP mode)

 * @BSS_CHANGED_PS: PS changed for this BSS (STA mode)

 * @BSS_CHANGED_TXPOWER: TX power setting changed for this interface

 */

enum ieee80211_bss_change {

	BSS_CHANGED_ASSOC		= 1<<0,
@@ -227,6 +228,7 @@ enum ieee80211_bss_change { 
	BSS_CHANGED_SSID		= 1<<15,

	BSS_CHANGED_AP_PROBE_RESP	= 1<<16,

	BSS_CHANGED_PS			= 1<<17,

	BSS_CHANGED_TXPOWER		= 1<<18,



	/* when adding here, make sure to change ieee80211_reconfig */

};
@@ -309,6 +311,7 @@ enum ieee80211_rssi_event { 
 * @ssid: The SSID of the current vif. Only valid in AP-mode.

 * @ssid_len: Length of SSID given in @ssid.

 * @hidden_ssid: The SSID of the current vif is hidden. Only valid in AP-mode.

 * @txpower: TX power in dBm

 */

struct ieee80211_bss_conf {

	const u8 *bssid;
@@ -341,6 +344,7 @@ struct ieee80211_bss_conf { 
	u8 ssid[IEEE80211_MAX_SSID_LEN];

	size_t ssid_len;

	bool hidden_ssid;

	int txpower;

};



/**
@@ -884,7 +888,8 @@ enum ieee80211_smps_mode { 
 *	powersave documentation below. This variable is valid only when

 *	the CONF_PS flag is set.

 *

 * @power_level: requested transmit power (in dBm)

 * @power_level: requested transmit power (in dBm), backward compatibility

 *	value only that is set to the minimum of all interfaces

 *

 * @channel: the channel to tune to

 * @channel_type: the channel (HT) type

net/mac80211/cfg.c

+32 −15
Original line number Diff line number Diff line
@@ -1996,33 +1996,46 @@ static int ieee80211_set_tx_power(struct wiphy *wiphy, 
				  enum nl80211_tx_power_setting type, int mbm)

{

	struct ieee80211_local *local = wiphy_priv(wiphy);

	struct ieee80211_channel *chan = local->_oper_channel;

	u32 changes = 0;

	struct ieee80211_sub_if_data *sdata;



	/* FIXME */

	if (local->use_chanctx)

		return -EOPNOTSUPP;

	if (wdev) {

		sdata = IEEE80211_WDEV_TO_SUB_IF(wdev);



		switch (type) {

		case NL80211_TX_POWER_AUTOMATIC:

		local->user_power_level = -1;

			sdata->user_power_level = IEEE80211_UNSET_POWER_LEVEL;

			break;

		case NL80211_TX_POWER_LIMITED:

		case NL80211_TX_POWER_FIXED:

			if (mbm < 0 || (mbm % 100))

				return -EOPNOTSUPP;

		local->user_power_level = MBM_TO_DBM(mbm);

			sdata->user_power_level = MBM_TO_DBM(mbm);

			break;

		}



		ieee80211_recalc_txpower(sdata);



		return 0;

	}



	switch (type) {

	case NL80211_TX_POWER_AUTOMATIC:

		local->user_power_level = IEEE80211_UNSET_POWER_LEVEL;

		break;

	case NL80211_TX_POWER_LIMITED:

	case NL80211_TX_POWER_FIXED:

		if (mbm < 0 || (mbm % 100))

			return -EOPNOTSUPP;

		/* TODO: move to cfg80211 when it knows the channel */

		if (MBM_TO_DBM(mbm) > chan->max_power)

			return -EINVAL;

		local->user_power_level = MBM_TO_DBM(mbm);

		break;

	}



	ieee80211_hw_config(local, changes);

	mutex_lock(&local->iflist_mtx);

	list_for_each_entry(sdata, &local->interfaces, list)

		sdata->user_power_level = local->user_power_level;

	list_for_each_entry(sdata, &local->interfaces, list)

		ieee80211_recalc_txpower(sdata);

	mutex_unlock(&local->iflist_mtx);



	return 0;

}
@@ -2032,8 +2045,12 @@ static int ieee80211_get_tx_power(struct wiphy *wiphy, 
				  int *dbm)

{

	struct ieee80211_local *local = wiphy_priv(wiphy);

	struct ieee80211_sub_if_data *sdata = IEEE80211_WDEV_TO_SUB_IF(wdev);



	if (!local->use_chanctx)

		*dbm = local->hw.conf.power_level;

	else

		*dbm = sdata->vif.bss_conf.txpower;



	return 0;

}

net/mac80211/chan.c

+2 −0
Original line number Diff line number Diff line
@@ -173,6 +173,8 @@ static int ieee80211_assign_vif_chanctx(struct ieee80211_sub_if_data *sdata, 
	rcu_assign_pointer(sdata->vif.chanctx_conf, &ctx->conf);

	ctx->refcount++;



	ieee80211_recalc_txpower(sdata);



	return 0;

}

net/mac80211/debugfs_netdev.c

+6 −0
Original line number Diff line number Diff line
@@ -168,6 +168,9 @@ IEEE80211_IF_FILE(rc_rateidx_mcs_mask_5ghz, 
IEEE80211_IF_FILE(flags, flags, HEX);

IEEE80211_IF_FILE(state, state, LHEX);

IEEE80211_IF_FILE(channel_type, vif.bss_conf.channel_type, DEC);

IEEE80211_IF_FILE(txpower, vif.bss_conf.txpower, DEC);

IEEE80211_IF_FILE(ap_power_level, ap_power_level, DEC);

IEEE80211_IF_FILE(user_power_level, user_power_level, DEC);



/* STA attributes */

IEEE80211_IF_FILE(bssid, u.mgd.bssid, MAC);
@@ -632,6 +635,9 @@ static void add_files(struct ieee80211_sub_if_data *sdata) 
	DEBUGFS_ADD(flags);

	DEBUGFS_ADD(state);

	DEBUGFS_ADD(channel_type);

	DEBUGFS_ADD(txpower);

	DEBUGFS_ADD(user_power_level);

	DEBUGFS_ADD(ap_power_level);



	if (sdata->vif.type != NL80211_IFTYPE_MONITOR)

		add_common_files(sdata);

net/mac80211/ieee80211_i.h

+10 −2
Original line number Diff line number Diff line
@@ -56,6 +56,9 @@ struct ieee80211_local; 
#define TU_TO_JIFFIES(x)	(usecs_to_jiffies((x) * 1024))

#define TU_TO_EXP_TIME(x)	(jiffies + TU_TO_JIFFIES(x))



/* power level hasn't been configured (or set to automatic) */

#define IEEE80211_UNSET_POWER_LEVEL	INT_MIN



/*

 * Some APs experience problems when working with U-APSD. Decrease the

 * probability of that happening by using legacy mode for all ACs but VO.
@@ -743,6 +746,9 @@ struct ieee80211_sub_if_data { 
	u8 needed_rx_chains;

	enum ieee80211_smps_mode smps_mode;



	int user_power_level; /* in dBm */

	int ap_power_level; /* in dBm */



	/*

	 * AP this belongs to: self in AP mode and

	 * corresponding AP in VLAN mode, NULL for
@@ -1117,8 +1123,7 @@ struct ieee80211_local { 
	int dynamic_ps_user_timeout;

	bool disable_dynamic_ps;



	int user_power_level; /* in dBm */

	int ap_power_level; /* in dBm */

	int user_power_level; /* in dBm, for all interfaces */



	enum ieee80211_smps_mode smps_mode;
@@ -1365,6 +1370,9 @@ void ieee80211_adjust_monitor_flags(struct ieee80211_sub_if_data *sdata, 
int ieee80211_do_open(struct wireless_dev *wdev, bool coming_up);

void ieee80211_sdata_stop(struct ieee80211_sub_if_data *sdata);



bool __ieee80211_recalc_txpower(struct ieee80211_sub_if_data *sdata);

void ieee80211_recalc_txpower(struct ieee80211_sub_if_data *sdata);



static inline bool ieee80211_sdata_running(struct ieee80211_sub_if_data *sdata)

{

	return test_bit(SDATA_STATE_RUNNING, &sdata->state);

CRA Git | Maintained and supported by SUSTech CRA and CCSE