linuxmsm/laptops-kernel
0
0
Fork 0
mirror of https://github.com/linux-msm/laptops-kernel.git synced 2026-08-13 14:19:53 -07:00
Code Issues Packages Projects Releases Wiki Activity

wifi: mac80211: Route (Re)association req/response to per-STA queue

Browse Source 
An association request or response frame is generally delivered to the
driver without a TX queue object. However, with drivers that do encryption
offload and couple the key with a transmit queue, this means that the
frames are not being encrypted.

Fix this by routing the association frames to the management TXQ.
This will allow the driver to set up the required resources before
transmitting the association frame, e.g., set up keys etc.

Signed-off-by: Ilan Peer <ilan.peer@intel.com>
Reviewed-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Link: https://patch.msgid.link/20260706222925.febcc62f485c.Iff932880e4b98232cbf6ba405fbb90d650a85381@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
This commit is contained in:
Ilan Peer
2026-07-07 10:16:31 +02:00
committed by Johannes Berg
parent 158438cd6a
commit ac798f757d
3 changed files with 19 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
include/linux/ieee80211.h
+11
View File
@@ -556,6 +556,17 @@ static inline bool ieee80211_is_reassoc_resp(__le16 fc)
cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_REASSOC_RESP);
}
/**
* ieee80211_is_assoc - check if (Re)association request/response frame
* @fc: frame control bytes in little-endian byteorder
* Return: whether or not the frame is an (re)association request or response
*/
static inline bool ieee80211_is_assoc(__le16 fc)
{
return ieee80211_is_assoc_req(fc) || ieee80211_is_reassoc_req(fc) ||
ieee80211_is_assoc_resp(fc) || ieee80211_is_reassoc_resp(fc);
}
/**
* ieee80211_is_probe_req - check if IEEE80211_FTYPE_MGMT && IEEE80211_STYPE_PROBE_REQ
* @fc: frame control bytes in little-endian byteorder
net/mac80211/ieee80211_i.h
+1 -3
View File
@@ -2470,9 +2470,7 @@ void __ieee80211_tx_skb_tid_band(struct ieee80211_sub_if_data *sdata,
static inline bool ieee80211_require_encrypted_assoc(__le16 fc,
struct sta_info *sta)
{
return (sta && sta->sta.epp_peer &&
(ieee80211_is_assoc_req(fc) || ieee80211_is_reassoc_req(fc) ||
ieee80211_is_assoc_resp(fc) || ieee80211_is_reassoc_resp(fc)));
return sta && sta->sta.epp_peer && ieee80211_is_assoc(fc);
}
/* sta_out needs to be checked for ERR_PTR() before using */
net/mac80211/tx.c
+7
View File
@@ -1309,10 +1309,17 @@ static struct txq_info *ieee80211_get_txq(struct ieee80211_local *local,
(info->control.flags & IEEE80211_TX_CTRL_PS_RESPONSE))
return NULL;
/*
* While (re)association request/response frames are not considered
* bufferable MMPDUs, use the TXQ abstraction for the transmission of
* these frames. This is specifically useful for drivers that might
* associate other resources with the TXQ, e.g., encryption keys etc.
*/
if (!(info->flags & IEEE80211_TX_CTL_HW_80211_ENCAP) &&
unlikely(!ieee80211_is_data_present(hdr->frame_control))) {
if ((!ieee80211_is_mgmt(hdr->frame_control) ||
ieee80211_is_bufferable_mmpdu(skb) ||
ieee80211_is_assoc(hdr->frame_control) ||
vif->type == NL80211_IFTYPE_STATION ||
vif->type == NL80211_IFTYPE_NAN ||
vif->type == NL80211_IFTYPE_NAN_DATA) &&