Remove the references to the TX IC lock - i ended up solving this
using net80211 to seralise encap+xmit, so now it's a non-issue.
This commit is contained in:
parent
648dfc1a29
commit
b94ab33751
@ -261,7 +261,6 @@ ath_ahb_attach(device_t dev)
|
||||
ATH_PCU_LOCK_INIT(sc);
|
||||
ATH_RX_LOCK_INIT(sc);
|
||||
ATH_TX_LOCK_INIT(sc);
|
||||
ATH_TX_IC_LOCK_INIT(sc);
|
||||
ATH_TXSTATUS_LOCK_INIT(sc);
|
||||
|
||||
error = ath_attach(device_id, sc);
|
||||
@ -271,7 +270,6 @@ ath_ahb_attach(device_t dev)
|
||||
ATH_TXSTATUS_LOCK_DESTROY(sc);
|
||||
ATH_RX_LOCK_DESTROY(sc);
|
||||
ATH_TX_LOCK_DESTROY(sc);
|
||||
ATH_TX_IC_LOCK_DESTROY(sc);
|
||||
ATH_PCU_LOCK_DESTROY(sc);
|
||||
ATH_LOCK_DESTROY(sc);
|
||||
bus_dma_tag_destroy(sc->sc_dmat);
|
||||
@ -315,7 +313,6 @@ ath_ahb_detach(device_t dev)
|
||||
ATH_TXSTATUS_LOCK_DESTROY(sc);
|
||||
ATH_RX_LOCK_DESTROY(sc);
|
||||
ATH_TX_LOCK_DESTROY(sc);
|
||||
ATH_TX_IC_LOCK_DESTROY(sc);
|
||||
ATH_PCU_LOCK_DESTROY(sc);
|
||||
ATH_LOCK_DESTROY(sc);
|
||||
|
||||
|
@ -283,7 +283,6 @@ ath_pci_attach(device_t dev)
|
||||
ATH_PCU_LOCK_INIT(sc);
|
||||
ATH_RX_LOCK_INIT(sc);
|
||||
ATH_TX_LOCK_INIT(sc);
|
||||
ATH_TX_IC_LOCK_INIT(sc);
|
||||
ATH_TXSTATUS_LOCK_INIT(sc);
|
||||
|
||||
/*
|
||||
@ -371,7 +370,6 @@ bad1:
|
||||
ATH_TXSTATUS_LOCK_DESTROY(sc);
|
||||
ATH_PCU_LOCK_DESTROY(sc);
|
||||
ATH_RX_LOCK_DESTROY(sc);
|
||||
ATH_TX_IC_LOCK_DESTROY(sc);
|
||||
ATH_TX_LOCK_DESTROY(sc);
|
||||
ATH_LOCK_DESTROY(sc);
|
||||
|
||||
@ -408,7 +406,6 @@ ath_pci_detach(device_t dev)
|
||||
ATH_TXSTATUS_LOCK_DESTROY(sc);
|
||||
ATH_PCU_LOCK_DESTROY(sc);
|
||||
ATH_RX_LOCK_DESTROY(sc);
|
||||
ATH_TX_IC_LOCK_DESTROY(sc);
|
||||
ATH_TX_LOCK_DESTROY(sc);
|
||||
ATH_LOCK_DESTROY(sc);
|
||||
|
||||
|
@ -940,26 +940,6 @@ struct ath_softc {
|
||||
#define ATH_TX_TRYLOCK(_sc) (mtx_owned(&(_sc)->sc_tx_mtx) != 0 && \
|
||||
mtx_trylock(&(_sc)->sc_tx_mtx))
|
||||
|
||||
/*
|
||||
* The IC TX lock is non-reentrant and serialises packet queuing from
|
||||
* the upper layers.
|
||||
*/
|
||||
#define ATH_TX_IC_LOCK_INIT(_sc) do {\
|
||||
snprintf((_sc)->sc_tx_ic_mtx_name, \
|
||||
sizeof((_sc)->sc_tx_ic_mtx_name), \
|
||||
"%s IC TX lock", \
|
||||
device_get_nameunit((_sc)->sc_dev)); \
|
||||
mtx_init(&(_sc)->sc_tx_ic_mtx, (_sc)->sc_tx_ic_mtx_name, \
|
||||
NULL, MTX_DEF); \
|
||||
} while (0)
|
||||
#define ATH_TX_IC_LOCK_DESTROY(_sc) mtx_destroy(&(_sc)->sc_tx_ic_mtx)
|
||||
#define ATH_TX_IC_LOCK(_sc) mtx_lock(&(_sc)->sc_tx_ic_mtx)
|
||||
#define ATH_TX_IC_UNLOCK(_sc) mtx_unlock(&(_sc)->sc_tx_ic_mtx)
|
||||
#define ATH_TX_IC_LOCK_ASSERT(_sc) mtx_assert(&(_sc)->sc_tx_ic_mtx, \
|
||||
MA_OWNED)
|
||||
#define ATH_TX_IC_UNLOCK_ASSERT(_sc) mtx_assert(&(_sc)->sc_tx_ic_mtx, \
|
||||
MA_NOTOWNED)
|
||||
|
||||
/*
|
||||
* The PCU lock is non-recursive and should be treated as a spinlock.
|
||||
* Although currently the interrupt code is run in netisr context and
|
||||
|
Loading…
x
Reference in New Issue
Block a user