mirror of
https://github.com/outbackdingo/mt76.git
synced 2026-10-24 13:26:48 +00:00
wifi: mt76: sdio: move mcu queue size check inside critical section
Even if it is not a real issue at the moment since concurrent access to mcu message queue is protected by mcu mutex, make the code more robust and move mcu queue free space check inside queue spinlock critical section. Signed-off-by: Lorenzo Bianconi <[email protected]> Signed-off-by: Felix Fietkau <[email protected]>
This commit is contained in:
committed by
Felix Fietkau
parent
b96ab5e620
commit
7ff11c7259
@@ -550,10 +550,7 @@ static int
|
||||
mt76s_tx_queue_skb_raw(struct mt76_dev *dev, struct mt76_queue *q,
|
||||
struct sk_buff *skb, u32 tx_info)
|
||||
{
|
||||
int ret = -ENOSPC, len = skb->len, pad;
|
||||
|
||||
if (q->queued == q->ndesc)
|
||||
goto error;
|
||||
int ret, len = skb->len, pad;
|
||||
|
||||
pad = round_up(skb->len, 4) - skb->len;
|
||||
ret = mt76_skb_adjust_pad(skb, pad);
|
||||
@@ -562,6 +559,12 @@ mt76s_tx_queue_skb_raw(struct mt76_dev *dev, struct mt76_queue *q,
|
||||
|
||||
spin_lock_bh(&q->lock);
|
||||
|
||||
if (q->queued == q->ndesc) {
|
||||
ret = -ENOSPC;
|
||||
spin_unlock_bh(&q->lock);
|
||||
goto error;
|
||||
}
|
||||
|
||||
q->entry[q->head].buf_sz = len;
|
||||
q->entry[q->head].skb = skb;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user