]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
mt76: move pad estimation out of mt76_skb_adjust_pad
authorLorenzo Bianconi <lorenzo@kernel.org>
Sat, 5 Sep 2020 09:26:04 +0000 (11:26 +0200)
committerFelix Fietkau <nbd@nbd.name>
Thu, 24 Sep 2020 16:10:19 +0000 (18:10 +0200)
Move frame pad computation out of mt76_skb_adjust_pad routine.
This is a preliminary patch to introduce sdio tx aggregation.

Tested-by: Sean Wang <sean.wang@mediatek.com>
Co-developed-by: Sean Wang <sean.wang@mediatek.com>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt76.h
drivers/net/wireless/mediatek/mt76/mt7615/sdio_txrx.c
drivers/net/wireless/mediatek/mt76/mt7615/usb_mcu.c
drivers/net/wireless/mediatek/mt76/mt7615/usb_sdio.c
drivers/net/wireless/mediatek/mt76/mt76x02_usb_core.c
drivers/net/wireless/mediatek/mt76/sdio.c
drivers/net/wireless/mediatek/mt76/tx.c

index bc06d67f7452fb824736e1e56a43c25f9f78b23f..72aa19e0e6ecc57e68e2273e9124499d2d367a79 100644 (file)
@@ -1043,7 +1043,7 @@ mt76u_bulk_msg(struct mt76_dev *dev, void *data, int len, int *actual_len,
        return usb_bulk_msg(udev, pipe, data, len, actual_len, timeout);
 }
 
-int mt76_skb_adjust_pad(struct sk_buff *skb);
+int mt76_skb_adjust_pad(struct sk_buff *skb, int pad);
 int mt76u_vendor_request(struct mt76_dev *dev, u8 req,
                         u8 req_type, u16 val, u16 offset,
                         void *buf, size_t len);
index ebae2a24c45ba870d0e6e7f89465410394bb6f89..4754b3abd0a2ca2c1eac89e57418746eae9772e0 100644 (file)
@@ -190,6 +190,8 @@ static int mt7663s_tx_run_queue(struct mt76_dev *dev, enum mt76_txq_id qid)
                                          &ple_sz))
                        break;
 
+               __skb_put_zero(e->skb, 4);
+
                err = __mt7663s_xmit_queue(dev, e->skb->data, e->skb->len);
                if (err)
                        return err;
index adbed373798e8f12e16198850baf14649adaecdf..4d8be366af31bfd4f4277e61497511eab59a54cd 100644 (file)
@@ -18,8 +18,7 @@ mt7663u_mcu_send_message(struct mt76_dev *mdev, struct sk_buff *skb,
                         int cmd, bool wait_resp)
 {
        struct mt7615_dev *dev = container_of(mdev, struct mt7615_dev, mt76);
-       int ret, seq, ep;
-       u32 len;
+       int ret, seq, ep, len, pad;
 
        mutex_lock(&mdev->mcu.mutex);
 
@@ -31,7 +30,8 @@ mt7663u_mcu_send_message(struct mt76_dev *mdev, struct sk_buff *skb,
 
        len = skb->len;
        put_unaligned_le32(len, skb_push(skb, sizeof(len)));
-       ret = mt76_skb_adjust_pad(skb);
+       pad = round_up(skb->len, 4) + 4 - skb->len;
+       ret = mt76_skb_adjust_pad(skb, pad);
        if (ret < 0)
                goto out;
 
index 56d82bd3a6158fd14a7bf255a8c7800a7550db0c..c24f26499537b09643b4316414d426c75e23c54e 100644 (file)
@@ -247,6 +247,7 @@ int mt7663_usb_sdio_tx_prepare_skb(struct mt76_dev *mdev, void *txwi_ptr,
        struct mt7615_dev *dev = container_of(mdev, struct mt7615_dev, mt76);
        struct sk_buff *skb = tx_info->skb;
        struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb);
+       int pad;
 
        if ((info->flags & IEEE80211_TX_CTL_RATE_CTRL_PROBE) &&
            !msta->rate_probe) {
@@ -262,9 +263,12 @@ int mt7663_usb_sdio_tx_prepare_skb(struct mt76_dev *mdev, void *txwi_ptr,
                u32 len = skb->len;
 
                put_unaligned_le32(len, skb_push(skb, sizeof(len)));
+               pad = round_up(skb->len, 4) + 4 - skb->len;
+       } else {
+               pad = round_up(skb->len, 4) - skb->len;
        }
 
-       return mt76_skb_adjust_pad(skb);
+       return mt76_skb_adjust_pad(skb, pad);
 }
 EXPORT_SYMBOL_GPL(mt7663_usb_sdio_tx_prepare_skb);
 
index 5d0a2857f193abafc711a323163cc9445865bba6..2c2f56112b576b5aaaca30b8ace0920e068b527f 100644 (file)
@@ -45,7 +45,7 @@ EXPORT_SYMBOL_GPL(mt76x02u_mac_start);
 
 int mt76x02u_skb_dma_info(struct sk_buff *skb, int port, u32 flags)
 {
-       u32 info;
+       u32 info, pad;
 
        /* Buffer layout:
         *      |   4B   | xfer len |      pad       |  4B  |
@@ -57,7 +57,8 @@ int mt76x02u_skb_dma_info(struct sk_buff *skb, int port, u32 flags)
               FIELD_PREP(MT_TXD_INFO_DPORT, port) | flags;
        put_unaligned_le32(info, skb_push(skb, sizeof(info)));
 
-       return mt76_skb_adjust_pad(skb);
+       pad = round_up(skb->len, 4) + 4 - skb->len;
+       return mt76_skb_adjust_pad(skb, pad);
 }
 
 int mt76x02u_tx_prepare_skb(struct mt76_dev *mdev, void *data,
index 7777d4944081ab9dd3b9092a4f9f427dc23be8dd..9a4d95a2a7072509c6749878c6078bc76d3b8b00 100644 (file)
@@ -226,12 +226,13 @@ mt76s_tx_queue_skb_raw(struct mt76_dev *dev, enum mt76_txq_id qid,
                       struct sk_buff *skb, u32 tx_info)
 {
        struct mt76_queue *q = dev->q_tx[qid];
-       int ret = -ENOSPC, len = skb->len;
+       int ret = -ENOSPC, len = skb->len, pad;
 
        if (q->queued == q->ndesc)
                goto error;
 
-       ret = mt76_skb_adjust_pad(skb);
+       pad = round_up(skb->len, 4) - skb->len;
+       ret = mt76_skb_adjust_pad(skb, pad);
        if (ret)
                goto error;
 
index bf7a6b3a8db0b5110b4cd6e7a7153976d57f0a86..44ef4bc7a46e5bdd270c464db81e9cf7276dc066 100644 (file)
@@ -589,13 +589,9 @@ u8 mt76_ac_to_hwq(u8 ac)
 }
 EXPORT_SYMBOL_GPL(mt76_ac_to_hwq);
 
-int mt76_skb_adjust_pad(struct sk_buff *skb)
+int mt76_skb_adjust_pad(struct sk_buff *skb, int pad)
 {
        struct sk_buff *iter, *last = skb;
-       u32 pad;
-
-       /* Add zero pad of 4 - 7 bytes */
-       pad = round_up(skb->len, 4) + 4 - skb->len;
 
        /* First packet of a A-MSDU burst keeps track of the whole burst
         * length, need to update length of it and the last packet.