From: "Guy, Wey-Yi" <wey-yi.w.guy@intel.com>
To: Stanislaw Gruszka <sgruszka@redhat.com>
Cc: Johannes Berg <johannes@sipsolutions.net>,
"Chatre, Reinette" <reinette.chatre@intel.com>,
"John W. Linville" <linville@tuxdriver.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"Berg, Johannes" <johannes.berg@intel.com>
Subject: Re: [PATCH 02/14] iwlwifi: unify scan start checks
Date: Fri, 10 Sep 2010 08:49:57 -0700 [thread overview]
Message-ID: <1284133797.6054.8.camel@wwguy-ubuntu> (raw)
In-Reply-To: <1284128807-11436-3-git-send-email-sgruszka@redhat.com>
On Fri, 2010-09-10 at 07:26 -0700, Stanislaw Gruszka wrote:
> From: Johannes Berg <johannes.berg@intel.com>
>
> Rather than duplicating all the checks and even
> in case of errors accepting the scan request
> from mac80211, we can push the checks to the
> caller and in all error cases reject the scan
> request right away (rather than accepting and
> then saying it was aborted).
>
> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
> Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Acked-by: Wey-Yi W Guy <wey-yi.w.guy@intel.com>
> ---
> drivers/net/wireless/iwlwifi/iwl-3945.h | 2 +-
> drivers/net/wireless/iwlwifi/iwl-agn-lib.c | 88 +++++--------------
> drivers/net/wireless/iwlwifi/iwl-agn.h | 2 +-
> drivers/net/wireless/iwlwifi/iwl-core.h | 2 +-
> drivers/net/wireless/iwlwifi/iwl-scan.c | 127 ++++++++++++++++-----------
> drivers/net/wireless/iwlwifi/iwl3945-base.c | 74 ++--------------
> 6 files changed, 108 insertions(+), 187 deletions(-)
>
> diff --git a/drivers/net/wireless/iwlwifi/iwl-3945.h b/drivers/net/wireless/iwlwifi/iwl-3945.h
> index bb2aeeb..98509c5 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-3945.h
> +++ b/drivers/net/wireless/iwlwifi/iwl-3945.h
> @@ -295,7 +295,7 @@ extern const struct iwl_channel_info *iwl3945_get_channel_info(
> extern int iwl3945_rs_next_rate(struct iwl_priv *priv, int rate);
>
> /* scanning */
> -void iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif);
> +int iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif);
>
> /* Requires full declaration of iwl_priv before including */
> #include "iwl-io.h"
> diff --git a/drivers/net/wireless/iwlwifi/iwl-agn-lib.c b/drivers/net/wireless/iwlwifi/iwl-agn-lib.c
> index 51a8d7e..f89dd81 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-agn-lib.c
> +++ b/drivers/net/wireless/iwlwifi/iwl-agn-lib.c
> @@ -1154,7 +1154,7 @@ static int iwl_get_channels_for_scan(struct iwl_priv *priv,
> return added;
> }
>
> -void iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> +int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> {
> struct iwl_host_cmd cmd = {
> .id = REPLY_SCAN_CMD,
> @@ -1174,57 +1174,20 @@ void iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> int chan_mod;
> u8 active_chains;
> u8 scan_tx_antennas = priv->hw_params.valid_tx_ant;
> + int ret;
> +
> + lockdep_assert_held(&priv->mutex);
>
> if (vif)
> ctx = iwl_rxon_ctx_from_vif(vif);
>
> - cancel_delayed_work(&priv->scan_check);
> -
> - if (!iwl_is_ready(priv)) {
> - IWL_WARN(priv, "request scan called when driver not ready.\n");
> - goto done;
> - }
> -
> - /* Make sure the scan wasn't canceled before this queued work
> - * was given the chance to run... */
> - if (!test_bit(STATUS_SCANNING, &priv->status))
> - goto done;
> -
> - /* This should never be called or scheduled if there is currently
> - * a scan active in the hardware. */
> - if (test_bit(STATUS_SCAN_HW, &priv->status)) {
> - IWL_DEBUG_INFO(priv, "Multiple concurrent scan requests in parallel. "
> - "Ignoring second request.\n");
> - goto done;
> - }
> -
> - if (test_bit(STATUS_EXIT_PENDING, &priv->status)) {
> - IWL_DEBUG_SCAN(priv, "Aborting scan due to device shutdown\n");
> - goto done;
> - }
> -
> - if (test_bit(STATUS_SCAN_ABORTING, &priv->status)) {
> - IWL_DEBUG_HC(priv, "Scan request while abort pending. Queuing.\n");
> - goto done;
> - }
> -
> - if (iwl_is_rfkill(priv)) {
> - IWL_DEBUG_HC(priv, "Aborting scan due to RF Kill activation\n");
> - goto done;
> - }
> -
> - if (!test_bit(STATUS_READY, &priv->status)) {
> - IWL_DEBUG_HC(priv, "Scan request while uninitialized. Queuing.\n");
> - goto done;
> - }
> -
> if (!priv->scan_cmd) {
> priv->scan_cmd = kmalloc(sizeof(struct iwl_scan_cmd) +
> IWL_MAX_SCAN_SIZE, GFP_KERNEL);
> if (!priv->scan_cmd) {
> IWL_DEBUG_SCAN(priv,
> "fail to allocate memory for scan\n");
> - goto done;
> + return -ENOMEM;
> }
> }
> scan = priv->scan_cmd;
> @@ -1331,8 +1294,8 @@ void iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> IWL_GOOD_CRC_TH_NEVER;
> break;
> default:
> - IWL_WARN(priv, "Invalid scan band count\n");
> - goto done;
> + IWL_WARN(priv, "Invalid scan band\n");
> + return -EIO;
> }
>
> band = priv->scan_band;
> @@ -1412,7 +1375,7 @@ void iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> }
> if (scan->channel_count == 0) {
> IWL_DEBUG_SCAN(priv, "channel count %d\n", scan->channel_count);
> - goto done;
> + return -EIO;
> }
>
> cmd.len += le16_to_cpu(scan->tx_cmd.len) +
> @@ -1420,30 +1383,21 @@ void iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> cmd.data = scan;
> scan->len = cpu_to_le16(cmd.len);
>
> - set_bit(STATUS_SCAN_HW, &priv->status);
> -
> - if (priv->cfg->ops->hcmd->set_pan_params &&
> - priv->cfg->ops->hcmd->set_pan_params(priv))
> - goto done;
> + if (priv->cfg->ops->hcmd->set_pan_params) {
> + ret = priv->cfg->ops->hcmd->set_pan_params(priv);
> + if (ret)
> + return ret;
> + }
>
> - if (iwl_send_cmd_sync(priv, &cmd))
> - goto done;
> + set_bit(STATUS_SCAN_HW, &priv->status);
> + ret = iwl_send_cmd_sync(priv, &cmd);
> + if (ret) {
> + clear_bit(STATUS_SCAN_HW, &priv->status);
> + if (priv->cfg->ops->hcmd->set_pan_params)
> + priv->cfg->ops->hcmd->set_pan_params(priv);
> + }
>
> - queue_delayed_work(priv->workqueue, &priv->scan_check,
> - IWL_SCAN_CHECK_WATCHDOG);
> -
> - return;
> -
> - done:
> - /* Cannot perform scan. Make sure we clear scanning
> - * bits from status so next scan request can be performed.
> - * If we don't clear scanning status bit here all next scan
> - * will fail
> - */
> - clear_bit(STATUS_SCAN_HW, &priv->status);
> - clear_bit(STATUS_SCANNING, &priv->status);
> - /* inform mac80211 scan aborted */
> - queue_work(priv->workqueue, &priv->scan_completed);
> + return ret;
> }
>
> int iwlagn_manage_ibss_station(struct iwl_priv *priv,
> diff --git a/drivers/net/wireless/iwlwifi/iwl-agn.h b/drivers/net/wireless/iwlwifi/iwl-agn.h
> index ab7c16f..e1f85ae 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-agn.h
> +++ b/drivers/net/wireless/iwlwifi/iwl-agn.h
> @@ -219,7 +219,7 @@ void iwl_reply_statistics(struct iwl_priv *priv,
> struct iwl_rx_mem_buffer *rxb);
>
> /* scan */
> -void iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif);
> +int iwlagn_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif);
>
> /* station mgmt */
> int iwlagn_manage_ibss_station(struct iwl_priv *priv,
> diff --git a/drivers/net/wireless/iwlwifi/iwl-core.h b/drivers/net/wireless/iwlwifi/iwl-core.h
> index f7b57ed..8dab074 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-core.h
> +++ b/drivers/net/wireless/iwlwifi/iwl-core.h
> @@ -111,7 +111,7 @@ struct iwl_hcmd_utils_ops {
> __le16 fc, __le32 *tx_flags);
> int (*calc_rssi)(struct iwl_priv *priv,
> struct iwl_rx_phy_res *rx_resp);
> - void (*request_scan)(struct iwl_priv *priv, struct ieee80211_vif *vif);
> + int (*request_scan)(struct iwl_priv *priv, struct ieee80211_vif *vif);
> };
>
> struct iwl_apm_ops {
> diff --git a/drivers/net/wireless/iwlwifi/iwl-scan.c b/drivers/net/wireless/iwlwifi/iwl-scan.c
> index 7727f09..6bd14bc 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-scan.c
> +++ b/drivers/net/wireless/iwlwifi/iwl-scan.c
> @@ -324,19 +324,68 @@ void iwl_init_scan_params(struct iwl_priv *priv)
> }
> EXPORT_SYMBOL(iwl_init_scan_params);
>
> -static int iwl_scan_initiate(struct iwl_priv *priv, struct ieee80211_vif *vif)
> +static int __must_check iwl_scan_initiate(struct iwl_priv *priv,
> + struct ieee80211_vif *vif,
> + bool internal,
> + enum nl80211_band band)
> {
> + int ret;
> +
> lockdep_assert_held(&priv->mutex);
>
> - IWL_DEBUG_INFO(priv, "Starting scan...\n");
> + if (WARN_ON(!priv->cfg->ops->utils->request_scan))
> + return -EOPNOTSUPP;
> +
> + cancel_delayed_work(&priv->scan_check);
> +
> + if (!iwl_is_ready(priv)) {
> + IWL_WARN(priv, "request scan called when driver not ready.\n");
> + return -EIO;
> + }
> +
> + if (test_bit(STATUS_SCAN_HW, &priv->status)) {
> + IWL_DEBUG_INFO(priv,
> + "Multiple concurrent scan requests in parallel.\n");
> + return -EBUSY;
> + }
> +
> + if (test_bit(STATUS_EXIT_PENDING, &priv->status)) {
> + IWL_DEBUG_SCAN(priv, "Aborting scan due to device shutdown\n");
> + return -EIO;
> + }
> +
> + if (test_bit(STATUS_SCAN_ABORTING, &priv->status)) {
> + IWL_DEBUG_HC(priv, "Scan request while abort pending.\n");
> + return -EBUSY;
> + }
> +
> + if (iwl_is_rfkill(priv)) {
> + IWL_DEBUG_HC(priv, "Aborting scan due to RF Kill activation\n");
> + return -EIO;
> + }
> +
> + if (!test_bit(STATUS_READY, &priv->status)) {
> + IWL_DEBUG_HC(priv, "Scan request while uninitialized.\n");
> + return -EBUSY;
> + }
> +
> + IWL_DEBUG_INFO(priv, "Starting %sscan...\n",
> + internal ? "internal short " : "");
> +
> set_bit(STATUS_SCANNING, &priv->status);
> - priv->is_internal_short_scan = false;
> + priv->is_internal_short_scan = internal;
> priv->scan_start = jiffies;
> + priv->scan_band = band;
>
> - if (WARN_ON(!priv->cfg->ops->utils->request_scan))
> - return -EOPNOTSUPP;
> + ret = priv->cfg->ops->utils->request_scan(priv, vif);
> + if (ret) {
> + clear_bit(STATUS_SCANNING, &priv->status);
> + priv->is_internal_short_scan = false;
> + return ret;
> + }
>
> - priv->cfg->ops->utils->request_scan(priv, vif);
> + queue_delayed_work(priv->workqueue, &priv->scan_check,
> + IWL_SCAN_CHECK_WATCHDOG);
>
> return 0;
> }
> @@ -355,12 +404,6 @@ int iwl_mac_hw_scan(struct ieee80211_hw *hw,
>
> mutex_lock(&priv->mutex);
>
> - if (!iwl_is_ready_rf(priv)) {
> - ret = -EIO;
> - IWL_DEBUG_MAC80211(priv, "leave - not ready or exit pending\n");
> - goto out_unlock;
> - }
> -
> if (test_bit(STATUS_SCANNING, &priv->status) &&
> !priv->is_internal_short_scan) {
> IWL_DEBUG_SCAN(priv, "Scan already in progress.\n");
> @@ -368,14 +411,7 @@ int iwl_mac_hw_scan(struct ieee80211_hw *hw,
> goto out_unlock;
> }
>
> - if (test_bit(STATUS_SCAN_ABORTING, &priv->status)) {
> - IWL_DEBUG_SCAN(priv, "Scan request while abort pending\n");
> - ret = -EAGAIN;
> - goto out_unlock;
> - }
> -
> /* mac80211 will only ask for one band at a time */
> - priv->scan_band = req->channels[0]->band;
> priv->scan_request = req;
> priv->scan_vif = vif;
>
> @@ -386,7 +422,8 @@ int iwl_mac_hw_scan(struct ieee80211_hw *hw,
> if (priv->is_internal_short_scan)
> ret = 0;
> else
> - ret = iwl_scan_initiate(priv, vif);
> + ret = iwl_scan_initiate(priv, vif, false,
> + req->channels[0]->band);
>
> IWL_DEBUG_MAC80211(priv, "leave\n");
>
> @@ -418,31 +455,13 @@ static void iwl_bg_start_internal_scan(struct work_struct *work)
> goto unlock;
> }
>
> - if (!iwl_is_ready_rf(priv)) {
> - IWL_DEBUG_SCAN(priv, "not ready or exit pending\n");
> - goto unlock;
> - }
> -
> if (test_bit(STATUS_SCANNING, &priv->status)) {
> IWL_DEBUG_SCAN(priv, "Scan already in progress.\n");
> goto unlock;
> }
>
> - if (test_bit(STATUS_SCAN_ABORTING, &priv->status)) {
> - IWL_DEBUG_SCAN(priv, "Scan request while abort pending\n");
> - goto unlock;
> - }
> -
> - priv->scan_band = priv->band;
> -
> - IWL_DEBUG_SCAN(priv, "Start internal short scan...\n");
> - set_bit(STATUS_SCANNING, &priv->status);
> - priv->is_internal_short_scan = true;
> -
> - if (WARN_ON(!priv->cfg->ops->utils->request_scan))
> - goto unlock;
> -
> - priv->cfg->ops->utils->request_scan(priv, NULL);
> + if (iwl_scan_initiate(priv, NULL, true, priv->band))
> + IWL_DEBUG_SCAN(priv, "failed to start internal short scan\n");
> unlock:
> mutex_unlock(&priv->mutex);
> }
> @@ -536,7 +555,6 @@ static void iwl_bg_scan_completed(struct work_struct *work)
> struct iwl_priv *priv =
> container_of(work, struct iwl_priv, scan_completed);
> bool internal = false;
> - bool scan_completed = false;
> struct iwl_rxon_context *ctx;
>
> IWL_DEBUG_SCAN(priv, "SCAN complete scan\n");
> @@ -549,16 +567,27 @@ static void iwl_bg_scan_completed(struct work_struct *work)
> IWL_DEBUG_SCAN(priv, "internal short scan completed\n");
> internal = true;
> } else if (priv->scan_request) {
> - scan_completed = true;
> priv->scan_request = NULL;
> priv->scan_vif = NULL;
> + ieee80211_scan_completed(priv->hw, false);
> }
>
> if (test_bit(STATUS_EXIT_PENDING, &priv->status))
> goto out;
>
> - if (internal && priv->scan_request)
> - iwl_scan_initiate(priv, priv->scan_vif);
> + if (internal && priv->scan_request) {
> + int err = iwl_scan_initiate(priv, priv->scan_vif, false,
> + priv->scan_request->channels[0]->band);
> +
> + if (err) {
> + IWL_DEBUG_SCAN(priv,
> + "failed to initiate pending scan: %d\n", err);
> + priv->scan_request = NULL;
> + priv->scan_vif = NULL;
> + ieee80211_scan_completed(priv->hw, true);
> + } else
> + goto out;
> + }
>
> /* Since setting the TXPOWER may have been deferred while
> * performing the scan, fire one off */
> @@ -571,19 +600,11 @@ static void iwl_bg_scan_completed(struct work_struct *work)
> for_each_context(priv, ctx)
> iwlcore_commit_rxon(priv, ctx);
>
> - out:
> if (priv->cfg->ops->hcmd->set_pan_params)
> priv->cfg->ops->hcmd->set_pan_params(priv);
>
> + out:
> mutex_unlock(&priv->mutex);
> -
> - /*
> - * Do not hold mutex here since this will cause mac80211 to call
> - * into driver again into functions that will attempt to take
> - * mutex.
> - */
> - if (scan_completed)
> - ieee80211_scan_completed(priv->hw, false);
> }
>
> void iwl_setup_scan_deferred_work(struct iwl_priv *priv)
> diff --git a/drivers/net/wireless/iwlwifi/iwl3945-base.c b/drivers/net/wireless/iwlwifi/iwl3945-base.c
> index 5b742eb..3b3686b 100644
> --- a/drivers/net/wireless/iwlwifi/iwl3945-base.c
> +++ b/drivers/net/wireless/iwlwifi/iwl3945-base.c
> @@ -2817,7 +2817,7 @@ static void iwl3945_rfkill_poll(struct work_struct *data)
>
> }
>
> -void iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> +int iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> {
> struct iwl_host_cmd cmd = {
> .id = REPLY_SCAN_CMD,
> @@ -2828,55 +2828,16 @@ void iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> u8 n_probes = 0;
> enum ieee80211_band band;
> bool is_active = false;
> + int ret;
>
> - cancel_delayed_work(&priv->scan_check);
> -
> - if (!iwl_is_ready(priv)) {
> - IWL_WARN(priv, "request scan called when driver not ready.\n");
> - goto done;
> - }
> -
> - /* Make sure the scan wasn't canceled before this queued work
> - * was given the chance to run... */
> - if (!test_bit(STATUS_SCANNING, &priv->status))
> - goto done;
> -
> - /* This should never be called or scheduled if there is currently
> - * a scan active in the hardware. */
> - if (test_bit(STATUS_SCAN_HW, &priv->status)) {
> - IWL_DEBUG_INFO(priv, "Multiple concurrent scan requests "
> - "Ignoring second request.\n");
> - goto done;
> - }
> -
> - if (test_bit(STATUS_EXIT_PENDING, &priv->status)) {
> - IWL_DEBUG_SCAN(priv, "Aborting scan due to device shutdown\n");
> - goto done;
> - }
> -
> - if (test_bit(STATUS_SCAN_ABORTING, &priv->status)) {
> - IWL_DEBUG_HC(priv,
> - "Scan request while abort pending. Queuing.\n");
> - goto done;
> - }
> -
> - if (iwl_is_rfkill(priv)) {
> - IWL_DEBUG_HC(priv, "Aborting scan due to RF Kill activation\n");
> - goto done;
> - }
> -
> - if (!test_bit(STATUS_READY, &priv->status)) {
> - IWL_DEBUG_HC(priv,
> - "Scan request while uninitialized. Queuing.\n");
> - goto done;
> - }
> + lockdep_assert_held(&priv->mutex);
>
> if (!priv->scan_cmd) {
> priv->scan_cmd = kmalloc(sizeof(struct iwl3945_scan_cmd) +
> IWL_MAX_SCAN_SIZE, GFP_KERNEL);
> if (!priv->scan_cmd) {
> IWL_DEBUG_SCAN(priv, "Fail to allocate scan memory\n");
> - goto done;
> + return -ENOMEM;
> }
> }
> scan = priv->scan_cmd;
> @@ -2971,7 +2932,7 @@ void iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> break;
> default:
> IWL_WARN(priv, "Invalid scan band\n");
> - goto done;
> + return -EIO;
> }
>
> if (!priv->is_internal_short_scan) {
> @@ -3006,7 +2967,7 @@ void iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
>
> if (scan->channel_count == 0) {
> IWL_DEBUG_SCAN(priv, "channel count %d\n", scan->channel_count);
> - goto done;
> + return -EIO;
> }
>
> cmd.len += le16_to_cpu(scan->tx_cmd.len) +
> @@ -3015,25 +2976,10 @@ void iwl3945_request_scan(struct iwl_priv *priv, struct ieee80211_vif *vif)
> scan->len = cpu_to_le16(cmd.len);
>
> set_bit(STATUS_SCAN_HW, &priv->status);
> - if (iwl_send_cmd_sync(priv, &cmd))
> - goto done;
> -
> - queue_delayed_work(priv->workqueue, &priv->scan_check,
> - IWL_SCAN_CHECK_WATCHDOG);
> -
> - return;
> -
> - done:
> - /* can not perform scan make sure we clear scanning
> - * bits from status so next scan request can be performed.
> - * if we dont clear scanning status bit here all next scan
> - * will fail
> - */
> - clear_bit(STATUS_SCAN_HW, &priv->status);
> - clear_bit(STATUS_SCANNING, &priv->status);
> -
> - /* inform mac80211 scan aborted */
> - queue_work(priv->workqueue, &priv->scan_completed);
> + ret = iwl_send_cmd_sync(priv, &cmd);
> + if (ret)
> + clear_bit(STATUS_SCAN_HW, &priv->status);
> + return ret;
> }
>
> static void iwl3945_bg_restart(struct work_struct *data)
> --
> 1.7.1
>
next prev parent reply other threads:[~2010-09-10 15:50 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-10 14:26 [PATCH 0/14 v3] iwlwifi: rewrite iwl-scan.c to avoid race conditions Stanislaw Gruszka
2010-09-10 14:26 ` [PATCH 01/14] iwlwifi: remove unused conf variables Stanislaw Gruszka
2010-09-10 15:49 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 02/14] iwlwifi: unify scan start checks Stanislaw Gruszka
2010-09-10 15:49 ` Guy, Wey-Yi [this message]
2010-09-10 14:26 ` [PATCH 03/14] iwlwifi: move scan completed flags handling Stanislaw Gruszka
2010-09-10 15:50 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 04/14] iwlwifi: cancel scan when down the device Stanislaw Gruszka
2010-09-10 15:50 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 05/14] iwlwifi: report scan completion when abort fail Stanislaw Gruszka
2010-09-10 15:51 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 06/14] iwlwifi: rework iwl_scan_cancel_timeout Stanislaw Gruszka
2010-09-10 15:52 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 07/14] iwlwifi: rewrite scan completion Stanislaw Gruszka
2010-09-10 15:52 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 08/14] iwlwifi: force scan complete after timeout Stanislaw Gruszka
2010-09-10 15:53 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 09/14] iwlwifi: assure we complete scan in scan_abort and scan_check works Stanislaw Gruszka
2010-09-10 15:54 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 10/14] iwlwifi: do not force complete scan too early Stanislaw Gruszka
2010-09-10 15:54 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 11/14] mac80211: wait for scan work complete before restarting hw Stanislaw Gruszka
2010-09-10 15:55 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 12/14] iwlwifi: cleanup scan initiate check Stanislaw Gruszka
2010-09-10 15:55 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 13/14] iwlwifi: use IWL_DEBUG_SCAN for debug scanning Stanislaw Gruszka
2010-09-10 15:56 ` Guy, Wey-Yi
2010-09-10 14:26 ` [PATCH 14/14] iwlwifi: apply settings when finishing scan Stanislaw Gruszka
2010-09-10 15:56 ` Guy, Wey-Yi
-- strict thread matches above, loose matches on Subject: below --
2010-09-13 12:46 [PATCH 0/14 v4] iwlwifi: rewrite iwl-scan.c to avoid race conditions Stanislaw Gruszka
2010-09-13 12:46 ` [PATCH 02/14] iwlwifi: unify scan start checks Stanislaw Gruszka
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1284133797.6054.8.camel@wwguy-ubuntu \
--to=wey-yi.w.guy@intel.com \
--cc=johannes.berg@intel.com \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=reinette.chatre@intel.com \
--cc=sgruszka@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).