From: Johannes Berg <johannes.berg@xxxxxxxxx>
This just prepares for passing the entire fragment
list to the driver. No significant changes, but the
TX throughput is calculated slightly differently
now and we blink only once for each MSDU.
Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx>
---
net/mac80211/tx.c | 28 ++++++++++++++++------------
1 file changed, 16 insertions(+), 12 deletions(-)
--- a/net/mac80211/tx.c 2011-11-16 15:16:45.000000000 +0100
+++ b/net/mac80211/tx.c 2011-11-16 15:17:19.000000000 +0100
@@ -1207,18 +1207,23 @@ ieee80211_tx_prepare(struct ieee80211_su
* Returns false if the frame couldn't be transmitted but was queued instead.
*/
static bool __ieee80211_tx(struct ieee80211_local *local,
- struct sk_buff_head *skbs,
+ struct sk_buff_head *skbs, int led_len,
struct sta_info *sta, bool txpending)
{
struct sk_buff *skb, *tmp;
struct ieee80211_tx_info *info;
struct ieee80211_sub_if_data *sdata;
unsigned long flags;
- int len;
+ __le16 fc;
+
+ if (WARN_ON(skb_queue_empty(skbs)))
+ return true;
+
+ skb = skb_peek(skbs);
+ fc = ((struct ieee80211_hdr *)skb->data)->frame_control;
skb_queue_walk_safe(skbs, skb, tmp) {
int q = skb_get_queue_mapping(skb);
- __le16 fc;
spin_lock_irqsave(&local->queue_stop_reason_lock, flags);
if (local->queue_stop_reasons[q] ||
@@ -1241,8 +1246,6 @@ static bool __ieee80211_tx(struct ieee80
info = IEEE80211_SKB_CB(skb);
- len = skb->len;
-
sdata = vif_to_sdata(info->control.vif);
switch (sdata->vif.type) {
@@ -1263,15 +1266,13 @@ static bool __ieee80211_tx(struct ieee80
else
info->control.sta = NULL;
- fc = ((struct ieee80211_hdr *)skb->data)->frame_control;
-
__skb_unlink(skb, skbs);
drv_tx(local, skb);
-
- ieee80211_tpt_led_trig_tx(local, fc, len);
- ieee80211_led_tx(local, 1);
}
+ ieee80211_tpt_led_trig_tx(local, fc, led_len);
+ ieee80211_led_tx(local, 1);
+
WARN_ON(!skb_queue_empty(skbs));
return true;
@@ -1341,6 +1342,7 @@ static bool ieee80211_tx(struct ieee8021
ieee80211_tx_result res_prepare;
struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb);
bool result = true;
+ int led_len;
if (unlikely(skb->len < 10)) {
dev_kfree_skb(skb);
@@ -1350,6 +1352,7 @@ static bool ieee80211_tx(struct ieee8021
rcu_read_lock();
/* initialises tx */
+ led_len = skb->len;
res_prepare = ieee80211_tx_prepare(sdata, &tx, skb);
if (unlikely(res_prepare == TX_DROP)) {
@@ -1363,7 +1366,8 @@ static bool ieee80211_tx(struct ieee8021
info->band = tx.channel->band;
if (!invoke_tx_handlers(&tx))
- result = __ieee80211_tx(local, &tx.skbs, tx.sta, txpending);
+ result = __ieee80211_tx(local, &tx.skbs, led_len,
+ tx.sta, txpending);
out:
rcu_read_unlock();
return result;
@@ -2119,7 +2123,7 @@ static bool ieee80211_tx_pending_skb(str
hdr = (struct ieee80211_hdr *)skb->data;
sta = sta_info_get(sdata, hdr->addr1);
- result = __ieee80211_tx(local, &skbs, sta, true);
+ result = __ieee80211_tx(local, &skbs, skb->len, sta, true);
}
return result;
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
