Skip to content

Commit d2515a9

Browse files
lkaufman-heegrumbach
authored andcommitted
iwlwifi: mvm: fix inconsistent lock in dqa mode
When working in DQA mode, there is a lockdep log warning about an inconsistent state of the mvmsta->lock and the mvm->queue_info_lock. Fix this. This mode is not activated for now. Signed-off-by: Liad Kaufman <[email protected]> Signed-off-by: Emmanuel Grumbach <[email protected]>
1 parent a6017b9 commit d2515a9

File tree

1 file changed

+4
-4
lines changed
  • drivers/net/wireless/intel/iwlwifi/mvm

1 file changed

+4
-4
lines changed

drivers/net/wireless/intel/iwlwifi/mvm/sta.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,7 @@ static int iwl_mvm_sta_alloc_queue(struct iwl_mvm *mvm,
296296

297297
lockdep_assert_held(&mvm->mutex);
298298

299-
spin_lock(&mvm->queue_info_lock);
299+
spin_lock_bh(&mvm->queue_info_lock);
300300

301301
/*
302302
* Non-QoS, QoS NDP and MGMT frames should go to a MGMT queue, if one
@@ -324,7 +324,7 @@ static int iwl_mvm_sta_alloc_queue(struct iwl_mvm *mvm,
324324
if (queue >= 0)
325325
mvm->queue_info[queue].setup_reserved = false;
326326

327-
spin_unlock(&mvm->queue_info_lock);
327+
spin_unlock_bh(&mvm->queue_info_lock);
328328

329329
/* TODO: support shared queues for same RA */
330330
if (queue < 0)
@@ -402,12 +402,12 @@ static void iwl_mvm_tx_deferred_stream(struct iwl_mvm *mvm,
402402

403403
__skb_queue_head_init(&deferred_tx);
404404

405+
/* Disable bottom-halves when entering TX path */
406+
local_bh_disable();
405407
spin_lock(&mvmsta->lock);
406408
skb_queue_splice_init(&tid_data->deferred_tx_frames, &deferred_tx);
407409
spin_unlock(&mvmsta->lock);
408410

409-
/* Disable bottom-halves when entering TX path */
410-
local_bh_disable();
411411
while ((skb = __skb_dequeue(&deferred_tx)))
412412
if (no_queue || iwl_mvm_tx_skb(mvm, skb, sta))
413413
ieee80211_free_txskb(mvm->hw, skb);

0 commit comments

Comments
 (0)