iwlwifi: mvm: use atomic interface iteration to avoid deadlock
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Mon, 11 Feb 2013 13:56:01 +0000 (15:56 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 12 Feb 2013 15:07:14 +0000 (16:07 +0100)
Using the non-atomic version creates a dependency between
mac80211's iflist_mtx and mvm->mutex. Use the atomic version
instead which doesn't take iflist_mtx but can't sleep, so
send the HCMD in ASYNC.

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/iwlwifi/mvm/mac80211.c
drivers/net/wireless/iwlwifi/mvm/power.c

index a6b05a02cfd4b192c86ccb63cc167e18d4c557c1..64acc4a597fb9df3709a3926169d6df868d3c05e 100644 (file)
@@ -475,7 +475,7 @@ static int iwl_mvm_mac_add_interface(struct ieee80211_hw *hw,
        if (mvm->vif_count > 1) {
                IWL_DEBUG_MAC80211(mvm,
                                   "Disable power on existing interfaces\n");
-               ieee80211_iterate_active_interfaces(
+               ieee80211_iterate_active_interfaces_atomic(
                                            mvm->hw,
                                            IEEE80211_IFACE_ITER_NORMAL,
                                            iwl_mvm_pm_disable_iterator, mvm);
index 63628739cf4a9494597b63c364a34f31f6a0ad55..5a92a4978795420acbd363ca0ca94037e47f5b28 100644 (file)
@@ -194,7 +194,7 @@ int iwl_mvm_power_disable(struct iwl_mvm *mvm, struct ieee80211_vif *vif)
                        cmd.id_and_color, iwlmvm_mod_params.power_scheme,
                        le16_to_cpu(cmd.flags));
 
-       return iwl_mvm_send_cmd_pdu(mvm, POWER_TABLE_CMD, CMD_SYNC,
+       return iwl_mvm_send_cmd_pdu(mvm, POWER_TABLE_CMD, CMD_ASYNC,
                                    sizeof(cmd), &cmd);
 }