mwifiex: delay skb allocation for RX until cmd53 over
authorZhaoyang Liu <liuzy@marvell.com>
Fri, 13 Mar 2015 12:07:59 +0000 (17:37 +0530)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 16 Mar 2015 16:12:41 +0000 (18:12 +0200)
This patch moves SKB allocation for RX packets from current
place i.e. after reading MP regs to place where we already
have read data from SDIO bus ie after cmd53.

mp_rx_aggr_setup has been modified accordingly to set
skb_arr to NULL.

Signed-off-by: Zhaoyang Liu <liuzy@marvell.com>
Signed-off-by: Shengzhen Li <szli@marvell.com>
Reviewed-by: Amitkumar Karwar <akarwar@marvell.com>
Reviewed-by: Cathy Luo <cluo@marvell.com>
Reviewed-by: Avinash Patil <patila@marvell.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/mwifiex/sdio.c
drivers/net/wireless/mwifiex/sdio.h

index fdeeb67f790ac4c42f4bac793aef65d0075a9737..330e9d06729df9651e32c881c17e8b23ac0f302b 100644 (file)
@@ -1197,7 +1197,7 @@ static int mwifiex_decode_rx_packet(struct mwifiex_adapter *adapter,
  * provided there is space left, processed and finally uploaded.
  */
 static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
-                                            struct sk_buff *skb, u8 port)
+                                            u16 rx_len, u8 port)
 {
        struct sdio_mmc_card *card = adapter->card;
        s32 f_do_rx_aggr = 0;
@@ -1205,10 +1205,9 @@ static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
        s32 f_aggr_cur = 0;
        s32 f_post_aggr_cur = 0;
        struct sk_buff *skb_deaggr;
-       u32 pind;
-       u32 pkt_len, pkt_type, mport;
+       struct sk_buff *skb = NULL;
+       u32 pkt_len, pkt_type, mport, pind;
        u8 *curr_ptr;
-       u32 rx_len = skb->len;
 
        if ((card->has_control_mask) && (port == CTRL_PORT)) {
                /* Read the command Resp without aggr */
@@ -1235,7 +1234,7 @@ static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
                dev_dbg(adapter->dev, "info: %s: not last packet\n", __func__);
 
                if (MP_RX_AGGR_IN_PROGRESS(card)) {
-                       if (MP_RX_AGGR_BUF_HAS_ROOM(card, skb->len)) {
+                       if (MP_RX_AGGR_BUF_HAS_ROOM(card, rx_len)) {
                                f_aggr_cur = 1;
                        } else {
                                /* No room in Aggr buf, do rx aggr now */
@@ -1253,7 +1252,7 @@ static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
 
                if (MP_RX_AGGR_IN_PROGRESS(card)) {
                        f_do_rx_aggr = 1;
-                       if (MP_RX_AGGR_BUF_HAS_ROOM(card, skb->len))
+                       if (MP_RX_AGGR_BUF_HAS_ROOM(card, rx_len))
                                f_aggr_cur = 1;
                        else
                                /* No room in Aggr buf, do rx aggr now */
@@ -1266,7 +1265,7 @@ static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
        if (f_aggr_cur) {
                dev_dbg(adapter->dev, "info: current packet aggregation\n");
                /* Curr pkt can be aggregated */
-               mp_rx_aggr_setup(card, skb, port);
+               mp_rx_aggr_setup(card, rx_len, port);
 
                if (MP_RX_AGGR_PKT_LIMIT_REACHED(card) ||
                    mp_rx_aggr_port_limit_reached(card)) {
@@ -1309,18 +1308,25 @@ static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
                curr_ptr = card->mpa_rx.buf;
 
                for (pind = 0; pind < card->mpa_rx.pkt_cnt; pind++) {
+                       u32 *len_arr = card->mpa_rx.len_arr;
 
                        /* get curr PKT len & type */
                        pkt_len = le16_to_cpu(*(__le16 *) &curr_ptr[0]);
                        pkt_type = le16_to_cpu(*(__le16 *) &curr_ptr[2]);
 
                        /* copy pkt to deaggr buf */
-                       skb_deaggr = card->mpa_rx.skb_arr[pind];
+                       skb_deaggr = mwifiex_alloc_dma_align_buf(len_arr[pind],
+                                                                GFP_KERNEL |
+                                                                GFP_DMA);
+                       if (!skb_deaggr)
+                               goto error;
+                       skb_put(skb_deaggr, len_arr[pind]);
+                       card->mpa_rx.skb_arr[pind] = skb_deaggr;
 
                        if ((pkt_type == MWIFIEX_TYPE_DATA ||
                             (pkt_type == MWIFIEX_TYPE_AGGR_DATA &&
                              adapter->sdio_rx_aggr_enable)) &&
-                           (pkt_len <= card->mpa_rx.len_arr[pind])) {
+                           (pkt_len <= len_arr[pind])) {
 
                                memcpy(skb_deaggr->data, curr_ptr, pkt_len);
 
@@ -1335,10 +1341,10 @@ static int mwifiex_sdio_card_to_host_mp_aggr(struct mwifiex_adapter *adapter,
                                        "type=%d len=%d max_len=%d\n",
                                        adapter->sdio_rx_aggr_enable,
                                        pkt_type, pkt_len,
-                                       card->mpa_rx.len_arr[pind]);
+                                       len_arr[pind]);
                                dev_kfree_skb_any(skb_deaggr);
                        }
-                       curr_ptr += card->mpa_rx.len_arr[pind];
+                       curr_ptr += len_arr[pind];
                }
                MP_RX_AGGR_BUF_RESET(card);
        }
@@ -1347,6 +1353,10 @@ rx_curr_single:
        if (f_do_rx_cur) {
                dev_dbg(adapter->dev, "info: RX: port: %d, rx_len: %d\n",
                        port, rx_len);
+               skb = mwifiex_alloc_dma_align_buf(rx_len, GFP_KERNEL | GFP_DMA);
+               if (!skb)
+                       goto error;
+               skb_put(skb, rx_len);
 
                if (mwifiex_sdio_card_to_host(adapter, &pkt_type,
                                              skb->data, skb->len,
@@ -1365,7 +1375,7 @@ rx_curr_single:
        if (f_post_aggr_cur) {
                dev_dbg(adapter->dev, "info: current packet aggregation\n");
                /* Curr pkt can be aggregated */
-               mp_rx_aggr_setup(card, skb, port);
+               mp_rx_aggr_setup(card, skb->len, port);
        }
 
        return 0;
@@ -1375,12 +1385,13 @@ error:
                for (pind = 0; pind < card->mpa_rx.pkt_cnt; pind++) {
                        /* copy pkt to deaggr buf */
                        skb_deaggr = card->mpa_rx.skb_arr[pind];
-                       dev_kfree_skb_any(skb_deaggr);
+                       if (skb_deaggr)
+                               dev_kfree_skb_any(skb_deaggr);
                }
                MP_RX_AGGR_BUF_RESET(card);
        }
 
-       if (f_do_rx_cur)
+       if (f_do_rx_cur && skb)
                /* Single transfer pending. Free curr buff also */
                dev_kfree_skb_any(skb);
 
@@ -1442,6 +1453,7 @@ static int mwifiex_process_int_status(struct mwifiex_adapter *adapter)
                     MWIFIEX_RX_DATA_BUF_SIZE)
                        return -1;
                rx_len = (u16) (rx_blocks * MWIFIEX_SDIO_BLOCK_SIZE);
+               dev_dbg(adapter->dev, "info: rx_len = %d\n", rx_len);
 
                skb = mwifiex_alloc_dma_align_buf(rx_len, GFP_KERNEL | GFP_DMA);
                if (!skb)
@@ -1538,24 +1550,11 @@ static int mwifiex_process_int_status(struct mwifiex_adapter *adapter)
                                        rx_len);
                                return -1;
                        }
-                       rx_len = (u16) (rx_blocks * MWIFIEX_SDIO_BLOCK_SIZE);
 
-                       skb = mwifiex_alloc_dma_align_buf(rx_len,
-                                                         GFP_KERNEL |
-                                                         GFP_DMA);
-
-                       if (!skb) {
-                               dev_err(adapter->dev, "%s: failed to alloc skb",
-                                       __func__);
-                               return -1;
-                       }
-
-                       skb_put(skb, rx_len);
-
-                       dev_dbg(adapter->dev, "info: rx_len = %d skb->len = %d\n",
-                               rx_len, skb->len);
+                       rx_len = (u16) (rx_blocks * MWIFIEX_SDIO_BLOCK_SIZE);
+                       dev_dbg(adapter->dev, "info: rx_len = %d\n", rx_len);
 
-                       if (mwifiex_sdio_card_to_host_mp_aggr(adapter, skb,
+                       if (mwifiex_sdio_card_to_host_mp_aggr(adapter, rx_len,
                                                              port)) {
                                dev_err(adapter->dev, "card_to_host_mpa failed:"
                                        " int status=%#x\n", sdio_ireg);
index 264bc9b9e02adfcb82226840b601a2aea0764085..6f645cf47369baddaa10bc6489837fa914a14a57 100644 (file)
@@ -573,9 +573,9 @@ mp_tx_aggr_port_limit_reached(struct sdio_mmc_card *card)
 
 /* Prepare to copy current packet from card to SDIO Rx aggregation buffer */
 static inline void mp_rx_aggr_setup(struct sdio_mmc_card *card,
-                                   struct sk_buff *skb, u8 port)
+                                   u16 rx_len, u8 port)
 {
-       card->mpa_rx.buf_len += skb->len;
+       card->mpa_rx.buf_len += rx_len;
 
        if (!card->mpa_rx.pkt_cnt)
                card->mpa_rx.start_port = port;
@@ -588,8 +588,8 @@ static inline void mp_rx_aggr_setup(struct sdio_mmc_card *card,
                else
                        card->mpa_rx.ports |= 1 << (card->mpa_rx.pkt_cnt + 1);
        }
-       card->mpa_rx.skb_arr[card->mpa_rx.pkt_cnt] = skb;
-       card->mpa_rx.len_arr[card->mpa_rx.pkt_cnt] = skb->len;
+       card->mpa_rx.skb_arr[card->mpa_rx.pkt_cnt] = NULL;
+       card->mpa_rx.len_arr[card->mpa_rx.pkt_cnt] = rx_len;
        card->mpa_rx.pkt_cnt++;
 }
 #endif /* _MWIFIEX_SDIO_H */