Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/linville/wirel...
authorJohn W. Linville <linville@tuxdriver.com>
Mon, 8 Feb 2010 21:38:38 +0000 (16:38 -0500)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 8 Feb 2010 21:38:38 +0000 (16:38 -0500)
Conflicts:
net/mac80211/scan.c

1  2 
drivers/net/wireless/ath/ath9k/hw.c
drivers/net/wireless/ath/ath9k/main.c
drivers/net/wireless/b43/b43.h
drivers/net/wireless/b43/main.c
drivers/net/wireless/iwlwifi/iwl-core.c
drivers/net/wireless/iwlwifi/iwl-sta.c
net/mac80211/driver-trace.h
net/mac80211/ibss.c
net/mac80211/scan.c

Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -345,6 -439,16 +345,13 @@@ static int __ieee80211_start_scan(struc
        if (local->scan_req)
                return -EBUSY;
  
 -      if (req != local->int_scan_req &&
 -          sdata->vif.type == NL80211_IFTYPE_STATION &&
 -          !list_empty(&ifmgd->work_list)) {
 -              /* actually wait for the work it's doing to finish/time out */
 -              set_bit(IEEE80211_STA_REQ_SCAN, &ifmgd->request);
++      if (!list_empty(&local->work_list)) {
++              /* wait for the work to finish/time out */
+               local->scan_req = req;
+               local->scan_sdata = sdata;
+               return 0;
+       }
        if (local->ops->hw_scan) {
                u8 *ies;
  
                local->hw_scan_req->ie = ies;
  
                local->hw_scan_band = 0;
++
++              /*
++               * After allocating local->hw_scan_req, we must
++               * go through until ieee80211_prep_hw_scan(), so
++               * anything that might be changed here and leave
++               * this function early must not go after this
++               * allocation.
++               */
        }
  
        local->scan_req = req;
                __set_bit(SCAN_HW_SCANNING, &local->scanning);
        else
                __set_bit(SCAN_SW_SCANNING, &local->scanning);
++
        /*
         * Kicking off the scan need not be protected,
         * only the scan variable stuff, since now
         * local->scan_req is assigned and other callers
         * will abort their scan attempts.
         *
--       * This avoids getting a scan_mtx -> iflist_mtx
--       * dependency, so that the scan completed calls
--       * have more locking freedom.
++       * This avoids too many locking dependencies
++       * so that the scan completed calls have more
++       * locking freedom.
         */
  
        ieee80211_recalc_idle(local);