Commit 95dac040 authored by Johannes Berg's avatar Johannes Berg Committed by John W. Linville

mac80211: small rate control changes

This patch fixes mac80211 to not rely on the rate control
algorithm to update sta->tx_retry_failed and sta->tx_retry_count
(even if we don't currently use them), removes a number of
completely unused values we don't even show in debugfs and
changes the code in ieee80211_tx_status() to not look up the
sta_info repeatedly.

The only behaviour change here would be not calling the rate
control function rate_control_tx_status() when no sta_info is
found, but all rate control algorithms ignore such calls anyway.
Signed-off-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 81c06523
...@@ -546,10 +546,11 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -546,10 +546,11 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
rcu_read_lock(); rcu_read_lock();
if (info->status.excessive_retries) {
sta = sta_info_get(local, hdr->addr1); sta = sta_info_get(local, hdr->addr1);
if (sta) { if (sta) {
if (test_sta_flags(sta, WLAN_STA_PS)) { if (info->status.excessive_retries &&
test_sta_flags(sta, WLAN_STA_PS)) {
/* /*
* The STA is in power save mode, so assume * The STA is in power save mode, so assume
* that this TX packet failed because of that. * that this TX packet failed because of that.
...@@ -558,8 +559,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -558,8 +559,6 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
rcu_read_unlock(); rcu_read_unlock();
return; return;
} }
}
}
fc = hdr->frame_control; fc = hdr->frame_control;
...@@ -567,8 +566,7 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -567,8 +566,7 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
(ieee80211_is_data_qos(fc))) { (ieee80211_is_data_qos(fc))) {
u16 tid, ssn; u16 tid, ssn;
u8 *qc; u8 *qc;
sta = sta_info_get(local, hdr->addr1);
if (sta) {
qc = ieee80211_get_qos_ctl(hdr); qc = ieee80211_get_qos_ctl(hdr);
tid = qc[0] & 0xf; tid = qc[0] & 0xf;
ssn = ((le16_to_cpu(hdr->seq_ctrl) + 0x10) ssn = ((le16_to_cpu(hdr->seq_ctrl) + 0x10)
...@@ -576,17 +574,19 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb) ...@@ -576,17 +574,19 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
ieee80211_send_bar(sta->sdata, hdr->addr1, ieee80211_send_bar(sta->sdata, hdr->addr1,
tid, ssn); tid, ssn);
} }
}
if (info->flags & IEEE80211_TX_STAT_TX_FILTERED) { if (info->flags & IEEE80211_TX_STAT_TX_FILTERED) {
sta = sta_info_get(local, hdr->addr1);
if (sta) {
ieee80211_handle_filtered_frame(local, sta, skb); ieee80211_handle_filtered_frame(local, sta, skb);
rcu_read_unlock(); rcu_read_unlock();
return; return;
} else {
if (info->status.excessive_retries)
sta->tx_retry_failed++;
sta->tx_retry_count += info->status.retry_count;
} }
} else
rate_control_tx_status(local->mdev, skb); rate_control_tx_status(local->mdev, skb);
}
rcu_read_unlock(); rcu_read_unlock();
......
...@@ -282,17 +282,6 @@ static void rate_control_pid_tx_status(void *priv, struct net_device *dev, ...@@ -282,17 +282,6 @@ static void rate_control_pid_tx_status(void *priv, struct net_device *dev,
spinfo->tx_num_xmit++; spinfo->tx_num_xmit++;
} }
if (info->status.excessive_retries) {
sta->tx_retry_failed++;
sta->tx_num_consecutive_failures++;
sta->tx_num_mpdu_fail++;
} else {
sta->tx_num_consecutive_failures = 0;
sta->tx_num_mpdu_ok++;
}
sta->tx_retry_count += info->status.retry_count;
sta->tx_num_mpdu_fail += info->status.retry_count;
/* Update PID controller state. */ /* Update PID controller state. */
period = (HZ * pinfo->sampling_period + 500) / 1000; period = (HZ * pinfo->sampling_period + 500) / 1000;
if (!period) if (!period)
......
...@@ -195,9 +195,6 @@ struct sta_ampdu_mlme { ...@@ -195,9 +195,6 @@ struct sta_ampdu_mlme {
* @tx_filtered_count: TBD * @tx_filtered_count: TBD
* @tx_retry_failed: TBD * @tx_retry_failed: TBD
* @tx_retry_count: TBD * @tx_retry_count: TBD
* @tx_num_consecutive_failures: TBD
* @tx_num_mpdu_ok: TBD
* @tx_num_mpdu_fail: TBD
* @fail_avg: moving percentage of failed MSDUs * @fail_avg: moving percentage of failed MSDUs
* @tx_packets: number of RX/TX MSDUs * @tx_packets: number of RX/TX MSDUs
* @tx_bytes: TBD * @tx_bytes: TBD
...@@ -273,10 +270,6 @@ struct sta_info { ...@@ -273,10 +270,6 @@ struct sta_info {
/* Updated from TX status path only, no locking requirements */ /* Updated from TX status path only, no locking requirements */
unsigned long tx_filtered_count; unsigned long tx_filtered_count;
unsigned long tx_retry_failed, tx_retry_count; unsigned long tx_retry_failed, tx_retry_count;
/* TODO: update in generic code not rate control? */
u32 tx_num_consecutive_failures;
u32 tx_num_mpdu_ok;
u32 tx_num_mpdu_fail;
/* moving percentage of failed MSDUs */ /* moving percentage of failed MSDUs */
unsigned int fail_avg; unsigned int fail_avg;
......
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