ath10k: avoid needless memset on TX path
authorMichal Kazior <michal.kazior@tieto.com>
Wed, 18 Sep 2013 12:43:19 +0000 (14:43 +0200)
committerKalle Valo <kvalo@qca.qualcomm.com>
Fri, 20 Sep 2013 05:18:09 +0000 (08:18 +0300)
This reduces number of memory accesses and
hopefully contributes to better performance in the
future.

Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath10k/htc.c
drivers/net/wireless/ath/ath10k/htt_tx.c
drivers/net/wireless/ath/ath10k/mac.c
drivers/net/wireless/ath/ath10k/txrx.c

index d0d72125df81dedab779fa710e74afe1e0560f29..3118d7506734267c8fcca7e80ad9eeba9269fce1 100644 (file)
@@ -103,10 +103,10 @@ static void ath10k_htc_prepare_tx_skb(struct ath10k_htc_ep *ep,
        struct ath10k_htc_hdr *hdr;
 
        hdr = (struct ath10k_htc_hdr *)skb->data;
-       memset(hdr, 0, sizeof(*hdr));
 
        hdr->eid = ep->eid;
        hdr->len = __cpu_to_le16(skb->len - sizeof(*hdr));
+       hdr->flags = 0;
 
        spin_lock_bh(&ep->htc->tx_lock);
        hdr->seq_no = ep->seq_no++;
index 454812879604ffb1d8d336c2b27e61544b4fc5cd..c4bbf74815f4ac8e4d479d6a35d6501b057af86f 100644 (file)
@@ -384,9 +384,11 @@ int ath10k_htt_mgmt_tx(struct ath10k_htt *htt, struct sk_buff *msdu)
        /* refcount is decremented by HTC and HTT completions until it reaches
         * zero and is freed */
        skb_cb = ATH10K_SKB_CB(txdesc);
+       skb_cb->htt.is_conf = false;
        skb_cb->htt.msdu_id = msdu_id;
        skb_cb->htt.refcount = 2;
        skb_cb->htt.msdu = msdu;
+       skb_cb->htt.txfrag = NULL;
 
        res = ath10k_htc_send(&htt->ar->htc, htt->eid, txdesc);
        if (res)
@@ -505,7 +507,6 @@ int ath10k_htt_tx(struct ath10k_htt *htt, struct sk_buff *msdu)
 
        skb_put(txdesc, desc_len);
        cmd = (struct htt_cmd *)txdesc->data;
-       memset(cmd, 0, desc_len);
 
        tid = ATH10K_SKB_CB(msdu)->htt.tid;
 
@@ -555,6 +556,7 @@ int ath10k_htt_tx(struct ath10k_htt *htt, struct sk_buff *msdu)
        /* refcount is decremented by HTC and HTT completions until it reaches
         * zero and is freed */
        skb_cb = ATH10K_SKB_CB(txdesc);
+       skb_cb->htt.is_conf = false;
        skb_cb->htt.msdu_id = msdu_id;
        skb_cb->htt.refcount = 2;
        skb_cb->htt.txfrag = txfrag;
index 5a56833c490791c400b2dd0d116678505e339ea1..0d367e46ba9a15a81f78de1ecd5980bb472d5922 100644 (file)
@@ -1757,7 +1757,9 @@ static void ath10k_tx(struct ieee80211_hw *hw,
                ath10k_tx_h_seq_no(skb);
        }
 
-       memset(ATH10K_SKB_CB(skb), 0, sizeof(*ATH10K_SKB_CB(skb)));
+       ATH10K_SKB_CB(skb)->is_mapped = false;
+       ATH10K_SKB_CB(skb)->is_aborted = false;
+       ATH10K_SKB_CB(skb)->htt.is_offchan = false;
        ATH10K_SKB_CB(skb)->htt.vdev_id = vdev_id;
        ATH10K_SKB_CB(skb)->htt.tid = tid;
 
index 37b819675d204f41fb6ffae69a638a638c25769e..f6fed31ac5453a38d3d0582361034da8ee322b92 100644 (file)
@@ -75,7 +75,6 @@ void ath10k_txrx_tx_unref(struct ath10k_htt *htt, struct sk_buff *txdesc)
        ath10k_report_offchan_tx(htt->ar, msdu);
 
        info = IEEE80211_SKB_CB(msdu);
-       memset(&info->status, 0, sizeof(info->status));
 
        if (ATH10K_SKB_CB(txdesc)->htt.discard) {
                ieee80211_free_txskb(htt->ar->hw, msdu);