wil6210: optimize index manipulation in wil_vring_reap_rx
authorVladimir Kondratiev <qca_vkondrat@qca.qualcomm.com>
Sun, 15 Mar 2015 14:00:16 +0000 (16:00 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Fri, 20 Mar 2015 06:33:16 +0000 (08:33 +0200)
Use temporal variable for often used vring->swhead;
and use proper index in debug printing - vring->swhead
used before was modified in wil_vring_advance_head
and then increased value was used in debug print

Signed-off-by: Vladimir Kondratiev <qca_vkondrat@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ath/wil6210/txrx.c

index 689081c2c37b22592bcb7b2a3a6d0d5a321bd844..3feb86c4779556829a16b0a6985f0cd9f8593284 100644 (file)
@@ -369,6 +369,7 @@ static struct sk_buff *wil_vring_reap_rx(struct wil6210_priv *wil,
        u16 dmalen;
        u8 ftype;
        int cid;
+       int i = (int)vring->swhead;
        struct wil_net_stats *stats;
 
        BUILD_BUG_ON(sizeof(struct vring_rx_desc) > sizeof(skb->cb));
@@ -376,24 +377,28 @@ static struct sk_buff *wil_vring_reap_rx(struct wil6210_priv *wil,
        if (unlikely(wil_vring_is_empty(vring)))
                return NULL;
 
-       _d = &vring->va[vring->swhead].rx;
+       _d = &vring->va[i].rx;
        if (unlikely(!(_d->dma.status & RX_DMA_STATUS_DU))) {
                /* it is not error, we just reached end of Rx done area */
                return NULL;
        }
 
-       skb = vring->ctx[vring->swhead].skb;
+       skb = vring->ctx[i].skb;
+       vring->ctx[i].skb = NULL;
+       wil_vring_advance_head(vring, 1);
+       if (!skb) {
+               wil_err(wil, "No Rx skb at [%d]\n", i);
+               return NULL;
+       }
        d = wil_skb_rxdesc(skb);
        *d = *_d;
        pa = wil_desc_addr(&d->dma.addr);
-       vring->ctx[vring->swhead].skb = NULL;
-       wil_vring_advance_head(vring, 1);
 
        dma_unmap_single(dev, pa, sz, DMA_FROM_DEVICE);
        dmalen = le16_to_cpu(d->dma.length);
 
-       trace_wil6210_rx(vring->swhead, d);
-       wil_dbg_txrx(wil, "Rx[%3d] : %d bytes\n", vring->swhead, dmalen);
+       trace_wil6210_rx(i, d);
+       wil_dbg_txrx(wil, "Rx[%3d] : %d bytes\n", i, dmalen);
        wil_hex_dump_txrx("Rx ", DUMP_PREFIX_NONE, 32, 4,
                          (const void *)d, sizeof(*d), false);