svn commit: r330154 - stable/11/sys/dev/iwm
Eitan Adler
eadler at FreeBSD.org
Thu Mar 1 04:56:50 UTC 2018
Author: eadler
Date: Thu Mar 1 04:56:49 2018
New Revision: 330154
URL: https://svnweb.freebsd.org/changeset/base/330154
Log:
MFC r306007:
[iwm] Improve reliability of iwm_release on disassociation a bit.
* We need to first call ivp->iv_newstate(), to enqueue the deauth/deassoc
mgmt frame, then flush the tx queue, before actually calling
iwm_release().
* cycling a wlan connection via wpa_cli frontend to wpa_supplicant, by
issuing disconnect and reconnect commandos works quite well.
(There is still an issue when disconnecting/reconnecting too quickly)
* Reassociating or roaming via wpa_supplicant is still broken.
Modified:
stable/11/sys/dev/iwm/if_iwm.c
Directory Properties:
stable/11/ (props changed)
Modified: stable/11/sys/dev/iwm/if_iwm.c
==============================================================================
--- stable/11/sys/dev/iwm/if_iwm.c Thu Mar 1 04:55:00 2018 (r330153)
+++ stable/11/sys/dev/iwm/if_iwm.c Thu Mar 1 04:56:49 2018 (r330154)
@@ -329,6 +329,8 @@ static int iwm_tx(struct iwm_softc *, struct mbuf *,
struct ieee80211_node *, int);
static int iwm_raw_xmit(struct ieee80211_node *, struct mbuf *,
const struct ieee80211_bpf_params *);
+static int iwm_mvm_flush_tx_path(struct iwm_softc *sc,
+ uint32_t tfd_msk, uint32_t flags);
static int iwm_mvm_send_add_sta_cmd_status(struct iwm_softc *,
struct iwm_mvm_add_sta_cmd_v7 *,
int *);
@@ -3512,7 +3514,6 @@ iwm_raw_xmit(struct ieee80211_node *ni, struct mbuf *m
* mvm/tx.c
*/
-#if 0
/*
* Note that there are transports that buffer frames before they reach
* the firmware. This means that after flush_tx_path is called, the
@@ -3522,23 +3523,21 @@ iwm_raw_xmit(struct ieee80211_node *ni, struct mbuf *m
* 3) wait for the transport queues to be empty
*/
int
-iwm_mvm_flush_tx_path(struct iwm_softc *sc, int tfd_msk, int sync)
+iwm_mvm_flush_tx_path(struct iwm_softc *sc, uint32_t tfd_msk, uint32_t flags)
{
+ int ret;
struct iwm_tx_path_flush_cmd flush_cmd = {
.queues_ctl = htole32(tfd_msk),
.flush_ctl = htole16(IWM_DUMP_TX_FIFO_FLUSH),
};
- int ret;
- ret = iwm_mvm_send_cmd_pdu(sc, IWM_TXPATH_FLUSH,
- sync ? IWM_CMD_SYNC : IWM_CMD_ASYNC,
+ ret = iwm_mvm_send_cmd_pdu(sc, IWM_TXPATH_FLUSH, flags,
sizeof(flush_cmd), &flush_cmd);
if (ret)
device_printf(sc->sc_dev,
"Flushing tx queue failed: %d\n", ret);
return ret;
}
-#endif
/*
* BEGIN mvm/sta.c
@@ -3894,6 +3893,8 @@ iwm_assoc(struct ieee80211vap *vap, struct iwm_softc *
static int
iwm_release(struct iwm_softc *sc, struct iwm_node *in)
{
+ uint32_t tfd_msk;
+
/*
* Ok, so *technically* the proper set of calls for going
* from RUN back to SCAN is:
@@ -3913,7 +3914,18 @@ iwm_release(struct iwm_softc *sc, struct iwm_node *in)
* back to nothing anyway, we'll just do a complete device reset.
* Up your's, device!
*/
- /* iwm_mvm_flush_tx_path(sc, 0xf, 1); */
+ /*
+ * Just using 0xf for the queues mask is fine as long as we only
+ * get here from RUN state.
+ */
+ tfd_msk = 0xf;
+ mbufq_drain(&sc->sc_snd);
+ iwm_mvm_flush_tx_path(sc, tfd_msk, IWM_CMD_SYNC);
+ /*
+ * We seem to get away with just synchronously sending the
+ * IWM_TXPATH_FLUSH command.
+ */
+// iwm_trans_wait_tx_queue_empty(sc, tfd_msk);
iwm_stop_device(sc);
iwm_init_hw(sc);
if (in)
@@ -4120,7 +4132,17 @@ iwm_newstate(struct ieee80211vap *vap, enum ieee80211_
if (((in = IWM_NODE(vap->iv_bss)) != NULL))
in->in_assoc = 0;
- iwm_release(sc, NULL);
+ if (nstate == IEEE80211_S_INIT) {
+ IWM_UNLOCK(sc);
+ IEEE80211_LOCK(ic);
+ error = ivp->iv_newstate(vap, nstate, arg);
+ IEEE80211_UNLOCK(ic);
+ IWM_LOCK(sc);
+ iwm_release(sc, NULL);
+ IWM_UNLOCK(sc);
+ IEEE80211_LOCK(ic);
+ return error;
+ }
/*
* It's impossible to directly go RUN->SCAN. If we iwm_release()
More information about the svn-src-stable-11
mailing list