Commit 3187ba0c authored by Ryder Lee's avatar Ryder Lee Committed by Johannes Berg

mac80211: add rate control support for encap offload

The software rate control cannot deal with encap offload, so fix it.
Signed-off-by: default avatarRyder Lee <ryder.lee@mediatek.com>
Link: https://lore.kernel.org/r/20210617163113.75815-3-nbd@nbd.nameSigned-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 03c3911d
...@@ -870,7 +870,6 @@ void ieee80211_get_tx_rates(struct ieee80211_vif *vif, ...@@ -870,7 +870,6 @@ void ieee80211_get_tx_rates(struct ieee80211_vif *vif,
int max_rates) int max_rates)
{ {
struct ieee80211_sub_if_data *sdata; struct ieee80211_sub_if_data *sdata;
struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb); struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb);
struct ieee80211_supported_band *sband; struct ieee80211_supported_band *sband;
...@@ -882,7 +881,7 @@ void ieee80211_get_tx_rates(struct ieee80211_vif *vif, ...@@ -882,7 +881,7 @@ void ieee80211_get_tx_rates(struct ieee80211_vif *vif,
sdata = vif_to_sdata(vif); sdata = vif_to_sdata(vif);
sband = sdata->local->hw.wiphy->bands[info->band]; sband = sdata->local->hw.wiphy->bands[info->band];
if (ieee80211_is_data(hdr->frame_control)) if (ieee80211_is_tx_data(skb))
rate_control_apply_mask(sdata, sta, sband, dest, max_rates); rate_control_apply_mask(sdata, sta, sband, dest, max_rates);
if (dest[0].idx < 0) if (dest[0].idx < 0)
......
...@@ -666,6 +666,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx) ...@@ -666,6 +666,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx)
u32 len; u32 len;
struct ieee80211_tx_rate_control txrc; struct ieee80211_tx_rate_control txrc;
struct ieee80211_sta_rates *ratetbl = NULL; struct ieee80211_sta_rates *ratetbl = NULL;
bool encap = info->flags & IEEE80211_TX_CTL_HW_80211_ENCAP;
bool assoc = false; bool assoc = false;
memset(&txrc, 0, sizeof(txrc)); memset(&txrc, 0, sizeof(txrc));
...@@ -707,7 +708,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx) ...@@ -707,7 +708,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx)
* just wants a probe response. * just wants a probe response.
*/ */
if (tx->sdata->vif.bss_conf.use_short_preamble && if (tx->sdata->vif.bss_conf.use_short_preamble &&
(ieee80211_is_data(hdr->frame_control) || (ieee80211_is_tx_data(tx->skb) ||
(tx->sta && test_sta_flag(tx->sta, WLAN_STA_SHORT_PREAMBLE)))) (tx->sta && test_sta_flag(tx->sta, WLAN_STA_SHORT_PREAMBLE))))
txrc.short_preamble = true; txrc.short_preamble = true;
...@@ -729,7 +730,8 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx) ...@@ -729,7 +730,8 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx)
"%s: Dropped data frame as no usable bitrate found while " "%s: Dropped data frame as no usable bitrate found while "
"scanning and associated. Target station: " "scanning and associated. Target station: "
"%pM on %d GHz band\n", "%pM on %d GHz band\n",
tx->sdata->name, hdr->addr1, tx->sdata->name,
encap ? ((struct ethhdr *)hdr)->h_dest : hdr->addr1,
info->band ? 5 : 2)) info->band ? 5 : 2))
return TX_DROP; return TX_DROP;
...@@ -763,7 +765,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx) ...@@ -763,7 +765,7 @@ ieee80211_tx_h_rate_ctrl(struct ieee80211_tx_data *tx)
if (txrc.reported_rate.idx < 0) { if (txrc.reported_rate.idx < 0) {
txrc.reported_rate = tx->rate; txrc.reported_rate = tx->rate;
if (tx->sta && ieee80211_is_data(hdr->frame_control)) if (tx->sta && ieee80211_is_tx_data(tx->skb))
tx->sta->tx_stats.last_rate = txrc.reported_rate; tx->sta->tx_stats.last_rate = txrc.reported_rate;
} else if (tx->sta) } else if (tx->sta)
tx->sta->tx_stats.last_rate = txrc.reported_rate; tx->sta->tx_stats.last_rate = txrc.reported_rate;
...@@ -3673,8 +3675,16 @@ struct sk_buff *ieee80211_tx_dequeue(struct ieee80211_hw *hw, ...@@ -3673,8 +3675,16 @@ struct sk_buff *ieee80211_tx_dequeue(struct ieee80211_hw *hw,
else else
info->flags &= ~IEEE80211_TX_CTL_AMPDU; info->flags &= ~IEEE80211_TX_CTL_AMPDU;
if (info->flags & IEEE80211_TX_CTL_HW_80211_ENCAP) if (info->flags & IEEE80211_TX_CTL_HW_80211_ENCAP) {
if (!ieee80211_hw_check(&local->hw, HAS_RATE_CONTROL)) {
r = ieee80211_tx_h_rate_ctrl(&tx);
if (r != TX_CONTINUE) {
ieee80211_free_txskb(&local->hw, skb);
goto begin;
}
}
goto encap_out; goto encap_out;
}
if (info->control.flags & IEEE80211_TX_CTRL_FAST_XMIT) { if (info->control.flags & IEEE80211_TX_CTRL_FAST_XMIT) {
struct sta_info *sta = container_of(txq->sta, struct sta_info, struct sta_info *sta = container_of(txq->sta, struct sta_info,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment