wil6210: relax spinlocks in rx reorder

In the Rx reorder mechanism, nothing is done in the interrupt
context, so there is no need to use 'irq' flavors of spinlock.
Rx done in NAPI context (tasklet), other manipulations - in the
thread context.

Having interrupts enabled makes it better for the OS in general.
Besides, if enslaved under bonding, bridge or team driver, Rx
won't work with interrupts disabled.

Signed-off-by: Vladimir Kondratiev <qca_vkondrat@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

1 files changed, 4 insertions, 3 deletions

diff --git a/drivers/net/wireless/ath/wil6210/wmi.c b/drivers/net/wireless/ath/wil6210/wmi.c
index 432ec55298f2..f3524a15b1ad 100644
--- a/drivers/net/wireless/ath/wil6210/wmi.c
+++ b/drivers/net/wireless/ath/wil6210/wmi.c
@@ -644,14 +644,15 @@ static void wmi_evt_addba_rx_req(struct wil6210_priv *wil, int id, void *d,
 }
 
 static void wmi_evt_delba(struct wil6210_priv *wil, int id, void *d, int len)
+__acquires(&sta->tid_rx_lock) __releases(&sta->tid_rx_lock)
 {
         struct wmi_delba_event *evt = d;
         u8 cid, tid;
         u16 reason = __le16_to_cpu(evt->reason);
         struct wil_sta_info *sta;
         struct wil_tid_ampdu_rx *r;
-        unsigned long flags;
 
+        might_sleep();
         parse_cidxtid(evt->cidxtid, &cid, &tid);
         wil_dbg_wmi(wil, "DELBA CID %d TID %d from %s reason %d\n",
                     cid, tid,
@@ -681,13 +682,13 @@ static void wmi_evt_delba(struct wil6210_priv *wil, int id, void *d, int len)
 
         sta = &wil->sta[cid];
 
-        spin_lock_irqsave(&sta->tid_rx_lock, flags);
+        spin_lock_bh(&sta->tid_rx_lock);
 
         r = sta->tid_rx[tid];
         sta->tid_rx[tid] = NULL;
         wil_tid_ampdu_rx_free(wil, r);
 
-        spin_unlock_irqrestore(&sta->tid_rx_lock, flags);
+        spin_unlock_bh(&sta->tid_rx_lock);
 }
 
 static const struct {