Commit 9bc383de authored by Johannes Berg's avatar Johannes Berg Committed by John W. Linville
Browse files

cfg80211: introduce capability for 4addr mode 



It's very likely that not many devices will support
four-address mode in station or AP mode so introduce
capability bits for both modes, set them in mac80211
and check them when userspace tries to use the mode.
Also, keep track of 4addr in cfg80211 (wireless_dev)
and not in mac80211 any more. mac80211 can also be
improved for the VLAN case by not looking at the
4addr flag but maintaining the station pointer for
it correctly. However, keep track of use_4addr for
station mode in mac80211 to avoid all the derefs.

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

include/net/cfg80211.h

+11 −0
Original line number Diff line number Diff line
@@ -1134,6 +1134,9 @@ struct cfg80211_ops { 
 *	by default -- this flag will be set depending on the kernel's default

 *	on wiphy_new(), but can be changed by the driver if it has a good

 *	reason to override the default

 * @WIPHY_FLAG_4ADDR_AP: supports 4addr mode even on AP (with a single station

 *	on a VLAN interface)

 * @WIPHY_FLAG_4ADDR_STATION: supports 4addr mode even as a station

 */

enum wiphy_flags {

	WIPHY_FLAG_CUSTOM_REGULATORY	= BIT(0),
@@ -1141,6 +1144,8 @@ enum wiphy_flags { 
	WIPHY_FLAG_DISABLE_BEACON_HINTS	= BIT(2),

	WIPHY_FLAG_NETNS_OK		= BIT(3),

	WIPHY_FLAG_PS_ON_BY_DEFAULT	= BIT(4),

	WIPHY_FLAG_4ADDR_AP		= BIT(5),

	WIPHY_FLAG_4ADDR_STATION	= BIT(6),

};



/**
@@ -1366,6 +1371,10 @@ struct cfg80211_cached_keys; 
 * @ssid_len: (private) Used by the internal configuration code

 * @wext: (private) Used by the internal wireless extensions compat code

 * @wext_bssid: (private) Used by the internal wireless extensions compat code

 * @use_4addr: indicates 4addr mode is used on this interface, must be

 *	set by driver (if supported) on add_interface BEFORE registering the

 *	netdev and may otherwise be used by driver read-only, will be update

 *	by cfg80211 on change_interface

 */

struct wireless_dev {

	struct wiphy *wiphy;
@@ -1379,6 +1388,8 @@ struct wireless_dev { 


	struct work_struct cleanup_work;



	bool use_4addr;



	/* currently used for IBSS and SME - might be rearranged later */

	u8 ssid[IEEE80211_MAX_SSID_LEN];

	u8 ssid_len;

net/mac80211/cfg.c

+8 −13
Original line number Diff line number Diff line
@@ -42,15 +42,6 @@ static bool nl80211_params_check(enum nl80211_iftype type, 
	if (!nl80211_type_check(type))

		return false;



	if (params->use_4addr > 0) {

		switch(type) {

		case NL80211_IFTYPE_AP_VLAN:

		case NL80211_IFTYPE_STATION:

			break;

		default:

			return false;

		}

	}

	return true;

}
@@ -107,12 +98,16 @@ static int ieee80211_change_iface(struct wiphy *wiphy, 
					    params->mesh_id_len,

					    params->mesh_id);



	if (params->use_4addr >= 0)

		sdata->use_4addr = !!params->use_4addr;



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

		return 0;



	if (type == NL80211_IFTYPE_AP_VLAN &&

	    params && params->use_4addr == 0)

		rcu_assign_pointer(sdata->u.vlan.sta, NULL);

	else if (type == NL80211_IFTYPE_STATION &&

		 params && params->use_4addr >= 0)

		sdata->u.mgd.use_4addr = params->use_4addr;



	sdata->u.mntr_flags = *flags;

	return 0;

}
@@ -827,7 +822,7 @@ static int ieee80211_change_station(struct wiphy *wiphy, 
			return -EINVAL;

		}



		if (vlansdata->use_4addr) {

		if (params->vlan->ieee80211_ptr->use_4addr) {

			if (vlansdata->u.vlan.sta)

				return -EBUSY;

net/mac80211/ieee80211_i.h

+2 −2
Original line number Diff line number Diff line
@@ -312,6 +312,8 @@ struct ieee80211_if_managed { 
	} mfp; /* management frame protection */



	int wmm_last_param_set;



	u8 use_4addr;

};



enum ieee80211_ibss_request {
@@ -459,8 +461,6 @@ struct ieee80211_sub_if_data { 
	int force_unicast_rateidx; /* forced TX rateidx for unicast frames */

	int max_ratectrl_rateidx; /* max TX rateidx for rate control */



	bool use_4addr; /* use 4-address frames */



	union {

		struct ieee80211_if_ap ap;

		struct ieee80211_if_wds wds;

net/mac80211/iface.c

+8 −4
Original line number Diff line number Diff line
@@ -752,7 +752,8 @@ int ieee80211_if_change_type(struct ieee80211_sub_if_data *sdata, 
		ieee80211_mandatory_rates(sdata->local,

			sdata->local->hw.conf.channel->band);

	sdata->drop_unencrypted = 0;

	sdata->use_4addr = 0;

	if (type == NL80211_IFTYPE_STATION)

		sdata->u.mgd.use_4addr = false;



	return 0;

}
@@ -810,6 +811,12 @@ int ieee80211_if_add(struct ieee80211_local *local, const char *name, 
	/* setup type-dependent data */

	ieee80211_setup_sdata(sdata, type);



	if (params) {

		ndev->ieee80211_ptr->use_4addr = params->use_4addr;

		if (type == NL80211_IFTYPE_STATION)

			sdata->u.mgd.use_4addr = params->use_4addr;

	}



	ret = register_netdevice(ndev);

	if (ret)

		goto fail;
@@ -820,9 +827,6 @@ int ieee80211_if_add(struct ieee80211_local *local, const char *name, 
					    params->mesh_id_len,

					    params->mesh_id);



	if (params && params->use_4addr >= 0)

		sdata->use_4addr = !!params->use_4addr;



	mutex_lock(&local->iflist_mtx);

	list_add_tail_rcu(&sdata->list, &local->interfaces);

	mutex_unlock(&local->iflist_mtx);

net/mac80211/main.c

+3 −1
Original line number Diff line number Diff line
@@ -328,7 +328,9 @@ struct ieee80211_hw *ieee80211_alloc_hw(size_t priv_data_len, 
	if (!wiphy)

		return NULL;



	wiphy->flags |= WIPHY_FLAG_NETNS_OK;

	wiphy->flags |= WIPHY_FLAG_NETNS_OK |

			WIPHY_FLAG_4ADDR_AP |

			WIPHY_FLAG_4ADDR_STATION;

	wiphy->privid = mac80211_wiphy_privid;



	/* Yes, putting cfg80211_bss into ieee80211_bss is a hack */

CRA Git | Maintained and supported by SUSTech CRA and CCSE